Neden “nesne başvurusu bir nesnenin örneğine ayarlanmadı” bize hangi nesneyi anlatmıyor?

Bir sistemi başlatıyoruz ve bazen NullReferenceExceptionmesajla ünlü istisnayı alıyoruz Object reference not set to an instance of an object.

Bununla birlikte, neredeyse 20 nesneye sahip olduğumuz bir yöntemde, bir nesnenin boş olduğunu söyleyen bir kütüğe sahip olmak gerçekten de hiçbir işe yaramaz. Bir seminerin güvenlik ajanı olduğunuzda, 100 katılımcının arasından birinin terörist olduğunu söylemek gibi. Bu gerçekten senin için hiçbir faydası yok. Hangi adamı tehdit eden adam olduğunu tespit etmek istiyorsanız daha fazla bilgi edinmelisiniz.

Aynı şekilde, hatayı kaldırmak istiyorsak hangi nesnenin boş olduğunu bilmemiz gerekir.

Şimdi, bir şey birkaç aydır kafamı kafamı karıştırdı ve bu:

.NET neden bize adını veya en azından null olan nesne başvurusunun türünü vermiyor? . Yansımayı veya başka bir kaynaktan türünü anlayamıyor mu?

Ayrıca, hangi nesnenin boş olduğunu anlamak için en iyi uygulamalar nelerdir? Bu bağlamlardaki nesnelerin çekilebilirliğini her zaman manuel olarak test etmeli ve sonucu kaydetmeli miyiz? Daha iyi bir yolu var mı?

Güncelleme: İstisna The system cannot find the file specifiedaynı yapıya sahip. İşleme ekleyip hata ayıklamadıkça hangi dosyayı bulamazsınız. Bu tür istisnalar daha akıllı hale gelebilir. .NET c:\temp.txt doesn't exist.bu genel mesaj yerine bize söylerse daha iyi olmaz mıydı ? Bir geliştirici olarak, oy kullanıyorum.

Temel NullReferenceExceptionolarak size söyler: yanlış yapıyorsunuz. Ne fazla ne eksik. Aksine, tam teşekküllü bir hata ayıklama aracı değildir. Bu durumda, her ikisini de yanlış yaptığınızı söyleyebilirim çünkü

• NullReferenceException var
• neden / nerede olduğunu bildiğiniz bir şekilde engellemediniz
• ve ayrıca belki: 20 nesne gerektiren bir yöntem biraz uzakta görünüyor

İşler ters gitmeden önce her şeyi kontrol etmenin ve geliştiriciye iyi bilgi vermenin büyük bir hayranıyım. Kısacası: çekleri kullanarak çek yazmayı ArgumentNullExceptionve hoşunuza gitti ve adı kendiniz yazın. İşte bir örnek:

void Method(string a, SomeObject b)
{
    if (a == null) throw ArgumentNullException("a");
    if (b == null) throw ArgumentNullException("b");

    // See how nice this is, and what peace of mind this provides? As long as
    // nothing modifies a or b you can use them here and be 100% sure they're not
    // null. Should they be when entering the method, at least you know which one
    // is null.
    var c = FetchSomeObject();
    if(c == null)
    {
        throw InvalidOperationException("Fetching B failed!!");
    }

    // etc.
}

Ayrıca Kod Sözleşmelerine de bakabilirsiniz , tuhaflıkları vardır ancak oldukça iyi çalışır ve bazı yazımlardan tasarruf etmenizi sağlar.

Gerçekten ne aramaya çalıştığını tam olarak göstermeli. "Bir sorun var. Bunu düzeltmelisin. Ne olduğunu biliyorum. Sana söylemeyeceğim. Bunu çözeceksin." Demesi gibi. Bu Yığın Taşması ile ilgili cevapların yarısı gibi, ironik bir şekilde.

Öyleyse ne kadar faydalı olurdu, örneğin, eğer bunu aldıysanız ...

Object reference (HttpContext.Current) not set to instance of an object

...? Koda girmeniz, adım adım ilerlemeniz ve aramaya çalıştığınız şeyin nulliyi olduğunu hesaplayın, ancak neden bize biraz yardım eli vermiyorsunuz?

Cevabı bulmak için kodda ilerlemenin genellikle yararlı olacağını kabul ediyorum (çünkü muhtemelen daha fazlasını öğreneceksiniz), ancak NullReferenceExceptionmetin yukarıdaki örnekte olduğu gibi olsaydı sık sık zamandan ve sıkıntıdan kurtuluruz .

Sadece söylüyorum.

Günlüğünüz bir yığın izlemesi içermelidir - bu, genellikle yöntemde hangi satırda sorun olduğunu gösteren bir ipucu verir. Yayınınızın PDB sembollerini içermesini sağlamanız gerekebilir, böylece hatanın hangi satırda olduğu hakkında bir fikriniz olur.

Verilmiş, bu durumda size yardımcı olmaz:

Foo.Bar.Baz.DoSomething()

Söyle sormayın böyle kod önlemek için prensip kutu yardım.

Bilginin neden dahil edilmediğine gelince, emin değilim - En azından bir hata ayıklama yapısında, gerçekten isterlerse çözebileceklerinden şüpheleniyorum. Bir çökme dökümü almak ve WinDBG'de açmak yardımcı olabilir.

İstisnalar, ölümcül olmayan istisnai durumları çağrı zincirine kadar bildiren bir araç olarak yaratılmıştır . Yani, bir hata ayıklama aracı olarak tasarlanmamıştır.

Bir boş gösterici istisnası, bir hata ayıklama aracı olsaydı, tam yerinde bir hata ayıklayıcının bağlanmasına izin vererek doğrudan suçlama hattına işaret ederek programın çalışmasını iptal ederdi. Bu programcıya mevcut tüm içerik bilgisini verir. (Boş bir işaretçi erişimi nedeniyle yapılan bir Segfault'un C de biraz kabaca olsa da, ne yaptığıdır.)

Bununla birlikte, boş gösterici istisnası, normal program akışında atılabilen ve yakalanabilen geçerli bir çalışma zamanı koşulu olarak tasarlanmıştır. Sonuç olarak, performans hususları dikkate alınmalıdır. İstisna mesajının herhangi bir şekilde özelleştirilmesi, çalışma zamanında yaratılma, birleştirme ve imha etme gibi string nesneler gerektirir. Bu nedenle, statik bir mesaj tartışmasız daha hızlıdır.

Çalışma zamanının suçlayıcı referansın adını verecek şekilde programlanamayacağını söylemiyorum. Bu yapılabilir. Bu sadece istisnaları onlardan daha yavaş yapacaktır. Biri yeterince önemserse, böyle bir özellik bile üretim kodunu yavaşlatmayacak, ancak daha kolay hata ayıklamaya izin verecek şekilde değiştirilebilir hale getirilebilir; ama nedense kimse yeterince umursamıyor gibi görünüyor.

Cromulent bence kafanın üstündeki çiviyi vurdu, ancak bariz bir nokta var ki, eğer bir tane elde NullReferenceExceptionederseniz başlatılmamış değişken (ler) iniz var. Bir yönteme yaklaşık 20 nesne iletildiğinin argümanının hafifletici olduğu söylenemez: kodun bir parçasının yaratıcısı olarak, bir kod tabanının geri kalanına uyumunu da içeren, yaptıklarından sorumlu olmak zorundasınız. değişkenlerin doğru ve doğru kullanımı vs.

zahmetli, sıkıcı ve bazen sıkıcı ama sonunda ödüller buna değer: birçoğu gigabayt ağırlığındaki kütük dosyalarında dolaşmak zorunda kaldığım zamanlardı ve neredeyse her zaman yardımcı oluyorlar. Ancak bu aşamaya gelmeden önce, hata ayıklayıcı size yardımcı olabilir ve bu aşamadan önce iyi planlama çok fazla acı kazandıracak (ve size kod çözümü için tamamen tasarlanmış bir yaklaşım anlamına gelmez: basit çizimler ve bazı notlar hiçbir şeyden daha iyi olmak.

Kodla ilgili olarak Object reference not set to an instance of an objectbizim isteyebileceğimiz değerlerde tahmin edemezsiniz: programlayıcı olarak bizim işimiz ve bu basitçe içinde başlatılmamış bir değişkeni geçtiğin anlamına gelir.

Hata ayıklayıcıyı kullanmayı öğrenin. Bu tam olarak onun için tasarlandığı türden bir şey. Söz konusu yönteme bir mola noktası koyun ve uzağa gidin.

Kodunuzda ilerleyin ve tüm değişkenlerinizin değerlerinin tam olarak ne olduğunu görün.

Düzenleme: Açıkçası ben henüz hata ayıklayıcı kullanarak başka kimsenin bahsetmediği için şok oldum.

@ Stijn'in cevabını bulmak ve kodunuza boş kontroller koymak istiyorsanız, bu kod pasajının yardımcı olması gerekir. İşte kod parçacıkları hakkında bazı bilgiler . Bu ayarları yaptıktan sonra, sadece argnulliki kez sekmeye basın, ardından boşluğu doldurun.

<CodeSnippet Format="1.0.0">
  <Header>
    <Title>EnsureArgNotNull</Title>
    <Shortcut>argnull</Shortcut>
  </Header>
  <Snippet>
    <Declarations>
      <Literal>
        <ID>argument</ID>
        <ToolTip>The name of the argument that shouldn't be null</ToolTip>
        <Default>arg</Default>
      </Literal>
    </Declarations>
    <Code Language="CSharp">
      <![CDATA[if ($argument$ == null) throw new ArgumentNullException("$argument$");$end$]]>
    </Code>
  </Snippet>
</CodeSnippet>