namespace PizzaApp
{
public partial class PizzaOrder : Form
{
// Initializing the toppings to its cost
// Double type since it could have decimal points
public double Small = 5.50;
public double Medium = 11.75;
public double Large = 15.00;
public double Pepperoni = 0.75;
public double Onion = 0.75;
public double Mushroom = 0.75;
public double BlackOlives = 0.75;
public double Pineapple = 0.75;
public double ExtraCheese = 0.75;
public static double Cost = 0.00; // Keeps track of total cost
public PizzaOrder()
{
InitializeComponent();
}
private void orderButton_Click(object sender, EventArgs e)
{
// If small size pizza radio button is checked, I do cost - 1.50 because 2 ingredients are free
if (smallRadioButton.Checked == true)
{
Cost = Small;
Cost -= 1.50;
}
// If medium size pizza radio button is checked, I do cost - 2.25 because 3 ingredients are free
if (mediumRadioButton.Checked == true)
{
Cost = Medium;
Cost -= 2.25;
}
// If large size pizza radio button is checked, I do cost - 3.00 because 4 ingredients are free
if (largeRadioButton.Checked == true)
{
Cost = Large;
Cost -= 3.00;
}
//Ingredrients if statements
if (pepperoniCheckBox.Checked == true)
{
Cost = Pepperoni;
}
if (onionCheckBox.Checked == true)
{
Cost = Onion;
}
if (mushroomCheckBox.Checked == true)
{
Cost = Mushroom;
}
if (blackOlivesCheckBox.Checked == true)
{
Cost = BlackOlives;
}
if (pineappleCheckBox.Checked == true)
{
Cost = Pineapple;
}
if (extraCheeseCheckBox.Checked == true)
{
Cost = ExtraCheese;
}
}
private void sizeGroupBox_Enter(object sender, EventArgs e)
{
}
private void ingredientsGroupBox_Enter(object sender, EventArgs e)
{
}
private void PizzaOrder_Load(object sender, EventArgs e)
{
}
private void priceLabel_Click(object sender, EventArgs e)
{
}
private void totalPriceTextBox_TextChanged(object sender, EventArgs e)
{
totalPriceTextBox.Text = Cost.ToString();
}
}
}
例如，當用戶拿小披薩時，6 種成分中有 2 種是免費的（中型是 3，大的是 4 種免費成分）。我找不到任何其他方法來實作這一點，所以我只是從比薩大小中減去了免費成分。我現在被困在如何強迫用戶至少選擇那些免費成分，這樣比薩餅的成本就不會低于初始價格。
uj5u.com熱心網友回復：
這是初學者代碼中的一個常見缺陷，即 UI 和資料/業務邏輯之間基本上是零抽象/分離。考慮使用適當的 OOP 方法，然后事情很快就會變得更加清晰。
例子：
public enum PizzaSize
{
Small,
Medium,
Large
}
public class PizzaTopping
{
public string Name { get; set; }
public decimal Price { get; set; }
}
public class PizzaOrder
{
private static readonly Dictionary<PizzeSize, decimal> pizzaPrices = new Dictionary<PizzeSize, decimal>
{
{ PizzaSize.Small, 5.5m },
{ PizzaSize.Medium, 11.75m },
{ PizzaSize.Large, 15m },
};
private static readonly Dictionary<PizzeSize, decimal> freeToppingsCounts = new Dictionary<PizzeSize, decimal>
{
{ PizzaSize.Small, 2 },
{ PizzaSize.Medium, 3 },
{ PizzaSize.Large, 4 },
};
public PizzaSize Size { get; set; }
public List<PizzaTopping> Toppings { get; set; } = new List<PizzaTopping>();
public decimal GetTotalPrice()
{
var totalPrice = pizzaPrices[this.Size];
var freeToppings = freeToppingsCounts[this.Size];
var counter = 0;
foreach (var topping in this.Toppings.OrderBy(x => x.Price))
{
counter ;
if (counter > freeToppings)
{
totalPrice = topping.Price;
}
}
return totalPrice;
}
}
public class PizzaOrderForm : Form
{
var order = GetPizzaOrder(); // create your order from the UI here
var totalPrice = order.GetTotalPrice();
}
（免責宣告：我很快在記事本中將其破解，這并不意味著完美，只是為了表明我的觀點。）
uj5u.com熱心網友回復：
我將首先定義一個類，該類包含對相關復選框的參考以及該澆頭的價格
public class Topping
{
public CheckBox Selector {get;}
public double Cost {get;}
public Topping(CheckBox selector, double cost)
{
this.Selector = selector;
this.Cost = cost;
}
}
然后Topping在您的表單中定義一個陣列，而不是為每個變數定義一堆離散變數：
public partial class PizzaOrder : Form
{
private Topping[] toppings = new Topping[]{
new Topping(pepperoniCheckBox, 0.75),
new Topping(onionCheckBox, 0.75),
new Topping(mushroomCheckBox, 0.75),
new Topping(blackOlivesCheckBox, 0.75),
// .. etc
};
這很有用，因為您現在可以確定所選澆頭的數量（所有選中的澆頭）
var numToppingsSelected = toppings.Where(x => x.Selector.Checked).Count();
您還可以獲得IEnumerable<Topping>選定的其中一個 -Cost在獲得剩余選擇的總成本之前，可以按順序訂購并跳過最便宜的 2 個。
var selectedToppings = toppings.Where(x => x.Selector.Checked).OrderBy(x => x.Cost);
var selectedToppingsExceptFirst2 = selectedToppings.Skip(2);
var costOfRemainingToppings = selectedToppingsExceptFirst2.Sum(x => x.Cost);
希望這能讓您在構建更合理的訂單按鈕處理程式的道路上走得更遠。如果您對如何執行此操作有任何后續問題，請告訴我。
uj5u.com熱心網友回復：
您的部分要求是用戶不能吃有一種配料的比薩餅，即使他們不想要其他五種配料中的任何一種嗎？因為如果沒有，那么您將引入一個不應該存在的約束，從而使其成為 XY 問題。
我將假設這個約束不應該存在。另請注意，我試圖對您的代碼進行最小的更改，而不是用更多的花里胡哨重新設計它，因為我認為這是初學者的任務。
我會從一開始就以不同的方式解決這個問題，但這將是一種更高級的方法，它需要從頭開始重寫整個解決方案，這在這里有點偏離主題，對您來說也不是一個好教訓。
這里的問題是您的任務需要考慮一組澆頭（“ [所選澆頭串列中的前兩個]是免費的”），但您的代碼僅單獨考慮每個澆頭復選框。就目前而言，您的代碼并未將澆頭集合視為一個組。
我注意到所有澆頭的價格都一樣。這是給定的嗎？或者澆頭有可能有不同的價格嗎？
如果所有配料的價格始終相同，則數學計算非常簡單。澆頭的總價為：
([amount_of_toppings] - 2) * [price_of_a_topping]
當amount_of_toppings小于或等于 2 時，您甚至不必計算它。
你怎么知道澆頭的數量？好吧，您數一下選中的框。
public double GetToppingsTotalPrice()
{
int counter = 0;
if(pepperoniCheckBox.Checked)
counter ;
if(onionCheckBox.Checked)
counter ;
// and so on
if(counter > 2)
return (counter-2) * this.toppingPrice;
else
return 0;
}
如果澆頭可以有不同的價格，那么您需要更多資訊。你如何決定哪些澆頭是免費的？您可以假設最便宜的將是免費的，但這應該在描述中確認。
讓我們相信最便宜的兩個將是免費的。這意味著我們需要首先找出選擇了哪些澆頭，它們各自的價格是多少，去掉兩個最低的價格，然后將剩余的價格相加。
在集合上執行這些型別的操作時，LINQ 是一種干凈且簡單的方法，因此我將在此解決方案中使用它。
public double GetToppingsTotalPrice()
{
List<double> addedToppingPrices = new List<double>();
if(pepperoniCheckBox.Checked)
addedToppingPrices.Add(this.Pepperoni);
if(onionCheckBox.Checked)
addedToppingPrices.Add(this.Onion);
// and so on
// Stop here if there aren't any paid toppings.
if(addedToppingPrices.Count() <= 2)
return 0;
return addedToppingPrices
.OrderBy(x => x) // Sort from cheapest to most expensive
.Skip(2) // Skip the first two
.Sum(x => x); // Sum the remaining numbers
}
轉載請註明出處，本文鏈接：https://www.uj5u.com/ruanti/317071.html
標籤：C＃
上一篇：在PackageManagerConsole中添加遷移時，EFCoreIEntityTypeConfiguration類中未命中斷點