C# Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.?
In meinem aktuellen Code muss ich eine Programm schreiben in welchem es sich zum Employee-Management handelt. Alles funktioniert so weit, doch beim Editieren der Daten kommt folgender Fehler "siehe Überschrift".
private void button3_Click(object sender, EventArgs e)
{
if (EmpIDTB.Text == "" || EmpNameTB.Text == "" || EmpPhoneTB.Text == "" || EmpAddTB.Text == "")
{
MessageBox.Show("Missing Information");
}
else
{
try
{
Con.Open();
string query = "update EmployeeTbl set Empname='" + EmpNameTB.Text + "',empadd='" + EmpAddTB.Text + "',emppos='"+EmpPosTB.SelectedItem.ToString()+"',empdob= '"+EmpDateTB.Value.Date+"',empphone='"+EmpPhoneTB.Text+"',empedu='"+EmpEduTB.SelectedItem.ToString()+"',empgen='"+EmpGenderTB.SelectedItem.ToString()+"' where Empid='"+EmpIDTB.Text+"';";
SqlCommand cmd = new SqlCommand(query, Con);
cmd.ExecuteNonQuery();
MessageBox.Show("Employee Update Successfully");
Con.Close();
populate();
}catch (Exception Ex)
{
MessageBox.Show(Ex.Message);
}
in dem Code ist der abschnitt mit "Edit". Ich verstehe nicht ganz woran es scheitert. Würde mich um Hilfen freuen.
3 Antworten
Ich würde den Fehler erstmal eingrenzen, in dem ich mit
try {
Con.Open();
} catch (Exception Ex) {
MessageBox.Show(Ex.Message);
}
Anfange und dann Zeile für Zeile hinzufüge und gucke, wo genau der Fehler ausgelöst wird.
Ich vermute den Fehler bei Objekt Con oder dem Funktionsaufruf populate().
Schon, aber manchmal entstehen Fehler erst im Zusammenspiel von 2 Zeilen. Wenn der Fehler also nicht direkt ersichtlich ist, ist das eine bewährte Methode zum Ermitteln von Errors. Zumal es abhängig davon ist, was der Debugger genau ausgibt.
Wenn es so offensichtlich ist für den Debugger, hätte die Exception e ja auch einfach beinhalten können: "ToString() not possible because EmpPosTB.SelectedItem is null!"
Also ich habe es jetzt mal schrittweise gemacht und erst bei ""',emppos='" + EmpPosTB.SelectedItem.ToString() +" popt der fehler auf.
Dann scheint die Methode ToString() nicht zu funktionieren, da EmpPosTB.SelectedItem nicht existiert bzw. null ist.
Unabhängig von dem Fehler:
Dein Code ist für SQL-Injections anfällig. Benutze die sicherlich vorhandenen Funktionen um Variablen anzugeben, so kann jeder der eines dieser Felder kontrolliert deine Datenbank mit den Rechten des Programmes verändern.
In welcher Zeile genau entsteht der Fehler?
Das passiert immer wenn ein Objekt nicht existiert (null ist) und du mit
objekt.methode()
darauf zugreifen willst.
Also ich habe es jetzt mal schrittweise gemacht und erst bei ""',emppos='" + EmpPosTB.SelectedItem.ToString() +" popt der fehler auf.
Eigentlich nicht.. Hab VS neu gestartet ging immer noch nicht, hab die Zeilen gelöscht nochmals 1zu1 hinzugefügt und auf einmal ging es... ich verstehe es manchmal nicht haha.. danke dir trotzdem
Beim nächsten mal einfach im Debugmodus starten dann kannst du zum Absturzzeitpunkt genau nachsehen welches objekt null ist
Den Debugger zu benutzen ist da doch viel einfacher 🙃