Здравейте,
Пиша във връзка с последната задача от Contests/PB-More-Exercises/Conditional Statements - More Exercises - Fuel Tank Part 2.
Реших задачата и judge ми дава 100/100. Обаче, за толкова лесна задача 121 реда код ми се виждат прекалено много. Моля за някаква препоръка/идея как да я реша "по-хитро".
Код: тук.
Аз имам същия проблем. Кодът ми работи и judge дава 100/100, но е 147 реда.
https://pastebin.com/Qhm6P6K8
Въпроса ми е има ли как да се опрости кода (конктретно в моя случай) без да се използва "switch statemnent" и "conditional operator (?:)", защото все пак задачата е от "More Conditional Statements Exercises", които аз се оптивам да решавам след първите две лекции, а иммено "Първи стъпки в програмирането" и "Проверки". На този етап още не сме взели вложени проверки, нито "switch statement" или "ternary operator (?:)". Благодаря предварително.
Voila.
One super compact fuel tank (100%):
using System;
namespace FuelTank2
{
class Program
{
static void Main(string[] args)
{
string fuelType = Console.ReadLine();
double fuelAmount = double.Parse(Console.ReadLine());
string clubCard = Console.ReadLine();
double gasolinePrice2 = clubCard == "Yes"
? (2.22 - 0.18) : 2.22;
double gasPrice2 = clubCard == "Yes"
? (0.93 - 0.08) : 0.93;
double dieselPrice2 = clubCard == "Yes"
? (2.33 - 0.12) : 2.33;
double result = 0;
if (fuelType == "Gasoline")
{
result = gasolinePrice2 * fuelAmount;
}
else if (fuelType == "Gas")
{
result = gasPrice2 * fuelAmount;
}
else if (fuelType == "Diesel")
{
result = dieselPrice2 * fuelAmount;
}
result = fuelAmount > 25
? result *= .90
: fuelAmount >= 20
? result *= .92
: result;
Console.WriteLine($"{result:f2} lv.");
}
}
}
And without ternary operator:
using System;
namespace FuelTank2
{
class Program
{
static void Main(string[] args)
{
string fuelType = Console.ReadLine();
double fuelAmount = double.Parse(Console.ReadLine());
string clubCard = Console.ReadLine();
double gasolinePrice = 2.22;
double gasPrice = 0.93;
double dieselPrice = 2.33;
double gasolinePriceWithClubCard = gasolinePrice - 0.18;
double gasPriceWithClubCard = gasPrice - 0.08;
double dieselPriceWithClubCard = dieselPrice - 0.12;
double result = 0;
if (fuelAmount > 25)
{
gasolinePrice *= .9;
gasPrice *= .9;
dieselPrice *= .9;
gasolinePriceWithClubCard *= .9;
gasPriceWithClubCard *= .9;
dieselPriceWithClubCard *= .9;
}
else if (fuelAmount >= 20)
{
gasolinePrice *= .92;
gasPrice *= .92;
dieselPrice *= .92;
gasolinePriceWithClubCard *= .92;
gasPriceWithClubCard *= .92;
dieselPriceWithClubCard *= .92;
}
if (clubCard == "Yes")
{
if (fuelType == "Gasoline")
{
result = gasolinePriceWithClubCard * fuelAmount;
}
else if (fuelType == "Gas")
{
result = gasPriceWithClubCard * fuelAmount;
}
else if (fuelType == "Diesel")
{
result = dieselPriceWithClubCard * fuelAmount;
}
}
else
{
if (fuelType == "Gasoline")
{
result = gasolinePrice * fuelAmount;
}
else if (fuelType == "Gas")
{
result = gasPrice * fuelAmount;
}
else if (fuelType == "Diesel")
{
result = dieselPrice * fuelAmount;
}
}
Console.WriteLine($"{result:f2} lv.");
}
}
}
Your solution reduced the code in half. That's what i was asking for. Thank you.