ローカル変数を定義する事の必要性。(参照渡)

Question

ユーザーが画面入力(POST)してきた値を元に、
BMI計算をし、そのBMIを元に体型を判定して、
判定結果を再度、画面に表示するというプログラムを作っています。

ユーザーの入力項目は
身長、体重の2種類のみです。

開発環境はwindows10で統合開発環境であるEclipseを使用
言語はJava、HTML5

-----【構成】-----
このプログラムはMVCモデルによって構成されています。

jspファイルが2つ
・ユーザ入力画面
・入力された情報を元に計算処理などをした結果画面
Javaクラスが3つ
・Logicクラス(BMI計算とその計算結果から体型を判定)
・プロパティクラス(JavaBeansのルールに沿ったもの)
→Healthクラス(フィールドは身長、体重、BMI、体型の4つ)
・コントローラークラス

---

-----【処理の流れ】-----
コントローラークラスから、
ユーザ入力画面をフォワード。
ユーザーが入力してきた値(POST)を
コントローラクラスでプロパティにつっこむ。
突っ込んだ後にLogicクラスをコントローラクラスで生成し、
計算や条件分岐処理。
その後、リクエストスコープに保存。(コントローラクラスから操作)
結果画面をコントローラクラスからフォワードする。

---

Logicクラスにて計算処理をした後、
if文にて条件分岐を行うロジックで疑問に思った事があります。
私は後述するコードを作成しました。

しかし、あるテンプレでは
String bodyType;
とローカル変数を定義してから、
各条件分岐後の処理において、私がコーディングしたsetShapeではなく、
bodyType =　"痩せ型";
bodyType =　…割愛します
の様にコーディングされています。
そして、このif文の処理が終わった後に
health.setShape(bodyType);
としています。

一体何が違うのでしょうか。

下記コードはLogicクラスのコードです。

public void execute(Health health){

               　//一部抜粋

		//BMI指数から体型を判定して設定
		if(bmi < 18.5) {
			health.setShape("瘦せ型");
		}else if(bmi >= 18.5 || bmi <25) {
			health.setShape("普通");
		}else {
			health.setShape("肥満");
		}
}

Comments

[m.ts10806]

抜粋されすぎていて流れが今ひとつ分かりませんけど、「あるテンプレ」がなんの事か分かりませんし、意図は書いた人しか分かりません。
自身でもっと検証できることもあるのでは？

[Aspiration]

流れを記載しました。
知る必要のない事、
知っても明らかに解決に繋がらない事について
は記載していません

情報量が多すぎても受けてが読みにくいだけだと判断しております。
単なる興味本位でしたら尚、お答えしかねます。

いかがでしょうか。

Answer 1

動作に差がないので機能的に違いはありません。
コーディング作法の好みによると思います。
つまり、ローカル変数を定義する必要性は、ありません。

実際に書いてみても、そのままでは気づきにくいのですが・・・

if(bmi < 18.5) {
            health.setShape("瘦せ型");
}else if(bmi >= 18.5 || bmi <25) {
            health.setShape("普通");
}else {
//            health.setShape("肥満");
}

と、

String bodyType;
if(bmi < 18.5) {
            bodyType = "瘦せ型";
} else if(bmi >= 18.5 || bmi <25) {
            bodyType = "普通";
} else {
//            bodyType = "肥満";
}
health.setShape(bodyType);

では、後者（ローカル変数を定義したもの）が、コンパイルエラーとなります。
そのため分岐などで設定もれが確認できるメリットがあります。

ちなみに、初期化漏れを検知したいので

String bodyType = null; // null初期化をしてしまう
if(bmi < 18.5) {
            bodyType = "瘦せ型";
} else if(bmi >= 18.5 || bmi <25) {
            bodyType = "普通";
} else {
//            bodyType = "肥満";
}
health.setShape(bodyType);

のように書いてしまっては意味がありません。

Comments

[Aspiration]

分岐処理のelse以下をコメントにする事で設定漏れを意図的に表し、
もしも、else以下を通った場合、
bodyTypeの変数は初期化されないいまま、
変数を渡す事になるのでコンパイルエラーになるという事ですね。

ご回答ありがとうございます。

[momon-ga]

そうですね。

elseが書いてあれば、通常の書き方でも違和感があるので気づくと思いますが
if(bmi < 18.5) {
bodyType = "瘦せ型";
} else if(bmi >= 18.5 || bmi <25) {
bodyType = "普通";
}

の場合だと、普通に未設定のままとか、ありますからね。