我的第一个程序的优化：C# 圣诞树控制台应用程序

Question

我刚开始学习 c# 所以我不知道我在做什么。 作为我的第一个 Projekt，我尝试制作一棵圣诞树。 它还没有完成，但我相信有很多可以优化的东西。 如果有人在这里很无聊，请看一下。 每次我看到一些可能的新事物时，我都会尝试一下，所以它很混乱！ 谢谢

        {

            //Variablen
            Program p = new Program();
            int zähler = 0;
            string breite;
            string höhe;
            int halbe__höhe;
            int halbe__breite;
            int zeilen = 0;
            string top = "";
            string bottom = "";
            char space = ' ';



            //Initialisiereungstext und eingabe von Höhe und Breite
            Console.WriteLine("Dieses Programm erstellt einen Grafischen Christbaum, dafür wird eine Eingabe von Höhe und Breite verwendet!\n" +
                "Das Programm wurde von xy in Auftrag der xy. \n" +
                "Sie werden anschließend aufgefordert zur Eingabe von Höhe und Breite\n" +
                "Geben die bitte ganze Zahlen ein! Die Angabe für die Breite muss für ein exaktes Ergebnis ungerade und geringer als die Höhe sein!");
            Console.Write("Höhe: ");
            breite = Console.ReadLine();

            Console.Write("Breite: ");
            höhe = Console.ReadLine();



            //Mathematische generierung
            halbe__breite = Convert.ToInt32(breite) / 2;
            halbe__höhe = Convert.ToInt32(höhe) / 2;
            top = "".PadRight(Convert.ToInt32(breite) - Convert.ToInt32(breite) / 2 - (Convert.ToInt32(breite) / 2) / 2 - 2); //Ich weis das ist echt ineffizient aber es funktioniert an dem Punkt und ich möchte es nur ungern wieder löschen...bitte um verständnis
            bottom = "".PadLeft(Convert.ToInt32(breite) - Convert.ToInt32(breite) / 2 - (Convert.ToInt32(breite) / 2) / 2 - 2);

            StringBuilder sb__top = new StringBuilder(top);
            StringBuilder sb__bottom = new StringBuilder(bottom);
            StringBuilder init__sb__top = new StringBuilder(top);
            StringBuilder init__sb__bottom = new StringBuilder(bottom);

            sb__top.Append(space, Convert.ToInt32(breite));

            Console.Clear();

            Console.WriteLine(sb__top.Length);



            //Grafische generierung
            while (true)
            {
                zähler++;
                for (int i = Convert.ToInt32(breite); i > 0; i -= 2)
                {
                    zeilen++;

                    sb__top[i++] = '#';

                    string s = Convert.ToString(sb__top);
                    char[] c = s.ToCharArray();
                    Array.Reverse(c);
                    s = new string(c);

                    Console.Write(sb__top);
                    Console.Write(s + "\n");

                    //Warum befinden sich zwischen den beiden Hälften leerzeichen????

                    if (zeilen > Convert.ToInt32(höhe))
                    {
                        return;
                    }

                }

                for (int i = 0; i < halbe__breite * 3; i++)
                {
                    Random rnd = new Random();
                    int rand1 = rnd.Next(1, Convert.ToInt32(breite));
                    int rand2 = rnd.Next(1, Convert.ToInt32(breite));
                    Console.SetCursorPosition(rand1, rand2);
                    Console.Write("8");
                }


                Thread.Sleep(2000000);
                return;
            }




        }```

Answer 1

你想优化什么？ 代码的可读性？ 表现？ 稳定？

您可以将一些计算值存储在局部变量中，因此您的机器必须计算它不超过一次。 为了便于阅读，将其存储在一个命名良好的变量中，并计算其值并将其返回到一个命名良好的 function 中。

例如，这可以为您节省大量解析，并且为了稳定性，您应该使用 TryParse。

在这一点上，在 for 循环中创建一个新的 Random 是多余的并且相当昂贵。 只需将其创建移到循环上方即可。

最后一个睡眠的重点是保持控制台打开？ 您可以添加另一个 Readline，然后在任何输入时退出应用程序。

我会用英语写变量和评论，你可能会接触到更多的人，因此更多的人可以帮助你。