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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Java

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

Q&A

解決済

3回答

6523閲覧

boolean型変数にsetterメソッドで値を入れようとするとNullPointerが発生する

tarutarupop

総合スコア20

Java

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

0グッド

0クリップ

投稿2017/10/31 08:24

お世話になっております。Screenクラスの26行目
player1.setOrder(false)
にjava.lang.NullPointerExceptionが発生し対処に困っております。Player1,2の宣言場所を場所を変えたりといろいろしてみたのですが、なぜ発生したのかわかりません。何が原因なのでしょうか?

Java

1package スタート画面; 2 3import 省略 4 5public class Screen extends JPanel { 6 public static CardLayout layout; 7 public static JPanel jp; 8 public static Player player1,player2 = new Player(); 9 private Random rand = new Random(); 10 public Screen() throws IOException { 11 //先攻後攻決める 12 int order = rand.nextInt(2); 13 if(order < 1){ 14 player1.setOrder(true); 15 player2.setOrder(false); 16 }else { 17 player1.setOrder(false); 18 player2.setOrder(true); 19 } 20 21 //親パネル(このクラスのインスタンス) 22 jp = new JPanel(); 23 jp.setBounds(135, 120, 930, 700); 24 layout = new CardLayout(); 25 jp.setLayout(layout); 26 27 //子パネル作成(別クラス JPanel継承) 28 EnterPlayer pane1 = new EnterPlayer(); 29 GameScreen pane2 = new GameScreen(15,15,1); 30 ResultScreen pane3 = new ResultScreen(); 31 32 jp.add(pane1); 33 jp.add(pane2); 34 jp.add(pane3); 35 36 jp.setOpaque(false); 37 } 38}

Java

1package enterPlayer; 2 3import java.awt.Color; 4import java.awt.image.BufferedImage; 5 6public class Player { 7 private Color c; 8 private String name; 9 private boolean order; 10 private BufferedImage color; 11 public BufferedImage getImage(){ 12 return color; 13 } 14 public Color getColor() { 15 return c; 16 } 17 public void setColor(Color c,BufferedImage color) { 18 this.c = c; 19 this.color = color; 20 } 21 public String getName() { 22 return name; 23 } 24 public void setName(String name) { 25 this.name = name; 26 } 27 public boolean getOrder() { 28 return order; 29 } 30 public void setOrder(boolean order) { 31 this.order = order; 32 } 33} 34

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

player1public static Player player1,player2 = new Player();として宣言されていますが、ここでplayer2にはnew Player()が代入されるのに対して、player1nullのままです。

投稿2017/10/31 08:28

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tarutarupop

2017/10/31 08:44

そうだったのですね...。このような書き方をしていたサンプルコードを見たことがあるのですが、このように書くメリットはあるのでしょうか?
guest

0

player1のインスタンスが生成されていないからです。

Java

1public static Player player1 = new Player(); 2public static Player player2 = new Player();

投稿2017/10/31 08:26

LouiS0616

総合スコア35658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

Java

1// Groovyのソースです。 2// 次のように書くと、初期化されるのは後ろの変数だけ 3int a, b, c = 2; 4System.out.println(a + ", " + b + ", " + c); 5// 0, 0, 2 6 7Object obja, objb, objc = new Object(); 8println(obja.toString() + ", " + objb.toString() + ", " + objc.toString()); 9// null, null, java.lang.Object@hoge 10 11// 宣言と同時に、全部同じ値で初期化したければ","じゃなくて"="を使う 12int d = e = f = 2; 13System.out.println(d + ", " + e + ", " + f); 14// 2, 2, 2 15 16Object objd = obje = objf = new Object(); 17println(objd.toString() + ", " + obje.toString() + ", " + objf.toString()); 18// java.lang.Object@huge, java.lang.Object@huge, java.lang.Object@huge 19 20// これで出来るけど、、、

変数は個別に宣言して、個別に初期化してください。
(大体のコーディング規約で禁止されています。Googleとか)
Google Java Style Guide - 4.8.2 Variable declarations

投稿2017/10/31 08:48

abs123

総合スコア1280

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問