２つの二次元配列を比較する

Question

import java.util.Scanner;
public class test
{
    public static void main(String [] args){
        Scanner input = new Scanner(System.in);
        String myGuess = input.nextLine();
        
        int row = myGuess.charAt(0) - 'A';
        int col = myGuess.charAt(1) - '1';
        
        int [][] guess = new int [6][6];
        char [][] ship = new char [6][6];
        
        if(guess[row][col] == ship[0][0]){
            System.out.println("valid");
        }
    }
}

```**入力がA1の場合のみifの中に入り込ませたいのですが、なにを入力しても正になってifに入ってしまいす。＝＝で比べるのが良くないのですか？
**

Comments

[maisumakun]

「入力がA1の場合のみifの中に入り込ませたい」とありますが、それと2つ確保してある2次元配列はどのように関係しますでしょうか。

[YukimasaYajima]

二次元配列shipの方で、表示を司ろうと思っています

[azuapricot]

[なにを入力しても正になってifに入る」

じゃあ、ifの手前でSystem.out.println()を使って guess[row][col] の中身と、 ship[0][0]の中身を見てみましたか？
デバッグはしましたか？

[YukimasaYajima]

どうでした、
guessでは０の表示だったのですが、
shipでは何もイニシャライズしてないから？println(ship[0][0])では空白でした。

これは自動的に０に変換ということですか？

[azuapricot]

何もしてないという理解はあるのですね、
比較対象となる配列に中身を入れないで　空っぽと空っぽで比較してますが、
これは何を意図してるのですか？

それとも、質問者さん的にはもう配列に中身は入っている想定、だったのですか？

[YukimasaYajima]

そうですね、たしかにイニシャライズするべきですね、
char[][] ship = new char[6][6];
ship[0][0] = 'A1';
とできないのはなぜですか？

[maisumakun]

Javaの文字リテラルには1文字しか入りません。'A1'の時点で文法エラーです。

[azuapricot]

’A1’　はchar型ではありません。
基礎からお勉強した方がよいのではないでしょうか。

[YukimasaYajima]

打ち込んでいる時点で分かっていましたが、再質問するために、しました。

[YukimasaYajima]

いやできました

[azuapricot]

char[][] ship = new char [6][6];
ship[0][0] = 'x';
System.out.println(ship[0][0]);

↓
出力結果：x

できますけど・・・

[azuapricot]

シングルクォートで囲い忘れてるとか？そんなわけないか

[YukimasaYajima]

脳みそが把握できてないことがちょっと多すぎるんで少し休憩しまーすｗ

[YukimasaYajima]

プログラムの意図を分かりやすくするために質問し直しますね

Answer 1

入力がA1の場合のみifの中に入り込ませたいのですが、なにを入力しても正になってifに入ってしまいす。＝＝で比べるのが良くないのですか？

newで作ったばかりの数値配列は、初期化子を指定しない場合すべてゼロクリアされています。なので、アクセスできるならどこでもguess[row][col]は0ですし、もちろんship[0][0]も0なので、条件は常に成立します。

Comments

[maisumakun]

空の配列の中身を調べているので、何をしたいのか意図がつかめません。

[YukimasaYajima]

なるほど、
すみません初期化をしていない場合、ということですか？

[YukimasaYajima]

shipの有るところを、guessで指定したら、if分の中に入り込ませたいです

[maisumakun]

そうですね、上のコードでは配列の中身を自動初期化に任せているので、全部0になっています。

（Javaの場合、「未初期化のままコードが進む」ということは基本的にないです。ローカル変数を代入する前に使おうとするとコンパイルエラーになりますし、フィールドや配列要素は0あるいはnullとしてデフォルトの初期化が行われます）

[maisumakun]

> shipの有るところを、guessで指定したら、if分の中に入り込ませたいです

では、それを意図したコード（shipやguessへの値設定も含む）を書いてください。

Answer 2

int [][] guess = new int [6][6];
char [][] ship = new char [6][6];

int i = 1;
for (int[] num : guess) {
    System.out.println(i + "番目のguessの中身↓");
    for (int num2 : num) {
         System.out.println(num2);
    }
    i++;
}
        
int j = 1;
for (char[] char1 : ship) {
     System.out.println(j + "番目のshipの中身↓");
     for (char char2 : char1) {
          System.out.println(char2);
     }
     j++;
}

↑の出力結果

 
1番目のguessの中身↓
0
0
0
0
0
0
2番目のguessの中身↓
0
0
0
0
0
0
3番目のguessの中身↓
0
0
0
0
0
0
4番目のguessの中身↓
0
0
0
0
0
0
5番目のguessの中身↓
0
0
0
0
0
0
6番目のguessの中身↓
0
0
0
0
0
0
1番目のshipの中身↓






2番目のshipの中身↓






3番目のshipの中身↓






4番目のshipの中身↓






5番目のshipの中身↓






6番目のshipの中身↓

---

入力がA1の場合のみifの中に入り込ませたい

とありますが、↑で出力した通り、質問者さんはまだどこにも Aと1という値を配列に入れていません。
ということはどちらの配列もまぁ空っぽのままです。

ということは現時点でのコードではいつまでたってもやりたいことはできません。

配列に値を入れないと何も始まりません。

Comments

[YukimasaYajima]

guess[row][col]でA1を入れたと思っていたのですが、入っていないということは、どのようにすればいいか教えてくれますでしょうか

[maisumakun]

> guess[row][col]でA1を入れたと思っていたのですが

どの処理で入ったと考えていました？

[YukimasaYajima]

guess[row][col]
違うのであれば指摘くれればいいです

[azuapricot]

if(guess[row][col] == ship[0][0]){

質問者さん的にはここでguessに row と col を入れてるつもりだったんじゃないでしょうか。

[azuapricot]

多分質問者さんがわかってないのは、二次元配列への値の入れ方じゃないですか？

[YukimasaYajima]

２つの配列のメモリに記憶されている場所同士を比較することをしようとしてました。

B1 is a hit
You sunk the Patrol Boat
ENEMY
1 2 3 4 5 6
___ ___ ___ ___ ___ ___
A |_X_|_O_|___|___|___|___|
B |_X_|___|___|___|___|___|
C |___|___|___|___|___|___|
D |___|___|___|___|___|___|
E |___|___|___|___|___|___|
F |___|___|___|___|___|___|
Please enter a guess in the form 'B5':