Могу ли я заменить эту инструкцию GOTO на C#

Question

используя goto кажется естественным здесь.

Проект должен читать pdf-файл, файл pdf может быть одним из следующих.

• Не Protected
• Protected с password1
• Protected с password2
• Protected с password3

Файл можно получить доступ только с правильным паролем, нет никакого способа узнать, какой пароль файл нужен заранее. мы должны попробовать все случаи (без пароля). если ни один из вышеперечисленных паролей не работает, выполните исключение.

PdfReader GetPdfReader(string filePath) 
{ 
    PdfReader r = null; 
    int Retries = 0; 
    start: try 
    { 
     switch (Retries) 
     { 
      case 0: r = new PdfReader(filePath); break; 
      case 1: r = new PdfReader(filePath, password1); break; 
      case 2: r = new PdfReader(filePath, password2); break; 
      case 3: r = new PdfReader(filePath, password3); break; 
     } 
    } 
    catch (BadPasswordException ex) 
    { 
     if (Retries == 3) throw ex; 
     Retries++; 
     goto start; 
    } 
    return r; 
} 

Вставить попытку/уловку работать, но выглядит уродливым, используя goto seem natural.

Два вопроса:

1. Должен ли я заменить этот Гото?
2. Есть ли элегантный способ заменить этот goto?

Благодаря

Answer 1

Я думаю, что вы должны заменить goto в этом случае, поскольку проблема заключается в том, что ваш код и повторные попытки затуманивают намерение.Лично я бы заменить код что-то вроде этого:

PdfReader GetPdfReader(string filePath) 
{ 
    PdfReader r = null; 

    string [] passwordsToTry = new string [] {null, password1, password2, password3}; 

    foreach(string password in passwordsToTry) 
    { 
     try 
     { 
      r = (password == null) ? 
       new PdfReader(filePath) 
       : new PdfReader(filePath, password); 
      if (r != null) 
       break; 
     } 
     catch(BadPasswordException){ } 
    } 
    return r; 
} 

мне код является более ясным, как это показывает:

1. Вы определили список паролей, чтобы попытаться не включая «None»
2. You не заботятся о BadPasswordExceptions кроме игнорировать их
3. Если вы получаете удар, то цикл выходит
4. Если вы не получаете хитов цикл завершается в конце

Другое дело, что ваш код с тремя паролями немного хрупкий, если вам приходится иметь дело с более или менее паролями. И я думаю, что использование переменной типа passwordsToTry прекрасно вписывается в инструкцию «try».

Answer 2

Готы, так же, как АЭС или бекон, по сути не является злом. Это вопрос того, что вы с ним делаете.

Текущий код довольно четко структурирован. Goto не используется для создания spaghetti code.

Сказав это, я все равно заменил бы его петлей while. Goto может быть скользким уклоном.

bool passwordIsOK = false; 
while (!passwordIsOK && Retries < 3) 
{ 
    // Existing code, without "goto start;". Set passwordIsOK = true when appropriate. 
    // See also @Sriram's comment about using "throw" rather than "throw ex". 
} 

Answer 3

Вы можете сделать while(true) с break:

PdfReader GetPdfReader(string filePath) 
{ 
    PdfReader r = null; 
    int Retries = 0; 
    while(true) 
    { 
     try 
     { 
      switch (Retries) 
      { 
       case 0: r = new PdfReader(filePath); break; 
       case 1: r = new PdfReader(filePath, password1); break; 
       case 2: r = new PdfReader(filePath, password2); break; 
       case 3: r = new PdfReader(filePath, password3); break; 
      } 
      break; 
     } 
     catch (BadPasswordException ex) 
     { 
      if (Retries == 3) throw ex; 
      Retries++; 
     } 
    } 
    return r; 
} 

Это позволяет избежать добавления каких-либо дополнительных переменных. This topic переваливает за/против goto.