Въпрос относно решението ми на Problem 3. Restaurant Discount от домашното към урок 6 - C# Conditional Statements and Loops - Exercises

katya93 25 Точки

Въпрос относно решението ми на Problem 3. Restaurant Discount от домашното към урок 6 - C# Conditional Statements and Loops - Exercises

Здравейте, имам следният проблем.

Написах решението на задачата във visual studio като при тестване с конзолата всичко е ОК. Проблемът е, че в judge ми дава 71/100 точки. Не мога да открия къде бъркам в кода или изпускам нещо може би? Понеже не могат да се видят тестовете, само ми изписва, че имам 3 грешни отговора. Единият от тях е в Zero теста.

В конзолата, обаче ми излиза отговора, а тук - нищо.

Ето и кода, който съм качила в judge - https://pastebin.com/0UDwQRuK

Имате ли идеи къде греша?

Благодаря предварително!

Тагове:

29/09/2017 00:51:13 Programming Fundamentals 29/09/2017 00:52:35

VasilValchanov 555 Точки

Best Answer

Трябва да е Platinum

а ти си написала Platinium в else if-a

29/09/2017 01:07:55 29/09/2017 01:08:34

katya93 25 Точки

Страхотно! Благодаря за помощта, точно това е бил проблема. Поправих и сега дава 100/100. :-)

29/09/2017 14:50:39

pavsavov 18 Точки

Мисля си,че и за последния else if трябва да му определиш граница от 100 до 120.

Тази проверка може и да е излишна,зависи от тестовете.

мое решение:

https://pastebin.com/u7YmusEG - в общи линии е същото

29/09/2017 08:20:35

katya93 25 Точки

Благодаря за предложението, но се оказа, че проблема е в една буква. VasilValchanov вече отговори къде ми е грешката. Тоест и без тази граница в judge дава 100/100.

И аз реших, че проблема е именно в нея, затова коригирах, но точките паднаха на 51/100. Тоест повече грешки. Сега не съм тествала да добавя проверката при работещ код дали ще върви, но мисля , че няма смисъл да се занимавам за тази задача. :-) Ще оставя кода така.

29/09/2017 14:54:24

AlenPaunov 32 Точки

Здравей, не съм чел подробно кода, не е много приятно за дебъгване, проверих ти само нулевия тест.
Грешката ти е в блока с
else if(groupSize >50 && groupSize <= 100)

else if (packageType == "Platinium")

трябва да е Platinum

Защо не пробваш да ползваш switch?

switch (packageType)

{

case "Platinum":

//code

break;

}

29/09/2017 09:33:59

katya93 25 Точки

Здравей, благодаря за отговора. Наистина грешката е беше в Platinium. Трябва да е Platinum. Не се сетих да ползвам switch-case, затова направих задачата само с if-else. Във форума има постнато решение със switch-case, но исках да разбера къде ми е на мен грешката. Явно може и без него. Просто кода става малко по-дълъг.

29/09/2017 14:56:32