java どうして自分のコードは期待する答えが出てこないのかわかりません。

Question

#［環境］
Java version 16.0.1
VS Code
Mac OS Big Surの環境です。
java -version
java version "16.0.1" 2021-04-20
Java(TM) SE Runtime Environment (build 16.0.1+9-24)
Java HotSpot(TM) 64-Bit Server VM (build 16.0.1+9-24, mixed mode, sharing)
MacBook:09_MinutesToYearsandDayCalculator $
#［エラー］
エラーメッセージはでないのですが、出てくる結果が期待しているものと違います。

［学習状況］
progateを３巡、ドットインストール１巡、スッキリJavaの本を８割読みました。現在、UdemyのJava Programming Masterclass for Software Developersというコースで演習問題に取り組んでいます
この問題はUdemyでの演習１8番目なので、まったくの初めてというわけではありません。
まだまだ初心者ですが真面目に取り組んでいるので、何卒、よろしくお願いします。
#［演習問題の内容］
同じ数が入っているかどうか調べよう！
hasShareddigitというメソッドを作りましょう。このメソッドはタイプint の引数が二つあります。
それらは１０から９９の間の数です。１０、９９は含みます。もしこの外の数ならfalseです。
(23,52)なら２を共通にもっているのでtrue
(45,67)ならなにも共通にもっていないのでfalse

#［わからない事］
(12,34)など、共通の数がないようにしても、「同じ数が入っています」と表示されます。
模範回答ほどすっきり書けていませんが、自分のコードも同じことをやっていると思います。
なにが、いけないのか教えていただけないでしょうか？
#［自分コード］

public class SharedDigit{
    public static boolean hasSharedDigit(int n1, int n2){

                if(n1<10 || n2<10 || n1 >99 || n2>99){
                    return false;
                } 
                int n1_1 = n1 % 10; // 38 →8  8→８
                int n1_2 = n1 / 10 ; // 38 →3 8→０
                int n2_1 = n1 % 10;// 38 →8  8→８
                int n2_2 = n1 / 10;// 38 →3 8→０
                if ( n1_1==n2_1 || n1_1==n2_2 || n1_2 ==n2_1 || n1_2== n2_2 ) {
                   return true; 
                }  else {
                    return false;
                } 
    }
}

public class Main {
    public static void main(String[] args) {
        
        boolean A = SharedDigit.hasSharedDigit(12, 34);
        if(A==true){
            System.out.println("同じ数が入っている。"); 
        } else {
            System.out.println("かぶっていない。"); 
        }
        
    }
}

#［ネットで見つけた模範回答コード］

public static boolean hasSharedDigit(int x,int y) {
		if((x > 99) || (x < 10) || (y > 99) || (y < 10)) {
			return false;
		}else {
		 return((x/10 == y/10)||(x/10 == y % 10)||(y/10 == x % 10)||(x%10 == y%10));
		}
	} 

Answer 1

int n1_1 = n1 % 10; // 38 →8  8→８
int n1_2 = n1 / 10 ; // 38 →3 8→０
int n2_1 = n1 % 10;// 38 →8  8→８
int n2_2 = n1 / 10;// 38 →3 8→０

n2_1、n2_2にも、n1の下から1桁目と2桁目が入ってしまっているためです。

int n2_1 = n2 % 10;// 38 →8  8→８
int n2_2 = n2 / 10;// 38 →3 8→０

とする必要があります。

Answer 2

int n1_1 = n1 % 10; // 38 →8 8→８

            int n1_2 = **n1** / 10 ; // 38 →3 8→０
            int n2_1 = **n1** % 10;// 38 →8  8→８
            int n2_2 = **n1** / 10;// 38 →3 8→０

全部n1に対して掛けてるからですね。

Comments

[talabagani]

ありがとうございます。ほんとうですね。

[m.ts10806]

デバッグ覚えると良いです。
メソッドの結果だけ見ても中身は追えませんので。

[talabagani]

デバッグ覚えるというのは、自分はこんなミスをしたみたいなことをメモしておくということですか？

[m.ts10806]

ちょっと違います。
デバッグとは起きてる問題を把握し調査し、解決策を考える作業です。

[hallen0225]

横から失礼します。
今回で言うと、n2_1、n2_2に想定通りの数値（3と4）が入っているかどうか確認していれば、その部分か、もしくはその手前で間違っているということが確認できたわけです。
メソッドの最終的な結果だけでなく、メソッドの中で、どこまでは正しく動いているか確認する必要があったわけですね。

その方法としては、n2_1に値を代入する部分の直後に"System.out.println("n2_1 : " + n2_1);"のように書いて、コンソール上でn2_1の値を確認するというのが今すぐできる方法だと思います。
（もちろん、完成したらこの行は削除します）
また、VSCodeにはデバッグ機能というものがあって、プログラムの実行を途中で止めて、その時点での変数の値などを確認することができます。

出来ればこの機能の使い方を覚えるとよいですね。

[m.ts10806]

補足ありがとうございます

[talabagani]

起きている問題を把握して、調査する、なるほど。そういう習慣をつけるように心がけようと思います。
VSCodeのデバッグ機能、教えて下さりありがとうございます。
ちょっと使ってみたのですが、使いこなすにはちょっと時間がかかりそうです。使えるようになったら、途中で変数の値などが手軽に確認出来ていいですね。
今は、デバッグ機能が使いこなせないのでSystem.out.printlnを使って確認するようにしようと思います。
ありがとうございます。
デバッグ機能って使えるようになるのはちょっと時間がかかると思いますか？

[hallen0225]

全く新しい言葉も色々出てくると思いますし、ある程度慣れは必要だと思います。
何にしても、使ってみないと使えるようにはならないので、この辺りの記事（https://qiita.com/12345/items/dc6184682838d9ae06f0）を参考に取りあえず使ってみるのもよいかと思います。

[m.ts10806]

基本はブレイクポイント（ここでプログラムの実行を止めるという点）を張って、その時点での変数の情報を見てみたり、想定の分岐を通るかを確かめるために使います。

[talabagani]

ちょっと苦戦しましたが、ブレークポイントで変数の動きが見れるようになりました。とても便利です。使うようにします。ありがとうございました。