Loading...
NikolayLiubomirov avatar NikolayLiubomirov 82 Точки

Simple Calculations - Задача 20. Дневна печалба - проблем

Ще съм много благодарен, ако някой ми каже защо програмата ми изкарва грешно число. Ред по ред пресмятах с калкулатора и не успях да открия грешка.

using System;

namespace DailyEarning
{
    class Program
    {
        static void Main()
        {
            byte daysPerMonth = byte.Parse(Console.ReadLine());
            double dollarsPerDay = double.Parse(Console.ReadLine());
            double dollarRate = double.Parse(Console.ReadLine());

            var monthSalary = daysPerMonth * dollarsPerDay;
            var yearSalary = monthSalary * 12 + monthSalary * 2.5;
            var tax = yearSalary * (25 / 100); 
            var netIncomeInDollars = yearSalary - tax;
            var netIncomeInLeva = netIncomeInDollars * dollarRate;
            var averageIncomePerDay = netIncomeInLeva / 365;

            Console.WriteLine(Math.Round(averageIncomePerDay, 2));
        }
    }
}

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

0
Programming Basics 15/03/2017 18:48:43
ThePSXHive avatar ThePSXHive 436 Точки
Best Answer

Според мен, досещането ти да използваш byte за daysPerMonth е много добро. Излишно е да се използва тип с по-голяма "вместимост". За грешката - не е от форматирането, а от фактът, че когато изчисляваш данъкът на следния ред

var tax = yearSalary * (25 / 100);

резултатът от делението е нула. Съответно, стойността на данъкът винаги е 0. Замени го с деление между целочисленна и дробна стойност

var tax = yearSalary * (25 / 100.0);

Останалото е наред, но форматирането на Luko193 е за предпочитане - по-кратко и си спестяваш извикване на метод.

0
NikolayLiubomirov avatar NikolayLiubomirov 82 Точки

Мерси за изчерпателният отговор, това е било!

1
Luko193 avatar Luko193 342 Точки

Незнам дали е от това но на последния ред за закръглянето използвай: 

Console.WriteLine("{0:F2}", averageIncomePerDay);

0
25/01/2017 22:55:07
NikolayLiubomirov avatar NikolayLiubomirov 82 Точки

Изглежда не е, вади същият грешен резултат като преди. 99.48, когато трябва да изкара 74.61, при вход

21

75.00

1.59

0
25/01/2017 23:19:17
fbinnzhivko avatar fbinnzhivko 1586 Точки

   Console.WriteLine(Math.Round(2.50, 2));    //2.5
   Console.WriteLine("{0:F2}", 2.50);                //2.50


{0:f2} показва втория знак след десетичната точка, дори и той да е нула. При Math.Round нулата бива скривана и ако Judge очаква да получи изход 2.50.

Math.Round би свършил някаква работа, само ако резултатът е (примерно) 2.503, но аз лично не виждам смисъл, защото {0:f3} спестява доста писане.

1
georgi.getz avatar georgi.getz 8 Точки

И аз много мразя като допускам такива грешки :Д махни скобите на (25/100) искаш първо да умножиш и после да делиш на 100

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