JavaScript 変数の代入

Question

回答のないJavaScriptドリルがあり、
JavaScript 学習中でコードを書いてみましたが、違う気がするので、
ご指摘、アドバイスいただけますと幸いです。

演習の中に、入力させるという指示がないので、
どう書くのがベストなのかよく分かりませんでした。

++演習+++++++++++++
可変動物を宣言します。 「牛」か何か他のものに設定してください。
牛と等しい場合に「mooooo」を出力するコードを記述します。
変数animalが牛以外の場合、「これは牛ではありません」と出力されるようにコードを変更します。
++++++++++++++++++

const cow = "mooooo";
const prtmessage = "これは牛ではありません";
let animal = 'yourchoice';
if (animal == cow){
  console.log(cow)
}else {
  console.log(prtmessage)
}

Comments

[miyabi_pudding]

> コードを書いてみましたが

その質問のコードは、質問者さんが全て考えて書いたものですか？
また、それまでの学習範囲で、
クラスとインスタンスに関しては書かれていませんでしたか？

[m.ts10806]

要件を満たせればそれが正解で、書き方は1つではないので「ベスト」というのはないですよ。その「ベストかどうか」の判断基準は何ですか？
観点もなしにアドバイスを得るのは難しいと思います。

[hiro_ike]

コードは、指定の文字列以外は自分で書いたものです。
上記の、演習と書かれた以外に指定はありませんでした。
演習の順番的に、この演習以降にクラス等が出てくるので、
たぶんまだクラス等は、使わないのではと思ってます。

[hiro_ike]

初心者なので、質問を正しく理解していない可能性があると思っています。
なんとなくで書いたコードなので、
理解が間違っているなど、ご指摘、アドバイスあればと思いました。

[m.ts10806]

実行結果が指示通りなら正解です。

[hiro_ike]

ご確認、コメントありがとうございます。

[m.ts10806]

現状で何が問題でしょうか。
実行して指示通りの結果が出れば良いわけですけど
先に書いたように同じゴールに向かう道は1つではないので、
「現状のコードでの明確な問題」を提示されない限り
アドバイスは得られないと思います。
プログラムは書いたとおりにしか動かないので、
「違う気がする」という気持ちだけの問題に対して何を応えればよいのか他者にはわかりません。
「想定通り動いてるならそれでいいですよ」だけで気が済むとしても私はそれだけを書こうとは思わないからこちらにコメントしています。
「どうなったら解決か」を明示してください。質問は編集できます。

[hoshi-takanori]

問題文が曖昧なのでぜんぜん間違ってませんが、牛と等しい場合とは animal === '牛' なのでは…。
また、もしも関数を習ったなら、animal を引数とする関数にするともっと良いですね。頑張ってください。

[m.ts10806]

ひとつ「指示を見た感想」だけだと
＞可変動物を宣言します
の時点で、「変数の代入値を手動で変えて出力を確認するんだろうな」ですけどね。クラスとかインスタンスとか全然先の話だと思います。

[hiro_ike]

コメントありがとうございます。
実際の演習が英文だったので、こちらに転記する際、訳した箇所と訳し忘れたものが混じってしまい、
混乱させて申し訳ないです。m(_ _ )m

[hoshi-takanori]

「可変動物を宣言します」って、もしかして declare variable 'animal' の訳だったりしますか？ それだと「変数 animal を宣言する」って意味になります。そして、(私の勝手な感覚ですが) 英語圏の課題の指示は大ざっぱなので、animal の値は適当な値を直接入れる (適宜書き換えて実行する) なり、入力させるなり、乱数で選ぶなり、自分の好きな方法でやれば良いと思います。(特に、習ってないからやっちゃダメということはなくて、むしろ推奨されるはず。)
あと、私の書き方も良くなかったですが、animal == cow と animal == 'cow' の違いはお分かりですよね？ 前者だと cow という変数の中身 (この場合だと牛の鳴き声) と比べることになるので…。

Answer 1

質問修正依頼にコメントがある通り、
期待する値を出力されている限り、それが正解のため、それ以上も以下もありません。
ただし、

可変動物を宣言します。

ここが表現されていないものと思われますので、ここに着目して、改変を試みます。

const animals = ['牛', '猫', '犬'];
const targetIndex = Math.floor( Math.random() * animals.length);

const cow = "mooooo";
const prtmessage = "これは牛ではありません";
// let animal = 'yourchoice';

if (animals[targetIndex] == '牛') {
  console.log(cow);
}else {
  console.log(prtmessage);
}

ターゲットの動物をランダム、ようは可変動物とするわけです。

Comments

[hiro_ike]

>可変動物を宣言します。
これがどう表現できるのかわかっていなかった為、大変助かりました。ありがとうございました。