javaでの簡易的なしりとり判定

簡単なしりとり判定のプログラムを製作しています。
とりあえず同じ単語が出たかどうかは考慮せず 2 つの単語からしりとりをミスしたかどうかを判定するものです。
先に出た単語 A とそれに続く単語 B がそれぞれ入力されます。
単語は必ず半角アルファベット小文字で構成された文字列です。
また、単語 A の最後の 1 文字は "n" ではないとします。

判定の基準は
・A の最後の 1 文字が B の最初の 1 文字と一致している。
・B の最後の 1 文字が "n" ではない。
２つのルールを満たす時は "OK" そうでない場合は "NG" と出力するプログラムです。

import java.util.*;

public class Main {
public static void main(String[] args ) throws Exception {
Scanner sc = new Scanner(System.in);
String A = sc.nextLine();
String B = sc.nextLine();

String Ausiro = A.substring(A.length() - 1);
String Bmae = B.substring(0,1);
String Busiro = B.substring(B.length() - 1);

if(Ausiro==Bmae&&!(Busiro=="n"))
System.out.println("Ok");
else
System.out.println("NG");
}
}

本当ならOKと出る場合でもNGと出てしまいます。
まだ始めたばかりなため基本的な用法がわかってない部分もあるのでどう直せば良いか教えてもらえると助かります。

行動規範の内容に同意します

回答3件

Javaでは、文字列の比較に == を使ってはいけません。
String#eqauls メソッドを用いてください。Ausiro == Bmae ⇒ Ausiro.equals(Bmae)など。

String#charAt を使って、文字の比較に落とし込むのもアリです。

コードの書き方について

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

投稿2019/03/13 14:45

編集2019/03/13 14:50

LouiS0616

総合スコア35660

otftrough

2019/03/14 04:13

質問に関係ない話で恐縮ですが、こちらのgif画像は他の質問でも転用することは可能でしょうか。非常に便利だと思うので、機会があれば私も使いたいのですが。

LouiS0616

2019/03/14 04:23

転用して貰って大丈夫です。現在、Python、Java、C、C++の4種類用意してあります。 https://teratail.storage.googleapis.com/uploads/contributed_images/56957fe805d9d7befa7dba6a98676d2b.gif https://teratail.storage.googleapis.com/uploads/contributed_images/f4ed904eff3506b68bb6e4c2b205ebd1.gif https://teratail.storage.googleapis.com/uploads/contributed_images/4c6e48a3bd0707d89f61b901fd1f8915.gif https://teratail.storage.googleapis.com/uploads/contributed_images/e378967f71829feb9e6187e5be60349f.gif

otftrough

2019/03/15 10:35

助かります。

行動規範の内容に同意します

Stringはequalsで判定してください。

java
1if(Ausiro.equals(Bmae)&&!Busiro.equals("n"))

また、一文字取って比較するならcharAt使った方がいい。

投稿2019/03/13 14:50

swordone

総合スコア20651

2250z

2019/03/13 15:25

import java.util.*; public class Main { public static void main(String[] args ) throws Exception { Scanner sc = new Scanner(System.in); String A = sc.nextLine(); String B = sc.nextLine(); char Ausiro = A.charAt(A.length()-1); char Bmae = B.charAt(0); char Busiro = B.charAt(B.length()-1); if(Ausiro.equals(Bmae)&&!Busiro.equals("n")) System.out.println("Ok"); else System.out.println("NG"); } } こういうことですか？

行動規範の内容に同意します

ベストアンサー

.String型で文字列の比較をするときはequalsで比較をします.
詳しくは以下のサイトを参考にしてください.
なぜStringの比較はequalsか?

java
1import java.util.*;
2
3public class Main {
4    public static void main(String[] args) throws Exception {
5        
6        Scanner sc = new Scanner(System.in);
7        String a = sc.nextLine();
8        String b = sc.nextLine(); 
9        
10        String aLast = A.substring(A.length() - 1);
11        String bFirst = B.substring(0,1);
12        String bLast = B.substring(B.length() - 1); 
13        
14        if(aLast.equals(bFirst) && (bLast != "n")){
15            System.out.println("OK");
16        }
17        else{
18            System.out.println("NG");
19        }
20
21    }
22}

また今回の場合比較する文字が一文字なのでline.charAt()を使った方がいいと思います.
.line.charAt()を使うとString型からchar型に変わるので,この場合は文字の比較の際「==」で比較します.

また "n"が'n'になることにも注意しましょう.

java
1import java.util.*;
2
3public class Main {
4    public static void main(String[] args) throws Exception {
5        
6        Scanner sc = new Scanner(System.in);
7        String a = sc.nextLine();
8        String b = sc.nextLine(); 
9        
10        char aLast = A.charAt(A.length() - 1);
11        char bFirst = B.charAt(0);
12        char bLast = B.charAt(B.length() - 1);
13        
14        if(aLast == bFirst && bLast != 'n'){
15            System.out.println("OK");
16        }
17        else{
18            System.out.println("NG");
19        }
20
21    }
22}