jsに限らず、for文の中のif文でのスマートな書き方

Question

for文の中でif文があるケースで該当するものがあればそれを代入。なければ、特定のキーの値を代入するケースで、次のように書いたのですが、もっとスマートな書き方や別の方法をもしご存知でしたら教えていただけないでしょうか。

var datas = ['hoge', 'fuga', 'piyo'];
var value = ''
var flag = false;

for (var key in datas) {
    if(datas[key] == 'fuga'){
        value = datas[key];
        flag = true;
        break;
    }
}

if (! flag) {
    value = datas[0];
}

Comments

[jun68ykt]

think49さんのご回答にある２つ目のコードにあるように、 for を途中で抜けるためにbreakを使うということも覚えておいて損はありません。break; がない元のコードですと、例えばもしdatas に1000個の要素があり、先頭の要素が "fuga" で、残り 999個が "hoge" だったとすると、先頭の1個目で合致した時点で flag が trueになって欲しい結果がその時点で決まっても、あとの 999個についても、if(datas[key] == 'fuga') の（無駄な）比較をすることになってしまいます。

[退会済みユーザー]

って、言ってる↑これ、全然ダメだから話になんねぇので覚えるだけ無駄だぞ。

Answer 1

状況次第だと思いますがこのケースだと、

var datas = ['hoge', 'fuga', 'piyo'];
var value;

value = datas[0];
for (var key in datas) {
    if(datas[key] == 'fuga'){
        value = datas[key];
    }
}

でしょうか。

あと、datasって、もしかして英単語のdataを忘れたのでしょうか。
dataが複数形ですよ(複数形が不規則変化の名詞）。

Comments

[yoshipu]

ありがとうございます。dataが複数形なのも初めて知りました。

[退会済みユーザー]

これは、ゴルフじゃ無いけど、大して勉強にもならない。

なぉ、dataは単複同形だが、データのまとまりのまとまりのようなものはネイティブでもdatasと表記するのでdatasが間違いと、いうわけではない。

[退会済みユーザー]

ちがった。単数形はdatumだったかな。

[退会済みユーザー]

やっぱそうだった。luckerお兄さんはさすが過ぎるな。

[think49]

> dataが複数形ですよ(複数形が不規則変化の名詞）。
「datalist要素」という前例はありますね。
https://momdo.github.io/html/form-elements.html#the-datalist-element
datalist要素においては "data" が単数(option要素)でないと "list" の意味が通りません。
「data(単数形/複数形)」は文脈次第というところでしょうか。

[退会済みユーザー]

うん、だからluckerお兄さんがネイティヴでもdatas使うって言ったやん？
文盲なん？

[otn]

> データのまとまりのまとまり

ファイルやデータベースが複数あるケースとかですね。

[退会済みユーザー]

例えば、そう言う事だね。

Answer 2

こんにちは。

配列の中に特定の値が含まれているかどうかの判定について、補足します。

ES2016 から Array.prototype.includes が追加され、Javascript にも実装されています。

const datas = ['hoge', 'fuga', 'piyo'];

console.log(datas.includes('fuga')); // => true
console.log(datas.includes('foo')); // => false

indexOfを使った場合と違うのは、NaN が含まれるかどうかを判定できることです。

const numbers = [1, 2, 3, NaN];

console.log(numbers.includes(NaN)); // => true
console.log(numbers.indexOf(NaN)); // => -1

以下、上記のデモです。

https://jsfiddle.net/jun68ykt/we5u8ar1/3/

参考になれば幸いです。

Comments

[退会済みユーザー]

うん、だから、それは質問者が聞いてる事じゃ、無い。
Could u understand?

[退会済みユーザー]

「jsに限らず、」が読めない非日本人ばっかと言う事で。

[think49]

方法論としては最良と思いますが、なぜ「低評価1」なのでしょう…。

[退会済みユーザー]

日本語が読めない人、更にマシマシ。

[jun68ykt]

@think49さん
コメントありがとうございます。

低評価をお付けになった方は、おそらく「お前はまだ修行が足りない」と仰りたいものと拝察しておりまして、大変ありがたいご鞭撻を頂戴したと受けとめており、今後一層、研鑽に努めてまいる所存でございます。

Answer 3

私はこの質問を読んだ時、2つの答えを用意出来ましたが、質問文のどちらにフォーカスすべきか判断できませんでした。
ベストアンサー結果を見る限りでは、前者が正解でしたが、どちらの答えも正解になりうるように読めました。

スマートな書き方

もっとスマートな書き方や別の方法をもしご存知でしたら教えていただけないでしょうか。

単純に質問文のコードをスマートに書きたいだけのパターン。

var array = ['hoge', 'fuga', 'piyo'];
var result = array.includes('fuga') ? 'fuga' : array[0];

console.log(result);  // "fuga"

for文内の条件式が偽だった場合を判定

for文の中でif文があるケースで該当するものがあればそれを代入。なければ、特定のキーの値を代入するケースで、

条件式の内容はどうでもよく、for文の外で条件式の真偽判定して代入値を制御したいパターン。

function sample1 (array) {
  var result = array[0];  // 初期値を与える

  for (let value of array) {
    if (value === 'fuga') {
      result = 'fuga';
      break;
    }
  }

  return result;
}

var array1 = ['hoge', 'fuga', 'piyo'],
    array2 = ['hoge', 'fuga2', 'piyo'];

console.log(sample1(array1)); // "fuga"
console.log(sample1(array2)); // "hoge"

while 文版。

function sample2 (array) {
  var i = array.length;

  while (--i > 0 && array[i] !== 'fuga');

  return array[i];
}

var array1 = ['hoge', 'fuga', 'piyo'],
    array2 = ['hoge', 'fuga2', 'piyo'];

console.log(sample2(array1)); // "fuga"
console.log(sample2(array2)); // "hoge"

Re: yoshipu さん

Comments

[jun68ykt]

"スマートな書き方" で挙げられているようなコードをさっと書けることも大事ですが、"for文内の条件式が偽だった場合を判定" のほうのコードにあるように、こういう場合は break; を使って、走査対象の配列要素のすべてを調べる前にループを抜けることができると思いつくことも大事ですね。

[退会済みユーザー]

それじゃ質問者は何も学べないってのと、
何で二択したんだ、要求判断力ねぇのか？ってのと、
何でjunはわかってる気になってんだ？っての度外視すると、
まぁ、ティン49兄さんが一番かすってるから今回だけはプラスしといてやる。

[think49]

To: jun68ykt さん
無駄を排除するのは大切ですね。
突き詰めると、if文も省略できる事に気が付いたので、コードを追記しておきました。

[退会済みユーザー]

またゴルフはじめやがった。

こいつら、仕事した事あんのかな？ 本当に。

[退会済みユーザー]

基本的なコードの書き方のイロハを知らずに小手先の技術ばっかかたりはじめるから、いつまでたっても一人前の仕事人になれん。

[退会済みユーザー]

お前ら全員の事言ってんだからな？

[jun68ykt]

@think49さん
> 突き詰めると、if文も省略できる事に気が付いたので、コードを追記しておきました。

なるほどです。そのwhileでも配列の全要素を読むことがなくなりますね。

ところで、

> 私はこの質問を読んだ時、2つの答えを用意出来ましたが、質問文のどちらにフォーカスすべきか判断できませんでした。

とのことですが、質問者さんが投稿された質問のタイトルや質問本文のテキストから文字通り読み取れる、回答を求められていることが、必ずしも質問者さんが本当に知りたいことではないという可能性が常にあると思いますので、そこをちゃんと質問者さんの言葉の裏にある真意を想像することで、

> 私はこの質問を読んだ時、2つの答えを用意出来ましたが、質問文のどちらにフォーカスすべきか判断できませんでした。

という逡巡は共感できますし、私もよくあります。

[退会済みユーザー]

なら、両方の視点から１つの答えを導けばいいじゃん。

[退会済みユーザー]

辿り着く場所が別だって言うなら、それはそもそも、少なくともどっちかが間違った考え方なんだよ。

Answer 4

今回は「あるかどうか」だけ調べればいいので、Array.prototype.indexOfを使いましょう（MDN）。

var datas = ['hoge', 'fuga', 'piyo'];
var value = ''

if(dates.indexOf('fuga')!== -1) {
  value='fuga';
} else {
  value=dates[0];
}

Comments

[maisumakun]

なお、for-inを配列に使うことは、余計なものをピックアップしたり、疎な配列の要素が抜けたりするので、あまりおすすめできません。

[otn]

「jsに限らず」という質問なので、(forのある)どの言語でも通用する書き方を求めているのかと思います。

[yoshipu]

ありがとうございます。参考になりました。

[退会済みユーザー]

これも、ゴルフ。

[maisumakun]

「もっとスマートな書き方」というのを、「for文の中でif文があるケース」自体をスマートに書き換えるもの、と解釈していました。

[退会済みユーザー]

else がある時点でスマートじゃないやっつけコード。

Answer 5

var datas = ['hoge', 'fuga', 'piyo'];
var value = (datas.indexOf('fuga') != -1) ? 'fuga': datas[0];
console.log(value);

Comments

[yoshipu]

ありがとうございます。参考になりました。目からウロコです。スマートです。

[miyabi-sun]

datas.indexOf('fuga')は見つからない場合-1を返すメソッドであり、
-1をif文や三項演算子の左辺に放り込むとtrueとして判定されます。
datasの3要素目にfugaが存在する場合、このコードでは取得出来ず要件を満たしていません。

[退会済みユーザー]

> miyabi-sun さん
ご指摘ありがとうございます

[bwz61366]

> miyabi-sun さん
miyabi-sun さんの、コメントの上2文には同意できますが、最後の行だけ意味が理解できません。
いったいどのような意味でしょうか。よろしければご返信お願いします_ _

[退会済みユーザー]

これは回答じゃなくて、ゴルフ。

[think49]

To: miyabi-sun さん
> datasの3要素目にfugaが存在する場合、このコードでは取得出来ず要件を満たしていません。
経過動作として、「代入処理が2回行われる」はなくなりますが、条件式が真なら、value = "fuga" という結果は変わりません。
どうしても、気になるなら Array#lastIndexOf を使えばいいですが、結果は変わりません。

質問者のコード
http://jsfiddle.net/5ka7mtyj/2/
Kosuke_Shibuyaさんのコード
http://jsfiddle.net/5ka7mtyj/5/
Array#lastIndexOf
http://jsfiddle.net/5ka7mtyj/7/

あえて、違いを見出すのなら、質問者のコードは等価比較なのに対して、Kosuke_Shibuyaさんのコードは厳密比較になっています。
質問者のコードは「1 == "1"」でtrueですが、Kosuke_Shibuyaさんのコードは「1 === "1"」でfalseです。

最後に、要素値に NaN があった場合に正しく判定されない、というjun68yktさんの指摘が気になりますね(最も、質問者のコードも同じ問題がありますが)。

[退会済みユーザー]

だからゴルファーはコンペでだけ腕を競い合ってよ。何の役にも立たないから。

[bwz61366]

> think49 さん
あ、やっぱりそんな感じですよね。JavaScriptは詳しくないので、勉強になりました_ _

[退会済みユーザー]

そんなJavaScriptでしか利用できない知識覚えてなんになるの…

Answer 6

for文の中でif文があるケースで該当するものがあればそれを代入。なければ、特定のキーの値を代入するケース

このケースだと、「なければ」の部分を読み替えて、最初に代入してあげるときれいになります。

var value = datas[0];

今回は関係ないですけど、flag って出てきたら、continue とか break とかと配列組み合わせるとイケるんじゃね？って考えるときれいに書けるケースが多いです。

Comments

[退会済みユーザー]

あー要件次第ですけど、このケースでも break 使ったほうが良いかもしれないですね＾＾；最初を取るか、最後を取るか？

[退会済みユーザー]

なんでそんななんとなくな感じで今までコード書いてきちゃったん？

[退会済みユーザー]

活いれといた。

[退会済みユーザー]

意味がわからん＾＾；ちゃんと回答読んでる？

そろそろ自分の回答修正すればよいかと。
ここのコメントでやり取りするのめんどい。

[退会済みユーザー]

意味がわからない？

…とか…とかだと…じゃね？だと…になる事がなんとなく多いです。時と場合によりけりなんで、この場合は…かもしれませんけどねー。マジ、どっちにしましょ？

↑これ、お前の回答の要約。凄まじいなんとなくな適当感漂ってるだろ、どう見ても。

[退会済みユーザー]

やっぱり、回答読んでないのね＾＾；

要件を読み替えて、初期値を
var value = datas[0];
とするときれいに書けるよって書いてるんだけど。

こっちにコメントしてないで、回答修正して収束すれば？

[退会済みユーザー]

まさか、マジでそっちが本題だったの？
それは本当の本当に要件次第だけど、
どうしてそんな、要件を捻じ曲げちゃうような適当な回答しちゃうの？

[退会済みユーザー]

まーさーかー、ここまで底レヴェルだとは、お釈迦様でもお富さん。

[退会済みユーザー]

この回答も理解出来てないし、自身の回答も回答になってないし、そろそろちゃんとしようよ。。。

なんか書きたいことがあるんだろうけど、書かんと分からんよ＾＾；

[退会済みユーザー]

なんで理解できてないと思うんだお前は。
こうすっとフラグいらなくなるっしょ？って言いたいだけだろ、お前は。
だから、それこそ要件次第で、今回聞かれてることには関係ねーだろ。

頭に赤飯でも詰まってんのか、お前は。

[退会済みユーザー]

> こうすっとフラグいらなくなるっしょ？って言いたいだけだろ、お前は。
> だから、それこそ要件次第で、今回聞かれてることには関係ねーだろ。

そそｗその理解で正しいわ。
書きたかったこと、今回の件とあんまりカンケーないのよ。

そこまでわかっててなんで理解できないかなぁ。。。
余談の扱いなんで、あんまり長く書けないから、ざっくり書いたのよ。
「今回は関係ないですけど」以下が30行も続いたら、おかしいでしょ。

[退会済みユーザー]

だーかーらー、

質問文をよく読んで、質問者のコード見て、質問者が知りたかった本当のタイヤブランコを理解しろってば。

[退会済みユーザー]

あのな、いまだに分かってないようだからあえて言うけど、
俺のほうがおまえら全員よりも遥かに綜合技能レヴェル上なんだからな。
いい加減、理解してくれ。

[退会済みユーザー]

自分でやってから言ってくれ＾＾；

> そろそろ自分の回答修正すればよいかと。
> ここのコメントでやり取りするのめんどい。

[退会済みユーザー]

おまえらが全員頭下げるっていうなら、教えてやらんことはないぞ。

[退会済みユーザー]

それはオレの求めてるタイヤブランコじゃないわ。

[退会済みユーザー]

赤飯の塩加減の事なんか知るか！

[退会済みユーザー]

> 書きたかったこと、今回の件とあんまりカンケーないのよ。

ここまでちゃんと理解した仲なのにｗ

Answer 7

なんでこんなバカ回答ばっかりに…

お題を変えちゃダメだろ。
何の勉強にもならない小手先技ばっか覚えてもしょうがねぇだろうに。
これだからふろんとえんじにぁーはダメなんだよ。

みやびも間違い探しとかやってないで
もっと教えるべきことを教えろよ。

Comments

[退会済みユーザー]

質問者としては運命の分かれ道だな。これから5年闇を彷徨う亡者になるか、5年分すっ飛ばして人間になるか。

[退会済みユーザー]

寺はロボしかいないから、気づかずに従ってるとロボ一直線だ。

[退会済みユーザー]

そういえば、みやびは転職して亡者を選んだんだっけ…

Answer 8

学習中でスマートな記述は書けませんが、forでなければいけないのですか？
判定処理が複雑になるようでしたらwhileでもいいと思います。

Comments

[yoshipu]

ありがとうございます。参考になりました。

[退会済みユーザー]

質問者が知りたいのはループに何を使うのかではなく、ループ内で分岐を使う場合の正しい書き方。