Zeile in Variable speichern mittels DataReader

Vegeto · Beiträge: 262

Hallo, habe nun dieses Thema geöffnet, weil es in dem anderen thema eine falsche überschrift hatte.(Tipp von TH69)

Okay... Also habe nun lange DataReader Objekt gelesen.
Doch ich schaffe es einfach nicht :/

Also ich möchte aus einer Tabelle(aus der DB, wird aus zwei Tabellen zusammen gejoint/leftjoin, mit NULL-Werten), jetzt möchte ich jede Zeile in eine Variable stecken. Doch einige Zeilen haben Zellen, die noch nicht gefüllt sind.

Jetzt habe ich das so gemacht:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:

        private void holen_btn_Click(object sender, EventArgs e)
        {
            SqlConnection Conn = new SqlConnection(conStr);
            Conn.Open();
            SqlCommand cmd = new SqlCommand();
            cmd.CommandText = SelectStr; 
            cmd.Connection = Conn;

            SqlDataReader dr = cmd.ExecuteReader(); 
                                  
            while (dr.Read())
            {
                int AN = dr.GetInt32(0);
                string Name = dr.GetString(1);
                //diese drei spalten haben viele Null-Values, weil sie noch nicht gefüllt wurden, doch eine Zeile hat in jeder Zeile was stehen.
                int Bank = dr.GetInt32(2);
                string BName = dr.GetString(3);
                int KNummer = dr.GetInt32(4);  

                string Ausgabetext = String.Format("{0},{1},{2}.{3},{4}",AN,Name,Bank,BName,KNummer);
                MessageBox.Show(Ausgabetext, "Korrekt");
            }            
            
            dr.Close();
            Conn.Close();
        }

Der Fehler kommt hier:

markieren

C#-Quelltext

1:	int AdressNummer = dr.GetInt32(0); //Die angegebene Umwandlung ist ungültig.

Ich weiß aber nicht woran das liegt, denn die erste Spalte besteht ausschließlich nur aus INT werten.

Ich weiß eigentlich will ich die ganze Zeile in EINE variable(vllt. List-Objekt) aber zunächst einmal will ich dieses Problem schaffen, denn ich denke es wird mir später weiter helfen.

LG

Ralf Jansen · Verfasst: Do 13.09.12 14:04

Der Wert ist entweder gar kein Int32 oder NULL. In beiden Fällen wird dein Code knallen. Der DataReader hat für letzteren Fall eine IsDBNull Funktion mit der du das testen kannst. Im anderen Fall solltest du dein Spaltendefinition checken ob du den die Spalte im SQL Server auch als Integer angelegt hast.

Vegeto · Beiträge: 262

Also Null ist es nicht.

habe mal eben in die Datenbank geguckt und du hast recht, er hat den Wert nvarchar(10).

Habe sofort den Code geändert in:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:

while (dr.Read())
            {
                int AN = Convert.ToInt32(dr[0]);
                string Name = Convert.ToString(dr[1]);
                //Diese drei spalten haben null values, habe bei Bankleitzahl versucht die Null Values zu bearbeiten doch dann kommen zwei fehler.
                //int Bank = Convert.ToInt32(dr[2]);
                string BName = Convert.ToString(dr[3]);
                int KNummer = Convert.ToInt32(dr[4]);
                if (!dr.IsDBNull(2))               
                    int BankLeitzahl = Convert.ToInt32(dr[2]);  // hier sagt er: Eine eingebettete Anweisung kann keine Deklaration und keine Anweisung mit Bezeichnung sein.                 

                string Ausgabetext = String.Format("{0},{1},{2},{3},{4}",AN,Name,Bank,BName,KNummer); // hier sagt er: Verwendung der nicht zugewiesenen lokalen Variablen "BankLeitzahl"

                MessageBox.Show(Ausgabetext, "Korrekt");
            }

Es tauchen diese beiden fehler auf :/
habe auch versucht die if anweisung in klammern ( { } ) zu setzen, funktoniert auch nicht, habe versucht

markieren

C#-Quelltext

1:
2:
3:

int Bank; 
if(!dr.IsDBNull(2))
Bank = Convert.ToInt32(dr[2]);  // Verwendung der nicht zugewiesenen lokalen Variablen "BankLeitzahl"...genau wie oben

hat auch nichts gebracht :/

LG

Dankesehr

Ralf Jansen · Verfasst: Do 13.09.12 14:38

Du must Bankleitzahl mit irgendwas initialisieren da ein int nicht null sein kann. Da du Bankleitzahl aber nur in dem Zweig des If setzt gibt es wege bis zur Verwendung von Bankleitzahl in dem nie ein Wert Bankleitzahl zugewiesen wird. Das darf nicht sein. Variablen müssen immer einen zugewiesen Wert haben bevor sie verwendet werden können.

Denk dir also eine Wert aus der in Bankleitzahl stehen soll wenn der Wert in der Datenbank dafür null wäre. Alternativ nimm einen Nullable int (also int?). Dann bist du aber im folgenden bei der Verwendung dieser Variablen immer gezwungen zu prüfen ob der nicht auch null ist. Ähnlich wie jetzt schon beim DataReader.

Vegeto · Beiträge: 262

Verstehe..

habe das demnach so gelöst:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:

            while (dr.Read())
            {
                int AN = Convert.ToInt32(dr[0]);
                string Name = Convert.ToString(dr[1]);
                int Bank = 0;
                string BName = "";
                int KNummer = 0;

                if (!dr.IsDBNull(2))               
                     Bank = Convert.ToInt32(dr[2]);               
                if (!dr.IsDBNull(3))
                    BName = Convert.ToString(dr[3]);             
                if (!dr.IsDBNull(4))
                    KNummer = Convert.ToInt32(dr[4]);
                 

                string Ausgabetext = String.Format("{0},{1},{2},{3},{4}",AN,Name,Bank,BName,KNummer);
                MessageBox.Show(Ausgabetext, "Korrekt");
            }

So zeigt er in der MessageBox, das was ich will nur halt die Null-Values ersetzt er mit 0, aber die habe ich ja zugewiesen. Das mit int? habe ich nicht hinbekommen :/ aber auch wenn sie sie mit If Abfange zeigt er ja irgendwo ja null an, denn wenn dort ein Null-Value vorhanden ist mit ich ihn ja mit IF ersetzen.

Okay Kommen wir zu part 2 (:
Jetzt will ich die ganze zeile in ein List-Objekt stecken oder ist ein Array besser dafür geeignet ? ? ?(Also ich würde sagen das einzelnen Variablen besser geeignet sind oder ? ?)
Denn ich möchte dieses Verfahren auf zwei typen benutzen, eine ist eine Tabelle und die andere eine Excel Datei, jz möchte ich die Werte in eine Variable packen und später in eine neue Datenbank in eine Zeile zusammen führen lassen.

Hoffe versteht was ich meine (:

LG

Ralf Jansen · Verfasst: Do 13.09.12 15:29

Schreibe dir eine Klasse mit deinen bisherigen Variablen als Properties. In deiner DataReader.Read Schleife erzeugst du eben dann diese Klasse weißt die Werte den Properties zu anstatt den Variablen und packst die Klasse dann in eine Liste. Also z.B. eine List<DeineNochAuszudenkendeKlasse>.

Vegeto · Beiträge: 262

Hmm...

So in etwa?

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:

//Die Klasse
    public class Zeile
    {
        public int AN { get; set; }
        public string Name { get; set; }
        public int Bank { get; set; }
        public string BName { get; set; }
        public int KNummer { get; set; }
    }

//Klasse wird instanziert
Zeile Zeilen = new Zeile();

        private void holen_btn_Click(object sender, EventArgs e)
        {
            //...Code

            SqlDataReader dr = cmd.ExecuteReader();
                         
            while (dr.Read())
            {
                Zeilen.AN = Convert.ToInt32(dr[0]);
                Zeilen.Name = Convert.ToString(dr[1]);
                Zeilen.Bank = Convert.ToInt32(dr[2]);
                Zeilen.BName = Convert.ToString(dr[3]);
                Zeilen.KNummer = Convert.ToInt32(dr[4]);

            } 
            List<Zeile> zeilen = new List<Zeile>();
          }

Ist das so richtig, ich weiß nciht ob ich überprüfen kann ob die Zeilen so neben einander stehen?

Lg

Ralf Jansen · Verfasst: Do 13.09.12 15:44

Klasse erzeugen gehört mit in die Schleife. Genauso wie das hinzufügen zu Liste.
Damit ist dann auch klar das du die Liste vorher erzeugen mußt und nicht nachher.

Zitat:

Ist das so richtig, ich weiß nciht ob ich überprüfen kann ob die Zeilen so neben einander stehen?

Was heißt das? Die Daten sind in der Liste erstmal in der Reihenfolge in der du die sie hinzugefügt hast. Wenn du die zu irgendeinem Zeitpunkt in einer bestimmten Reihenfolge brauchst solltest du die dann sortieren.

Vegeto · Beiträge: 262

Jetzt komme ich garnicht mehr mit :/

habe nun dies getan:

markieren

C#-Quelltext

1:
2:
3:
4:
5:

 while (dr.Read())
            {
                List<Zeile> Zeilen = new List<Zeile>();
                Zeilen.Add(dr["AN"];  //Fehler:Die beste Übereinstimmung für die überladene System.Collections.Generic.List<ClassLibrary1.Zeile>.Add(ClassLibrary1.Zeile)-Methode hat einige ungültige Argumente.
            }

Komisch...Hätte nie gedacht das es so schwierig wird :/
Danke bis hierhin.

Was muss ich den nun tun? Und was meinst du mit sortieren, sie sind doch schon in der Klasse sortiert, bekomme ich keine Zeile in dieser Reihnfolge?

LG

Ralf Jansen · Verfasst: Do 13.09.12 16:02

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:

List<Zeile> zeilen = new List<Zeile>();
while (dr.Read())
{
   Zeile zeile = new Zeile();
   // hier jetzt Daten in zeile einfügen
   zeilen.Add(zeile);
}

Zitat:

Und was meinst du mit sortieren, sie sind doch schon in der Klasse sortiert, bekomme ich keine Zeile in dieser Reihnfolge?

Die Daten sind in der Liste genau in der Reihenfolge wie du sie aus der DB holst. Also wenn du einen Order By in deinem SQL hast dann genau so. Sonst so wie die Datenbank das dann zufällig liefert.
Und wenn du sie später irgendwo anders sortiert brauchst dann mußt du halt die zeilen Liste sortieren.

Wenn dich das verwirrt ignorier das erstmal. Du hattest was von Zeilen ~nebeneinander~ geschrieben. Und natürlich haben die Zeilen irgend eine Reihenfolge solange du aber nirgendwo eine eindeutig definierst dann ist die halt potentiell zufällig.

Vegeto · Beiträge: 262

Ahhh Dankeschön

(Danke habe bis hierhin eine menge gelernt

, bin in der nächsten woche in der berufschule, hoffe das ich dort genau so viel lerne

nebenbei werde ich auch in sämtlichen foren und büchern lesen

)

So:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:

            List<Zeile> zeilen = new List<Zeile>();
            while (dr.Read())
            {
                Zeile Zeilen = new Zeile();
                Zeilen.AN = Convert.ToInt32(dr[0]);
                Zeilen.Name = Convert.ToString(dr[1]);
                Zeilen.Bank = Convert.ToInt32(dr[2]);  //Fehler: Ein Objekt kann nicht von DBNull in andere Typen umgewandelt werden.
                Zeilen.BName = Convert.ToString(dr[3]);
                Zeilen.KNummer = Convert.ToInt32(dr[4]);
                row.Add(Zeilen);
            } 
            

            dataGridView1.DataSource = zeilen.ToList();

Doch nun muss ich irgendwie die Null-Values bearbeiten, habe ein try-Catch-Block drum herum gemacht, doch hat auch nichts gebracht, dann zeigt er dauerhaft den Fehler an: "Ein Objekt kann nicht von DBNull in andere Typen umgewandelt werden."

ich möchte das er durchgeht und wenn NULL-values vorhanden sind, soll er eine Meldung geben, es fehlen Daten, doch trotzdem soll er die Zeile anzeigen die keine Null-Value haben.

Wie kann ich das festhalten, wieder mit einer if-anweisung?

Lg

Ralf Jansen · Verfasst: Do 13.09.12 16:23

.. und dem bekannten Test mit IfDBNull.

Vegeto · Beiträge: 262

Ralf Jansen hat folgendes geschrieben :

.. und dem bekannten Test mit IfDBNull.

Weiß leider nicht wie ich diesen Aufbauen soll.

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:

if(DBNull..//weiß nicht mehr weiter :/
while (dr.Read())
            {
                Zeile Zeilen = new Zeile();
                Zeilen.AN = Convert.ToInt32(dr[0]);
                Zeilen.Name = Convert.ToString(dr[1]);
                Zeilen.Bank = Convert.ToInt32(dr[2]);
                Zeilen.BName = Convert.ToString(dr[3]);
                Zeilen.KNummer = Convert.ToInt32(dr[4]);
                row.Add(Zeilen);
            }

gibts du mir ein TPP

Lg

Ralf Jansen · Verfasst: Do 13.09.12 16:28

Du hast es bei direkten Variablen doch auch hinbekommen? Das ist jetzt nicht anders.

Vegeto · Beiträge: 262

Also muss ich das auch für jeden Zeile einzelnen machen, ich kann nicht einfach vor die While Schleife ein If setzen und der soll dann alle Zeilen überprüfen?

Edit:
SO?

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:

while (dr.Read())
            {
                Zeile Zeilen = new Zeile();
                
                Zeilen.AN = Convert.ToInt32(dr[0]);
                Zeilen.Name = Convert.ToString(dr[1]);
                if(!dr.IsDBNull(2))
                Zeilen.Bank = Convert.ToInt32(dr[2]);
                if (!dr.IsDBNull(3))
                Zeilen.BName = Convert.ToString(dr[3]);
                if (!dr.IsDBNull(4))
                Zeilen.KNummer = Convert.ToInt32(dr[4]);
                row.Add(Zeilen);
            } 
            
            dr.Close();
            ApConn.Close();
            dataGridView1.DataSource = row.ToList();

So zeigt er aber jede zeile an und ersetzt die Null-values mit Null oder Leer String...
Doch er soll dann nur die vollständige zeile anzeigen?

Lg

Ralf Jansen · Verfasst: Do 13.09.12 16:37

Natürlich. Wenn dein if um alles fehlschlägt wurde ja gar keine Property gesetzt und nicht nur die die nicht zuweisbar (weil null) sind. Das würde nur Sinn machen wenn du gar keine Zeile anlegen willst wenn eine der Properties der Zeile nicht ok wären.

Vegeto · Beiträge: 262

Okay Dankeschön...

Danke bis hierhin habe sehr viel gelernt und geschafft, bin gerade zuhause angekommen, werde morgen ein feedback mit meinem arbeitskoleggen halten.. melde mich dann bei euch...

DANKE an alle...

Lg

Vegeto · Beiträge: 262

Guten Morgen..

Ich bräuchte mal wieder euere Hle (:

Mein Code:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:

            List<Zeile> row = new List<Zeile>();
            
            while (dr.Read())
            {
                Zeile Zeilen = new Zeile();               
                
                Zeilen.KName = Convert.ToString(dr[1]);
                if(!dr.IsDBNull(2))
                Zeilen.Bank = Convert.ToInt32(dr[2]);
                if (!dr.IsDBNull(3))
                Zeilen.BName = Convert.ToString(dr[3]);
                if (!dr.IsDBNull(4))
                Zeilen.KNummer = Convert.ToInt32(dr[4]);

                row.Add(Zeilen);


            }            
            dr.Close();
            ApConn.Close();


// das ist der Code von meiner ClassLibary, zum hinzufügen der datensätze in DB
public void InsertBank(Zeile row)
        {
            string anweisung = String.Format("INSERT INTO Zahlung" + 
                "(KName, Bank, BName, KNummer) Values" + 
                "('{0}','{1}','{2}','{3}')", row.KName ,row.Bank, row.BName, row.KNummer);

            using (SqlCommand cmd = new SqlCommand(anweisung, this.ZaConn))
            {
                cmd.ExecuteNonQuery();
            }

        }

Doch irgendwie bekomme ich das nicht hin das er die einfügt.

Habe auch das hier probiert:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:

Zeile Zeilen = new Zeile {KName = Convert.ToString(dr[1]),
                if(!dr.IsDBNull(2))
                Zeilen.Bank = Convert.ToInt32(dr[2]),
                if (!dr.IsDBNull(3))
                Zeilen.BName = Convert.ToString(dr[3]),
                if (!dr.IsDBNull(4))
                Zeilen.KNummer = Convert.ToInt32(dr[4])};

zahlung.Insert(Zeilen); //Methode aus meiner ClassLibary... Doch funktoniert nicht da kein IF benutzt werden darf.

Doch hier darf ich kein if verwenden :/

Weiß jmd bescheid, wie ich den inhalt von List wieder geben kann?
Dachte es würde mit List[1] <--- für den ersten punkt der liste funktionieren, doch es funktioniert nicht

Das problem ist ja, dachte mir dann ich stecke sie in einzelnen Variablen, doch die Variablen sind auch nur in der While-Schleife vorhanden, kann sie ja ausserhalb nicht ansprechen.

Lg

Ralf Jansen · Verfasst: Fr 14.09.12 15:23

Du sprichst(fragst) in rätseln.
Dort wo du im ersten Code row.Add aufrufst kannst du stattdessen oder zusätzlich dein zahlung.Insert aufrufen. Da hast du ja ein korrektes Zeile Objekte(mit Namen Zeilen). Wenn du es an anderer Stelle brauchst dann sag wo und wie das mit dem anderen Code zusammenhängt.

Achte bei deinen Verwendung von Bezeichner auf die richtige Verwendung von Singular und Plural. Und einheitliche Groß/Kleinschreibung wäre auch nett (Variablen klein, Methoden und Typen groß).

List<Zeile> row - da gehts um eine Liste enthält also mehrere Objekte da wäre der Plural angebracht
Zeile Zeilen = new Zeile(); - hier wiederum geht um ein einzelnes Objekt also sollte der Bezeichner auch im Singular geschrieben werden.

Wenn du das wie hier durcheinander wirfst bringst du dich nur selbst (und natürlich andere die deinen Code lesen oder damit arbeiten müssen) durcheinander.

Vegeto · Beiträge: 262

Danke sehr...

ich habe das projekt momentan nicht bei mir, bin in der Berufssschule.

Werde das sofort ausprobieren, wenn ich die gelegenheit habe.

Ich soll also die Bezeichner ändern, damit die Kunden/Leute eine überschaubaren Code bekommen.

Aber du hast verstand was ich erreichen will oder?

Lg

	Mitgliederliste
	Gruppen
	Das Team
	Richtlinien
	Synonyme