Loading...
edgelord avatar edgelord 0 Точки

C# 11. Manipulating Arrays (Methods Exercises)

Директно от judge свалих файла с условията. CSharp-Fundamentals Exercise. Мъчих го 4 часа и стана с 285 реда и 11 метода. Имам чувството, че съм направил доста излишни неща. Дали може някой с повече време да даде мнение за оптимизиране? Гледал съм да кръщавам всички методи и променливи читаво :D

Ето кода: 

https://pastebin.com/GjJTjKx6

Благодаря на който се занимае! :^)

Тагове:
0
Fundamentals Module 28/07/2019 02:56:37
svetoslav_0 avatar svetoslav_0 1009 Точки
Best Answer

Здравей!

Смятам, че задачата е решена и то правилно. Обемът на кода е станал толкова голям, защото си разписал доста неща, които езикът предлага наготово, което всъщност не само, че не е лошо, а е и градивно. Полезно е, ще знаеш как точно рабоят тези функции, които езикът предлага. В общи линии с тези знания, които имаш на този етап, не бих казал, че имаш излишни неща, освен това задачата е със звездичка, така че е нормално.

Предлагам ти следните оптимизации: Линк 
Не малка част от които е възможно да не разбираш за сега, други най-вероятно ще ти се сторят съвсем логични. Дори и да не разбереш всичко от сега, според мен ще ти бъде полезно да ги разгледаш отново след няколко лекции. Разбира се има още какво да се желае, това е най-основното.

Съвет: когато имаш два разлчини входа в даден момент, например min или max. Забелазах, че правиш проверка за едното, а другото слагаш с else, което на този етап е добре и работи, но е лош навик. След време ще работиш по големи проекти, които ще се ползват от потребители, на които не може да се има доверие. По-добре би било да си конструираш проверките if-elseif вместо if-else. Например:

if (command == "min") 
{
    // do smth
}
else if (command == "max") 
{
    // do smth else 
}
else
{
    Console.WriteLine("Invalid command.");
}

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

0
edgelord avatar edgelord 0 Точки

Благодаря за отзива и отделеното време!  Ще имам предвид тези неща.

0
Можем ли да използваме бисквитки?
Ние използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Можете да се съгласите с всички или част от тях.
Назад
Функционални
Използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Използваме „сесийни“ бисквитки, за да Ви идентифицираме временно. Те се пазят само по време на активната употреба на услугите ни. След излизане от приложението, затваряне на браузъра или мобилното устройство, данните се трият. Използваме бисквитки, за да предоставим опцията „Запомни Ме“, която Ви позволява да използвате нашите услуги без да предоставяте потребителско име и парола. Допълнително е възможно да използваме бисквитки за да съхраняваме различни малки настройки, като избор на езика, позиции на менюта и персонализирано съдържание. Използваме бисквитки и за измерване на маркетинговите ни усилия.
Рекламни
Използваме бисквитки, за да измерваме маркетинг ефективността ни, броене на посещения, както и за проследяването дали дадено електронно писмо е било отворено.