質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.32%

【GAS】複数のシートを一つのシートにまとめたいです。

受付中

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 2,375

kazuyahonda

score 4

 前提・実現したいこと

プログラム、GAS、teratail初心者です。

1.GASで複数のシートの値を一つのシートにまとめるプログラムを作成しています。

2.コピーの範囲は2行目から最終行、列は16列です。

3.貼り付け先のシートには、2行目、3列目からの貼り付けをしようとしています。

4.一つずつシートをコピペをしていくと、処理速度が遅くなったため、
全てのシートの値を一度配列に格納して、setValuesでまとめて書き出したいと考えています。

5.配列に格納はできましたが、
書き出しの際に以下のエラーメッセージが発生し、
書き出しがうまくいきません。

以下がコードとエラーメッセージとなります。
こちら解決策をご教授いただけますとありがたいです。

※質問に足りない情報などありましたらご指摘お願いいたします。

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

◆エラーメッセージ
2 は無効な高さです。6にする必要があります。

 該当のソースコード

GAS

  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var sh_all = spreadsheet.getSheetByName('all'); //コピー先のシート名all

  var copy_column_range = 16 //コピーする列数


  //全てのシートをallシートにコピー。
  var c_sheet_cnt = 2; //コピーするシート数
  var last_l_row = 2; //何行目からコピーするか※すべてのシート

  var dt_all_sheet = []; // 全てのシートのデータの配列
 //全てのシートのデータを配列に格納してから書き出し
  for (var i=0;i<c_sheet_cnt;i++){
    var sheet = spreadsheet.getSheets()[i]; 
    var new_l_row = sheet.getLastRow(); //各シートの最終行を取得

    //シートの値を配列に格納
     var copyValue = sheet.getRange(2,1,new_l_row-1,copy_column_range).getValues();
     dt_all_sheet.push(copyValue); // シートの値を配列に格納
     var last_l_row = last_l_row + new_l_row-1;
   }

  //Browser.msgBox(last_l_row);
  //sheet内容を書き出し
  sh_all.getRange(2,3,last_l_row,copy_column_range+2).setValues(dt_all_sheet);

 試したこと

sh_all.getRange(2,3,last_l_row,copy_column_range+2).setValues(dt_all_sheet);


が違うとエラーが出ていたため、範囲が合っていないと思い下記方法を試しました。

仮シートを二つ作成。
仮シート1には
2行目の1列~16列に1
3行目の1列~16列に2

仮シート2には
2行目の1列~16列に3
3行目の1列~16列に4
の値があります。

・last_l_rowをdt_all_sheet.lengthにした。
→2 は無効な範囲です。18 にする必要があります。とエラー

・直接数字を打ち込んでみた。
→何を打ち込んでも2は無効な高さです~にしてくださいと表示。

他も試しましたが、うまく行かない状況です。
こちらご回答いただけますと幸いです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

0

concatを使って2次元配列を結合するところがポイントになると思います。
あとシート名all自体は結合の範囲から除く必要があると思ったので分岐を入れました。
コピーするシート数も自動的にカウントさせるほうがいいかと思います。
添削が下手で恐縮ですができました。

function myFunction() {
  var spreadsheet = SpreadsheetApp.getActiveSpreadsheet();
  var sh_all = spreadsheet.getSheetByName('all'); //コピー先のシート名all
  var copy_column_range = 16 //コピーする列数
  //全てのシートをallシートにコピー。
  var c_sheet_cnt = spreadsheet.getSheets().length; //コピーするシート数
  var last_l_row = 2; //何行目からコピーするか※すべてのシート
  var dt_all_sheet = []; // 全てのシートのデータの配列
 //全てのシートのデータを配列に格納してから書き出し
  for (var i=0;i<c_sheet_cnt;i++){
    var sheet = spreadsheet.getSheets()[i];
    if (sheet.getName() === "all") {
      continue;
    }
    var new_l_row = sheet.getLastRow(); //各シートの最終行を取得

    //シートの値を配列に格納
     var copyValue = sheet.getRange(2,1,new_l_row-1,copy_column_range).getValues();
     var dt_all_sheet = dt_all_sheet.concat(copyValue); 
     var last_l_row = last_l_row + new_l_row-1;
   }
  //sheet内容を書き出し
  sh_all.getRange(2,3,dt_all_sheet.length,copy_column_range).setValues(dt_all_sheet);
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.32%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る