GASでExcelのデータ表からSQLのINSERT文を自動生成したいです

Question

前提・実現したいこと

タイトル通りです。

発生している問題・エラーメッセージ

困っているのは、INSERT文を生成する時に、文字列であれば''をつけ、そうでなければ''をつけないという様にif文で条件分岐させたのですが、日付だけ文字列として認識されず''がつかずにINSERT文が生成されてしまうというものです。日付にも''をつけたいのですが、数値や文字列かどうか確かめるためのisNaNなどの関数を見つけられなかったのでどうしたらいいか詰まっています。
宜しくお願いいたします。

該当のソースコード

function addInsert() {
  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var sheet1 = spreadsheet.getSheetByName("シート1");
  var lastColumn = sheet1.getLastColumn();
  var lastRow = sheet1.getLastRow();

  // INSERT文のカラムを入れるところまで書く
  var sentence = 'INSERT INTO ' + sheet1.getRange(1,2).getValue() + '(';
  // スプレッドシートの2行目にカラムが入っているのでそれを取り出す
  for (var i = 1;i <= lastColumn;i++) {
    if(i != lastColumn) {
      sentence += sheet1.getRange(2,i).getValue() + ',';
    } else {
      sentence += sheet1.getRange(2,lastColumn).getValue() + ')';
    }
  }
  
  // VALUESのあとの文を追加する
  sentence += ' VALUES';
  // 3行目からデータが記述されているのでそれらを行ごとに、最後の列まで取り出す
  for (var i = 3;i <= lastRow;i++) {
    for (var j = 1;j <= lastColumn;j++) {
      if(sheet1.getRange(i,1).getValue() == '') break;
      // 数値ではなかった場合にシングルクォーテーションで文字をくくるようにする
      if (j == 1) {
        sentence += '(';
      }
      
      // 最後のカラムではない場合
      if (j != lastColumn) {
        if (!isNaN(sheet1.getRange(i,j).getValue())) {
          sentence += sheet1.getRange(i,j).getValue() + ',';
        } else {
          sentence += "'" + sheet1.getRange(i,j).getValue() + "',";
        }
        //最後のカラムかつ最後の行、つまり最後のセルである場合。 )で閉じる必要がある
      } else if (j == lastColumn && i == lastRow) {
        if (!isNaN(sheet1.getRange(i,j).getValue())) {
          sentence += sheet1.getRange(i,j).getValue() + ')';
        } else {
          sentence += "'" + sheet1.getRange(i,j).getValue() + "')";
        }
        // 最後のカラムである場合 ),で閉じてピリオドを入れる必要がある
      } else if (j == lastColumn) {
        if (!isNaN(sheet1.getRange(i,j).getValue())) {
          sentence += sheet1.getRange(i,j).getValue() + '),';
        } else {
          sentence += "'" + sheet1.getRange(i,j).getValue() + "'),";
        }
      }
    }
  }
}

以下が実際に生成されたSQL文です

INSERT INTO facility(id,name,type_id,capacity,insert_date,update_date,user_id) VALUES(1,'会議室001',1,816,Sat Jan 01 2000 00:00:00 GMT+0900 (日本標準時),Tue Jan 01 2002 00:00:00 GMT+0900 (日本標準時),'A0001'),(2,'会議室002',1,506,Sun Jan 02 2000 00:00:00 GMT+0900 (日本標準時),Wed Jan 02 2002 00:00:00 GMT+0900 (日本標準時),'A0002'),(3,'会議室003',1,450,Mon Jan 03 2000 00:00:00 GMT+0900 (日本標準時),Thu Jan 03 2002 00:00:00 GMT+0900 (日本標準時),'A0003')

スプレッドシートはこんな感じです。

テーブル名 facility
id name type_id capacity insert_date update_date user_id
1 会議室001 1 816 2000/1/1 0:00 2002/1/1 0:00 A0001
2 会議室002 1 506 2000/1/2 0:00 2002/1/2 0:00 A0002
3 会議室003 1 450 2000/1/3 0:00 2002/1/3 0:00 A0003

Answer 1

isNaNではなく、typeofやinstanceof使って判定すればいいです。

if(typeof sheet1.getRange(i,j).getValue() === "string") {
    //文字列
}
else if(typeof sheet1.getRange(i,j).getValue() === "number") {
    //数値
}
else if(sheet1.getRange(i,j).getValue() instanceof Date) {
    //日時
}

Comments

[heysuky]

回答ありがとうございます。
試したのですが、console.logで日付の入っているセルの型を調べたところ、objectと出ていたのでそれだとうまくいきませんでした。
何か解決方法ありますでしょうか？

[draq]

typeof だけで調べてませんか？Date型はtypeofではobjectを返すので、instanceof 使って調べてください。

[heysuky]

すみません、返信きてたことに気づかず送ってしまってました。

何かを見落としていたのか、instanceof Dateでやってみたらうまくいきました。
自分のコードよりわかりやすくて助かりました。ありがとうございます。