[C#] Datagrid probleem

[tha_crazy]

Hey.
Ik ben nu bezig met een project waarbij ik gebruik moet maken van een datagrid.
Dit werkt in combinatie van C# met een Access Datagrid.
Nu werkt echter als als ik een rij toevoeg het niet
ik krijg dan deze error "an object reference is not set to an instance of an object"

In de database worden 3 dingen opgeslagen
- customer ID (cid)
- Project description (Description)
- Status (status)

Status kan 2 waarden hebben:
- In progress
- Closed

Ik heb zelf ook al flink rond zitten zoeken maar ik kan er niks op vinden
heeft iemand een idee ?
dit is de code:

C#:

         private void dgvProjects_UserAddedRow(object sender, DataGridViewRowEventArgs e)
        {
           
            Companys tmp = cbxCompanyName.SelectedItem as Companys;
            
            try
            {
                string qryInsertSecValues = "INSERT INTO Projects (Cid, Description, Status) VALUES (" + tmp.ID.ToString() + ", '" + e.Row.Cells[0].Value.ToString() + "', '" + e.Row.Cells[1].Value.ToString() + "')";
                OdbcCommand myodbccommand = new OdbcCommand(qryInsertSecValues);
                myodbccommand.Connection = TabStripApp.Singleton.Connector.Connection;
                myodbccommand.ExecuteNonQuery();
            }
            catch (Exception Ex)
            {
                MessageBox.Show(Ex.ToString());
            }
            
            
        }

Ik hoop voldoende informatie te hebben verschaft
zo niet, zeg het even

Groetjes, Vincent

[Boss]

Op welke regel krijg je de foutmelding?

Klinkt in ieder geval alsof de myodbccommand niet goed aangemaakt wordt. Als je daarna de .connection set krijg je een fout.

Om de fout beter te vinden misschien even de try..catch commenten?

[tha_crazy]

Ik heb er even een screenshot van gemaakt
http://www.ugf-hq.com/files/error.JPG

Maar de odbccommand zou moeten werken
Wij hebben dezelvde methode ook al gebruikt bij andere delen van de applicatie (user management enzo) en daar werkt hij prima

En ik zal even proberen zonder de catch method

[Creepy]

Blijft nog steeds de vraag: op welke regel in de code die je hierboven geeft treedt die melding nu eigenlijk op? In de afbeelding die je geeft staat regel 246 maar dat zegt ons natuurlijk helemaal niks.

En wat heb je nu zelf al geprobeerd om dit op te lossen? "Ik heb al gezocht" zegt ons alweer helemaal niks Al aan het debuggen geslagen?

[ Voor 26% gewijzigd door Creepy op 12-09-2007 11:55 ]

[Depress]

Ik denk dat het fout gaat op deze regel:

Companys tmp = cbxCompanyName.SelectedItem as Companys;

Het selected item kan niet worden gekopierd omdat het Companys type niet compatible is met het type selectedItem.

[L-VIS]

Ik gok op regel acht in jouw voorbeeld. Maar waarom gebruik je geen sqlParameters?

[tha_crazy]

Creepy schreef op woensdag 12 september 2007 @ 11:54:
Blijft nog steeds de vraag: op welke regel in de code die je hierboven geeft treedt die melding nu eigenlijk op? In de afbeelding die je geeft staat regel 246 maar dat zegt ons natuurlijk helemaal niks.

En wat heb je nu zelf al geprobeerd om dit op te lossen? "Ik heb al gezocht" zegt ons alweer helemaal niks Al aan het debuggen geslagen?

hij begint bij mij met de void op regel 239.
de query staat op 246.

Ik ben net even aan het debuggen geweest ja en daarop kwam uit dat e.Row.Cells[0].Value.ToString() NULL was en e.Row.Cells[1].Value.ToString() ook NULL was.

En voor zoeken
ik heb google afgespeurt op alles wat met datagridview en useraddedrows te maken heeft maar daar ben ik op niks uitgekomen

L-VIS schreef op woensdag 12 september 2007 @ 11:57:
Ik gok op regel acht in jouw voorbeeld. Maar waarom gebruik je geen sqlParameters?

Hoe bedoel je sqlParameters ?

[L-VIS]

Hier meer info over SqlParameter. Zo ben je minder gevoelig voor Sql injectie. Misschien is dit minder gevaarlijk bij een datagrid, maar het is een good practice om je aan te leren dit te gebruiken.

Maar gaat dat event niet af als er een nieuwe lege rij wordt aangemaakt?

[Niemand_Anders]

ik heb google afgespeurt op alles wat met datagridview en useraddedrows te maken heeft maar daar ben ik op niks uitgekomen

Op zich logisch, want deze fout heeft ook niets met een datagrid of database te maken. Zoals de foutmelding al aangeeft probeer je een property of methode aan te roepen van een object welke niet geinitialiseerd is.

C#:

string a = null;
int length = a.Length; //raises NullReferenceException

Als Cells[0] geen value heeft (null), dan kun je daar ook geen string van maken.
Wat je kunt doen is statements zoals

C#:

string value = (Cells[0].Value == null ? string.Empty : Cells[0].Value.ToString());

gebruiken.

[sig69]

Wederom een mooie reden om geen generieke exceptions af te vangen:

C#:

            catch (Exception Ex) 
            { 
                MessageBox.Show(Ex.ToString()); 
            } 

Als deze er niet had gestaan had Visual Studio (ik neem toch aan dat je die gebruikt) keurig naar de juiste regel gesprongen en waren de eerste vijf replies om te achterhalen waar het nu precies fout gaat niet nodig geweest.

[tha_crazy]

Ook zonder de catch kwam hij met die error zoals ik hierboven heb beschreven.

Nu zijn we echter met een andere methode bezig en dat is via validating.
Echter voegt die nu continu nieuwe rijen bij.

C#:

        private void dgvProjects_RowValidated(object sender, DataGridViewCellEventArgs e)
        {
            DataGridViewRow row = dgvProjects.Rows[e.RowIndex]; 
            if (row.Tag == null || (bool)row.Tag == false)
                OnInsert(row);
            else
                OnUpdate(row);
        }

        private void dgvProjects_RowValidating(object sender, DataGridViewCellCancelEventArgs e)
        {
            DataGridViewRow row = dgvProjects.Rows[e.RowIndex];
            if (row.IsNewRow == true) return;

            e.Cancel = false;
            foreach (DataGridViewCell cell in row.Cells)
                if (cell.Value == null)
                {
                    e.Cancel = true;
                    break;
                }
        }

        private void OnInsert(DataGridViewRow e)
        {
            Companys tmp = cbxCompanyName.SelectedItem as Companys;
            string qryInsertSecValues = "INSERT INTO Projects (Cid, Description, Status) VALUES (" + tmp.ID.ToString() + ", '" + e.Cells[0].Value.ToString() + "', '" + e.Cells[1].Value.ToString() + "')";
            OdbcCommand myodbccommand = new OdbcCommand(qryInsertSecValues);
            myodbccommand.Connection = TabStripApp.Singleton.Connector.Connection;
            myodbccommand.ExecuteNonQuery();
        }

        private void OnUpdate(DataGridViewRow e)
        {
            MessageBox.Show("update");
        }

[TeeDee]

Ja, en nu? Krijg je een andere foutmelding of blijft de eerste foutmelding naar voren komen?
Stap eens met de Debugger door je

C#:

private void OnInsert(DataGridViewRow e) 

heen en bekijk (ik gok) je hele SqlQuery eens?

[ Voor 46% gewijzigd door TeeDee op 13-09-2007 15:13 ]

[Creepy]

Wat in mijn eerste post hier ook stond dus:

En wat heb je nu zelf al geprobeerd om dit op te lossen? "Ik heb al gezocht" zegt ons alweer helemaal niks Al aan het debuggen geslagen?

Een vraag dumpen zonder aan te geven wat je zelf al hebt gedaan is hier eigenlijjk not done. We verwachten hier dat je minstens zelf al aan het debuggen bent geslagen (wat TeeDee ook aangeeft).