作者Moneyyyyy (擋我者死)
看板C_Sharp
標題[問題] 新手想問關於get; set;和陣列
時間Thu Apr 18 17:56:17 2019
我需要先有十個Sale物件
fields需要inventoryNum庫存數量、saleAmount銷售額、taxOwed所欠的稅
皆由使用者輸入
當銷售額使用set存取器設定好後，taxOwed也被計算出來
(銷售額低於100稅率是0.08，大於一百超出額是0.06)
所以我寫了如下的程式碼
Main method 裡面
{
Sale[] sales = new Sale[10];
for (int i = 0; i < sales.Length; i++)
{
sales[i] = new Sale();
sales[i].InventoryNum = ReadLine();
sales[i].SaleAmount = ReadLine();
}
}
然後class中長這樣
{
private int inventoryNum; //庫存數量
private double saleAmount; //銷售額
private double taxOwed;
public double SaleAmount
{
get { return saleAmount; }
set
{
saleAmount = value;
if (saleAmount <= 100)
{
taxOwed = 0.08 * saleAmount;
}
else
{
taxOwed = 0.08 * 100 + 0.06 * (saleAmount - 100);
}
}
}
}
其他部分沒有詳細寫出來，這樣是可以跑的
1. 想問一般情況大家會在set裡面寫兩行以上的程式碼嗎？（我知道官方建議是一行
2. 會有什麼缺點嗎？
然後延伸部分是，當我今天要使用c#內建的排序方法去將銷售額排出順序時
會出現問題，因為Array.Sort(這邊必須放陣列)
但是我的sales[0].taxOwed和sales[1].taxOwed和其他的taxOwed並不是一整個陣列
所以我將原本建立Sales物件陣列(如上面敘述的程式)，改成
Sale sales = new Sale(10); (並且讓fiels都是陣列)
class中的程式碼長這樣
{
private int[] inventoryNum; //庫存數量
private double[] saleAmount; //銷售額
private double[] taxOwed;
public Sale(int size)
{
inventoryNum = new int[size];
saleAmount = new double[size];
taxOwed = new double[size];
}
public int[] InventoryNum
{
get { return inventoryNum; }
set { inventoryNum = value; }
}
public double[] SaleAmount
{
get { return saleAmount; }
set
{
saleAmount = value;
for (int i = 0; i < 10; i++)
{
if (saleAmount[i] <= 100)
{
taxOwed[i] = 0.08 * saleAmount[i];
}
else
{
taxOwed[i] = 0.08 * 100 + 0.06 * (saleAmount[i] - 100)
;
}
}
}
}
}
但是這樣的程式碼存入銷售額的時候，並不會去計算taxOwed
推測是set中不會進入for迴圈
想請問
3.是不是只能另外寫一個calc method去計算，然後在main method裡面呼叫？
4.關於這兩種陣列的寫法
Sale[] sales = new Sale[10]; //第一種：物件是陣列
Sale sales = new Sale(10); //第二種：實際上是fields是陣列
有沒有什麼差別？或是使用時機？優缺點等等
如果對內容有建議,麻煩鞭小力一點
謝謝各位大大
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 49.197.94.181
※ 文章網址: https://webptt.com/m.aspx?n=bbs/C_Sharp/M.1555581380.A.05B.html
1F：推 s4300026: 1.官方沒有建議只能一行吧？ set的用意是有些變數在存04/18 19:16
2F：→ s4300026: 檔前要預先處理，如果直接呼叫變數，則會有漏處理的情況04/18 19:16
已經回覆在下一篇
3F：推 jass970991: 你這樣寫啊不哪天class傳到其他地方 又決定修改這個cl04/18 21:35
4F：→ jass970991: ass 就越變越大坨 為什麼不好好的處理完再傳進去04/18 21:35
因為原本他這樣寫
The tax should be calculated whenever
the amount of the sale is set.
可能是我理解有誤，因為我原本以為要在設定好銷售額的同時就設定好taxowed，不能自
己
另外呼叫method這樣，所以才想說可不可以直接放在set裡
※ 編輯: Moneyyyyy (49.197.94.181), 04/18/2019 22:01:01
5F：推 jass970991: 那我會把他們兩個用tuple 或另設一個class來readonly04/18 22:07
6F：→ jass970991: ， 當你要設定一個值的時候會強迫另外一個也要賦值04/18 22:07
7F：→ jass970991: 同時做好一個calculator, 只要輸入 amount 自動產生04/18 22:07
8F：→ jass970991: 一組可以直接用的tax amount pair04/18 22:07
9F：→ jass970991: 把ds和邏輯分開真的非常重要04/18 22:08
10F：→ Moneyyyyy: 喔喔，學到一課，謝謝，我來查查tuple是什麼~04/18 22:37
11F：推 wezmag: 有個嚴重的缺點，錢請用decimal，用double會出問題04/19 19:28
已修正，謝謝
12F：→ forewero: 我自己寫除了預設值以外不會在model 做事，不是不行，但04/20 10:51
13F：→ forewero: 不直觀；權責沒分清後續接手很痛苦04/20 10:51
14F：推 Litfal: 當你的屬性值只和class內的資料相關時，就可以寫在model裡 04/20 11:43
15F：→ Litfal: 而你的稅務計算聽起來就很專業XD, 又有8%,100,6%這種與類04/20 11:47
16F：→ Litfal: 無關的魔術數字，建議另外拉一個Service或Helper class算04/20 11:47
請問，所謂的 model 是指getter, setter嗎？
17F：推 vi000246: 如果只是amount = salse x price之類簡單的邏輯04/21 23:40
18F：→ vi000246: 可以寫在set或get裡 複雜一點的算式就不建議了04/21 23:40
謝謝各位大大的建議
※ 編輯: Moneyyyyy (49.197.94.181), 04/22/2019 21:16:13