Loading...
krum_43 avatar krum_43 756 Точки

Промлем при решаване на задачата за изчисляване на комисионни в 3 различни града -модула Programing bacik C#.

Може ли някой да ми каже защо този код не работи?

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace trade_comissions
{
    class Program
    {
        static void Main(string[] args)
        {
            var city = Console.ReadLine().ToLower();
            var sales = double.Parse(Console.ReadLine());
            var comission = 0.0;

            if (city == "Sofia")
            {
                if( sales>=0 && sales <= 500) comission=0.05;
                else {
                    if (sales>=500 && sales <= 1000) comission = 0.07;
                    else {
                        if (sales>=1000 && sales <= 10000) comission = 0.08;
                        else  comission = 0.12;
                         }
                     }                
            }
            else if (city =="Plovdiv")
            {
                if (sales >= 0 && sales <= 500) comission = 0.055;
                else {
                    if (sales >= 500 && sales <= 1000) comission = 0.08;
                    else {
                        if (sales >= 1000 && sales <= 10000) comission = 0.12;
                        else comission = 0.145;
                    }
                }
            }
                    else if (city =="Varna")
            {
                if (sales >= 0 && sales <= 500) comission = 0.045;
                else {
                    if (sales >= 500 && sales <= 1000) comission = 0.075;
                    else {
                        if (sales >= 1000 && sales <= 10000) comission = 0.1;
                        else comission = 0.13;
                    }
                }
            }
            else if (comission > 0) Console.WriteLine("{0:f2}" ,comission * sales);
            else Console.WriteLine("error");
        }
    }
}
 

Тагове:
0
Programming Basics
ad2bg avatar ad2bg 123 Точки

Правиш City ToLower, а после в сравненията имаш главни букви...   Трябва да сравняваш със "sofia", "varna", "plovdiv"...

Моето решение тук:

https://github.com/ad2bg/2017-10-28---Conditions/blob/master/TradeCommissions/TradeCommissions.cs

0
krum_43 avatar krum_43 756 Точки

Да това е грешка,но и без да дам ToLower пак не ми работи странно защо?!

0
ad2bg avatar ad2bg 123 Точки

Този ред

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

... не трябва да е част от if - else if  веригата, понеже като ти се изпълни по-горно условие -> следващите (включително този) се пропускат. 

0
krum_43 avatar krum_43 756 Точки

Да така е просто не сьм го сьобразил.

0
melik avatar melik 285 Точки

Направи това което колегата ad2bg е казал или махни .ToLower на "city".

Примерно решение.

0
05/11/2017 12:30:33
krum_43 avatar krum_43 756 Точки

Този код сьщо не работи както трабва  ?!?...

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace trade_comissions
{
    class Program
    {
        static void Main(string[] args)
        {
            var city = Console.ReadLine().ToLower();
            var sales = double.Parse(Console.ReadLine());
            var comission = 0.0;

            if (city == "sofia")
            {
                if( sales>=0 && sales <= 500) comission=0.05;
                else {
                    if (sales>=500 && sales <= 1000) comission = 0.07;
                    else {
                        if (sales>=1000 && sales <= 10000) comission = 0.08;
                        else  comission = 0.12;
                         }
                     }                
            }
            else if (city =="plovdiv")
            {
                if (sales >= 0 && sales <= 500) comission = 0.055;
                else {
                    if (sales >= 500 && sales <= 1000) comission = 0.08;
                    else {
                        if (sales >= 1000 && sales <= 10000) comission = 0.12;
                        else comission = 0.145;
                    }
                }
            }
                    else if (city =="varna")
            {
                if (sales >= 0 && sales <= 500) comission = 0.045;
                else {
                    if (sales >= 500 && sales <= 1000) comission = 0.075;
                    else {
                        if (sales >= 1000 && sales <= 10000) comission = 0.1;
                        else comission = 0.13;
                    }
                }
            }
            else  if (comission > 0) Console.WriteLine("{0:f2}" ,comission * sales);
            else Console.WriteLine("error");
        }
    }
}
 

0
melik avatar melik 285 Точки

Много else-a имаш и най-вероятно там нещо се бърка, в момента нямам време да дебъгвам. Примерно решение без switch-case. На пръв поглед виждам една грешка. Това else  if (comission > 0) Console.WriteLine("{0:f2}" ,comission * sales);
            else Console.WriteLine("error"); махни else  и остави само if и else

https://pastebin.com/gBzaYmnQ

0
05/11/2017 12:54:20
krum_43 avatar krum_43 756 Точки

Мерси колега,

След  корекзиите judja ми даде 100-те точки.

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