gas while文使い方

Question

前提・実現したいこと

while文によって空白を追加したい

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

while文が機能していないのか空白が追加されません。

該当のソースコード

function compare(data) {
const nullarray = [""]
let newarray = []
for(var i=1; i <300 ; i++){
if(data[i][15] !== "ok"){
newarray.push(data[i])
}
else{
continue
}
let matchCount = 0
for(var j=1; j < 301 ; j++){
if(i !== j && data[i][1] === "SinglePhrase" && data[j][1] === "SinglePhrase" && data[i][15] !== "ok" && data[j][15] !== "ok"){
if (data[i][12] === data[j][12] ) {
newarray[newarray.length - 1] = newarray[newarray.length - 1].concat(data[j]);
data[j].push("ok")
matchCount += 1
}
}

           }

}

while (newarray[newarray.length - 1] < 105) {
newarray[newarray.length - 1] = newarray[newarray.length - 1].concat(nullarray);
}

  return newarray
           }

試したこと

位置を変えたりした。

Comments

[Zuishin]

> while (newarray[newarray.length - 1] < 105) {
> newarray[newarray.length - 1] = newarray[newarray.length - 1].concat(nullarray);
> }

多分ここで無限ループしますね。
GAS は時間が切れたら強制終了するので、他のところが正常に動いたとしてもここより先には進まないはずです。

[gomagoma00]

何か解決策はございますか？
if文の内側のループが終わったときにこの処理をさせたいのですが。

[Zuishin]

問題がここだけとは限らないしこちらにはデータも動作環境もないので全ての問題を解決することはできません。おそらくここだけの話ではないと思っています。

[papinianus]

確認したところ、複数の要素をもつ array は < で比較するとき(つまり Numer にするとき) NaN になるので、while に入れないですね。なので while 内が実行されない。されたところで最終行しか補正されないので、修正案の回答を書きました。

Answer 1

function compare(data) {
    //const nullarray = [""]
    let newarray = []
    let maxmatchCount = 0;
    for(var i=1; i <20 ; i++){
        if(data[i][15] !== "ok"){
            newarray.push(data[i])
        }
        else{
            continue
        }
        let matchCount = 0
        for(var j=1; j < 20 ; j++){
            if(i !== j && data[i][1] === "SinglePhrase" && data[j][1] === "SinglePhrase" && data[i][15] !== "ok" && data[j][15] !== "ok"){
                if (data[i][12] === data[j][12] ) {
                    newarray[newarray.length - 1] = newarray[newarray.length - 1].concat(data[j]);
                    data[j].push("ok")
                    matchCount += 1
                }
            }

        }
        maxmatchCount = Math.max(maxmatchCount,matchCount);
    }

    for(let i = 0; i < newarray.length; i++) {
        while (newarray[i].length < 15 * (maxmatchCount +1)) {
            newarray[i].push("");
        }
    }
    return newarray
}
function t() {
    const book = SpreadsheetApp.getActive();
    const data = book.getActiveSheet().getDataRange().getValues();
    const result = compare(data);
    const format = result;
    book.getSheetByName("1").getRange(1, 1, format.length, format[0].length).setValues(format);
}

A	B	C	D	E	F	G	H	I	J	K	L	M	N	O
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	1	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	30	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	30	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	a	2	3	4	5	6	7	8	9	0	1	40	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	40	3	4
	a	2	3	4	5	6	7	8	9	0	1	9	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	10	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	1	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	30	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	30	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	20	3	4
	a	2	3	4	5	6	7	8	9	0	1	40	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	40	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	9	3	4
	SinglePhrase	2	3	4	5	6	7	8	9	0	1	10	3	4

Answer 2

コメントにもありますし、別の質問でもかいたようにデータサンプルがなければ確認のしようがないです。

あらためて書きますが

• サンプルデータを開示してください。
• compare の呼び出し元の関数を開示してください。
• 別質問のほうの t() を実行してください。
• やりたいことを具体化してください。
  率直に申し上げますが compare でつくっているデータ形式は異常です。フレーズという名前からして、単語をまとめあげているのではないかと思うのですが、出現頻度によって、横方向の長さが違うというのは変ですし、本来縦方向に意味が揃っている値(singlePhraseがあるB列)をよこにずらすとセル関数などであつかいづらいかたちになって、収拾がつかないです。
  105 までふやそうとしていますが、出現頻度によっては 105 でおさまる(7回)とはいえないですよね。早晩破綻します。逆にデータが最大で 60 しかないのに 105 つくるのは無意味です。
  その while でやっているのが別質問の squarize です。
  とてもおすすめできないですが、仮に重複キーワードをまとめあげしたいのであっても、セルを横にのばすのではなく、各セルごとに改行で連結していくとか、数値であれば足すとかいった方法で四角形の構造をくずさずにまとめる必要があると考えます。