C# - Избегание повторения кода: Вложенный, если Statement

Question

Мне нужно посмотреть, доступен ли путь, но только во время выполнения. Поэтому, если моя программа проверила, что файл не существует, он устанавливает для invalidPath bool значение «true», чтобы он не пересказывал запрос «File.Exists ...» еще раз, как только программа запускается.

Проблема: если я делаю это так, как показано ниже, я не вижу возможности избежать избыточности кода. Мне нужно показать сообщение, чтобы информировать пользователя о недостающем файле, но не очень элегантно использовать одну и ту же строку кода для обоих случаев «если».

private bool invalidPath = false 

if (!invalidPath) 
    { 
     if (File.Exists(temp)) 
     { 
      //do code    
     } 
     else 
     { 
      Show.MessageBox("no file found") 
      invalidPath = true 
     } 
    } 
    else 
    { 
     Show.Messagebox("no file found") /*<---thats redundant*/ 
    } 

Надеюсь, кто-то может дать мне представление.

Answer 1

Есть ли причина, по которой вы не хотите комбинировать логические операторы в одном блоке if?

т.е.

private bool invalidPath = false 

if (!invalidPath && File.Exists(temp)) { 
    //do code    
} 
else { 
    Show.MessageBox("no file found") 
    invalidPath = true 
} 

Answer 2

Вы не можете использовать один булевы держать как информацию (Path проверяется и существует файл). Таким образом, вы должны использовать второе логическое значение.

псевдо-код будет выглядеть следующим образом:

bool valid = false; 
bool checked = false; 

if(!checked) 
{ 
    valid = File.Exists("bla"); 
    checked = true; 
} 

if(!valid) 
{ 
    MessageBox("Path does not exist"); 
} 

Однако, ваша стратегия может иметь проблемы. Например, пользователь может удалить или переименовать путь во время работы вашей программы.

Answer 3

Попробуйте это:

  bool invalidPath = false; 
      bool fileExistsflag = false;    
      if (!invalidPath) 
      { 
       if (File.Exists(temp)) 
       { 
        //do code 
        fileExistsflag = true; 
       } 
       else 
       { 
        invalidPath = true; 
       } 
      } 
      if (!fileExistsflag || invalidPath) 
      { 
       MessageBox.Show("no file found"); 
      }