GAS : 最後の一文が原因でエラー発生。googleカレンダーからスプレッドシートに出力する

Question

前提・実現したいこと

GASで、googleカレンダーから一括で過去のデータを引っ張ってきて、スプレッドシートに出力させようとしています。
スプレッドシートに出力する際に、エラーメッセージが発生しました。
エラー内容は、以下の通りです。

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

Exception: データの行数が範囲の行数と一致しません。データは 657 行ですが、範囲は 18137.行です。

該当のソースコード

// 参考URL : https://note.com/w_yoshida/n/n84b5fcbccb2b

var ss = SpreadsheetApp.getActiveSpreadsheet();
var listEventSheet = ss.getSheetByName("イベント一覧取得");
var updateEventSheet = ss.getSheetByName("イベント更新");
//実行メニューを作成
function onOpen() {
 var ui = SpreadsheetApp.getUi();
 var menu = ui.createMenu("GAS実行");
 menu.addItem("イベント取得", "listEvent");
 menu.addItem("イベント更新", "modifyEvent");
 menu.addToUi();
}
function listEvent(){
  var today = new Date();
  var tmpDate = new Date();
  var newDateMonth = tmpDate.getMonth() - 3;
  var newDateYear = tmpDate.getYear();
  if (newDateMonth == 0) {
    newDateMonth = 12
    newDateYear = newDateYear - 1;
   }
   if (newDateMonth == -1) {
    newDateMonth = 11
    newDateYear = newDateYear - 1;
   }
   if (newDateMonth == -2) {
    newDateMonth = 10
    newDateYear = newDateYear - 1;
   }
  if (newDateMonth < 10) {
  newDateMonth = '0' + newDateMonth;
　　}
  var pastDate = newDateYear + '/' + newDateMonth + '/01 00:00:00'; // 3ヶ月前の日付
  var memberId = [ // 指定したいカレンダーIDを配列に入れる
    '*****@group.calendar.google.com', 
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
    '*****@group.calendar.google.com'
  ];
    
  var lastColum = listEventSheet.getLastColumn();
  var lastRow = listEventSheet.getLastRow();
  var startRow = 5;
  var dataColumnCount = 9;
  var dataCount = 0;
 //イベントを取得する対象のカレンダーIDを取得
//  for(var m = 0; m < memberId.length; m++) { // memberId がオブジェクト配列だから、 forで回さないといけない
//    var calendar = CalendarApp.getCalendarById(memberId[m]); //指定したIDのカレンダーを取得
//    var schedules = calendar.getEventsForDay(today); 
 
 //シートのリスト出力部をクリア
  listEventSheet.getRange(startRow, 1, listEventSheet.getLastRow(), dataColumnCount).clear();
 
 //予定の一覧（今日〜3か月前までの分）を取得
 //停止ユーザーのカレンダーを指定してもエラーになるので注意
    for(var k = 0; k < memberId.length; k++) {
      var eventList = Calendar.Events.list(memberId[k], {
        timeMin: new Date(pastDate).toISOString(),
        timeMax: new Date().toISOString(),
        singleEvents: false,
        //orderBy: "startTime", //"orderBy"は"singleEvents"がtrueでないと使用できない
     maxResults: 2500 //APIの上限は2500
      });
 
     var values = [];
      if(eventList){
        for(var i = 0; i < eventList.items.length; i++){
     
     //まれに予定ではないレコードが含まれるので、"htmlLink"が存在しないレコードはスキップする
          if(!eventList.items[i].htmlLink)
          {
            continue;
          }
          
          var value = [];
          var strDate = '';
          var strStartTime = '';
          
     //終日予定の場合は"event.start.date"を、通常の予定の場合は"event.start.dateTime"を保持している
          if(eventList.items[i].start.dateTime){
            var startDate = new Date(eventList.items[i].start.dateTime);
            //var startDate = eventList.items[i].start.dateTime;
            
            //getMonthは0～11を返すので1を加算
            var strMonth = startDate.getMonth() + 1;
            
            
            //"時"の頭0が省略されるので付与
            var strHour = startDate.getHours();
            if(strHour < 10){
              strHour = '0' + strHour;
            }
            
            //"分"の頭0が省略されるので付与
            var strMinute = startDate.getMinutes();
            if(strMinute < 10){
              strMinute = '0' + strMinute;
            }
            
            strDate = startDate.getFullYear() + '-' + strMonth + '-' + startDate.getDate();
            strStartTime = strHour + ':' + strMinute;
          }
          else
          {
            strDate = eventList.items[i].start.date;
          }
     
          value.push(strDate);
          value.push(strStartTime);
          value.push(eventList.items[i].summary);
          value.push(eventList.items[i].organizer.email);
          value.push(eventList.items[i].visibility);
          value.push(eventList.items[i].guestsCanModify);
          value.push(eventList.items[i].recurrence);
          value.push(eventList.items[i].recurringEventId);
          value.push(eventList.items[i].id);
          
          values.push(value);
          dataCount++;
        }
        
        values.sort();
      }
     }
//  }
   //取得したデータをスプレッドシートにセット
   listEventSheet.getRange(startRow, 1,dataCount, dataColumnCount).setValues(values);
}

試したこと

最後の一文のdataCountを、values.lengthに変更してみました。その場合、カレンダーIDが入っている配列、memberIdの一番最後のデータのみ取り出され出力されてしまう結果となりました。

//取得したデータをスプレッドシートにセット
listEventSheet.getRange(startRow, 1, values.length, dataColumnCount).setValues(values);

補足情報（FW/ツールのバージョンなど）

お手上げ状態なので、何卒よろしくお願いいたします。

Comments

[m.ts10806]

>GAS
タグあります。
https://teratail.com/tags/Google+Apps+Script

[nanainu]

ご指摘ありがとうございます！

Answer 1

ざっと見た限りでは コード上で、

for(var k = 0; k < memberId.length; k++) {

のループ内に　var values = [] の記述があるのが良くなさそうです。

これだとmemberId毎に values がリセットされてしまい、forループを抜けて 書き込むときには最後の memberId分しか残ってない状態です。

エラーメッセージで データ（valuedsのサイズ）が 、行の範囲（dataCount) より大幅に小さい数なのもその為です。

var dataCount = 0; の下あたりに、var values = [] を移動してみてください。

Comments

[nanainu]

おっしゃった通りに修正したら、解決しました！！
わかりやすいご説明で大変助かりました。
ありがとうございます！