[問題] 新手想問關於get; set;和陣列

看板C_Sharp (C#)作者 (擋我者死)時間6年前 (2019/04/18 17:56), 6年前編輯推噓6(6012)
留言18則, 7人參與, 6年前最新討論串1/2 (看更多)
我需要先有十個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://www.ptt.cc/bbs/C_Sharp/M.1555581380.A.05B.html

04/18 19:16, 6年前 , 1F
1.官方沒有建議只能一行吧? set的用意是有些變數在存
04/18 19:16, 1F

04/18 19:16, 6年前 , 2F
檔前要預先處理,如果直接呼叫變數,則會有漏處理的情況
04/18 19:16, 2F
已經回覆在下一篇

04/18 21:35, 6年前 , 3F
你這樣寫啊不哪天class傳到其他地方 又決定修改這個cl
04/18 21:35, 3F

04/18 21:35, 6年前 , 4F
ass 就越變越大坨 為什麼不好好的處理完再傳進去
04/18 21:35, 4F
因為原本他這樣寫 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

04/18 22:07, 6年前 , 5F
那我會把他們兩個用tuple 或另設一個class來readonly
04/18 22:07, 5F

04/18 22:07, 6年前 , 6F
, 當你要設定一個值的時候會強迫另外一個也要賦值
04/18 22:07, 6F

04/18 22:07, 6年前 , 7F
同時做好一個calculator, 只要輸入 amount 自動產生
04/18 22:07, 7F

04/18 22:07, 6年前 , 8F
一組可以直接用的tax amount pair
04/18 22:07, 8F

04/18 22:08, 6年前 , 9F
把ds和邏輯分開真的非常重要
04/18 22:08, 9F

04/18 22:37, 6年前 , 10F
喔喔,學到一課,謝謝,我來查查tuple是什麼~
04/18 22:37, 10F

04/19 19:28, 6年前 , 11F
有個嚴重的缺點,錢請用decimal,用double會出問題
04/19 19:28, 11F
已修正,謝謝

04/20 10:51, 6年前 , 12F
我自己寫除了預設值以外不會在model 做事,不是不行,但
04/20 10:51, 12F

04/20 10:51, 6年前 , 13F
不直觀;權責沒分清後續接手很痛苦
04/20 10:51, 13F

04/20 11:43, 6年前 , 14F
當你的屬性值只和class內的資料相關時,就可以寫在model裡
04/20 11:43, 14F

04/20 11:47, 6年前 , 15F
而你的稅務計算聽起來就很專業XD, 又有8%,100,6%這種與類
04/20 11:47, 15F

04/20 11:47, 6年前 , 16F
無關的魔術數字,建議另外拉一個Service或Helper class算
04/20 11:47, 16F
請問,所謂的 model 是指getter, setter嗎?

04/21 23:40, 6年前 , 17F
如果只是amount = salse x price之類簡單的邏輯
04/21 23:40, 17F

04/21 23:40, 6年前 , 18F
可以寫在set或get裡 複雜一點的算式就不建議了
04/21 23:40, 18F
謝謝各位大大的建議 ※ 編輯: Moneyyyyy (49.197.94.181), 04/22/2019 21:16:13
文章代碼(AID): #1Sk4d41R (C_Sharp)
文章代碼(AID): #1Sk4d41R (C_Sharp)