javascriptで、特定の文字を含む場合の要素の位置の取得方法がわかりません。

Question

前提・実現したいこと

javascriptで、特定の文字を含む場合の要素の位置の取得方法がわかりません。
どうすればよいでしょうか？

例えば、
["abc", "def", "ghi"]
という配列があったとき、
"e"
が含まれているものを探したら、
1
が返ってくるようにしたいです。

試したこと

現状、下記のように書きましたが上手くいきません。
function findIndexOfStr(array, str) {
for(i=0; i<array.length; i++){
return array[i].indexOf(str);
}
}

補足情報（FW/ツールのバージョンなど）

https://codeprep.jp/books/54/chapters/6/sections/4
↑この問題です。

Comments

[ku__ra__ge]

関数内にreturnが無いので、何も結果が帰ってこないのは当たり前だと思いますが……。

[ppos]

ありがとうございます。追記いたしました。

それでも望む結果が返ってこず...

Answer 1

文字列が含まれているかどうかの条件判断をした上で、含まれていればreturn i;ですね。

あと、どこにも含まれていない場合に何を返すのか決めないと。

Comments

[ppos]

ありがとうございます！！！あともう少しだけ、良ければ教えて頂けますと幸いです..！

>文字列が含まれているかどうかの条件判断をした上で、含まれていればreturn i;ですね。
↑この部分、記述できました。

>あと、どこにも含まれていない場合に何を返すのか決めないと。
↑、自分なりに書こうとしてみたのですが、上手く実装されず...。
(最後のif分の部分です。undefindではなく-1にしたいのです。)
なぜうまくいかないのでしょうか、どうすれば良いのでしょうか..？

function findIndexOfStr(array, str) {
for (i=0; i<array.length; i++) {
if (array[i].indexOf(str)>-1) {
return i;
break;
}
if (array.length<=i) {
return -1;
}
}
}

[otn]

return -1するのは、全ての要素をチェックし終わった時ですね。

あと、return の次にbreakを書くのは無意味です。

[ppos]

こちらで実現できました！
わかりやすいFB、誠にありがとうございました。

function findIndexOfStr(array, str) {
for (i=0; i<array.length; i++) {
if (array[i].indexOf(str)>-1) {
return i;
}
}
return -1;
}

[otn]

はい。それで正解です。

Array.findIndexを使うと、コンパクトに書けます。やっていることは同じです。
function findIndexOfStr(array, str) { return array.findIndex((s)=>s.indexOf(str)>-1) ;}

Answer 2

Array.prototype.findIndex() - JavaScript | MDN

Comments

[ppos]

ありがとうございます。