I have code which works pretty well. I just need some of your opinions on how to write it better with fewer line of code. I want to use a ternary operator but I couldn't make it work so I did an if else
instead.
Basically, I want to insert a price on my database. The price column is a nullable decimal. Even in my C# code, it's a nullable decimal as well.
if (items.Price == null)
{
SqlParameter[] p =
{
new SqlParameter("@purchaseorder", items.PurchaseOrderItems),
new SqlParameter("@modelnumber", items.ModelNumber),
new SqlParameter("@quantity", items.Quantity),
new SqlParameter("@Price", DBNull.Value),
new SqlParameter("@Description", items.Description),
new SqlParameter("@OrderNumber", items.OrderNumber),
new SqlParameter("@Unit", items.Unit)
};
i = Dal.ExecuteNonQuery(sql,p);
}
else
{
SqlParameter[] p =
{
new SqlParameter("@purchaseorder", items.PurchaseOrderItems),
new SqlParameter("@modelnumber", items.ModelNumber),
new SqlParameter("@quantity", items.Quantity),
new SqlParameter("@Price", items.Price),
new SqlParameter("@Description", items.Description),
new SqlParameter("@OrderNumber", items.OrderNumber),
new SqlParameter("@Unit", items.Unit)
};
i = Dal.ExecuteNonQuery(sql,p);
}
return i;
Can this be written without the else
block?