Loading...
Ipetrov1994 avatar Ipetrov1994 1 Точки

Тестова задача "Harvest"

Колеги, моля някой да ми обясни, защо форматът данни, който съм задал не е четим?

ето това е кода ми:

 

namespace harvest
{
    class Program
    {
        static void Main(string[] args)
        {
            var kvMLoze = double.Parse(Console.ReadLine());
            var grapeForKvM = double.Parse(Console.ReadLine());
            var wineNeeded = double.Parse(Console.ReadLine());
            var workers = double.Parse(Console.ReadLine());

            var wineProd = (kvMLoze * grapeForKvM) * 0.4;
            var wine = wineProd / 2.5;
            if (wine > wineNeeded)
            {
                var vineLeft = wine - wineNeeded;
                Console.WriteLine("Good harvest this year! Total wine: {0} liters.", Math.Floor(wine));
                Console.WriteLine("{0} liters left -> {1} liters per person.", Math.Ceiling(vineLeft), Math.Ceiling(vineLeft / workers));
            }
            else
            {
                Console.WriteLine("It will be a tough winter! More {0} liters wine needed.", Math.Floor(wineNeeded - wine));
            } 
        }  
    }
}

Тагове:
0
Programming Basics 27/08/2016 13:07:47
AntyfrizZz avatar AntyfrizZz 238 Точки

Здравей,

 

До колкото разбирам, някой ти е направил забележка, че кодът ти не е четим. Ето защо и аз мисля, че е така:

1) Трябва да си кръщаваш променливите по начин, по който дори и някой незапознат с проблема да разбере за какво служат

    1.1) kvMLoze ?? Трябваше да изтегля условието, за да разбера, че това е крадратурата на лозето. Кръстил си я на български, това не трябва да го правиш. Удачно име за тази променлива е vineyardArea или нещо подобно.

    1.2) grapeForKvM - тук пък си използвал българо английски. Може би grapesPerSquareMeter

    1.3) wineProd - също не ми е ясно за какво служи тази променлива, без да прочета условието.

    1.4) vineLeft - и тази не е именувана правилно. Според мен си искал да напишеш wineLeft.

2) Добре е да оставяш по някой празен ред, за по добра четимост на кода. Например аз бих оставил един след последния ред, който чете от конзолата и един преди иф-а.

 

И една забележка към условието. Не знам дали има тестове, в които произведеното вино ще е равно на необходимото, но в момента на теб това условие ти влиза в else тялото. Може би трябва да влиза в if-а  (wine >= wineNeeded)

 

Поздрави!

1
Ipetrov1994 avatar Ipetrov1994 1 Точки

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

0
27/08/2016 13:12:48
AntyfrizZz avatar AntyfrizZz 238 Точки

Здравей,

 

Приеми го като градивна критика :Д

Липсва ти using System;

 

Поздрави!

0
Ipetrov1994 avatar Ipetrov1994 1 Точки

сега пък джъджа ми го прие като 100/100 , но програмата пак ми дава грешка и се чупи, да не съм пипнал някоя настройка без да искам..? 

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