Member 13510990 Ответов: 0

Может ли кто-нибудь помочь мне с некоторыми идеями по улучшению этого кода?


   List<string> loggbok = new List<string> { };
   DateTime datum = DateTime.Now;
   Console.WriteLine(datum);

   string titel = "";
   string logg = "";
   string sparad = "";

   bool isRunning = true;
   while (isRunning)
       try
       {
           {
               Console.WriteLine("\n\t Välkommen till loggboken");

               Console.WriteLine("\n\t[1] Skriv nytt inlägg i loggboken\n" +
                                 "\t[2] Sök inlägg i loggboken\n" +
                                 "\t[3] Skriv ut alla loggar" +
                                 "\t[4] Avsluta programmet");
               Console.Write("\tVälj: ");

               int menyVal = Convert.ToInt32(Console.ReadLine());

               switch (menyVal)
               {
                   case 1:
                       Console.Clear();
                       Console.Write("Skriv titel på ditt inlägg: ");
                       titel = Console.ReadLine();
                       Console.Write("Skriv ditt inlägg: ");
                       Console.WriteLine(datum);
                       Console.WriteLine(titel);
                       Console.WriteLine(sparad);
                       sparad = Console.ReadLine();
                       Console.WriteLine("Alla dina inlägg är nu sparade!");
                       logg = datum + "\n" + titel + "\n" + sparad;

                       loggbok.Add(logg);
                       break;

                   case 2:
                       Console.Clear();
                       Console.WriteLine("Skriv in ett sökOrd");

                       var sökObjekt = Console.ReadLine();

                       var hittatItem = false;
                       foreach (var item in loggbok)
                       {
                           if (item.Contains(sökObjekt))
                           {
                               hittatItem = true;

                               Console.WriteLine(item);
                           }
                       }
                       if (!hittatItem)
                       {
                           Console.WriteLine("Felaktig inmatning...Vänligen försök igen!");
                       }
                       break;

                   case 3:
                       Console.Clear();
                       Console.WriteLine("HÄR ÄR ALLA DINA SPARADE INLOGG: ");
                       foreach (string loggboken in loggbok)
                       {
                           Console.WriteLine(loggboken);
                       }
                       Console.WriteLine("Tryck på valfri knapp på tangentbordet för att gå" +
                           " tillbaka till meny");
                       Console.ReadKey();
                       break;

                   default:

                       break;

                  case 4:
                       return;
               }
           }
       }

       catch
       {
           Console.Clear();
           Console.WriteLine("Nu blev det fel, gör ett val i menyn...");
       }
}


Что я уже пробовал:

Программа работает нормально. Наверное, все необходимые функции работают. Так что я просто сделал несколько предложений по улучшению. Не стесняйтесь указывать, даже если что-то не так. Потому что я новичок.

BillWoodruff

Поскольку я не говорю на вашем языке, то, что я могу сказать, ограничено, но я предлагаю вам добавить краткое описание на английском языке к вашему сообщению о том, что делает ваша программа.

Очевидно, что вы можете сделать много рефакторинга с помощью строк, которые вы используете ... если это важно для вас ?

Member 13510990

Моя программа должна иметь меню, и я должен использовать переключатель.
Во-вторых, я должен использовать список, в то время как каждый отдельный журнал должен быть массивом.

BillWoodruff

"используйте список, в то время как каждый отдельный журнал должен быть массивом."

Код, который я вижу сейчас, содержит один список строк. Разве это не то, чего ты хочешь ?

А как насчет сохранения журнала ?

Richard MacCutchan

В случае 1 вы записываете в консоль поля, которые не имеют никаких значений.

Perić Željko

Интересно, когда эта программа будет меняться состояние выполняется переменной значение false, и конец.

0 Ответов