4

この行の(C#)コード

if (!currentLap.S1.HasValue)

私に与えています

System.NullReferenceException:オブジェクト参照がオブジェクトのインスタンスに設定されていません。

currentLap変数がインスタンス化されていることを確認し(数行前に使用されており、ローカル変数であるため)、次のプロパティがあります。

private double? _s1;

[DefaultValue(null)]
[JsonConverter(typeof(ShortDoubleConverter))]
public double? S1
{
    get { return _s1; }
    set { _s1 = value; }
}

どうすればNullReferenceExceptionをスローできますか?リリースモードでの最適化と関係があるのでしょうか?

ありがとう、ステボ

編集:

これが完全なメソッドコードです。

public void Update(DriverData driverData)
    {
        LapInfo currentLap = this.CurrentLap;

        if (currentLap != null &&
       this.LastDriverData != null &&
       driverData.TotalLaps != this.LastDriverData.TotalLaps &&
       driverData.InPits &&
       driverData.Speed < 10 &&
       !this.LastDriverData.InPits)
        {
            currentLap.Escaped = true;
        }    

        this.LastDriverData = driverData;

        if ((currentLap == null || currentLap.Lap != driverData.LapNumber) &&
            !this.Laps.TryGetValue(driverData.LapNumber, out currentLap))
        {
            currentLap = new LapInfo() { Lap = driverData.LapNumber, Parent = this, Class = driverData.Class };
            this.Laps.Add(driverData.LapNumber, currentLap);

            int lapsCount = 0, completedDriverLaps = 0, cleanLaps = 0;
            this.TotalLaps = driverData.TotalLaps;

            //if it's not the first lap
            if (driverData.TotalLaps > 0)
            {
                //previous lap
                if (this.CurrentLap == null || !this.CurrentLap.Escaped)
                {
                    this.CompletedLaps++;

                    if (this.CurrentLap == null || !this.CurrentLap.MaxIncident.HasValue)
                        this.CleanLaps++;
                }
            }

            foreach (DriverLapsInfo laps in this.Parent.LapsByVehicle.Values)
            {
                lapsCount += laps.TotalLaps;
                completedDriverLaps += laps.CompletedLaps;
                cleanLaps += laps.CleanLaps;
            }

            this.Parent.Parent.SetLapsCount(driverData, lapsCount, driverData.Class, completedDriverLaps, cleanLaps);
        }

        this.CurrentLap = currentLap;

        //add incidents
        if (driverData.Incidents != null)
        {
            foreach (IncidentScore incident in driverData.Incidents)
            {
                this.CurrentLap.MaxIncident = Math.Max(this.CurrentLap.MaxIncident ?? 0, incident.Strength);
                this.CurrentLap.Incidents++;
                this.Incidents++;
            }                
        }

        LapInfo previousLap = null;
        if ((this.PreviousLap == null || this.PreviousLap.Lap != driverData.TotalLaps) &&
            this.Laps.TryGetValue(driverData.TotalLaps, out previousLap))
        {
            this.PreviousLap = previousLap;

            if (!this.PreviousLap.Date.HasValue)
            {
                this.PreviousLap.Date = DateTime.UtcNow;
            }
        }

        if (currentLap.Position == 0)
            currentLap.Position = driverData.Position;

        if (driverData.CurrentS1 > 0)
        {                
            **if (!currentLap.S1.HasValue)**
            {
                this.UpdateBestS1(driverData.BestS1);
                this.Parent.Parent.UpdateBestS1(driverData.BestS1, driverData.UniqueName);
                currentLap.UpdateS1(driverData.CurrentS1, driverData);

                //reset the best split set at the finish line
                if (this.PreviousLap != null && this.PreviousLap.SplitBest < 0)
                    this.PreviousLap.SplitBest = 0;
            }

            if (driverData.CurrentS2.HasValue && driverData.CurrentS1.HasValue && !currentLap.S2.HasValue)
            {
                double s2 = driverData.CurrentS2.Value - driverData.CurrentS1.Value;    
                this.UpdateBestS2(s2);
                this.Parent.Parent.UpdateBestS2(s2, driverData.UniqueName);
                currentLap.UpdateS2(s2, driverData);                           
            }
        }            

        if (this.PreviousLap != null)
        {
            if (driverData.LastLap > 0)
            {
                if (!this.PreviousLap.S3.HasValue && driverData.LastS2.HasValue)
                {
                    double s3 = driverData.LastLap.Value - driverData.LastS2.Value;
                    this.UpdateBestS3(s3);
                    this.Parent.Parent.UpdateBestS3(s3, driverData.UniqueName);                        
                    this.PreviousLap.UpdateS3(s3, driverData);
                }

                if (!this.PreviousLap.LapTime.HasValue)
                {
                    double? bestLap = this.Parent.Parent.BestLap;
                    this.PreviousLap.UpdateLapTime(driverData, 0);
                    this.Parent.Parent.UpdateBestLap(this.PreviousLap, driverData.BestLap, driverData);
                    this.UpdateBestLap(driverData.BestLap, this.PreviousLap);
                    this.PreviousLap.UpdateLapTime(driverData, bestLap);
                }
            }

            else
            {
                if (this.PreviousLap.SplitBest.HasValue)
                    this.PreviousLap.UpdateBestSplit();
                if (this.PreviousLap.SplitSelf.HasValue)
                    this.PreviousLap.UpdateSelfSplit();
            }
        }

        if (driverData.InPits)
        {
            switch (driverData.Sector)
            {
                case Sectors.Sector1:
                    if (previousLap != null)
                        previousLap.InPits = true;
                    break;
                case Sectors.Sector3:
                    currentLap.InPits = true;
                    break;
            }
        }

        //lap to speed
        if (currentLap.TopSpeed < driverData.Speed)
        {
            driverData.TopSpeedLap = driverData.Speed;
            currentLap.UpdateTopSpeed(driverData.Speed);
        }
        else
            driverData.TopSpeedLap = currentLap.TopSpeed;

        //overall top speed
        if (this.TopSpeed < driverData.Speed)
        {
            driverData.TopSpeed = driverData.Speed;
            this.TopSpeed = driverData.Speed;
            this.Parent.Parent.UpdateTopSpeed(this.TopSpeed, driverData);
        }

        else
            driverData.TopSpeed = this.TopSpeed; 

    }

コードがその行に到達し、currentLapがnullになる方法は地球上にありません。

それとも私は夢中になりますか?:)

4

2 に答える 2

10

.HasValuenullable 参照が null の場合はスローしませんが、null の場合はスローしa.b.HasValueますa。私はそれを疑うcurrentLap == null。null ではないことは確かだとおっしゃっていますがcurrentLap、それが最も可能性の高い説明だと思います。もっとコードを投稿できますか?

アップデート:

コードを投稿していただきありがとうございます。

これはスローしません:

void Main() {
    var f = new Foo();

    Console.WriteLine (f.S1);
    Console.WriteLine (f.S1.HasValue);
}

class Foo {
    private double? _s1 = null;

    public double? S1 {
        get { return _s1; }
        set { _s1 = value; }
    }   
}

ミニマルな複製を作ってみませんか?(問題を示す最小限のコード)

于 2012-11-28T20:42:57.627 に答える
2

前のコード行を見てください :) - デバッガーは、NullReferenceException が実際にスローされた行の次の行を強調表示することがよくあります。

于 2012-11-28T21:11:16.023 に答える