JavaScriptの関数について

Question

JavaScriptの関数の引数

JavaScriptでコードを書いてる時に関数で悩み事が発生しました。
function Stone()かfunction Stone(Month)
一応どちらも正しく動作はするのですが、どちらで書くのがプログラムとして綺麗で正しいのか分からないため質問させていただきます。
この場合どちらのほうがいいのでしょうか。

追記:サンプルありで一番簡略化されており、とても分かりやすかったのでベントアンサーを決めさせていただきました。様々なやり方がありとても参考になりました。回答してくださった方々ありがとうございました。

1のソースコード

var Monthtemp = prompt("誕生月を入力");
var Month = Monthtemp * 1;
Stone(Month);

function Stone()
{
　switch(Month)
　{
	case 1: console.log("誕生石:"  + "ガーネット"); break;
	case 2: console.log("誕生石:"  + "アメシスト"); break;
	case 3: console.log("誕生石:"  + "アクアマリン"); break;
	case 4: console.log("誕生石:"  + "ダイヤモンド"); break;
	case 5: console.log("誕生石:"  + "エメラルド"); break;
	case 6: console.log("誕生石:"  + "真珠"); break;
	case 7: console.log("誕生石:"  + "ルビー"); break;
	case 8: console.log("誕生石:"  + "ペリドット"); break;
	case 9: console.log("誕生石:"  + "サファイア"); break;
	case 10: console.log("誕生石:" +"オパール"); break;
	case 11: console.log("誕生石:" +"トパーズ"); break;
	case 12: console.log("誕生石:" +"ラピスラズリ"); break;
　}
}

2のソースコード

var Monthtemp = prompt("誕生月を入力");
var Month = Monthtemp * 1;
Stone(Month);

function Stone(Month)
{
　switch(Month)
　{
	case 1: console.log("誕生石:"  + "ガーネット"); break;
	case 2: console.log("誕生石:"  + "アメシスト"); break;
	case 3: console.log("誕生石:"  + "アクアマリン"); break;
	case 4: console.log("誕生石:"  + "ダイヤモンド"); break;
	case 5: console.log("誕生石:"  + "エメラルド"); break;
	case 6: console.log("誕生石:"  + "真珠"); break;
	case 7: console.log("誕生石:"  + "ルビー"); break;
	case 8: console.log("誕生石:"  + "ペリドット"); break;
	case 9: console.log("誕生石:"  + "サファイア"); break;
	case 10: console.log("誕生石:" +"オパール"); break;
	case 11: console.log("誕生石:" +"トパーズ"); break;
	case 12: console.log("誕生石:" +"ラピスラズリ"); break;
　}
}

Answer 1

グローバル変数を参照するか、関数のパラメータ変数を参照するか
の違いでしょうか。言語によらず後者の方が好まれる気がします。
ちなみに、自分だったらこんな感じで(入力チェックは省略してます)

var Monthtemp = prompt("誕生月を入力");
var Month = Monthtemp * 1;

console.log("誕生石:"  + getStone(Month));

function getStone(m)
{
  const stones = ["dummy","ガーネット","アメシスト","アクアマリン","ダイヤモンド","エメラルド","真珠","ルビー","ペリドット","サファイア","オパール","トパーズ","ラピスラズリ"];
  return stones[m];
}

Comments

[yasutomi]

"ラピスラズリ]; は "ラピスラズリ"]; ですね。

[takasima20]

ご指摘ありがとうございます。(汗

Answer 2

2に1票
但し引数に使う変数名は変えたい

オマケ情報として、"use strict"を使っておくと汚いコードは書き辛くなります。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Strict_mode

Answer 3

どちらが良いかと言われたらStone関数に
引数が指定されている2のソースコード

ちなみに私なら下記のように書きます。
書き方は人それぞれですが、
1から12の数字以外が入力されたときの処理は
必ず入れたほうが良いです。

function Stone(Month) {
　switch(+Month) {
    case 1: console.log("誕生石:"  + "ガーネット"); break;
    case 2: console.log("誕生石:"  + "アメシスト"); break;
    case 3: console.log("誕生石:"  + "アクアマリン"); break;
    case 4: console.log("誕生石:"  + "ダイヤモンド"); break;
    case 5: console.log("誕生石:"  + "エメラルド"); break;
    case 6: console.log("誕生石:"  + "真珠"); break;
    case 7: console.log("誕生石:"  + "ルビー"); break;
    case 8: console.log("誕生石:"  + "ペリドット"); break;
    case 9: console.log("誕生石:"  + "サファイア"); break;
    case 10: console.log("誕生石:" + "オパール"); break;
    case 11: console.log("誕生石:" + "トパーズ"); break;
    case 12: console.log("誕生石:" + "ラピスラズリ"); break;
    default: console.log('1から12の数字を入れてください。'); break;
　}
}
Stone(prompt("誕生月を入力"));

Answer 4

質問の意図と違うかもしれませんが、プログラムの綺麗さで言うなら
分岐が多い場合等はjsonデータなどに格納しておいて取り出した方が良いかと思います。

Comments

[退会済みユーザー]

そういうやり方もあるんですね。ありがとうございます。