如何使代码在 FOR 循环中运行？而不是那么多的IF答案

【问题标题】：How do I make the code run in a FOR loop? Instead of so many IF如何使代码在 FOR 循环中运行？而不是那么多的IF
【发布时间】：2019-12-25 07:24:27
【问题描述】：

如何在这段代码中使用FOR 循环？而是使用大量IF 问题。我已经尝试了很多方法来使它工作，但没有成功。我很乐意通过智能循环帮助我解决这个问题。这适用于计算架子位置和抽屉位置的软件。我需要你帮我解决这个问题。我想要这样的东西：

            for (int i = 1; i <= NishCount; i++)
            {
                lblNish + i + .Text = nish + i + .ToString();
            }

这是代码：

            if (NishCount == 1 && MdfCount == 0)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = "ENTER VALUES";

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = "ENTER VALUES";
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 2 && MdfCount == 0)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = "ENTER VALUES";
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 1 && MdfCount == 1)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = "ENTER VALUES";

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 2 && MdfCount == 1)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 1 && MdfCount == 2)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = "ENTER VALUES";

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = MDF2.ToString();
            }
            if (NishCount == 2 && MdfCount == 2)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = MDF2.ToString();
            }
            if (NishCount == 1 && MdfCount == 0 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = "ENTER VALUES";

                lblSho.Text = shocev.ToString();

                lblMDF1.Text = "ENTER VALUES";
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 2 && MdfCount == 0 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = shocev.ToString();

                lblMDF1.Text = "ENTER VALUES";
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 1 && MdfCount == 1 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = "ENTER VALUES";

                lblSho.Text = shocev.ToString();

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 2 && MdfCount == 1 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = shocev.ToString();

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = "ENTER VALUES";
            }
            if (NishCount == 1 && MdfCount == 2 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = lblSho.ToString();

                lblSho.Text = "ENTER VALUES";

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = MDF2.ToString();
            }
            if (NishCount == 2 && MdfCount == 2 && SNCheckBox.Checked == true)
            {
                lblNish1.Text = nish1.ToString();
                lblNish2.Text = nish2.ToString();

                lblSho.Text = shocev.ToString();

                lblMDF1.Text = MDF1.ToString();
                lblMDF2.Text = MDF2.ToString();
            }

【问题讨论】：

你需要在这里使用 switch case... For 循环在这里没有用。
怎么办？你能写代码吗？
你能不能把字符串文字改成英文，这样我们就可以更容易地看出哪些是相同的，哪些是不同的？（最好是变量名）你可以edit你的问题。
请阅读docs.microsoft.com/en-us/dotnet/csharp/language-reference/… 了解 C# 中的 switch case。尝试在您的代码中使用它并在此处提问您是否遇到任何问题。
@Sweeper 我把它改成英文了。

标签： c# c#-4.0 c#-3.0 c#-2.0

【解决方案1】：

你可以从初始化它们开始：

lblNish1.Text = "הזן מידות";
lblNish2.Text = "הזן מידות";

lblSho.Text = "הזן מידות";

lblMDF1.Text = "הזן מידות";
lblMDF2.Text = "הזן מידות";

然后根据需要设置它们：

if (NishCount >= 1) 
    lblNish1.Text = nish1.ToString();

if (NishCount >= 2) 
    lblNish2.Text = nish2.ToString();

if (SNCheckBox.Checked)
    lblSho.Text = shocev.ToString();

if (MdfCount >= 1) 
    lblMDF1.Text = MDF1.ToString();

if (MdfCount >= 2) 
    lblMDF2.Text = MDF2.ToString();

如果我们知道您的所有代码，可能会有更优雅的解决方案。

【讨论】：

@אריפרץ 这很容易阅读，在这种情况下可能就是您所需要的。
@אריפרץ 有很多好的方法，如果你争得够狠，其中任何一种都可能是“最好的”。我认为这没问题，除非你想彻底改变你的代码结构。

【解决方案2】：

不要使用 for 循环。您的逻辑简化为以下

            lblNish1.Text = nish1.ToString();
            lblNish2.Text = "ENTER VALUES";

            lblSho.Text = "ENTER VALUES";

            lblMDF1.Text = "ENTER VALUES";
            lblMDF2.Text = "ENTER VALUES";

            if (NishCount == 2)
            {
                lblNish2.Text = nish2.ToString();
            }
            switch(MdfCount)
            {
                case 1:
                    lblMDF1.Text = MDF1.ToString();
                    break;
                case 2:
                    lblMDF1.Text = MDF1.ToString();
                    lblMDF2.Text = MDF2.ToString();
                    break;
            }
            if(SNCheckBox.Checked && !((NishCount == 1) && (MdfCount == 2)))
            {
                lblSho.Text = shocev.ToString();
            }

【讨论】：

非常感谢，可以给我发邮件吗？这是我的邮件：ari.perez25.06@gmail.com
如果我有50次，我会写50个案例？
很难说减少代码量的最佳方法。通常最好的方法是为每个条件设置一个不同的位。所以你有 1,2,4,8,16,32,64,128,256（等等），其中每个位都是不同的条件。如果你使用 long，那么你就有 64 位和 64 个条件。您仍然需要 64 个 if 语句，但每个 if 中只需要一行代码。您的代码在每个 if 语句中重复了重复代码。