キーボード入力後の分岐　Javaで数独の問題を作成

総合スコア80850

2019/08/19 04:46

ご指摘ありがとうございます。引数の部分修正させていただきました。

2019/08/19 04:49

インデントもお願います。ただ、修正したことにより状況はどうなったのでしょうか。

2019/08/19 05:04

インデントのほうも修正させていただきました！結果として、スキャナーを用いることはできたのですが、 createメソッド内の　if文の中に３つあるdata2が使用されていない。とエラーメッセージが表示されます。 for文内で用いておりますがそこがうまくいっていない点です。

2019/08/19 05:12

>インデントのほうも修正させていただきました！えっと、どこがでしょうか・・そんなに変わった印象を持てませんでした。 IDE使っているのでしたら手で修正するのではなくコードフォーマット機能を利用したほうが早く確実です >if文の中に３つあるdata2が使用されていない。とエラーメッセージが表示されますこれは単に警告かと思います。というか手元ではそのような警告はできていませんが・・ cleanして再ビルドしてみては？

2019/08/19 05:17

あぁ分かった・・インデント部分は質問のコードに手を入れるのではなく手元のコードを揃えてからコピペしてください。全角空白混じってますよ。

2019/08/19 05:32 編集

あといまのif文の作りではelseに入ったときにdata2が定義されていない状態になります。空でもいいのでifの外で定義したうえで各分岐に値を入れるべきですね。ただその場合だとelseに入ったときにNullPointerExceptionがでると思うので、ifを抜けるのではなくcreate()を実行させてreturnするなど工夫が必要です。

2019/08/19 05:39

インデントに関しては、フォーマットを使い修正することができました！教えていただきありがとうございます。上記３つ目のコメントに関してですが、今から修正を行いますので、今しばらくお待ちいただければ幸いです。

2019/08/19 05:43

別回答でも挙げられていますが、あくまで変数のスコープの問題ですね（もちろん未定義のパターンがあることも問題）

2019/08/19 06:26

if文以前に、data２を定義し、if文の中では初期化を行おうとしたのですが data2[9][9] = { { 1, 1, 1, 0, 1, 1, 0, 0, 0 }, { 0, 0, 0, 1, 0, 0, 1, 0, 0 }, { 1, 1, 1, 1, 0, 1, 0, 1, 0 }, { 0, 0, 0, 0, 1, 0, 0, 1, 0 }, { 1, 1, 1, 0, 1, 0, 1, 0, 1 }, { 0, 0, 0, 0, 0, 1, 0, 0, 1 }, { 0, 1, 1, 1, 0, 1, 1, 1, 0 }, { １, 0, 0, １, 0, 0, 0, 0, 0 }, { 1, 0, 0, 0, 0, 0, 1, 1, 1 } }; 初期化方法は、こう行った風に定義させていただいたんですが、間違っていますでしょうか…？ if文内の３つのdata２をs上記のように修正したところ、警告として・配列定数はイニシャライザーにおいてのみ使用可能です。と表示されている現状です。

2019/08/19 06:30 編集

普通に int[][] data2; だけで良いのでは。で、if内の[ int[][] ]は外す。

2019/08/19 06:40

そちらの方法も試させていただいたのですが、変わらずエラーが発生してしまいました… 修正させていただいたものを再度質問の上部に記載させていただきましたので、かくにんしていただければさいわいです。

2019/08/19 06:46

編集するたびにコードが滅茶苦茶になっているような。 create()でプロパティのdata1を利用するなら引数で渡す必要ないですし、引数で受け取ったx使われてないですし。「配列定数はイニシャライザーにおいてのみ使用可能です」は文言の通りです。初期化にしか使えませんので、別の方法で配列内容の情報を作ってください。

2019/08/19 06:54

一番簡単なのは、結局「定義」するだけで中身が改変されるわけではないデータということでfinalキーワードでもつけてプロパティにしておいて適宜data2に代入することでしょうか。データとなるものをあまりコードに直接持っておくのはメンテナンス性に乏しいのでJSONなど別の形で持っておいたほうが良いとは思います

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

ベストアンサー

追記

＞変数定数はイニシャライザーにおいてのみ使用可能です

配列に中かっこ　{}←を用いて値を代入できるのは、宣言と同時に行う際のみになります。

java
1//OK
2String[] hoge = {"hoge1","hoge2"...};
3
4//コンパイルエラー（変数定数は・・・）
5String[] fuga;
6fuga = {"fuga1","fuga2"...};

全体を見直さずに簡単に解決する方法としては
if文の外で最終的に使いたい配列(今回ならばdata2）を定義（宣言のみ）
if文内で実際の中身をdata3（一例）として定義、中身を入れる
最後にdata2 = data3としてみてはどうでしょう。とりあえずエラーは消えるかと。

for文内のdata2とif文内のdata2が別物の扱いを受ける

質問時点でのコードでは、createメソッドの引数としてdata2を受け取っていましたが
現在のコードだとif文中で初めてdata2を定義しています。
if文中で宣言した変数はその外で扱うことができないため、for文中のdata2は未定義の状態です。

java
1    if(条件) {
2    String hoge = "hoge";
3    //hoge
4    System.out.println(hoge);
5    }
6
7    //コンパイルエラー
8    System.out.println(hoge);

投稿2019/08/19 05:29

編集2019/08/19 06:59

総合スコア419

2019/08/19 06:39

回答ありがとうございます！修正前の質問内容、現在の内容ともに、if文より前に変数data2を定義させていただきました。その際に、if文内で　初期化を行いたいのですが、二次元配列を初期化してもエラーが出てしまう状態です。その修正したプログラムを再度質問内に記載しましたので、見ていただければ幸いです。

2019/08/19 06:46

編集日時が2019/08/19 14:36 になっており、恐らく編集が反映されていないものと思われます。

2019/08/19 06:50

大変失礼しました。おっしゃる通りでした！今一度修正させていただきました。

2019/08/19 06:58

追記しました。