Stream.Foreachの間のNULLポインタ例外

Question

私は無限Nosの中央値をシミュレートしようとしています。これは私のコード

です

package hard;
import java.util.PriorityQueue;
import java.util.Queue;
import java.util.Random;
import java.util.stream.Stream;

public class Median
{
static Queue<Integer> minHeap;

static Queue<Integer> maxHeap;

{
    minHeap = new PriorityQueue<>();

    maxHeap = new PriorityQueue<>();
}

private static void add(Stream<Integer> randomNos)
{

    randomNos.forEach(no ->
    {
        if (minHeap.size() == 0 || no < minHeap.peek())
        {
            minHeap.add(no);
        }
            else
            {
                maxHeap.add(no);
            }

            int diff = minHeap.size() - maxHeap.size();

            if (Math.abs(diff) > 1)
            {
                if (diff < 0)
                {
                    minHeap.add(maxHeap.peek());
                }
                else
                {
                    maxHeap.add(minHeap.peek());
                }
            }
        });

}
private static int median()
{
    int median = 0;

    if ((minHeap.size() + maxHeap.size()) % 2 == 0)
    {
        median = (minHeap.peek() + maxHeap.peek()) / 2;
    }
    else
    {
        if (minHeap.size() < maxHeap.size())
        {
            median = maxHeap.peek();
        }
        else
        {
            median = minHeap.peek();
        }
    }

    return median;
}

public static void main(String[] args)
{
    add(new Random().ints(1, 100000).boxed());
    System.out.println(median());
}

}

.

次の例外を取得します。

Exception in thread "main" java.lang.NullPointerException
at hard.Median.lambda$0(Median.java:25)
at hard.Median$$Lambda$2/1929600551.accept(Unknown Source)
at java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:183)
at java.util.stream.IntPipeline$4$1.accept(IntPipeline.java:250)
at java.util.Random$RandomIntsSpliterator.forEachRemaining(Random.java:1044)
at java.util.Spliterator$OfInt.forEachRemaining(Spliterator.java:693)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:512)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:502)
at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)
at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:418)
at hard.Median.add(Median.java:23)
at hard.Median.main(Median.java:75)

.

デバッガを実行し、この例外がフラッシュされた後に、同じ整数を繰り返し繰り返しの2つの値があることがわかりました。

Answer 1

この

{
    minHeap = new PriorityQueue<>();

    maxHeap = new PriorityQueue<>();
}

.

はインスタンスイニシャライザです。あなたは

が欲しいかもしれません

static {
    minHeap = new PriorityQueue<>();

    maxHeap = new PriorityQueue<>();
}

.

クラスの初期化時に実行されるように。

しかしながら、PriorityQueueオブジェクトは空になることに注意してください。peekはとにかくnullを返し、<比較は失敗します。使用する前に、PriorityQueueオブジェクトにいくつかのオブジェクトを追加することができます。