java - グラフの頂点を決定する関数

Question

Java の NullPointerException に問題があります。私の関数は頂点の番号を取得し、頂点を返します (私はすでにそのようなクラスを持っています)。これが私のコードです

public class Graph {
private Set<Vertex> vertices;

public Vertex getVertexByNumber(int number) {
        Vertex answer = new Vertex(number);
        for( Vertex v : vertices ) {
            if(v.getNumber() == number) {
                answer = v;
                break;
            }
        }
        answer = null;

        return answer;
    }

Answer 1

なぜanswer = null;ですか？

このステートメントのため、メソッドgetVertexByNumberは常に を返しnullます。

Answer 2

あなたが何をしようとしているのか、私にはよくわかりません。ただし、値を持つ頂点がない場合に関数で新しい頂点を作成する場合は、次のnumberように記述できます。

public Vertex getVertexByNumber(int number) {
    Vertex answer;

    for (Vertex v : vertices) {
        if (v.getNumber() == number) {
            answer = v;
            break;
        }
    }

    return answer == null ? new Vertex(number) : answer;
}

書くこともできます

public Vertex getVertexByNumber(int number) {

    for (Vertex v : vertices) {
        if (v.getNumber() == number) {
            return v;
        }
    }

    return new Vertex(number);
}

それ以外の場合は、初期値を に与える必要はありませんanswer。宣言するだけで済みます。

public Vertex getVertexByNumber(int number) {
    Vertex answer;

    for (Vertex v : vertices) {
        if (v.getNumber() == number) {
            answer = v;
            break;
        }
    }

    return answer;
}

また

public Vertex getVertexByNumber(int number) {

    for (Vertex v : vertices) {
        if (v.getNumber() == number) {
            return v;
        }
    }

    return null;
}