Может ли кто-нибудь помочь мне с некоторыми идеями по улучшению этого кода?
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, и конец.