コンポジションの学習練習をしています。模範回答と同じように書いたのにエラーが出てしまいました。

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 $

パソコン MacBook(Retina, 12-inch,2017), memory 8GB

#［エラー］
MacBook:37_Composition $ javac Main.java
MacBook:37_Composition $ java Main

Exception in thread "main" java.lang.NullPointerException: Cannot invoke "Bed.make()" because "this.bed" is null
at Bedroom.makeBed(Bedroom.java:14)
at Main.main(Main.java:15)

MacBook:37_Composition $

エラーはコンパイルの後に出ました。

#［学習状況］
2021年５月中旬から独学でJavaを学習しはじめました。
progateを３巡、ドットインストール１巡、スッキリJavaの本を８割読みました。現在、UdemyのJava Programming Masterclass for Software Developersというコースで動画を見たり、演習問題に取り組んでいます。
この問題はUdemyでの演習35番目なので、まったくの初めてというわけではありません。
まだまだ初心者ですが真面目に取り組んでいるので、何卒、よろしくお願いします。

#［演習問題の内容］
コンポジションの学習をしましょう。
クラスを５つ作ります。

Bed,Ceiling,Lamp,Wall、　
そしてこれらのクラスを取り込んだBedroomというクラスです。

#［わからない事］
BedroomクラスのmakeBed()メソッドを呼んでいるところでエラーがでています。
模範解答も同じようにコードを書いており、何が問題なのかわかりません。

#［自分コード］

public class Main {
    public static void main(String[] args) {
        
        Wall wall1= new Wall("West");
        Wall wall2= new Wall("East");
        Wall wall3= new Wall("South");
        Wall wall4= new Wall("North");

        Ceiling ceiling = new Ceiling(12,55);
        Bed bed = new Bed ("Modern", 4,3,2,1);
        Lamp lamp = new Lamp("Classic", false, 75);

        Bedroom bedRoom = new Bedroom("Roba room", wall1, wall2, wall3, wall4, 
        ceiling, bed, lamp);
        bedRoom.makeBed();

        bedRoom.getLamp().turnOn();
    }
    
}

public class Bedroom {
    private String name; private Wall wall1; private Wall wall2; private Wall wall3;
    private Wall wall4; private Ceiling ceiling; private Bed bed; private Lamp lamp;

   public Bedroom(String name, Wall wall1, Wall wall2, Wall wall3, Wall wall4, 
                                           Ceiling ceiling, Bed bed, Lamp lamp){
       this.name=name; this.wall1=wall1; this.wall2=wall2; this.wall3=wall3;
       this.wall4=wall4; this.lamp=lamp;
   }
   public Lamp getLamp() {
       return lamp;
   }
   public void makeBed(){
    bed.make();
    System.out.println("Bedroom -> Making bed | ");
   }
}

public class Bed {
    private String style; private int pillow;private int height;
    private int sheets;private int quilt;
  
    public Bed(String style,int pillow,int height,int sheets,int quilt){
        this.style=style;this.pillow=pillow;this.height=height;
        this.sheets=sheets;this.quilt=quilt;
    }
    public void make(){
        System.out.println("Bed -> Making | ");
    }
    public String getStyle() {
        return style;
    }
    public int getPillow() {
        return pillow;
    }
    public int getHeight() {
        return height;
    }
    public int getSheets() {
        return sheets;
    }
    public int getQuilt() {
        return quilt;
    }
}

public class Ceiling {
    private int height;
    private int paintedColor;
    
    public Ceiling(int height,int paintedColor){
        this.height=height;
        this.paintedColor=paintedColor;
    }
    public int getHeight() {
        return height;
    }
    public int getPaintedColor() {
        return paintedColor;
    }

}

public class Lamp {
    private String style;
    private boolean battery;
    private int globRating;
  
   public Lamp(String style,boolean battery,int globRating){
      this.style=style;
      this.battery=battery;
      this.globRating=globRating;
   }
   public void turnOn(){
    this.battery=true;
    System.out.println("Lamp -> Turning on");
   }
   public boolean isBattery(){
     return battery;
   }
   public String getStyle() {
       return style;
   }
   public int getGlobRating() {
       return globRating;
   }
}

public class Wall {
    private String direction;

    
    public Wall(String direction){
        this.direction=direction;
    }

    public String getDirection() {
        return direction;
    }

}

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

public class Bedroom {
    private String name;
    private Wall wall1;
    private Wall wall2;
    private Wall wall3;
    private Wall wall4;
    private Ceiling ceiling;
    private Bed bed;
    private Lamp lamp;
 
    public Bedroom(String name, Wall wall1, Wall wall2, Wall wall3, Wall wall4, Ceiling ceiling, Bed bed, Lamp lamp) {
        this.name = name;
        this.wall1 = wall1;
        this.wall2 = wall2;
        this.wall3 = wall3;
        this.wall4 = wall4;
        this.ceiling = ceiling;
        this.bed = bed;
        this.lamp = lamp;
    }
 
    public Lamp getLamp() {
        return lamp;
    }
 
    public void makeBed(){
        System.out.println("Bedroom -> Making bed | ");
        bed.make();
    }
}
 
public class Wall {
    private String direction;
 
    public Wall(String direction) {
        this.direction = direction;
    }
 
    public String getDirection() {
        return direction;
    }
}
 
 
public class Ceiling {
    private int height;
    private int paintedColor;
 
    public Ceiling(int height, int paintedColor) {
        this.height = height;
        this.paintedColor = paintedColor;
    }
 
    public int getHeight() {
        return height;
    }
 
    public int getPaintedColor() {
        return paintedColor;
    }
}
 
 
public class Bed {
    private String style;
    private int pillows;
    private int height;
    private int sheets;
    private int quilt;
 
    public Bed(String style, int pillows, int height, int sheets, int quilt) {
        this.style = style;
        this.pillows = pillows;
        this.height = height;
        this.sheets = sheets;
        this.quilt = quilt;
    }
 
    public void make(){
        System.out.println("Bed -> Making | ");
    }
 
    public String getStyle() {
        return style;
    }
 
    public int getPillows() {
        return pillows;
    }
 
    public int getHeight() {
        return height;
    }
 
    public int getSheets() {
        return sheets;
    }
 
    public int getQuilt() {
        return quilt;
    }
}
 
 
public class Lamp {
    private String style;
    private boolean battery;
    private int globRating;
 
    public Lamp(String style, boolean battery, int globRating) {
        this.style = style;
        this.battery = battery;
        this.globRating = globRating;
    }
 
    public void turnOn(){
        System.out.println("Lamp -> Turning on");
    }
 
    public String getStyle() {
        return style;
    }
 
    public boolean isBattery() {
        return battery;
    }
 
    public int getGlobRating() {
        return globRating;
    }
}

#［スクリーンショット］

Answer 1

Exception in thread "main" java.lang.NullPointerException: Cannot invoke "Bed.make()" because "this.bed" is null

Bedroomクラス内にはbedと言う名前のフィールドがありますが、宣言されているだけでどこでもインスタンス化されておらず、すなわちデフォルト値のnullのままです。makeBedメソッドでnullのままmakeを呼び出そうとしているので、「Cannot invoke "Bed.make()" because "this.bed" is null」、「this.bedがnullなのでBed.make()を呼び出せませんよ。」とそのままの意味でエラーメッセージは伝えています。

模範解答の方のコードではBedroomのコンストラクタ内で this.bed = bed;としているのでBedroomのインスタンスの参照がセットされるかたちですが、質問者talabaganiさんのものではそうはなっていません。もう一度目を皿のようにし、それでも分からなければエディターなどの力も借りて再確認してみてください。

Comments

[dodox86]

「同じように書いた（はず）」と言うのは本人の主観でしかないので、模範解答のテキストをコピー＆ペーストできるようであればそれをファイルに保存し、自分のファイルと比較用のツールを使って（macOSのターミナルのコマンドで言うとdiffなど）突き合せ、照合するくらいの意気込みでやっても良いと思います。

[talabagani]

diffコマンドというものがあるんですね。
便利ですね。
早速使ってみようとして、５つに別れていたファイルを一つまとめてf1.javaとしました。そして模範回答もf2.javaとしました。ターミナルに　diff f1.java f2.javaといれてみました。わーとたくさん返ってきたのですが、比較してくれているように見えません。使い方あっていますか？
もっとシンプルなテキストエディタに貼ってもう一度やってみました。今度はff1 ff2というファイル名前にしてみました。コマンドを打つと、なんと、なんの反応もありませんでした。
その時の様子をスクリーンショットで追加しました。

[dodox86]

ターミナルのdiffコマンドは常に使える汎用的な「ファイル比較」ツールの一例として挙げたものでしたが、macOS含むUN*Xのターミナルでのコマンド操作に慣れていない方にとっては最初はちょっと（かなり）面喰うものであったかもしれません。双方のファイルに大きく差があるとそれはそれで確認が大変です。その点、取り上げたことは不適当だったかもしれませんが、diffの出力はteratailでも「ここが違いますよ」と回答で指摘するときにも使われることがあるので、覚えておくと良いです。時間のある時にでも「1. 同じファイルであったらどうなるのか」「2. 少しだけファイルであったらどうなるのか」を試してみて慣れておくことをお勧めします。

Visual Studio Codeを使っているのであれば、Visual Studio Codeと言うエディター自体がもともと内蔵しているファイル比較機能が使えるはずです。恐らくこちらの方が段違いに見易いはずです。「Visual Studio Code diff」などとキーワード検索してみてください。

コメントの当初の主旨としては、「目視で確認してOKであったというのは主観による自己申告でしかないことを理解し、"比較"と言う機能に特化したツールを使って確実に確認しましょう。」と言うことです。空白の有無を無視する、違う部分を色分け表示する、など、知っておくと得しかありません。
[一部誤記修正]

[dodox86]

[2021/07/06 07:21]のコメントより：
> わーとたくさん返ってきたのですが、比較してくれているように見えません。

わーっと出力されたときは何らかの違いがあったときです。何も出力されないのは何の違いも無いときです。

[kazuma-s]

> diff f1.java f2.javaといれてみました。わーとたくさん返ってきたのですが、

diff -u -b -w f1.java f2.java と入れてみてください。どうなりますか？

[talabagani]

少しだけ違うファイルを用意してdiffコマンドを使ってみました。こんな具合に表示されると理解することができました。

Visual Studio Code diff 早速使ってみました。とてもみやすくて便利なので、
これから使うようにしようと思います。
ありがとうございます。

[talabagani]

diff -u -b -w f1.java f2.java と入れてみると、わーとたくさん出てきました。+ や-が表示されています。-は元のファイルにしかないものなんですよね。これはみやすくて便利ですね。ありがとうございます。

Answer 2

Bedroomのコンストラクタでbedは引数として渡されてますが自身のフィールドにセットされてません。
宣言だけされてるのでnullです。nullからmakeを呼び出そうとして出ているNullPointerException。

コードインデントやフォーマットが滅茶苦茶なので分かりづらくなってるのでは。
コードフォーマット掛ければ多少は見やすくなりますよ。

public class Bedroom {
	private String name;
	private Wall wall1;
	private Wall wall2;
	private Wall wall3;
	private Wall wall4;
	private Ceiling ceiling;
	private Bed bed;
	private Lamp lamp;

	public Bedroom(String name, Wall wall1, Wall wall2, Wall wall3, Wall wall4,
			Ceiling ceiling, Bed bed, Lamp lamp) {
		this.name = name;
		this.wall1 = wall1;
		this.wall2 = wall2;
		this.wall3 = wall3;
		this.wall4 = wall4;
		this.lamp = lamp;
	}

	public Lamp getLamp() {
		return lamp;
	}

	public void makeBed() {
		bed.make();
		System.out.println("Bedroom -> Making bed | ");
	}
}

Comments

[talabagani]

インテンド、それなりに気をつけていたんですが、めちゃくちゃですか、、、
（スクリーンショットでは、一つの画面に全てのコードが入るように場所を節約してちょっとインテンドがおかしくなっています。）
コードフォーマットなるもの、初めて知りました。早速、ちょっと調べてみてVSCodeにPrettierという拡張機能を入れてみました。設定がいまいち話からず現在苦戦中ですが、調べて使えるようにしたいと思います。
初心者的にはテキストエディタを使えるようにするのがハードルが高いです。
最近も、スニペットのカスタマイズ設定がわからず、こちらで大変お世話になってどうにか使えるようにしたばかりです。

[m.ts10806]

＞インテンド、それなりに気をつけていたんですが、めちゃくちゃですか、、、

変数宣言や代入が横に並んでる時点で「あー」ってなります。
私が回答に提示したフォーマット後のコード見るとその違いは明白かと思います。

Javaならeclipse使った方が楽です。特に凝った設定もプラグインもなくフォーマット機能があります。

[m.ts10806]

いずれにしても、要件を満たすことではなく「お手本のとおりに」を目指すから出来ません。そこは認識してください。目的は書籍やお手本の通りにすることではないですよね。
今書いているコードが何を実現するものかなぜ作られるものなのか常に考えて作ってください。

[dodox86]

@質問者さん

字下げはインテンド(intend)ではなく、インデント(indent)です。
初心の頃は敷居が高いかもしれませんが、コードの書き方には好み、プログラミング言語ごとに差異はあるものの一般的な「標準（coding standardなどと呼ばれる）」があって、そうしたものに沿って書いていくと、世間で読み易いと言われるかたちに収束していきます。エディターの書式整形機能もそういうものに沿っていることが多いです。余裕ができたらそういったものにも目を向けていくと良いと思います。

https://github.com/future-architect/coding-standards/blob/master/documents/forJava/Java%E3%82%B3%E3%83%BC%E3%83%87%E3%82%A3%E3%83%B3%E3%82%B0%E8%A6%8F%E7%B4%84.md

[talabagani]

リンクありがとうございます。
読んで取り入れられるものは早速取り入れて書くようにしようと思います。
ありがとうございました。

[m.ts10806]

もちろんコードの可読性は重要ですが、質問者さんの場合はコードを書くことが目的になっている部分の改善が最優先と思います。

Answer 3

エラー文に「 "this.bed" is null」って書いてありますよね？

public class Bedroom {
    private String name; private Wall wall1; private Wall wall2; private Wall wall3;
    private Wall wall4; private Ceiling ceiling; private Bed bed; private Lamp lamp;

   public Bedroom(String name, Wall wall1, Wall wall2, Wall wall3, Wall wall4, 
                                           Ceiling ceiling, Bed bed, Lamp lamp){
       this.name=name; this.wall1=wall1; this.wall2=wall2; this.wall3=wall3;
       this.wall4=wall4; this.lamp=lamp;
+      this.bed=bed;
   }

Comments

[talabagani]

ありがとうございます。
ちなみにこちらdiffコマンドで比較したものですか？

[Zuishin]

高評価していましたが、質問者の次の一言で外しました。

> ちなみにこちらdiffコマンドで比較したものですか？

[退会済みユーザー]

>ちなみにこちらdiffコマンドで比較したものですか？
いいえ。diffを使うまでもなく、回答に書いてある通り「"this.bed" is null」というエラー文からの判断です。