Здравейте,
Пиша във връзка с последната задача от Contests/PB-More-Exercises/Conditional Statements - More Exercises - Fuel Tank Part 2.
Реших задачата и judge ми дава 100/100. Обаче, за толкова лесна задача 121 реда код ми се виждат прекалено много. Моля за някаква препоръка/идея как да я реша "по-хитро".
Код: тук.
ето 69 реда код ако те притеснява обема на кода
Здравей,
може да разгледаш и този код ..
string fuelType = Console.ReadLine();
double fuelQty = Double.Parse(Console.ReadLine());
string discountCard = Console.ReadLine();
double fuelPrice = 0;
switch (fuelType)
{
case "Gas":
fuelPrice = discountCard == "Yes" ? 0.93 - 0.08 : 0.93;
break;
case "Gasoline":
fuelPrice = discountCard == "Yes" ? 2.22 - 0.18 : 2.22;
break;
case "Diesel":
fuelPrice = discountCard == "Yes" ? 2.33 - 0.12 : 2.33;
break;
}
if (fuelQty > 25)
{
fuelPrice *= 0.90;
}
else if (fuelQty >= 20)
{
fuelPrice *= 0.92;
}
double result = fuelQty * fuelPrice;
Console.WriteLine($"{result:f2} lv.");
Поздрави :)
В бейсик курса, shorthand референциите минават ли в judge?
Здравей,
може да опитате и да кажете дали минават.
Поздрави :)
Естествено ще минат защото judge чете генерирания стринг и отговаря ли на критерия, но предполагам, прескачане на материал ще доведе до преписване на код и неразбиране на функционалността му. Judge ще приеме отговорите за верни ако логиката в кода е изпълнена и с отделни методи или клас, но то пак си е прескачане на материал. Ще се прецакаме :)
Аз имам същия проблем. Кодът ми работи и 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.