for 内でswitchを呼び出して特定の値の際に配列に格納する方法

Question

宜しくお願い致します。

やりたいこと

配列に入っている特定の数値をswitchで処理し、新たな配列にその数値に対応する文字列を格納したいです。
尚、他の処理の関係上、for文の中にswitch文を入れて処理させることを考えております。

fruitが7なら「りんご」の文字列をfruitsに格納する
fruitが20なら「みかん」の文字列をfruitsに格納する
fruitが9なら「いちご」の文字列をfruitsに格納する

for (var i = 0; i < data.length; i++) {
markerLatLng = {lat: data[i]['lat'], lng: data[i]['lng']};
marker[i] = new google.maps.Marker({
position: markerLatLng,
map: map
});

var fruits = [];

var fruit = json[i]['fudotype'];   //配列fruitに値が格納されていることはconsole.logで確認済み

switch( fruit ) {
case 7:
    fruits.push = 'みかん';
    break;
case 20:
    fruits.push = 'りんご';
    break;
case 9:
    fruits.push = 'いちご';
    break;
default:
    break;
  }

console.log(fruits);  // コンソールで[]のみが表示され希望をしている文字列が格納されていない

infoWindow[i] = new google.maps.InfoWindow({ 

    content: '<p>fruits</p>' 


});

わからないこと

・現状でfruitsの配列にうまく文字列が入っていない（もしくは指定の仕方が間違っている？）ので、switchで特定の配列に文字列を入れる方法を知りたいです。
→console.log(fruits);　で意図した文字列を表示させたいです。

・そもそもfor文の中にswitch文をネストするような形でコードがかけるのかどうか

・他に良い方法があれば上記の方法ではなくても大丈夫です。

宜しくお願い致します。

Answer 1

fruits.push = 'みかん';という書き方が適当ではないと思われます。

fruits.push('みかん');と、メソッドを呼んでください。

Comments

[tomato01]

ご回答を頂き誠にありがとうございます。
ご指摘の点を変更したのですがconsole.logの結果は[]のままでございました。
何か他におかしい箇所などはありそうでしょうか。

[maisumakun]

上の文章に書かれた「fruitが１なら」とswitch内部の値が一致していないのですが、これはなにか理由がありますでしょうか。

[tomato01]

すみません、誤字です。正しい文に修正致しました。失礼しました。

[maisumakun]

fruitの値は本当に数値の「7」でしょうか。文字列の「'7'」だったりはしないでしょうか（+friutのように変換してしまうこともできますが）。

[tomato01]

ご回答をいただきありがとうございます。case '7':やcase "7":などに変更はしてみたのですがやはり空の状態でした。

Answer 2

jsonという変数(配列?)次第だと思いますが・・・

const json=[{fudotype:7},{fudotype:8},{fudotype:9}];
const fruits=[];
const list={7:'みかん',20: 'りんご',9:'いちご'};
json.forEach(x=>{
  var fruit=x.fudotype;
  if([7,9,20].indexOf(fruit)>-1) fruits.push(list[fruit]);
});
console.log(fruits);

追記

もしJSONが表記のように[]で囲まれているのでしたらそれは配列です。
配列が数値をもつ場合は工夫が必要ですが、単に配列に1個しかデータがないなら
fudotype[0]を参照すればいいでしょう

const json=[{fudotype:[7]},{fudotype:[8]},{fudotype:[9]}];
const fruits=[];
const list={7:'みかん',20: 'りんご',9:'いちご'};
json.forEach(x=>{
  var fruit=x.fudotype[0];
  if([7,9,20].indexOf(fruit)>-1) fruits.push(list[fruit]);
});
console.log(fruits);

Comments

[tomato01]

丁寧なご回答を頂き誠にありがとうございます。

ご指摘の点などを確認し、試しておりますがまだうまく表示ができておりません。

私の知識不足ですみません。json の中身について、配列fruitのconsole.logのログは下記のように表示されております。

[7]0: 7length:
1__proto__: Array(0)
[7]0: 7length:
1__proto__: Array(0)
[2]0: 2length:
1__proto__: Array(0)
[6]0: 6length:
1__proto__: Array(0)

尚、JSONの元データは下記のような形式で記載がされておりました。（これはネスト状態ということでしょうか。このような場合、数値の取得は異なってきますでしょうか。）

jsonファイル
[
　{ "fudotype": [ 7 ] .. }

　{ "fudotype": [ 20 ] ..}

　{ "fudotype": [ 9 ] ..}
]

宜しくお願い致します。

[yambejp]

調整分追記しました

[tomato01]

ご回答を頂き誠にありがとうございます。

ご指摘の通り、[0]を指定したところ、無事に文字列が挿入できました！

どうもありがとうございました。

ご回答を頂きました皆様、どうもありがとうございました。

Answer 3

var fruits = [];

がforループの中にあるようなので毎回初期化されてしまっているのではないでしょうか。

Comments

[tomato01]

ご回答を頂きありがとうございました。

Answer 4

一つ気になったのは、for ループの 終端}は質問コードに示していないだけで忘れてないよね？という点

switchで特定の配列に文字列を入れる方法を知りたいです。

yambejp さん同様、json の中身についての疑問が拭えません。

質問に示されるコード：
for ループは data の長さで回していることから、json も dataと__同じかそれ以上の長さの配列__であり、各配列要素には {fudotype: 数値, ... } というオブジェクトを持っているのかな（fruit = json[i]['fudotype'] が成立する条件）…と感じました。

上記の中身であれば、switch の分岐方法には問題ないです（maisumakun さんの指摘通り、分岐後の処理は__fruits.push()に要訂正__ですが）。
異なるのであればエラーも考えられます。

for文の中にswitch文をネストするような形でコードがかけるのかどうか

これについては、「書けます」（switch の中で switch も可能）
ただし、__混乱しやすく、他者に褒められることはない__ですけどね。

もし、書く場合は、break; 文で switchブロックを抜けるのか、for ループを抜けるのかをしっかり意識する必要がありますので、しっかりとインデントを揃えて一定の可読性を保ったコードに仕上げることが大事になります。

Comments

[tomato01]

丁寧なご回答を頂き誠にありがとうございます。

>一つ気になったのは、for ループの 終端}は質問コードに示していないだけで忘れてないよね？

こちらは、コードを記載していないのですが、実際は閉まっております。失礼しました。

ご指摘の点などを確認し、試しておりますがまだうまく表示ができておりません。

私の知識不足ですみません。json の中身について、配列fruitのconsole.logのログは下記のように表示されております。

[7]0: 7length:
1__proto__: Array(0)
[7]0: 7length:
1__proto__: Array(0)
[2]0: 2length:
1__proto__: Array(0)
[6]0: 6length:
1__proto__: Array(0)

尚、JSONの元データは下記のような形式で記載がされておりました。（これはネスト状態ということでしょうか。このような場合、数値の取得は異なってきますでしょうか。）

jsonファイル
[
　{ "fudotype": [ 7 ] .. }

　{ "fudotype": [ 20 ] ..}

　{ "fudotype": [ 9 ] ..}
]

宜しくお願い致します。