一次元配列から二次元配列へ変換したのち、シートにデータを貼り付けたい。

Question

実現したいこと

for文で作った一次元配列のデータを二次元配列に変換し、シートの指定カ所に一気に貼り付けたいです。

前提

GoogleスプレッドシートのGASで一覧表のようなものを作っています。
C列・D列のデータを二次元配列として取得し、for文で加工して1つの文字列として一次元配列に作り直し、B列に一気に入れる、という作業をしたいです。

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

Exception: The parameters (number[]) don't match the method signature for SpreadsheetApp.Range.setValues.

該当のソースコード

let njigenText = [ichijigenText];
sheet.getRange(2,2,njigenText.length,1).setValues(njigenText);

試したこと

C列・D列のデータを取得することは出来て、文字列の加工もできました。
文字列で一次元配列を作ることも出来ました。
ネットで調べたところ、一次元配列ではsetValuesは使えない、とのことでした。

なので、
let njigenText = [ichijigenText];
で二次元化してみました。

しかし、
sheet.getRange(2,2,njigenText.length,1).setValues(njigenText);
で貼り付けられず、エラーが出てしまいます。

njigenText が二次元配列化に失敗しているのかと思い、
Logger.log(njigenText [0][10]
等で見てみましたが、問題は無いようです。

引数等が間違っているのかと思い、getRangeの数字を調整したりしたのですが、どうにもうまくいきません。

今現在は、setValuesではなくsetValueで、for文で回して1つずつ配列をセルに書き込んでごまかしています。
しかし、今後データが増えることを考えると、for文は使いたくありません。

なにとぞお知恵を拝借できないでしょうか？
よろしくお願いいたします。

Answer 1

既に回答が出ていますが、エラーになるのは修正方法が回答者様の想定と違っているからだと思います。
質問者様に修正後のスクリプトの全体を示していただければと思いますが。

例えば、このように全体をお示ししてみるとどうでしょうか。
※ 回答を修正して、もう一つの参考例を追記しました。

function myFunction() {
  const ss = SpreadsheetApp..getActiveSpreadsheet();
  const sheet = ss.getSheetByName('メインシート');
  //C,D列の値を取得
  const values = sheet.getRange(1, 3, sheet.getLastRow(), 2).getValues();
  //データの見出し行（1行目）を削除
  values.shift();
  //C,D列の値を★で結合した一次元配列に変換 
  const ichijigenText = values.map(v => v.join('★'));
  console.log(ichijigenText);
  //二次元配列に変換
  const nijigenText = ichijigenText.map(v => [v]);
  console.log(nijigenText);
  //B列（2行目以降）に値を記入
  if (nijigenText.length > 0) {
    sheet.getRange(2, 2, nijigenText.length, 1).setValues(nijigenText);
  }
}

なお、今回は、
一次元配列と二次元配列の変換がご質問の主旨でしたので上のような例としてますが、
一次元配列は二次元配列への変換用に一時的に生成されているだけなので、
最初から列を結合した二次元配列を生成することもできますので参考まで。

function myFunction() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = ss.getSheetByName('メインシート');
  //C,D列を結合した二次元配列を生成
  const nijigenText = sheet.getRange(2, 3, sheet.getLastRow() - 1, 2)
    .getValues().map(v => [v.join('★')]);
  //B列に値を記入
  if (nijigenText.length > 0) {
    sheet.getRange(2, 2, nijigenText.length, 1).setValues(nijigenText);
  }
}

Comments

[退会済みユーザー]

色々ご提示いただいて試してはいるのですが、何をしても「～is not a function」的なエラーが出てしまいます。

コードの流れ的には、これでほぼ揃っています（他に行見出しの設定などありますが割愛しました）。
改めて皆様のお知恵を拝借したいです。

let ss = SpreadsheetApp.getActiveSpreadsheet();
let mainSheet = SpreadsheetApp.getActiveSpreadsheet() .getSheetByName('メインシート);
ss.setActiveSheet(mainSheet);

let lastRow = mainSheet.getLastRow();

let nijigenDisplayValues = mainSheet.getRange(2,3,lastRow,2).getDisplayValues();

//文字列の加工
let i;
let ichijigenText = {};
for(i=0; i<lastRow; i++){
ichijigenText[i] = nijigenDisplayValues[i][0] + '★' + nijigenDisplayValues[i][1];
}

let nijigenText = [ichijigenText];

//現在はこの方法でごまかしていますが、本音は一発で貼れるsetValuesを使いたいです。
for(i=0;i<lastRow;i++){
mainSheet.getRange(i + 2, 2).setValue(nijigenText[0][i]);
}

[YAmaGNZ]

let ichijigenText = {};
こちらの定義でichijigenText は連想配列となっています。
let ichijigenText = [];
for(i=0; i<lastRow; i++){
ichijigenText.push(nijigenDisplayValues[i][0] + '★' + nijigenDisplayValues[i][1]);
}
と配列として定義してください。

[YellowGreen]

全体としてお示ししたスクリプトを質問者様にご提示いただいたスクリプトの動作にあわせてみました。

[退会済みユーザー]

ご回答いただきありがとうございます。
連想配列とは気が付きませんでした。皆様のご意見を伺っていて、GASを使うならJavaScriptもきちんと勉強しないとダメだと痛感しました。これから頑張ります。

さて、ご提示いただいた普通の配列の書き方をしたところ、無事動きました！

let nijigenText = [];
ichijigenText.forEach(value => nijigenText .push([value]));
mainSheet.getRange(2,2,lastRow,1).setValues(nijigenText);

これでOKでした。
forEachの部分が全然意味不明ですが、これから勉強してみます。
こんな小さな質問にお付き合いいただき、本当にありがとうございました。

[YellowGreen]

解決に導いたのはYAmaGNZ様だと思いますので、ベストアンサーは辞退させていただければと思います。

[退会済みユーザー]

承知いたしました。
しかし、私としては皆様にベストアンサーを差し上げたい気持ちです。
今後勉強する方向性も含め、色々な気付きを得ることが出来、大変有意義でした。

[退会済みユーザー]

追記の参考例を拝見しました。
早速その通りに書いて動かしてみたのですが、無事動きました！

ご提示いただいた書き方のほうがコードがスマートですね。アロー関数というのが初耳で、使うこと自体、全然思いも付きませんでした。mapというのも馴染みが無いです。
今ちょっと調べましたが、JavaScriptのほうで便利なメソッドがたくさんあるんですね。これはますますJavaScriptを勉強した方がいいと思いました。

発展編という感じに受け取りました、とても勉強になりました。
本当にありがとうございました。

[YellowGreen]

参考にしていただけてよかったです。
配列に対する様々なメソッドとアロー関数を組み合わせるとスッキリしたコードになるので、
for (let i = 0; i < values.length; i++) {
for (let j = 0 ; j < values[0].length; j++) {
...values[i][j]...;
}
}
などインデックスを利用した煩雑なコードを書くことが減りました。
繰り返し処理でインデックスを利用したい時は、
values.forEach((v, i) => ...);
とすると i をインデックスとして使えます。

[退会済みユーザー]

こちらこそ、コードの書き方っていろいろあるんだな、ということを教えていただき、本当にありがたいです。

アロー関数はいきなりは使えなさそうです……何かサンプルを作って練習したほうが良さそうですね。
土日の楽しみが増えました。ありがとうございました。

Answer 2

2次元配列をsetValuesで入力する場合、njigenText [0][10]で言う所の[0]が行で[10]が列として処理されるので縦一列の範囲に横一列のデータが入力出来ずにエラーになっています。
なので配列内の行と列を入れ替える必要があります。

  let njigenText = [ichijigenText];
  const transpose = a => a[0].map((_, c) => a.map(r => r[c]));
  njigenText = transpose(njigenText);

Comments

[退会済みユーザー]

ご回答いただきありがとうございます。

const transpose = a => a[0].map((_, c) => a.map(r => r[c]));
njigenText = transpose(njigenText);

この2行の部分で、

TypeError: a[0].map is not a function

というエラーが出てしまいます。aやcなど、何も宣言せず使ってよろしいのでしょうか？

[bebebe_]

let ichijigenText = {};
を
let ichijigenText = [];
にしてみてください

[退会済みユーザー]

ご指摘ありがとうございます。
その通りでした。連想配列は間違っていました。

Answer 3

1列に入れるということは
[[項目１,項目２,項目３,項目４]]
といった配列ではなく
[[項目１],[項目２],[項目３],[項目４]]
という配列にしなくてはなりません。

なので下記のように２次元配列に変換することになります。

let njigenText = [];
ichijigenText.forEach(value =>  njigenText.push([value]));

動作するサンプルとしては下記のようになります。

  let values = [["項目１","項目２"],["項目３","項目４"]];
  let ichijigenText = values.flat();
  Logger.log(ichijigenText);

  let nijigenText  = [];
  ichijigenText.forEach(value =>  nijigenText  .push([value]));
  Logger.log(nijigenText  );

Comments

[退会済みユーザー]

ご回答いただきありがとうございます。

forEachの行で、

TypeError: ichijigenText.forEach is not a function

というエラーが出てしまいます。valueはそのままvalueと書いてよろしいのでしょうか？

[YAmaGNZ]

ichijigenTextは1次元の配列なのではないですか？

[退会済みユーザー]

度々申し訳ありません。2行目の
let ichijigenText = values.flat();
の部分で

TypeError: syuttaikinText.flat is not a function
のエラーが出てしまいます。

ichijigenTextは一次元配列だと思います。
ichijigenText[0]　や　ichijigenText[1]　でログが取れます。

[YAmaGNZ]

syuttaikinTextというものが突然出てきましたが、どのように取得しているのですか？
そのあたりの取得からスプレッドシートへのセットまでのソースは開示できないですか？

[退会済みユーザー]

ごめんなさい、突然出てきたものは業務用の変数名なのでスルーしてください。