GASで抽選人数をカスタマイズし、当選者が重複しない関数を作りたいです。

Question

前提・実現したいこと

指定したシートで、行及び当選者数をカスタマイズして、抽選を行い、当選者のセルの背景色を黄色にしたいです。
しかし、当選者が重複しないようにしたいのですが、上手くできなくて質問させていただきました。

該当のソースコード

function Roll() {
  var sheet = SpreadsheetApp.openById("リンク先").getSheetByName("test2");
  var lastRow = sheet.getLastRow();　
   for (var j = 0; j < 10;j++) {　　//当選者数を設定
  var row = Math.ceil(Math.random() * (lastRow-1)) + 1;   
  var tag = sheet.getRange(row, 1).setBackground("yellow"); 
  }
}

試したこと

抽選する際に、getvalueを使って配列に格納し、当選者をリストアップして比較したかったですが、上手く格納できなくて失敗しました。
また、getvalueの際に、背景色を設定できなくて困ってます。

Answer 1

抽選する際に、getvalueを使って配列に格納し、当選者をリストアップして比較したかったですが、上手く格納できなくて失敗しました。

こちらで対応してみました。

function myFunction() {
  function Roll() {
    var sheet = SpreadsheetApp.openById("リンク先").getSheetByName("test2");
    var lastRow = sheet.getLastRow();
    var l = [];
    for (var j = 0; j < 20;j++) {　　//当選者数を設定
      var row = Math.ceil(Math.random() * (lastRow-1)) + 1;
      while(l.indexOf(row) >= 0) {
        row = Math.ceil(Math.random() * (lastRow-1)) + 1;
      }
      l.push(row);
      var tag = sheet.getRange(row, 1).setBackground("yellow"); 
    }
  }
  Roll();
}

本題とはズレますが、「Math.ceil(Math.random() * (lastRow-1)) + 1;」lastRow-1やランダム値に+1すると微妙に範囲がズレますが、意図している挙動なのでしょうか？？

---

行番号自体は重複していないのですが、アンケートなどを回答した者が重複してアンケートに参加した場合、異なる行の番号に同じ者の名前が重複してしまいます。

追記を受けて回答を変更しました。
また、元のコードを活かすとかなり冗長になったため書き直しております。
わからない部分もあるかもしれませんが、コメントもつけてますし、よく読めばそんなに難しい事はありません。

function myFunction2() {
  // 重複を除去したい値が入っている列数を指定（B列だったら2）
  var key_column_number = 2;
  
  // 色を塗りたい数
  var n = 5;
  
  function Roll() {
    var sheet = SpreadsheetApp.openById("xxxx").getSheetByName("test2");
    var lastRow = sheet.getLastRow();
    
    // 対象の値を全て取得
    var values = sheet.getRange(1,1,lastRow, key_column_number).getValues();
    
    // 指定した列の値が重複した行は除く
    var unique_values = values.reduce(function(x, y, index) {
      if (x.names.indexOf(y[1]) < 0) {
        x.row_nubers.push(index + 1);
        x.names.push(y[1]);
      }
      return x;
    }, {row_nubers: [], names: []});
    var row_numbers = unique_values.row_nubers;
    
    // 行番号をランダムで並び替え
    row_numbers.sort(function() {return  Math.random() - .5});

    // 先頭から色を塗りたい行数分だけ行番号を取得
    var target_row_numbers = row_numbers.slice(0,n);
    
    // 色塗り
    target_row_numbers.forEach(function(v) {
      sheet.getRange(v, 1).setBackground("yellow");
    });
  }
  Roll();
}

Comments

[koroaisu]

ご回答、誠にありがとうございます！
「Math.ceil(Math.random() * (lastRow-1)) + 1;」につきまして、ランダムでセルを取得する関数を調べてそのまま使っていました。ご指摘ありがとうございます！

[koroaisu]

お忙しい中すみません、ご教授の関数を試してみましたが、一つの列の中に同じ値のセルが存在した場合(例:A1、A18の値=thankyou)、当選者が重複してしまいます。indexOfではなく、もしかしたら、配列の長さに応じて、valueごとに比較した方がいいでしょうか。

[yamap55]

保持しているものは行番号なので重複する事はないと思います。Logger.log(l)などとして値を見ていただければ確認できます。

> もしかしたら、配列の長さに応じて、valueごとに比較した方がいいでしょうか。

ここは何を言っているかわかりません。（配列とは何？valueごとに何と何を比較？）

[koroaisu]

説明が下手で申し訳ございません。
行番号自体は重複していないのですが、アンケートなどを回答した者が重複してアンケートに参加した場合、異なる行の番号に同じ者の名前が重複してしまいます。
そのため、一度抽選した当選者は配列に格納し、抽選するたびに、すでに当選した者は配列にpushせず、再度抽選を行う形にできればと思います。

[koroaisu]

親切で丁寧なご回答、誠にありがとうございます！
ご教授頂いたコードを消化してみます。
本当に大変、助かりました。ありがとうございます！