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

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

ただいまの
回答率

88.93%

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

解決済

回答 8

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,503

yoshipu

score 90

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];
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • jun68ykt

    2018/07/21 12:20 編集

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

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2018/07/21 12:48 編集

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

    キャンセル

回答 8

+6

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

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が複数形ですよ(複数形が不規則変化の名詞)。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 12:58

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

    キャンセル

  • 2018/07/21 13:08

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

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

    キャンセル

  • 2018/07/21 13:26

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

    キャンセル

+4

こんにちは。

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

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/

参考になれば幸いです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 10:56

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

    キャンセル

  • 2018/07/21 11:09

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

    キャンセル

  • 2018/07/21 12:24

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

    キャンセル

+4

私はこの質問を読んだ時、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 さん

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 14:53 編集

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

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

    ところで、

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

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

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

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

    キャンセル

  • 2018/07/21 15:16

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

    キャンセル

  • 2018/07/21 15:18 編集

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

    キャンセル

+3

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 07:31

    これも、ゴルフ。

    キャンセル

  • 2018/07/21 07:53

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

    キャンセル

  • 2018/07/21 07:58

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

    キャンセル

checkベストアンサー

+2

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 12:46

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

    キャンセル

  • 2018/07/21 15:28

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

    キャンセル

  • 2018/07/21 15:33

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

    キャンセル

+1

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

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

var value = datas[0];

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 14:36

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

    キャンセル

  • 2018/07/21 14:37

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

    キャンセル

  • 2018/07/21 14:42

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

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

    キャンセル

-1

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/20 23:36

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

    キャンセル

  • 2018/07/21 07:39

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

    キャンセル

-8

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/07/21 08:09

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

    キャンセル

  • 2018/07/21 08:10

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

    キャンセル

  • 2018/07/21 08:11

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

    キャンセル

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

  • ただいまの回答率 88.93%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る