Effectuer une action en fonction du résultat d'une tickbox

Question

Une fois n'est pas coutume, je reviens vers vous car je bloque... Globalement et pour expliquer le but, le sheet concerné me sert à traiter les prêts de matériel. https://docs.google.com/spreadsheets/d/1clsBBaHXftBXimCepfCpO6EhhwKpjllsJweycsA7XHA/edit?usp=sharing Ce que je souhaite, c'est que lorsque je clique sur mon bouton "DELIVER SELECTED", cela envoie un mail avec les infos du prêt à l'utilisateur concerné. La partie mail est fonctionnelle mais pas implémentée dans ce fichier de test. Là où je bloque, c'est que j'ai essayé de faire une boucle incrémentielle pour prendre les infos et envoyer le mail (en l'occurrence j'ai remplacé l'action d'envoyer un mail par effectuer une copie dans des cellules) pour chaque ligne jusqu'à la dernière ligne concernée cochée, et j'ai seulement la première ligne prise en compte... Voici le code concerné: // Deliver List A - BOUTON: "DELIVER SELECTED" function deliverA() { var spreadsheet = SpreadsheetApp.getActive(); var sheetDeliverA = spreadsheet.getSheetByName("Deliver List A"); //Début boucle-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- for (var i=9;i&lt;=13;i++){ var LoanToA = sheetDeliverA.getRange('C'+i).getValue(); var MailA1 = sheetDeliverA.getRange('D'+i); var MailA2 = sheetDeliverA.getRange('E'+i).getValue(); var LoanAssetA = sheetDeliverA.getRange('F'+i).getValue(); var TickBoxA = sheetDeliverA.getRange('I'+i).getValue(); //Début condition 1_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_- if(TickBoxA === true) { //Début condition 2_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_- if(MailA2 !== "") { MailAfull = MailA2; } else { MailAfull = MailA1; //DEBUT MANIP A FAIRE (FUTUR ENVOI MAIL)----------------------------------- sheetDeliverA.getRange('L'+i).setValue(LoanAssetA); sheetDeliverA.getRange('M'+i).setValue(MailAfull); sheetDeliverA.getRange('N'+i).setValue(LoanByA); sheetDeliverA.getRange('O'+i).setValue(LoanToA); //FIN MANIP A FAIRE-------------------------------------------------------- //Fin condition 2_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_- }; //Fin condition 1_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_- }; //Fin boucle-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=- }; Browser.msgBox("Loan(s) delivered successfully & mail(s) sent"); }; Pouvez-vous m'aider (encore et encore...)...

сталиевич · Accepted Answer

Commence par travailler tes boucles comme ceci (exécute la fonction dans l'éditeur de script)

function architecture() {
  var spreadsheet = SpreadsheetApp.getActive();
  var sheetDeliverA = spreadsheet.getSheetByName("Deliver List A");
  var sheetReservationA = spreadsheet.getSheetByName("Reservation List");
  var lastRow1 = sheetDeliverA.getLastRow();
  var lastRow2 = sheetReservationA.getLastRow();

  for (var i=9;i<=lastRow1;i++){
    var TickBoxA = sheetDeliverA.getRange('F'+i).getValue();
    var DeliveredA = sheetDeliverA.getRange('G'+i).getValue();

    if((TickBoxA == true) && (DeliveredA != 'Ok')){
      var LoanToA = sheetDeliverA.getRange('C'+i).getValue();

      Logger.log(LoanToA)

      for (var j=1;j<=lastRow2;j++){
        if (sheetReservationA.getRange('A'+j).getValue() == LoanToA) {
          var CourseNbA = sheetReservationA.getRange('B'+j).getValue();

          Logger.log(CourseNbA)

        }
      }

    }
  }
}

après tu remplis à l'intérieur des boucles

сталиевич · Answer

J'ai une erreur : LoanByA n'est pas défini ! et donc a priori le script s'arrête là à la première occurrence...

сталиевич · Answer

Quand je lance le script, j'ai bien autant de résultats que de cases cochées ...

сталиевич · Answer

Pas la peine de faire une boucle, il suffit d'ajouter dans ton tableau =iferror(query('Reservation List'!$A$1:$H;"select D,E,F where A='"& C9 &"' and B='"& $B$9 &"' ";0);"pas trouvé !") que j'ai mis en colonne Q pour le moment ET ... certaines personnes ne sont pas trouvées !...

сталиевич · Answer

J'ai par principe de toujours faire par formule quand je peux plutôt que par script. Pourquoi ? parce que le script s'exécute côté serveur et qu'il vaut mieux minimiser cette pratique sachant qu'en plus du temps du script il faut y ajouter les temps de transferts. Non, on peut faire une boucle dans une boucle sans problème. Et dans ton cas, il est même préférable de d'abord charger la table en mémoire car balayer à chaque pas d'une boucle la feuille par script peut amener des temps de réponses significatifs (au moins sensibles, voire rédhibitoires si la table est importante) Je vais regarder ce point pour t'aider bien qu'il ne me plaise pas...

сталиевич · Answer

Il faut indenter correctement un code pour qu'il soit lisible pour charger les données de Reservation List var sheetReservationA = spreadsheet.getSheetByName("Reservation List") // ici je charge les données de Reservation List var reseravtionList = sheetReservationA.getDataRange().getValues(); il faudrait en faire de même Deliver List A pour éviter d'aller en continu chercher les infos dans la feuille je ne comprends pas qu'on ne ferme pas ici ce que tu appelles la condition 2 if(MailA2 !== "") { MailAfull = MailA2; } else { MailAfull = MailA1; // je ne comprends pas qu'on ne ferme pas ici } ... je continuerai plus tard car je m'y perds dans les données du fait du maque d'indentation...

сталиевич · Answer

C'est ce que tu as fait ici, mais sans exploiter ni le lastRow, ni le tableau readingGender var readingGender = sheetReservationA.getRange("D1:D15").getValues(); var lastRow = sheetReservationA.getLastRow(); par contre il ne faut pas le faire à chaque fois, mais une fois pour toutes...

сталиевич · Answer

N'ayant plus retrouvé le code sur lequel j'avais commencé à travailler ... je te laisse gérer la suite, d'autant que je ne vois pas les recommandations ci-dessus reprises....

сталиевич · Answer

Je dirais plutôt que le mail est envoyé 15 fois à la même personne ! parce que je crois que la boucle s'appuie sur la mauvaise page, mais comme le code n'est toujours pas indenté, ce n'est donc pas très lisible !

J'avais commencé à travailler sur un code qui a été effacé !

сталиевич · Answer

Déjà une des raisons, est que les 2 boucles ont la même variable i ... et donc la seconde influe sur la première. Commence par changer cela...