Блог

Вопрос 1> Должен ли я проверять NULL в следующем случае?

 public interface INewClass {}
public class NewClass : INewClass {}

public void FunMeA(INewClass obj)
{
    NewClass n = (NewClass) obj;
    ... // Should I check (n == null) here?
    // call methods defined inside INewClass (updated from NewClass to INewClass)
    ...
}

A concrete example,

public void FunMeB(IAsyncResult itfAR)
{
    AsyncResult ar = (AsyncResult) itfAR;
    ... // Should I check (ar == null) here?
    // access ar.AsyncDelegate
    ...
}
  

Вопрос 2> Я только начинаю переходить на C # с C .
При написании кода на C я знаю, когда следует выполнить проверку.
Однако я полностью потерялся в мире C #. Итак, вопрос в том, существует ли
общее правило, которое может подсказать мне, когда я ДОЛЖЕН проверять значение NULL?

Спасибо

При выполнении этого:

 NewClass n = (NewClass) obj;
  

В этом нет смысла, потому что, если он не будет приведен, он выдаст недопустимое приведенное исключение.

Если у вас есть какие-либо сомнения относительно того, действительно ли вы можете его привести, вы хотите сделать:

 NewClass n = obj as NewClass;
  

затем

 if(n != null) ...
  

Выполняемое вами приведение называется прямым приведением, при котором система будет предполагать, что оно может быть выполнено. n = obj as NewClass это называется косвенным приведением и предназначено для тех сценариев, где вы хотите сообщить программе «Эй, я думаю, это сработает, но если нет, не переключайтесь и не создавайте исключение…Я разберусь с этим, если это не сработает.»

Использование is vs as при приведении.

В зависимости от того, какой сценарий вы хотите, один будет лучше другого. Технически с точки зрения производительности as предпочтительнее. .Net будет использовать атомарную попытку приведения к желаемому типу и возвращать null, если это не так, где с is ему придется дважды обойти дерево наследования, чтобы увидеть, соответствует ли оно этому типу, а затем привести его. Итак, в большинстве случаев, если вы хотите увидеть, хотите ли вы привести и использовать этот тип, лучше:

 var obj = o as type
if (obj != null)
  

в отличие от

 if(o is type)
{
   var obj = (type); 
}
  

Если приведение завершится неудачей, ваш код выдаст исключение.

Используйте is оператор, чтобы увидеть, сработает ли приведение, и as для приведения (которое не будет выдавать и возвращать значение null, если приведение завершится неудачей).

Итак:

 if(obj is NewClass)
{
  //Yay, can cast!
}
  

Или:

 NewClass nc = obj as NewClass;
if(nc != null)
{
  //Yay!
}
  

Если вы хотите проверить наличие null, вам следует сделать

 NewClass n = obj as NewClass ;
if (n != null)
{

...

}
  

Но в целом вы могли бы проверить значение null, если это что-то значит в контексте, и вы можете предпринять альтернативные действия. В большинстве случаев вам следует просто позволить ему вызывать исключение NullReferenceException, а не проглатывать его, чтобы вы могли быстрее определить причину нулевой ссылки.

Если вы это сделаете NewClass n = (NewClass)obj , вы получите исключение, если оно не будет приведено должным образом.

Если вы хотите, чтобы была возможность, чтобы оно было null, без создания исключения, вы бы хотели сделать, NewClass n = obj as NewClass; а затем проверить, n равно ли это null.

Вы можете проверить, сработает ли приведение заранее, используя is оператор. if(obj is NewClass)

Надеюсь, это поможет.

Если параметр является обязательным, вы всегда должны проверять null и выдавать ArgumentException или ArugmentNullException .

Лучший способ проверки этого параметра здесь:

 AsyncResult ar = itfAR as AsyncResu<
if(ar == null)
{
    // even better: Use a resource here
    throw new ArgumentNullException("Parameter itfAR must not be null and must be of type AsyncResult"); 
}
  

Я предпочитаю проверять значение to на null перед приведением. Например.

 if (obj == null) 
{
  throw new ArgumentNullException("obj", "The argument must has a value specified");
}
  

Вы путаете проблемы.

1. Вы всегда должны проверять аргументы общедоступного метода. Поэтому вы должны сначала сделать:

   public WhateEverMethod(элемент IElement) { if (элемент IElement == null) { создать новое исключение ArgumentNullException(…); } }

2. Как только вы подтвердите, что элемент не равен null, вы проверите, можете ли вы выполнить приведение. По сути, вы можете пойти двумя путями:

   Элемент element = IElement как элемент;

   if (element == null) //приведение не удалось { создать новое исключение InvalidCastException(…); }

или

 if (!(iElement is Element))
{
    throw new InvalidCastException(...);
}

Element element = (Element)iElement;