Show Dialogfeld nur einmal in einer for-Schleife - Closed

Question

Dieser Code funktioniert gut, außer mehrere Dialogfeld Eingabeaufforderung, wenn es mehrere leere Textbox, aber ich möchte nur einmal aufgefordert.

Zum Beispiel, wenn ich 1,1 eingeben, (Null), (Null), d, g, Dialogfeld wird zweimal aufgefordert, da gibt es zwei leere Textfelder, aber ich brauche es nur einmal zu bestätigen.

Wie kann ich dieses Problem lösen?

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
    { 
     for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
     { 
      if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
      { 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      } 
     } 
    } 

Answer 1

Ich fand einen anderen Weg, um es zu lösen.

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
    { 
     int tt = 0; 
     for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
     { 
      if (tt == 0) 
      { 
       if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
       { 
        MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
        tt = 1; 
       } 
      } 
     } 
    } 

Answer 2

Sie einfach eine Fahne vorstellen kann:

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    bool hasEmpty = false; 

    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      hasEmpty = true; 
     } 
    } 

    if (hasEmpty) { 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
    } 
} 

Answer 3

Warum nicht brechen aus der Schleife, so dass sie die Überprüfung nicht mehr reagiert?

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

return wird auch in dieser Situation funktionieren.

---

Testen Sie, ob BeforeSave läuft zweimal:

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    MessageBox.Show("Test"); // <-- 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

Wie yo sehen kann ich eine neue „Test“ Nachricht an der Spitze des Verfahrens hinzugefügt (außerhalb der Schleife), wenn Sie doppelte siehe " Testen Sie "Nachrichten", wenn Sie den Code verwenden, bedeutet dies, dass BeforeSave zweimal ausgeführt wird.

In diesem Fall müssen Sie sehen, warum es zweimal ausgeführt wird, und dann das beheben. Wenn das nicht reparierbar ist, dann könnte es einige syncronization Lösung sein ... wie:

private int canSave; 

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{ 
    if (Interlocked.CompareExchange(ref canSave, 0, 1) != 1) 
    { 
     // Any cancelation logic that's appropiate here 
     return; 
    } 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; // <-- 
     } 
    } 
} 

Dann sehen Sie canSave 1 den Code zu ermöglichen 0 zu laufen, es zu verbieten. Die Interlocked Operation wird sicherstellen, dass der Code in BeforeSave nicht erneut ausgeführt wird, bis Sie canSave auf 1 irgendwo im Code setzen (es setzt es automatisch auf 0, wenn es ausgeführt wird - keine Chance für mehrere Threads, es zu vermasseln).

Obwohl ich Ihnen eine Lösung zur Kontrolle der doppelten Ausführung von BeforeSave geben, wenn es zweimal als erwartet ausgeführt wird, zeigt, dass es irgendwo woanders ein Problem gibt, und Sie sollten versuchen, das zu beheben (es sei denn, es ist Code von Drittanbietern)).

Answer 4

Verwenden Sie entweder eine Flagge oder verwenden Sie Linq.

public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) { 
    bool flag = false; 
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
     if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
     { 
      flag = true; 
      break; 
     } 
    } 
    if (flag) 
     MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
} 

Ich weiß nicht genug von den verwendeten Objekten einer Linq-Lösung

Answer 5

Nach Art und Weise bieten sollte für Sie arbeiten, auch wenn Ihre Methode Beforemehrmal aufgerufen zu werden.

private bool _isMessageBoxShown; 
public void BeforeSave(BCE.AutoCount.Invoicing.Sales.SalesOrder.SalesOrderBeforeSaveEventArgs e) 
{  
    for (int i = 0; i < e.MasterRecord.DetailCount; i++) 
    { 
    if (String.IsNullOrEmpty(e.MasterRecord.GetDetailRecord(i).YourPONo.ToString())) 
    { 
     if(!_isMessageBoxShown) 
     { 
      _isMessageBoxShown = true; 
      MessageBox.Show("You left Your PO No empty. Please check it carefully."); 
      break; 
     } 
    } 
    }    
} 

So stellen Sie sicher, dass, wenn Sie Ihre messagebox wollen beim nächsten Mal gezeigt werden, Sie _isMessageBoxShown = false; einstellen müssen.