03.Flowers ot Nested Conditional Statements - More Exercises - Софтуерен университет

Jimipage 7 Точки

03.Flowers ot Nested Conditional Statements - More Exercises

Пържите 2 входа излизат, но третият не. Според мен става грешка с отстъпката за розите и после с отстъпката за повече от 20 цветя, но мисля че съм я записал вярно. Някой идея какво да променя? https://pastebin.com/43r7tQrm ето и линк за условието https://judge.softuni.bg/Contests/Practice/Index/1663#2

Тагове:

12/05/2019 12:23:36 Programming Basics 12/05/2019 12:29:38

inkarnasion 278 Точки

Честно казано прекалено усложнено решение, защо са ти толкова проверки? Особенно с това да проверяваш дали е празник или не, излишно утежняваш кода, ето ти преработено решение, погледни го мисля че е по сбито и разбираемо....

https://pastebin.com/yChJJa8t

Поздрави.

12/05/2019 14:15:48

Jimipage 7 Точки

На мен ми се вижда по - сложно ;) но работи. Но все пак не мога да си видя грешката на мойто, дава ми 87/100 и си мисля че трябва да ми е вярно.

12/05/2019 14:25:05

inkarnasion 278 Точки

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

"За закупени 10 или повече рози през зимата – 10% от цената на целият букет."

В твоят код си казал:

case "N":

......

} else if ("Winter".equals(season) || "Autumn".equals(season)) {
    priceAll = countOfChrysanthemum * 3.75 + countOfRoses * 4.50 + countOfTulips * 4.15;
    if (countOfRoses >= 10) {
        priceAll = priceAll - (priceAll * 0.10);
    }
}

В този си вид на кода всеки път когато имаш 10 рози в теста ще влизаш в проверката за розите,и съответно правиш намаление на цената, и това е защото си сложил този if в проверка която проверява дали е есен или зима, така ВИНАГИ ще ти гърми....

Пусни си примерните тестове от условието. При тест 2 имаш 10 рози и тогава ти е верен кода, но при тест 3 ОТНОВО имаш 10 рози и съответно влизаш в проверката за тези цветя и им правиш намаление, независимо че сезона не е Зима а е Есен, така си структурирал самият код, ако закоментираш реда if (countOfRoses >= 10) то тогава ще ти мине третия нулев тест обаче ще гръмне тест 2, и пак ще ти даде 87/100 като този път обаче грешката ще е в тест 2.. Това ти е проблема, надявам се че си схванал обяснението ми, и може да си промениш кода така че да стане верен. Поздрави.

12/05/2019 14:58:14

Jimipage 7 Точки

Мого благодаря за обяснението! Сега вече го схванах!

12/05/2019 15:23:48

Покажи всички коментари