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

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

新規登録して質問してみよう
ただいま回答率
85.47%
Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

4回答

1294閲覧

if文の条件式を変数に格納するなどして動的にしたい

T_mari

総合スコア73

Google Apps Script

Google Apps ScriptはGoogleの製品と第三者のサービスでタスクを自動化するためのJavaScriptのクラウドのスクリプト言語です。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

2クリップ

投稿2022/09/29 05:13

編集2022/09/29 06:40

前提

データソースから複数(可変数)条件で抽出するためQUERY関数ではなくGASでスクラッチで記述しています。

積上集計のデータソースを生成するにあたりQUERY関数のGROUP BY句を利用するのですが、
データソースが増えてくるとQUERY関数の挙動が不安定になる為、
元データからselect句(query関数)で抽出、group byで積上数値を加算
→データソースをupdate
のような構成を想定し制作しています。
(具体的な工程としてはgroup byが必要なデータとそうでないものを一旦切り分け、計算後に再結合して戻す)

実現したいこと

以下のようなif文の条件式のand条件が動的に増減するようにしたい

GAS

1if(currentValues[i][0] !==202207 && currentValues[i][0] !==202208){ 2//処理内容 3}

該当のソースコード

GAS

1function splitSrcData() { 2 const srcSheet = //分割元のデータが入っているスプレッドシート 3 const uniqueValues = Array.from(new Set(dbSheet.getRange('A2:A').getValues().flat())).filter(String); //[202207,202208,202209]のような一意の数値型の一次元配列の返り値を別シートより抽出 4 const currentValues = srcSheet.getRange(1, 1, srcSheet.getLastRow(), 10).getValues(); 5 let term = ''; //条件格納用の変数 6 for (var i = 0; i < uniqueValues.length; i++) {//条件式を生成 7 term + `currentValues[i][0] !==` + uniqueValues.shift() + `&&` 8 } 9 let extractArray = []; 10 for (var i = 0; i < currentValues.length; i++) { 11 if (terms) { 12 extractArray.push(currentValues[i]); 13 } else { 14 } 15 }

試したこと

条件式を変数にいれてfor文で生成を試みましたが、バッククォートで囲うなどしてもダメでした。
(条件式を文字列として認識してしまっているため)

GAS

1 for (var i = 0; i < uniqueValues.length; i++) { 2 term + `currentValues[i][0] !==` + uniqueValues.shift() + `&&` 3 }

知識不足で恐縮ですがお知恵を拝借できれば幸いです。

補足情報

早々に回答、修正質問を下さった方、本当にありがとうございます。
自身で試行錯誤している間に論理演算子がおかしなことになっていたので修正いたしました。
パイプでor句としていたところは、&&でand句とすることが想定していた挙動となります。

また欲しい結果が分からないというコメントを受け、前提で実現したいことの具体的内容を追記しました。
uniqueValuesの等号、不等号については上述の通り元データから加工が必要なデータとそうでないデータを切り分ける為に使用します。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

int32_t

2022/09/29 05:30

条件は「currentValues[i][0] が uniqueValues のどれとも一致しないなら」ですか?
Cocode

2022/09/29 06:34 編集

ちょっとどういう結果がほしいのかあんまりわかってなくて申し訳なのですが、 https://jsfiddle.net/5nr6jkfs/ ↑こういうことですか?(簡易サンプル) 複数あるuniqueValuesの数値と一つも重複していないcurrentValuesだけ、配列に格納する?
guest

回答4

0

ベストアンサー

GASでも filter と includes が使えるので以下でどうでしょう?

javascript

1const extractArray = currentValues.filter(([v]) => !uniqueValues.includes(v));

上記のコードを簡単なサンプルで試してみました:

イメージ説明

投稿2022/09/29 07:26

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Cocode

2022/09/29 07:28

これ綺麗です!
退会済みユーザー

退会済みユーザー

2022/09/29 07:56

ありがとうございます〜😇
T_mari

2022/09/30 08:18

filterメソッドで複数条件を絞り込めるとは知らず驚きました…! コードの読みやすさと、indexOfとforの繰り返し処理よりも実行速度も早そうなのでBAとさせて頂きます! ありがとうございましたm(__)m
guest

0

eval() を使えば実行時に生成したコードを実行することは可能ではありますが、eval() を使うのはほとんどのケースで不適切です。

uniqueValues のそれぞれと比較することを繰り返して条件に合うかどうか調べられます。この場合は indexOf() を使うと簡単です。また、ある配列の中から条件に合うものだけを抜き出すのは filter() を利用できます。

js

1let extractArray = currentValues.filter(current => uniqueValues.indexOf(current[0]) == -1);

投稿2022/09/29 06:43

int32_t

総合スコア20941

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

CurrentValuesのそれぞれ1番目の数値が、UniquValuesのどれとも一致しないCurrentValuesのみ配列に格納する、という処理でしたら.indexOf()がおすすめです。

コード例

javascript

1function splitSrcData() { 2 const uniqueValues = [1, 2, 3]; 3 const currentValues = [ 4 [0, 99, 99, 99, 1], 5 [1, 99, 99, 99, 99], 6 [12, 99, 99, 99, 99], 7 [2, 99, 99, 99, 99], 8 [99, 99, 99, 99, 99] 9 ]; 10 let results = []; 11 12 for (const arr of currentValues) { 13 if (uniqueValues.indexOf(arr[0]) == -1) { // currentValues[i][0]がuniqueValuesのどれとも一致しなかったら 14 results.push(arr); // currentValues[i]を、別の配列に格納する 15 } 16 } 17 18 return results; 19} 20 21console.log(splitSrcData()); 22// [[0, 99, 99, 99, 1], [12, 99, 99, 99, 99], [99, 99, 99, 99, 99]]

解説

Array.indexOf(value)は、配列の中に指定したvalueが存在したら、そのインデックス番号を返します。
存在しない場合-1を返します。

javascript

1let arr = [100, 200, 300]; 2arr.indexOf(100) // 0 3arr.indexOf(400) // -1

なので、uniqueValues.indexOf(currentValues[i][0]) == -1をしてやれば、「どれとも一致しない」が判定できます。

また今回の場合、for (let i = 0; i < array.length; i++)と書くよりも、
for (let val of array)というfor...of構文を使ってやった方が、見やすいコードになると思います。

投稿2022/09/29 06:07

編集2022/09/29 06:42
Cocode

総合スコア2314

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

if(currentValues[i][0] !==202207 || currentValues[i][0] !==202208){
この条件は意味がないのではないですか?
currentValues[i][0]が202207の時はcurrentValues[i][0] !==202208が真となり
currentValues[i][0]が202208の時はcurrentValues[i][0] !==202207が真となり
currentValues[i][0]がどのような数字でも真となります。
if(currentValues[i][0] !==202207 && currentValues[i][0] !==202208){
であれば分かりますが・・・

もしかしてcurrentValues[i][0]の値がuniqueValuesに含まれていない場合ってことですか?
そうなのであればわざわざ条件文を構築しようとせずに

GAS

1if (uniqueValues.indexOf(currentValues[i][0]) == -1) 2{ 3 // 処理 4}

でいけませんか?

投稿2022/09/29 05:57

編集2022/09/29 06:06
YAmaGNZ

総合スコア10288

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.47%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問