Stirng型からint型への変換で躓きました

Question

前提・実現したいこと

掲題の通り、String型 → int型へ変換しようと思っています。
いろいろとググれば出てきますが、やりたいこととは一致していないので質問させていただきます。

やりたいこととしては、JSP/サーブレットの一環で入力された文字を数値に変換する、ということですが
例えば「りんご」と入力されれば「0」、「いちご」と入力されれば「1」、それ以外はnullを返します

どうしてもうまくいかないので解決方法を教えてください

発生している問題・エラーメッセージ

非互換オペランド型 int型とString型

とエラーメッセージが表示され実行できない状態です。

該当のソースコード

(略)
		String food = request.getParameter("Food");
		int foodnum = Integer.parseInt(food);

		if (foodnum == "りんご") {
			food = 0;
		} else if(foodnum == "いちご") {
			food = 1;
		} else {
			food = (Integer) null;　　//food = null;にしたかったけどint型をnullにできないためこのように記載
		}

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

Eclipse IDE 4.14.0

追記）　int food　をint foodnum に修正しました

Comments

[K_3578]

＞いろいろとググれば出てきますが、やりたいこととは一致していないので･･･
ならば、どういうキーワードでググってどういう記事を見たのか追記して頂けますか？

追記:質問とは関係ありませんが、以前の質問は解決したのでしょうか？

[dodox86]

コンパイルエラーに関する問題、質問を除き、コンパイルエラーが無いソースコードで質問をするのは最低条件です。

[dodox86]

そして退会。

[K_3578]

あら、退会されてしまったのか･･･残念だ

Answer 1

まず、そもそも同名の変数は定義できません。

String paramFood = request.getParameter("Food");
Integer food = null;

if ("りんご".equals(paramFood)) {
    food = 0;
} else if("いちご".equals(paramFood)) {
    food = 1;
}

Comments

[momon-ga]

ソースが変更された・・・が、

> foodnum = (Integer) null;
元のコードでもそうですが、
int 型の変数にnullいれようとするとコンパイルは通りますが、実行時に例外が発生します。

Answer 2

• 上部でString foodとint foodという、同名の変数を宣言してしまっているため、これではコンパイルを通りません。
• "りんご"や"いちご"をInteger.parseIntにかけたところで、意味のある値を返しません。
• food == "りんご"のように、文字列を==で比較するのは適切ではありません。"りんご".equals(food)のようにequalsメソッドを使いましょう。

Comments

[maisumakun]

> 非互換オペランド型 int型とString型

エラーメッセージの通りです。intのfoodnumと"りんご"や"いちご"は、比較演算自体が成立しません。

（すでに上で触れたように、Integer.parseInt("りんご")とすること自体が妥当ではありません）

Answer 3

1. 変数名が重複している

変数foodが2つ宣言されています。
型違いであっても同じ変数名を宣言することは出来ません。
(普通に人間が見てもどっちの型のfoodだかわからない)

1. 文字列の比較方法がおかしい

文字列が等しいかを確認する
https://java-code.jp/183
String(Java 11)

nullを返したいなら最初からInteger型使って
Optionalで返せばいいのではないでしょうか。

Answer 4

そもそもfoodという変数名がStringとintで被ってしまっていますし、

int food = Integer.parseInt(food);

の(food)には数字(数値ではなく数字、つまり数字の文字列)が格納されていないとダメです。
なので変更するならば、int型の変数名をfood_number等に変え、Integer.parseInt(food);は消すと動くと思います。