質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.51%

  • Java

    15311questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • if

    233questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 143

2250z

score 3

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

判定の基準は
・A の最後の 1 文字が B の最初の 1 文字と一致している。
・B の最後の 1 文字が "n" ではない。
2つのルールを満たす時は "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と出てしまいます。
まだ始めたばかりなため基本的な用法がわかってない部分もあるのでどう直せば良いか教えてもらえると助かります。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+2

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

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

コードの書き方について

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+1

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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/14 00: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");
    }
    }
    こういうことですか?

    キャンセル

checkベストアンサー

0

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

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 aLast = A.substring(A.length() - 1);
        String bFirst = B.substring(0,1);
        String bLast = B.substring(B.length() - 1); 

        if(aLast.equals(bFirst) && (bLast != "n")){
            System.out.println("OK");
        }
        else{
            System.out.println("NG");
        }

    }
}

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

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

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 aLast = A.charAt(A.length() - 1);
        char bFirst = B.charAt(0);
        char bLast = B.charAt(B.length() - 1);

        if(aLast == bFirst && bLast != 'n'){
            System.out.println("OK");
        }
        else{
            System.out.println("NG");
        }

    }
}

変数の名前の付け方について

今回の質問には関係ないですが, 変数名をAmae Ausiro Bmaeにするより
.aFirst aLast bFirstにした方が後で見たとき読みやすいと思います.
変数名なのでどちらでも構いませんが...

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/14 13:39

    変数名に突っ込むなら、Javaでは変数名は小文字始まりのキャメルケースなので、aMaeなどになるかと思います。
    まあ、英語でaLast,bFirstのほうが読みやすいですが。

    キャンセル

  • 2019/03/14 13:46

    確かにそうですね.ご指摘ありがとうございます。
    そちらの方が読みやすいので訂正しました.

    キャンセル

同じタグがついた質問を見る

  • Java

    15311questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • if

    233questions

    if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。