Discussion :

[arobase37]

Bonjour,
Je viens de passer un moment sur un bout de script mais qui me donne du fil à retordre !

A la base, c'est simple. Si la valeur de colonne B est égale à 2 (plus tard, ce sera en fonction de la date du jour, une autre histoire), la valeur de la cellule de la colonne A doit être modifiée en "Terminé".
Le truc c'est que mon script implémente le "Terminé" partout, je n'arrive pas à lui indiquer de ne modifier que la cellule en face avant de recommencer le check sur la ligne du dessous.

Si quelqu'un a une idée, ça m'intéresse :-)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
function myFunction() {
var ss = SpreadsheetApp.getActiveSpreadsheet();
var sheet = ss.getSheetByName("Suivi ventes");
var lastRow = sheet.getLastRow();
var startRow = 2;  
 
//récupère la date de fin  
var range = sheet.getRange(2, 2,lastRow-startRow+1,1 );
var numRows = range.getNumRows();
var date_fin_values = range.getValues();
 
 
// définit la zone de texte à modifier  
 range = sheet.getRange(2, 1,lastRow-startRow+1,1 );
var cell_emplacement = range.activate();  
 
 
for (var i = 0; i <= numRows - 1; i++) {
var date_fin = date_fin_values[i][0];
  var cell = cell_emplacement[i];
  if (date_fin == 2) {
     cell_emplacement.setValue("Terminé");   
   }
 }
}

Merci merci

[Bktero]

Bonjour,

Un code comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
function terminate() {
  var sheet = SpreadsheetApp.getActiveSheet();
  var rows = sheet.getLastRow();
  Logger.log(rows + ' rows to process');
 
  for (var i = 1; i <= rows; ++i) {
    var b = sheet.getRange(i, 2, 1, 1);
    var a = sheet.getRange(i, 1, 1, 1);
    var value = (b.getValue() == 2) ? 'Terminé' : 'A faire';
    a.setValue(value);
  }
}

?

[arobase37]

Merci beaucoup !
C'est parfait :-)

[Bktero]

Après quelques essais, il semble que cette version soit plus performantes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
function terminate() {
  var sheet = SpreadsheetApp.getActiveSheet();
  var rows = sheet.getLastRow() - 1; // -1 to skip header
 
  var values_b = sheet.getRange(2, 2, rows, 1).getValues();
 
  for (var i = 1; i <= rows; ++i) {
    var value_a = (values_b[i] == 2) ? 'Terminé' : 'A faire';
    var cell_a = sheet.getRange(i + 1, 1);
    cell_a.setValue(value_a);
  }
}

En fait, les getValue() sont lents et tu as intérêt à les sortir de la boucle. Voir https://yagisanatode.com/2017/12/13/...and-wrong-way/

[arobase37]

Par contre, la nouvelle valeur est modifiée dans la ligne du dessus :/
Je ne vois pas où corriger.

[Bktero]

Au temps pour moi !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var value_a = (values_b[i - 1] == 2) ? 'Terminé' : 'A faire';

Les Array de JavaScript ont des indices partant de 0 alors que les Range de Google Sheet ont des indices partant de 1. Ça fait des jeux assez inhabituels sur les indices...

[arobase37]

Idem, les indices me font perdre les pédales et comme je débute :-)
Merci beaucoup