javaのwhileとif文で簡単な英和辞典のモデルを作ってみましたがうまくいきません

Question

お世話になっております。

秀和システムのjava入門書籍にて独習中です。

教科書の章末演習に手こづっており質問をするに至りました。どうかご教授下さいませ。

問題はキーボードをタイプして繰り返し英単語を入力し、その日本語訳を表示するプログラムを作成する
というものです。下記プログラムを組みました。

package Pass;
import lib.Input;
public class Pass10_01_3 {
public static void main(String[] args) {
String s,wa;
while((s=Input.getString())!=null){
if(s=="dog"){
wa="犬";
}else if(s=="cat"){
wa="猫";
}else if(s=="mouse"){
wa="ねずみ";
}else if(s=="rabbit"){
wa="うさぎ";
}else{
wa="?";
}
System.out.println(wa);

問題文の通りキーボードをタイプしてdog,cat,mouse,rabbitと打ち込んでみましたが？としか表示されません。
何が間違っているか教えていただけたら幸いです。

Answer 1

文字列は == では比較できません。
String#equalsメソッドで比較してください。

参考 : https://www.javadrive.jp/start/string/index4.html

後辞書を作るのであれば、Mapを使うと簡単に実現できます。

Comments

[gomakasu423]

ありがとうございます。
確認してみます

Answer 2

文字列どうしを比較するときにはequalsメソッドを利用せねばなりません。

if(s.equals("dog")){ 
    wa="犬";
}

他も同様です。

コードの書き方について

teratailには、上記のようにコードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

Comments

[gomakasu423]

ありがとうございます。
出来ました。

[swordone]

にしても、この「Stringを==で比較する」という間違いがあとを絶ちませんねぇ…
いっそ+を文字列連結としたように、==もObjects.equalsでの比較とかそんなふうにサポートしてくれるといいんですがねえ。

[LouiS0616]

IDEを使っていれば、警告が出て気付けると思うんですけどね。
私自身は==でハマることによって初めて参照値と言うものを意識したので、個人的には挙動はこのままでいいんじゃないかと思っています。
むしろ一貫性があって無用な混乱を防げるかと。

---
teratailに同様の質問が溢れているのは確かなので、投稿前に確認を促す仕組みがあると便利かもしれませんね。

[swordone]

ただこのパターン、原因は「Stringを==で比較している」で共通なのに、各々がやりたいこと、問題視していることがバラバラだから、うまいこと検索できそうなキーワードがない気がする

[LouiS0616]

確かに。