if (selected == NULL) {
return;
}
deviq.com
C#:if (selected == NULL) { return; }Guard Clause | DevIQ
Complexity in code makes it harder to understand what the code is doing. The smallest unit of our code tends to be the function or method.
private void button1_Click(object sender, EventArgs e)
{
con.Open();
SqlCommand komutguncelle=new SqlCommand("Update modeller set mfiyat=@a1 where id=@a2",con);
komutguncelle.Parameters.AddWithValue("@a1",textBox4.Text);
komutguncelle.Parameters.AddWithValue("@a2",textBox1.Text);
komutguncelle.ExecuteNonQuery();
con.Close();
MessageBox.Show("Fiyat bilgisi güncellendi");
}
Bir yerde sorgularına m_ad ve m_fiyat yazmışsın mad ve mfiyat yerine.
Guard clause kullan demiştim, null olma ihtimaline karşı. Kullanmamışsın. NULL dönmüş bir şey. Sende onu kullanmaya çalışmışsın. Muhtemelen SelectedItem null geliyor.
private void modelbox_SelectedIndexChanged(object sender, EventArgs e)
{
baglanti.Open();
SqlCommand komut = new SqlCommand("select mfiyat from modeller where mad ="+listBox1.SelectedItem.ToString(),baglanti);
SqlDataReader read=komut.ExecuteReader();
while (read.Read())
{
listBox1.Items.Add(read[2]);
}
baglanti.Close();
}Olur. Google kullanmayı öğrenmelisin. Sana 2 döküman gönderdim, kod örneği gönderdim, üzerine bir rehber gönderdim konuna yardımcı olması için. Birazda senin araştırma yapman gerekiyor bu noktadan sonra.
// Eğer seçili item null'sa fonksiyondan
// çıkacak.
if(listBox1.SelectedItem == null) {
return;
}
baglanti.Open();
SqlCommand command = new SqlCommand($"select mfiyat from modeller where mad = '{listBox1.SelectedItem.ToString()}'", baglanti);
SqlDataReader sqlDataReader = command.ExecuteReader();
Bende veritabanından fiyatları çektim zaten.
