Shinodari
commented
2 months ago Transport Class Bug ที่ Child Class ลองทบทวน Access Modifier ของ field และ property อีกครั้ง
protected int id;
public int ID { get { return id; } }
protected string invoiceno;
public string InvoiceNo { get { return invoiceno; } set { invoiceno = value; } }
protected DateTime deliverydate;
public DateTime DeliveryDate { get { return deliverydate; } set { deliverydate = value; } }
protected Supplier supplier;
public Supplier Supplier { get { return supplier; } set { supplier = value; } }
protected List<Truck> trucklist;
public List<Truck> TruckList { get { return trucklist; } }
protected Storage storage;
public Storage Storage { get { return storage; } }
protected Dictionary <Product,int> quantityofproductlist;
public Dictionary<Product, int> QuantityOfProductList { get { return quantityofproductlist; }set { quantityofproductlist = value; } }field ต้องไม่มี Access Modifier Property ตาม Diagram ต้องเป็น protected
ตาม code ข้างล่างนี้
protected virtual string TableName { get { return GetType().Name.ToLower(); } }
protected virtual string TableName1 { get { return GetType().Name.ToLower()+"truck"; } }
protected virtual string TableName2 { get { return GetType().Name.ToLower() + "quantityofproductlist"; } }
protected virtual string ColumnName
{
get
{
string typeName = GetType().Name;
string tableName = typeName.Substring(0, 1).ToUpper() + typeName.Substring(1).ToLower() + "ID";
return tableName;
}
}คาดน่าจะเป็น local variable ดังนั้นไม่ควรใช้ protected modifier และ virtual ใช้สำหรับ Inheritance เรื่อง Polymorphism ดังนั้นไม่ควรใช้ เพราะจะมีผลกับ child class get method ไม่ควรใช้กับ variable ที่ไม่ได้เป็น property
แต่เมื่อดู code โดยละเอียด พบว่ามีความพยายามใช้ super class ทำหน้าที่แทน child class ซึ่งตามหลัก OOP ไม่ควรทำ
ตามความเห็นข้างต้น จึงพบว่า Create, Change, Remove ไม่ได้เป็น abstraction ตาม Diagram
คิดว่าจะน่าต้องทำใหม่
Hibino02
Conversate about Model Package