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

Trade Comissions - задача 8 от Complex Conditional Statements

Може ли някой да ми каже къде бъркам? Това ми е кода и ми изкарва 35 от 100т.

 

using System;


namespace _08.TradeComissions
{
    class Program
    {
        static void Main(string[] args)
        {
            string town = Console.ReadLine().ToLower();
            double sales = double.Parse(Console.ReadLine());
            var comission = -1.0;

            if (town == "sofia")
            {
                if (0 <= sales || sales <= 500) comission = 0.05;
                else if (500 < sales || sales <= 1000) comission = 0.07;
                else if (1000 < sales || sales <= 10000) comission = 0.08;
                else if (sales > 10000) comission = 0.12;

            }

            else if (town == "varna")
            {
                if (0 <= sales || sales <= 500) comission = 0.045;
                else if (500 < sales || sales <= 1000) comission = 0.075;
                else if (1000 < sales || sales <= 10000) comission = 0.1;
                else if (sales > 10000) comission = 0.13;

            }

            else if (town == "plovdiv")
            {
                if (0 <= sales || sales <= 500) comission = 0.055;
                else if (500 < sales || sales <= 1000) comission = 0.08;
                else if (1000 < sales || sales <= 10000) comission = 0.12;
                else if (sales > 10000) comission = 0.145;
            }

            else
            {
                Console.WriteLine("error");
            }
            
            if (comission >=1.0)
            {
                Console.WriteLine("{0:f2}", sales * comission);
            }

            else
            {
                Console.WriteLine("error");
            }
        }
    }
}
 

Тагове:
0
Programming Basics
borislav9212 avatar borislav9212 745 Точки

Здравей,

Допускаш две грешки:

  1. Трябва да използваш логическия оператор "&&"(и) вместо "||"(или). Трябва да отговаря и на двете условия и да е по-голпмо, и да е по-малко, тоест "&&"(и)
  2. if (comission > 0) трябва да е така, да започва от "0" а не така -> if (comission >=1.0)
1
amilotinova avatar amilotinova 1 Точки

Мерси много :)

1
hwfbcisod avatar hwfbcisod 80 Точки

Към отговора на колегата ще добавя, че в случая метода String.ToLower()  е ненужен, а дори повече от това - вреди ти. В заданието на задачата input-а започва с главна буква e.g. "Sofia","Plovdiv","Varna". По този начин, когато конвертираш user input-а към lower case програмата ти въобще не влиза в условните конструкции. Друг съвет е да зададеш като условие sales > 0 вместо comission > -1.0 и да използваш променливи от тип decimal, когато правиш валутни изчисления, тъй като при променливи от тип double се получават грешки. Позволих си да редактирам кода ти и получих 100% в judge системата: http://pastebin.com/Bva5sccv

 

П.С: Моя грешка, ToLower() не беше причината, поради която нямаш output v judge-a :)

 

0
14/06/2016 15:15:07
DoroteyaDoncheva avatar DoroteyaDoncheva 11 Точки

Ето го и моето работещо решение, ако все още имаш нужда от помощ. http://pastebin.com/ztZzZbhP

2
daniel.a.mihaylov avatar daniel.a.mihaylov 6 Точки

Мерси доротея ,решението ми е същото ама се оказа ,че къдравите скоби са ми по малко и за това не ми дава 100/100 :)

-2
versenjentorix avatar versenjentorix 0 Точки

На мен ми помогна.Благодаря ти!

0
sasho10 avatar sasho10 2 Точки

http://pastebin.com/VvmWXwYu thats my code

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