c# - 如果还有最好的选择？

Question

我正在做一个项目，需要建立一个流程。我附上了一张显示我的流程的图片。在这一刻，我完全不知道如何写这个......你们能帮帮我吗？！目前我有这段代码：但是在路径上 Set=Yes|Complete=Yes 它不会再继续了

if (item.IsSet == "J")
{
    DialogResult isComplete = MessageBox.Show("Zijn alle bijlagen meegeleverd?", "Bijlagen compleet?", MessageBoxButtons.YesNo, MessageBoxIcon.Question);
    if (isComplete == DialogResult.Yes)
    {
        // Break out of this loop and continue
    }
    if (isComplete == DialogResult.No)
    {
        // STATUS NIET INGELEVERD
        MessageBox.Show("STATUS NIET INGELEVERD"); 
    }
}
if (item.IsSet == "N")
{
    if (item.DeliveryMethod == "N")
    {
        DialogResult geaccepteerd = MessageBox.Show("Let op: Dit boek is nieuw geleverd. Accepteer je de kwaliteit van dit boek?", "Kwaliteit", MessageBoxButtons.YesNo, MessageBoxIcon.Warning);
        if (geaccepteerd == DialogResult.Yes)
        {
            // STATUS GEACCEPTEERD
            MessageBox.Show("STATUS GEACCEPTEERD");
        }
        if (geaccepteerd == DialogResult.No)
        {
            // STATUS NIET GEACCEPTEERD
            MessageBox.Show("STATUS NIET GEACCEPTEERD");
        }
    }
    if (item.DeliveryMethod == "G")
    {
        // STATUS GEACCEPTEERD
        MessageBox.Show("STATUS GEACCEPTEERD");
    }
}

Answer 1

我认为这会做到。

if (item.IsSet == "J")
{
  if (MessageBox.Show("Zijn alle bijlagen meegeleverd?", "Bijlagen compleet?", MessageBoxButtons.YesNo, MessageBoxIcon.Question) == DialogResult.Yes)
  {
    if (item.DeliveryMethod == "N")
    {
      if (MessageBox.Show("Let op: Dit boek is nieuw geleverd. Accepteer je de kwaliteit van dit boek?", "Kwaliteit", MessageBoxButtons.YesNo, MessageBoxIcon.Warning) == DialogResult.Yes)
      {
         // STATUS GEACCEPTEERD
         MessageBox.Show("STATUS GEACCEPTEERD");
      }
      else
      {
         // STATUS NIET GEACCEPTEERD
         MessageBox.Show("STATUS NIET GEACCEPTEERD");
      }
    }    
  }
  else 
  {
    MessageBox.Show("STATUS NIET INGELEVERD"); 
  }
}
else
{
  if (item.DeliveryMethod == "N")
  {
    if (MessageBox.Show("Let op: Dit boek is nieuw geleverd. Accepteer je de kwaliteit van dit boek?", "Kwaliteit", MessageBoxButtons.YesNo, MessageBoxIcon.Warning) == DialogResult.Yes)
    {
       // STATUS GEACCEPTEERD
       MessageBox.Show("STATUS GEACCEPTEERD");
    }
   else
    {
       // STATUS NIET GEACCEPTEERD
       MessageBox.Show("STATUS NIET GEACCEPTEERD");
    }
  }
}

Answer 2

好的，根据您的评论和流程图，这是我简化它的建议

if (item.IsSet)
{
    DialogResult isComplete = MessageBox.Show("Complete set?", "complete set?", MessageBoxButtons.YesNo, MessageBoxIcon.Question);
    if (isComplete == DialogResult.No)
        // Break out
}
if(item.IsNew)
{
    DialogResult goodQuality = MessageBox.Show("Is the quality good", "quality", MessageBoxButtons.YesNo, MessageBoxIcon.Warning);
    if (goodQuality == DialogResult.No)
        //not accepted (break)
}
//if reached here, accepted