2010-10-30 50 views
2

我正在使用Visual Studio 2008(C#)。有没有更好的方法来编写这段代码?我是一个可怜的程序员。更好的方式来编写这个属性和构造函数代码?

#region Properties 
public string ItemCode 
{ 
    get { return _itemCode; } 
    set { _itemCode = value; } 
} 

public string ItemName 
{ 
    get { return _itemName; } 
    set { _itemName = value; } 
} 

public decimal? StockInHand 
{ 
    get { return _stockInHand; } 
    set { _stockInHand = value; } 
} 

public decimal? AlertLevelQty 
{ 
    get { return _alertLevelQty; } 
    set { _alertLevelQty = value; } 
} 

public string Unit 
{ 
    get { return _unit; } 
    set { _unit = value; } 
} 

public decimal? Discount 
{ 
    get { return _discount; } 
    set { _discount = value; } 
} 

public string WhetherInPercent 
{ 
    get { return _whetherInPercent; } 
    set { _whetherInPercent = value; } 
} 

public int CategoryID 
{ 
    get { return _categoryID; } 
    set { _categoryID = value; } 
} 
#endregion 

#region Constructors 
public ItemMaster() 
{ } 

public ItemMaster(string argItemName, string argItemCode, decimal? argStockInHand, decimal? argAlertLevelQty, string argUnit, decimal? argDiscount, string argWhetherInPercent, int argCategoryID) 
{ 
    this.ItemName = argItemName; 
    this.ItemCode = argItemCode; 
    this.StockInHand = argStockInHand; 
    this.AlertLevelQty = argAlertLevelQty; 
    this.Unit = argUnit; 
    this.Discount = argDiscount; 
    this.WhetherInPercent = argWhetherInPercent; 
    this.CategoryID = argCategoryID; 
} 
#endregion 

回答

2

没有什么是你的代码错误。

我是一个懒惰的程序员,所以我经常在我的代码中使用自动属性。

public string Unit { get; set; } 
+0

可能没有任何问题,但只是想知道如何给它专业的触摸。 – RKh 2010-10-30 17:36:22

+0

我更喜欢使用自动方法。它写得更干净更快。国际海事组织,尽可能保持简单。 – Kamyar 2010-10-30 17:56:30

+0

@Kamyar:这只是一个偏好问题。在某些情况下,自动属性根本无法使用。 – Lorenzo 2010-10-30 18:00:15

2

所有在你的榜样的属性可以用“自动属性”被写,例如:

public int CategoryId { get; set; } 
+0

对构造函数没有什么可说的。 – 2010-10-30 17:27:57

+0

如何改进构造函数代码? – RKh 2010-10-30 17:30:54

+2

我更喜欢'public ItemMaster(string ItemName,...){this.ItemName = ItemName; ''。不需要预先加入'arg','this.'就可以改变。但是如果没有真正了解设计,就没有太多可以说的东西了,它有许多相关的参数,但这本身并不意味着太多。 – 2010-10-30 17:30:58

0

如果没有需要修改的属性诠释您的get/set,你同`公共字符串ItemCode {获得;设置;}
但是,如果你必须改变这些值,你必须为它定义一个私有变量,并像你在问题中那样定义get/set。

相关问题