Optimierung redundante else-Anweisung

Question

ich diesen Code-Block haben:

if (duplicateVoucherChecker(voucher)) 
{ 
    DialogResult dialogResult = Messages.Question("Podany bon był już użyty na tej stacji", "Uwaga", false); 

    if (dialogResult == DialogResult.No) 
    { 
     ViewTyped.PaymentValueEditor.Focus(); 
    } 
    else if (dialogResult == DialogResult.Yes) 
    { 
     Vouchers.Add(voucher); 
     Payment.OriginalToPay = Payment.ToPay; 
     ViewTyped.PaymentNumberEditor.Focus(); 
     ViewTyped.ChangeEditor.Focus(); 
    } 
} 
else 
{ 
    Vouchers.Add(voucher); 
    Payment.OriginalToPay = Payment.ToPay; 
    ViewTyped.PaymentNumberEditor.Focus(); 
    ViewTyped.ChangeEditor.Focus(); 
} 

Wie kann ich dies nicht überflüssig machen?

Ich möchte zweite else-Anweisung loswerden, aber ich möchte immer noch diesen else-Block getan werden, wenn duplicateVoucherChecker(voucher)false zurückgibt.

Answer 1

Sie können eine bool verwenden, um zu verfolgen, ob das Dialogfeld nicht angezeigt wurde oder das Ergebnis war Ja und nur diese Logik, wenn es true ist.

bool dialogNotShownOrYesPicked = true; 
if (duplicateVoucherChecker(voucher)) 
{ 
    DialogResult dialogResult = Messages.Question(
     "Podany bon był już użyty na tej stacji", "Uwaga", 
     false); 
    if (dialogResult == DialogResult.No) 
    { 
     ViewTyped.PaymentValueEditor.Focus(); 
    } 
    if (dialogResult != DialogResult.Yes) 
    { 
     dialogNotShownOrYesPicked = false; 
    }  
} 

if(dialogNotShownOrYesPicked) 
{ 
    Vouchers.Add(voucher); 
    Payment.OriginalToPay = Payment.ToPay; 
    ViewTyped.PaymentNumberEditor.Focus(); 
    ViewTyped.ChangeEditor.Focus(); 
} 

Answer 2

Sie können sinnvolle Variablen zum Speichern eines Status verwenden. Wenn Sie Code wiederholen, der auch darauf hinweist, dass Sie wiederverwendbare Methoden erstellen könnten, z. AddVoucher.

In diesem Fall könnten Sie eine bool Variable als Flag verwenden:

bool duplicateVoucher = duplicateVoucherChecker(voucher); 
bool addVoucher = !duplicateVoucher; 
if(!addVoucher) 
{ 
    DialogResult dialogResult = Messages.Question("Podany bon był już użyty na tej stacji", "Uwaga", false); 
    addVoucher = dialogResult == DialogResult.Yes; 
    if (dialogResult == DialogResult.No)  
    { 
     ViewTyped.PaymentValueEditor.Focus(); 
    } 
} 
if(addVoucher) 
{ 
    Vouchers.Add(voucher); 
    Payment.OriginalToPay = Payment.ToPay; 
    ViewTyped.PaymentNumberEditor.Focus(); 
    ViewTyped.ChangeEditor.Focus(); 
} 

Answer 3

Nested if Aussagen sind vom Konzept her die gleichen wie einfach ANDing diese Operationen:

if (duplicateVoucherChecker(voucher) && 
    Messages.Question("Podany bon był już użyty na tej stacji", "Uwaga",false) 
     == DialogResult.No) 
{ 
    ViewTyped.PaymentValueEditor.Focus(); 
} 
else 
{ 
    Vouchers.Add(voucher); 
    Payment.OriginalToPay = Payment.ToPay; 
    ViewTyped.PaymentNumberEditor.Focus(); 
    ViewTyped.ChangeEditor.Focus(); 
} 

Beachten Sie, dass dies den Code läuft in Frage jederzeit Messages.Question gibt einen anderen Wert als No zurück, anstatt nur, wenn es Yes zurückgibt. Wenn Messages.Question einen anderen Wert als Yes oder No zurückgeben kann und Sie den betreffenden Code nicht ausführen sollten, ist der Code, den Sie haben, ungefähr so ​​gut, wie Sie möchten; Im besten Fall könnte man den duplizierten Code in eine Methode umwandeln.

Answer 4

Sie müssen nur den Zustand der dialogResult im Falle eines doppelten Gutscheins speichern.

var addVoucher = true; 
if (duplicateVoucherChecker(voucher)) 
{ 
    DialogResult dialogResult = Messages.Question("Podany bon był już użyty na tej stacji", "Uwaga", false); 
    addVoucher = (dialogResult == DialogResult.Yes); 
    if (dialogResult == DialogResult.No) 
    { 
     ViewTyped.PaymentValueEditor.Focus(); 
    } 
} 
if (addVoucher) 
{ 
    Vouchers.Add(voucher); 
    Payment.OriginalToPay = Payment.ToPay; 
    ViewTyped.PaymentNumberEditor.Focus(); 
    ViewTyped.ChangeEditor.Focus(); 
}