Здравейте колеги.
Имам сериозен проблем с тази задача и той се изразява, според VS, в липсата на една затваряща скоба ( } ), след реда на който печатам.
16 пъти броих, 18 пъти ги проверявах тези скоби, но уви. Надявам се някой да открие ключа от палатката. Ето и кода: http://pastebin.com/cjAFzHEB
Грешката ти е накрая. Казваш else ,но преди това нямаш if към него . Няма как след if else да напишеш команда и след това да кажеш отново else.Те трябва да бъдат навързани. Пробвай по този начин.
if ((sales < 0) || (town != "Sofia" && town != "Plovdiv" && town != "Varna"))
{
Console.WriteLine("error");
return;
}
Сложи тази проверка да бъде първа. Ако се въведат невалидни данни ще изпише error и програмата ще приключи (зареади return),без да прави останалите ти проверки .
проблема е в елса защото е въздушен не следва след иф
Забелязах също,че не си сметнал както трябва процентите. Винаги,когато става въпрос за пари използвай decimal.
Заповядай редактирания ти код:
Това е проблемният ред;
Console.WriteLine("{0:f2}", sales * comission);
Благодаря за решението, чак сега го видях.
Обаче се случва нещо интересно. Както се вижда скрийна по-долу } скоба ми я отчита като {
:D
Някой знае ли защо се случва така?
Открих следните проблеми при твоето решение (няма да ти предлагам мое за да не си мислиш, че твоя начин на мислене е грешен).
1. Не присвояваш стойност на "double comission = 0;" - това се прави с оператора "+=", който добавя дясната част на равенството към лявата;
2. направил си грешно деление на процентите: например с "comission = 0.7" - в условието е дадено, че тя е 7%, които са равни на 0.07, а не както си написал 0.7. Объркал си всички преубразувания от проценти в числа с една дума;
3. проверката за градовете: трябва да използваш знака "&&"(И), а не "||"(ИЛИ), защото цикъла ще спре, ако някой от градовете е напр. "Sofia", а целта е едновременно и трите градове в случая да не са София, Варна и Пловдив И(&&) комисионната да е по-малка от 0;
4. в условието има вариант при, който градовете да са с малка буква.
