GASのfor文がうまく動いてくれない

Question

前提・実現したいこと

for (var flg = 0; flg < 3; flg=flg+1) {

flgが3よりも小さい間、繰り返しを行いたいのですが、
i=1の場合までの処理は結果として戻ってくるのですが、
その後の結果が出てくれません。

下記の条件を通過させる為にはどうしたら良いでしょうか？

else if (i = 2) {
var valueColumn = 12 ;
}

else if (i = 2) {
var setRow = 4 ;
}

該当のソースコード

function mainMessage(event) {

//スプレッドのURLを指定
var ss = SpreadsheetApp.openById("スプレッドシートのID");
// 一覧を連想配列化
var sheet1 = ss.getSheetByName('シート2');
var sheet2 = ss.getSheetByName('シート1');
var kvRow = 2;        // 開始する行数を定義（ヘッダは省く） 
var lastRow = sheet1.getLastRow();

//******************************************************
var i=0
Logger.log(i);
for (var flg = 0; flg < 3; flg=flg+1) {
　　　　　
var keyColumn = 1;    // キーの列を定義

if (i == 0) {
var valueColumn = 2 ; 
}
else if (i == 1)  {
var valueColumn = 11 ; 
}
else if (i == 2)  {
var valueColumn = 12 ; 
}
else {

}

var kvData = sheet1.getRange(kvRow, keyColumn, sheet1.getLastRow(), valueColumn).getValues();
Logger.log(kvData);
var kvDic = {};
for each(var kv in kvData) {
kvDic[kv[keyColumn - 1]] = kv[valueColumn - 1];
}

// 値をセットする
var sheet2 = ss.getSheetByName('シート1');
var startRow = 2;     // 開始する行数を定義（ヘッダは省く） 
var targetColumn = 1; // キーになるコードの列を指定
var targetData = sheet2.getRange(startRow, targetColumn, sheet2.getLastRow(), 1).getValues();

Logger.log(targetData);
var setData = [];

for each(var targetRow in targetData) {
setData.push([kvDic[targetRow[0]]]);
}

if (i == 0) {
var setRow = 2 ;  
}
else if (i == 1)  {
var setRow = 3 ;  
}
else if (i == 2)  {
var setRow = 4 ;  
}
else {

}     

var setColumn = 2; // 挿入したい列数
sheet2.getRange(setRow, setColumn, sheet2.getLastRow(), 1).setValues(setData);
i=flg;
Logger.log(flg);
Logger.log(i);
}   

ログ

試したこと

GAS初心者、プログラム未経験のため、対策がわかりません。
はじめは条件式の誤りかと思いましたが、何度やってもうまくいきません。
どこかの設定が抜けているのではと思いますが、それすらもわかりません。

Comments

[papinianus]

で==にしたらどうなったのですか。図が改まってませんよ。

Answer 1

ifで代入をしないでください。最低でも=は2つ==です。話はそれからです。

できればまともな開発環境を使ってください。webstormをすすめときます。

function mainMessage(event) {

//スプレッドのURLを指定
    var ss = SpreadsheetApp.openById("スプレッドシートのID");
// 一覧を連想配列化
    var sheet1 = ss.getSheetByName('シート2');
    var sheet2 = ss.getSheetByName('シート1');
    var kvRow = 2;        // 開始する行数を定義（ヘッダは省く） 
    var lastRow = sheet1.getLastRow();

//******************************************************
    var i = 0
    Logger.log(i);
    for (var flg = 0; flg < 3; flg = flg + 1) {

        var keyColumn = 1;    // キーの列を定義

        if (i = 0) {
            var valueColumn = 2;
        } else if (i = 1) {
            var valueColumn = 11;
        } else if (i = 2) {
            var valueColumn = 12;
        } else {

        }

        var kvData = sheet1.getRange(kvRow, keyColumn, sheet1.getLastRow(), valueColumn).getValues();
        Logger.log(kvData);
        var kvDic = {};
        for each(var kv
    in
        kvData
    )
        {
            kvDic[kv[keyColumn - 1]] = kv[valueColumn - 1];
        }

// 値をセットする
        var sheet2 = ss.getSheetByName('シート1');
        var startRow = 2;     // 開始する行数を定義（ヘッダは省く） 
        var targetColumn = 1; // キーになるコードの列を指定
        var targetData = sheet2.getRange(startRow, targetColumn, sheet2.getLastRow(), 1).getValues();

        Logger.log(targetData);
        var setData = [];

        for each(var targetRow
    in
        targetData
    )
        {
            setData.push([kvDic[targetRow[0]]]);
        }

        if (i = 0) {
            var setRow = 2;
        } else if (i = 1) {
            var setRow = 3;
        } else if (i = 2) {
            var setRow = 4;
        } else {

        }

        var setColumn = 2; // 挿入したい列数
        sheet2.getRange(setRow, setColumn, sheet2.getLastRow(), 1).setValues(setData);
        i = flg;
        Logger.log(flg);
        Logger.log(i);
    }
}

Answer 2

最後のi=flgが変数宣言の頭varが無いので、グローバルになっているのでは？

Comments

[papinianus]

おなじコードをみているとしたら12行目のvar i = 0ではだめでしょうか？
またグローバルだとして、意図通り動作しない理由がわかりませんので補足してくださると助かります。

[hiroshi0240]

確かに。
よく読めていませんでした。
ただIf文で=になっているので、最終的にiが2に代入されている状態かとは思うのですが、その後flgで代入されているとしたらログ上は0,1が残るかと思ったのですが、そうで無い理由は？と思いまして。変数宣言して再定義してしまえばとりあえずは変わるかなと

[papinianus]

その疑問への答えはおそらくログを出したときと掲示のコードが違うから、だと思います。
13行目でログ出力を試みているのに、1が出力されたことになっている時点で、確実に違います。
また、現時点でのコードではi = i + 1もありません。
またこのif / else if ブロックを実行すると必ずiは1になります。なので2が繰り返しログ記録されているのは、図にあるとおり、i = i + 1がログに出されているのだと想像できます。
最終的に、冒頭に書いたようにログ時点とはコードが違う、と考えます。

[macaron_xxx]

>if / else if ブロックを実行すると必ずiは1になります。
0では？
あ、i = 0 でfalseとみなされるのか…。

[hiroshi0240]

ま、いずれにせよpapinianusさんの言う通り、「ifで代入をしないでください。」ってことですね。小生の回答で皆様を混乱させお騒がせいたしました。

[退会済みユーザー]

意図通り動作しない理由についてですが、
私はfor文が３回実行を条件としているのですが、2回目の実行結果がスプレッドで
吐き出されたら、なぜか3回目を通らず処理が終わってしまうのを困っています。

[hiroshi0240]

ログを見る限りは3回実行されてますよね。そういう問題では無く、if文の中でiが0の時と表現したいのであれば、i==0と書かないと表現出来ないと言うことです。i=0のままでは、iは0ですと言い切っていることになるんです。

[退会済みユーザー]

for文で3回実行でログでは動いているはずなのに、3回目の結果がスプレッドに書き出されないので、
ものは試しにfor文を外してみた処、『データの行数が範囲の行数と一致してません』というエラーで
ひっかっており、これを治した処正常に結果が出るようになりました。

Answer 3

for (var flg = 1; flg < 3; flg+1) {

flgの値が０のまま変更されないので、これでは無限ループになってしまいますね
ああ、flgの内容は１のまま変わらない、ですね

Comments

[退会済みユーザー]

すみません、flg = 1で、flgに1の値が代入されているわけではないのでしょうか？
何が足りないのでしょうか？

[y_waiwai]

flg+1
では、flgの内容に１が足されますが、その結果はどこにも代入されてません。
これを実行しても flg は1のままです
やるなら、
flg=flg+1
とか、
flg++
とかやらないと、flgの内容は増えません

[退会済みユーザー]

書き方に間違いがあったことは、わかりましたが、
『flg=flg+1』と『flg++』のどちらも試しても、変化がなく、
1万行のマスタを参照している為、マスタ件数が多いのが原因かと思って、
わざとデータを１件にしてみても、問題が改善されませんでした。

[y_waiwai]

そもそも、mainMessage関数が実行されているのか、をチェックしてみれば。

[退会済みユーザー]

チェックをかけてみますと、
for文の条件とその中でうごくif文を同じflgを変数に見ていたで、
for文は、変数flgを
if文は、変数iを
それぞれ見るようにして、
for文を実行する前にvar i = 1; を代入し、ログでflgとiの
動きを確認した処、for文は１づつ増えていることを確認することができました。
これでfor文はうまく回っていると思います。
しかし、iという変数については　if文を抜けたら、i = i + 1; と
書いているのですが、ずっとiには1が入ったままになってしまいます。

[papinianus]

最新の状態に質問を修正してください

[退会済みユーザー]

papinianusさん、最新の状態で質問を修正しました。
ご確認いただけますか？よろしくお願いします。

[y_waiwai]

>　if文を抜けたら、i = i + 1; と
そういう箇所は見当たりませんが

このままではコードが見づらいので、質門を編集し、<code>ボタンで、出てくる’’’の枠の中にコードを貼り付けてください