Как обрабатывать неинициализированные локальные переменные

Question

После прочтения this Eric Lippert Статья, я понимаю, что компилятор C# не нравится, если мы оставляем локальными переменными неинициализированные.

Когда я сталкивался с этой «проблемой» время от времени, я смотрел на некоторые из моих старых кодов и смог отсеять большую часть ситуации, где на самом деле не нужны неинициализированные (SomeClass obj = null) локальные переменные.

Но я столкнулся с ситуацией, когда я не знаю, как реорганизовать код.

public void DoSomething(string foo) { 

    SomeClass obj; // = null; 

    try { 
     obj = SomeClass.CreateItem(target); 
    } catch(CustomException ex) { 
     // notify UI of error 
    } 

    if (obj != null) { 
     // do something with `obj` 
    } 
} 

SomeClass.CreateItem может не работать из-за внешних факторов. Если это так, я хочу уведомить пользователя, если не хочу выполнить действие.

C# компилятор не хочет, чтобы оставить obj неинициализированным, поэтому я обычно назначить null к нему.

Это чувствует, как «взломать» сейчас и мой вопрос:

Есть недостаток дизайна в коде выше?

И если есть, как мне обращаться со ссылками во время компиляции, когда я не могу определить, будут ли они указывать на существующий объект во время выполнения?

Answer 1

Код // do something with obj`` должен находиться внутри блока try`.

Что вы пытаетесь сделать, это запустить код, который может или не может быть успешным, а затем запустить другой код , только если предыдущий код преуспел. Это, как правило, очень сильный знак того, что другой код является частью одного и того же логического блока, который зависит от того, что он не является исключением. Если есть исключение, строящее этот объект, вы хотите, чтобы этот код был пропущен, что является именно тем поведением, которое вы получаете, включив его в блок try.

Answer 2

Вы можете реорганизовать это инкапсулировать весь код в Try/улова, связанных с этим объектом, и если вы действительно действительно нужно что-то делать, если он не работает, то вы можете использовать логическое значение, чтобы связать это с остальной частью вашего код:

public void DoSomething(string foo) 
    { 

     bool failedCreation = false; 

     try 
     { 
      SomeClass obj = SomeClass.CreateItem(target); 
     } 
     catch (CustomException ex) 
     { 
      // notify UI of error 
      failedCreation = true; 
     } 

     if (failedCreation) 
     { 
      // do other stuff. 
     } 
    } 

Но это не похоже на то, что вы имеете в виду. Я бы просто инкапсулировал все в try/catch и делал с ним.

Answer 3

Я бы реорганизовать код так:

private SomeClass TryToCreateItem() 
{ 
    try 
    { 
     return SomeClass.CreateItem(target); 
    } 
    catch(CustomException ex) 
    { 
     // notify UI of error 
    } 
    return null; 
} 

public void DoSomething(string foo) 
{ 
    SomeClass obj = TryToCreateItem(); 
    if (obj != null) { 
     // do something with `obj` 
    } 

"метод Extract" мой любимый рефакторинга.