Question

I am using Visual Studio 2008 (C#). Is there any better way to write this code? I am a poor programmer.

#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

Answer 1

Nothing is wrong with your code.

I am a lazy programmer so I often use automatic properties in my code.

public string Unit { get; set; }