格納した配列が空白の場合は処理を実行しないようにしたい（GAS）

Question

前提・実現したいこと

配列が空白の場合、Slackへ通知しないようにしたいです。

現状は空白の場合でも処理するようにしているのですが、nullだと対応できないようなので困っております。
※一部のコードは削除して記載しています。

該当のソースコード

// シートを取得し、progressValに格納
var Sheet = spreadsheet.getSheetByName("シート名");
var progressVal = Sheet.getRange(1,1,Sheet.getLastRow(),Sheet.getLastColumn()).getValues();

//1行ごとに処理
for(i=0; i<progressVal.length; i++){
  //各情報を取得
  date = new Date(progressVal[i][0]);
  code = progressVal[i][1];
  Status = progressVal[i][2];
  Submission = progressVal[i][10];

  //処理対象かをで確認する（セルが 1=対象・0=対象外）
  if (Submission == "1" ) {
    //処理対象の場合は各情報を格納する
    dateArray.push([Utilities.formatDate(date,'Asia/Tokyo','yyyy/MM/dd'),code,Status]);
  } else {
    //デバック用
    Logger.log((i+0) + "行目は処理対象外");
  }
}

// Slack通知用
if (dateArray[1] == null) {
  // 空白のため処理は必要ない。自分宛にその内容のメールを送信する。
  MailApp.sendEmail(toAddress,subject,main);
} else {
  // Slackのチャンネル宛に必要事項を記載し投稿する。
  UrlFetchApp.fetch(slackURL, slackOptions);
}

自身で対応できていない事

dateArray.push([Utilities.formatDate(date,'Asia/Tokyo','yyyy/MM/dd'),code,Status]);

上記で配列に格納した情報が空白（処理対象外のみだった場合）に、SlackにslackOptions（main）で指定した定型文だけ送られてしまうため
dateArray[1]が空白の場合はSlackに送らないようにしたいですが出来ず困っています。

▼空白状態のイメージ（処理対象があった場合は、線の間にセルから取得した情報が記載されます。）

================================================

================================================

Answer 1

今のコードでも、通知すべきデータが 0 行のときは、通知されないはずなので、他の部分や元のデータに想定外のところがあるのかもしれないです。
可能な限り、シート(の構成)と他のコード部分も共有していただきたいです。

---

コードの全体像が見えてないですが、おそらく

var dateArray = [];

みたいなのがあって、かつ K 列(Submission) が全て 0 だと通知したくない、ということですよね。
とりあえずは、こういう感じじゃないかと思います。

// Slack通知用
if (dateArray.length < 1) {
  // 空白のため処理は必要ない。自分宛にその内容のメールを送信する。
  MailApp.sendEmail(toAddress,subject,main);
} else {
  // Slackのチャンネル宛に必要事項を記載し投稿する。
  UrlFetchApp.fetch(slackURL, slackOptions);
}

修正すべきポイントとしては

• dateArray[1] と 1 を判定していますが、配列は 0 indexed なので、1 を見ると、Submission が 1 つだけの(通知すべき行が 1 行しかない)ときにも通知が抑制されます。
• dateArray[0] == null でも動きますが、javascript で == を使うと危険なので、=== にすべきですし、=== にするなら dateArray[0] === undefined にしないと誤判定します。配列なので、長さが 0 である、を判定するのが適切だと思います。
• ただ dateArray[1] == null でも dateArray に何も入っていないときは通知されないと考えられますので、他のところに何らかの問題があって通知がされているのではないかと思います。(そもそも 1/0 を入れる列を取り違えている(K列を見るはずではない)など、別の要素がありそうです)

Comments

[退会済みユーザー]

dateArray[1] === undefined　に修正した所、正常に作動しました！
年末年始の貴重なお時間でご回答いただきありがとうございます。
大変助かりました。

Answer 2

dateArray[1] == nullを!dateArray[1]にするといいと思います。

function test(text, cond)
{
  if (!text) {
    console.log(cond);
  }
}

function myFunction() {
  test(null, 'null');
  test('', '空文字列');
  test(' ', '空白文字');
}

結果：
null
空文字列

もし空白文字も真にしたいなら

  if (!text || text.trim() == '') {

とするしかないかな。

結果：
null
空文字列
空白文字

Comments

[itagagaki]

低評価には理由を書きなさい