Programming Basics C# Dancehall Exercise Lab/Прости операции и пресмятания.

Denislava91 5 Точки

Programming Basics C# Dancehall Exercise Lab/Прости операции и пресмятания.

Здравейте, колеги,

Исках да ви попитам дали според вас може вместо да работим в сантиметри и да превръщаме дължината, ширината на залата, гардероба и пейката, да превърнем пространството необходимо за всеки един танцъор в м2 и да си спестим доста редове излишно пресмятане, защото ние търсим броя на танъорите, въпреки че насоките ни казват друго?

Задачата би изглеждала така:

using System;

public class Program
{
   public static void Main()
   {

       double L=double.Parse(Console.ReadLine());
       double W=double.Parse(Console.ReadLine());
       double A=double.Parse(Console.ReadLine());

       double areaDanceHall=L*W;
       double areaWordDrobe=A*A;
       double areaBench=areaDanceHall/10;
       double areaTotal=areaDanceHall-areaWordDrobe-areaBench;

       double dancers=areaTotal/0.7040;

       Console.WriteLine(Math.Floor(dancers));


       }
}

Какво мислите?

Благодаря и успешна седмица на всички!

Денислава

Тагове:

11/11/2019 11:24:47 Programming Basics with C#

Borislav_Vulkov 54 Точки

да естествено, че можеш няма драма :)
като цяло това е й идеята, да се пише възможно най - малко код без да имаш повторения и в същото време да е лесен за четене. Ти си се справила с това :)
Едиствено бих те посъветвал да не пишеш "магически" числа или стрингове в кода (както е направено тук: double dancers=areaTotal/0.7040). Добра практика е тези неща да се изнасят като константи и след това да ги ползваш там където ти е необходимо.

Примерно: const double ConstName = 0.7040;

и след това да я използваш: double dancers=areaTotal/ConstName;

Има две основни причини за това:
1) Даваш име на тази стойност. Това помага на всеке който чете този код да знае какво си имала в предвид. Примерно аз в тази момент нямам идея защо делиш на 0.7040. Да ти си стигнала до този извод че трбва да е толкова но не се знае защо. С две думи правиш кода мого по четим и ясен.
2) Има много ситуации в които ще ти се налага тези числа/стингове и т.н. да ги ползваш на много места в тоя код. Тогава е много по - удобно да имаш константи защото ако решиш да промениш нещо трябва да го промениш само на едно място. В противен случай ще трябва да минаваш по кода и да го променяш навсякъде. Сама разбираш че това може да доведе до много грешки, примерно ако забравиш да го промениш на някое място.

Надявам се да съм ти бил полезен,
Боби :)

11/11/2019 13:38:16

Denislava91 5 Точки

Много благодаря за изчерпателния коментар:)

Съгласна съм и със забележката за магическите числа, ще го имам предвид:)

Поздрави:)

Денислава

11/11/2019 14:07:19

Във форума е въведено ограничение, което позволява на потребителите единствено да разглеждат публикуваните въпроси.

Programming Basics C# Dancehall Exercise Lab/Прости операции и пресмятания.