Forum
 
Fragen
 
Suchen
 
Über Uns
 
 

SQL Abfrage
Neue Frage »

29.03.2013, 12:06 Auf diesen Beitrag antworten »
InformaTiger SQL Abfrage

Hallo,
ich möchte gerne in meinem Programm eine Datenbank (eine externe .accdb) auslesen. Wie mache ich das mit SQL Abfrage in C#?

Ich habe hier ein Beispiel, wie ich es schon verwendet habe. Es funktioniert allerdings eben nicht.

code: 
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:

public string[] LeseDatenbankAusTKundenAnmeldedaten()
          {
              string[] aAnmeldedaten = new string[2];

              // Erstell wieder eine Verbindung zur Datenbank
              aDatenbankverbindung = new OleDbConnection(aVerbindungspfad);
              aDatenbankverbindung.Open();
              aSQLBefehl = aDatenbankverbindung.CreateCommand();

              // ist der Befehl zum Auslesen der Tabelle Kunden
              aSQLBefehl.CommandText = "SELECT Name, Passwort FROM Kunden";

              // leitet das auslesen der Datenbank ein
              OleDbDataReader aDatenSet = aSQLBefehl.ExecuteReader();

              for (int i = 0; i < 2; i++)
              {
                  if (i == 1)
                  {
                     aAnmeldedaten[0] = aDatenSet.GetString(2);
                  }
                  else if (i == 2)
                  {
                     aAnmeldedaten[1] = aDatenSet.GetString(16);
                  }
              }
              aDatenbankverbindung.Close();
              return aAnmeldedaten;
          }


Zur Erklärung:

Zitat:
aSQLBefehl, und aDatenbankverbindung sind am Anfang der Klasse deklarierte OleDbConnection bzw. OleDbCommand Variablen.


Lg
InformaTiger
 
 
29.03.2013, 12:55 Auf diesen Beitrag antworten »
eulerscheZahl

Habe bisher nicht mit Datenbanken gearbeitet, kann daher auch nicht viel beitragen.
Was mir aufgefallen ist:
code: 
1:
2:
3:
4:
5:
6:
7:

              for (int i = 0; i < 2; i++)
              {
                  if (i == 1)
                  //...
                  else if (i == 2)
                  //...
              }

i ist entweder 0 oder 1, der Fall i==2 tritt also nie ein.
29.03.2013, 13:38 Auf diesen Beitrag antworten »
InformaTiger

Ups,
habe das ausgebessert funktioniert jedoch immer noch nicht unglücklich

Lg
InformaTiger
30.03.2013, 14:23 Auf diesen Beitrag antworten »
Karlito

Hallo gib mal bitte die Definitionen von aSQLBefehl und aDatenbankverbindung. Und die genaue Fehlermeldung...

VG,

Karlito
 Anzeige
 
30.03.2013, 19:17 Auf diesen Beitrag antworten »
Airblader

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

for (int i = 0; i < 2; i++) {
    if (i == 1) {
        aAnmeldedaten[0] = aDatenSet.GetString(2);
    } else if (i == 2) {
        aAnmeldedaten[1] = aDatenSet.GetString(16);
    }
}


ist sowieso völlig unsinnig. Wieso nicht gleich so (Korrektur des oben angesprochenen Fehlers eingerechnet)?

code: 
1:
2:

aAnmeldedaten[0] = aDatenSet.GetString(2);
aAnmeldedaten[1] = aDatenSet.GetString(16);
30.03.2013, 22:32 Auf diesen Beitrag antworten »
InformaTiger

@Karlito: Wenn man das wie Airblader erwähnt ausbessert kommt folgende Fehlermeldung:

Zitat:
InvalidOperationException wurde nicht behandelt. Keine Daten für die Zeile/Spalte.


Wobei ich dazu sagen muss, dass in der verbundenen Datenbank auf Spalte 2 / 16 sehr wohl Daten sind... geschockt

Zur Definition von aSQLBefehl und aDatenbankverbindung:

code: 
1:
2:

private static OleDbConnection aDatenbankverbindung = null; // Erstellt eine Datenbankverbindung
private static OleDbCommand aSQLBefehl = null; // Gibt der Datenbank Befehle


@Airblader: Frag ich mich auch gerade... verwirrt

edit: desweiteren habe ich jetzt in meiner Access- Datenbank mit der ich mich im Programm verbinde, "SELECT Name, Passwort FROM Kunden" versucht und er hat mir exakt die zwei gewollten Spalten angegeben.

Lg
InformaTiger
31.03.2013, 00:11 Auf diesen Beitrag antworten »
InformaTiger

Was mir persönlich dann noch mal mehr zu schaffen macht ist das ich so einen "auslese" Code im selben Programm schon mal geschrieben habe, er dort funktioniert und hier nicht. böse traurig
Dieser Code folgt:

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

private void LeseDatenbankAusTNachrichten(string mName)
          {
              aSQLBefehl = aDatenbankverbindung.CreateCommand();

                // ist der Befehl zum Auslesen der Tabelle Nachrichten
              aSQLBefehl.CommandText = "SELECT * FROM Nachrichten";

                // leitet das auslesen der Datenbank ein
              OleDbDataReader aDatenSet = aSQLBefehl.ExecuteReader();

                while(aDatenSet.Read())
              {
                  if (aDatenSet.GetString(2) == aEmpfaengerName)
                  {
                      MessageBox.Show("Von: " + aDatenSet.GetString(1) + "\nAm: " + aDatenSet.GetDateTime(3) + "\nNachricht: " + aDatenSet.GetString(4));
                  }
              }
              aDatenSet.Close();
          }


Ich verstehe allerdings eben auch nicht woran der Fehler liegen könnte... unglücklich

Lg
InformaTiger
31.03.2013, 12:45 Auf diesen Beitrag antworten »
Airblader

Ich kenne die API dazu nicht, aber wenn sie ansatzweise der von OracleSQL unter Java ähnelt und das Query

code: 
1:

SELECT Name, Passwort FROM Kunden


lautet, dann sollten die obigen zwei Zeilen so ausshen:

code: 
1:
2:

aAnmeldedaten[0] = aDatenSet.GetString(0);
aAnmeldedaten[1] = aDatenSet.GetString(1);


Vielleicht aber auch mit 1 und 2 statt 0 und 1 – je nachdem, wo diese API zu zählen beginnt.

Ist die Kapitalisierung von Methodennamen unter C# eigentlich Standard? Sieht in meinen Java- und Javascript-geprägten Augen fürchterlich aus.

Die Namensgebung mit dem "a"-Prefix finde ich übrigens auch seltsam. Wofür soll das stehen?

air
31.03.2013, 13:09 Auf diesen Beitrag antworten »
InformaTiger

Leider ist mir das auch schon in den Sinn gekommen, hab das auch versucht. Leider trotzdem dieselbe Fehlermeldung.

Zu dem "a"-Prefix: das ist eine Programmierkonvention an unserer Schule.

Zu der Kapitalisierung von Methodennamen: Du meinst, dass die Anfangsbuchstaben des nächsten Wortes Großgeschrieben werden? Das ist auch eine Konvention unserer Schule. Ob das ansonsten so gänglich ist weiß ich nicht...

Lg
InformaTiger
31.03.2013, 14:11 Auf diesen Beitrag antworten »
Airblader

Zitat:
Original von InformaTiger
Zu der Kapitalisierung von Methodennamen: Du meinst, dass die Anfangsbuchstaben des nächsten Wortes Großgeschrieben werden?


Nein, CamelCase ist tatsächlich ein verbreiteter Standard. Typischerweise beginnen Variablen- und Methodennamen aber mit einem Kleinbuchstaben ("readLine", "getProeprty", ...), während die Großschreibung des ersten Buchstabens für Klassen reserviert ist.
Da die Schreibweise schon in den Klassen, die du verwendest, so enthalten zu sein scheint, ist es vermutlich ein C#-Standard.

Zurück zum Thema. Kann es sein, dass dir noch ein aDatenset.Read(); fehlt? Der Fehler, den ich im letzten Beitrag angesprochen habe, dürfte übrigens dennoch eine Rolle spielen, d.h. da sollte dennoch 0 und 1 (bzw. 1 und 2) stehen.

air
P.S.: Dass du am Unterrichtsstandard nichts ändern kannst ist klar – aber gewöhne dir sinnlose Prefixes bitte nicht an. Augenzwinkern
31.03.2013, 16:10 Auf diesen Beitrag antworten »
Karlito

Hallo zusammen...

@Air: Groß geschriebene Methodennamen sind unter C# tatsächlich üblich. Auch sieht man oft die ungarische Notation. In der Firma, in der ich einmal gearbeitet habe, war das auch gang und gäbe (und nicht ganz mein Geschmack). Man kann sich über die Ästhetik streiten. Ich verstehe deinen Standpunkt, aber lass ihn mal selbst entscheiden ob er sich das angewöhnt... Schlussendlich entscheided oft eh dann der Chef oder der Auftraggeber über den anzuwendenden Codestyle.

Aber danke für die Tipps. Es freut mich wenn das Forum hier weiter belebt wird und deine Beiträge waren bisher wertvoll.

VG,

Karlito
31.03.2013, 17:52 Auf diesen Beitrag antworten »
Airblader

Um vielleicht einem Missverständnis entgegenzuwirken: Wenn großgeschriebene Methodennamen C#-Standard sind, so sei es. Mein "Rat", sich etwas nicht anzugewöhnen, bezog sich auf die offenbar sinnlosen "a"-Präfixe – die sind ja hoffentlich kein C#-Standard; fiele mir jedenfalls schwer, das zu glauben. Augenzwinkern Ungarische Notation ist das Eine, aber die Präfixe sollten ja auch noch einen Sinn erfüllen (der sich mir hier entzieht).
In der Schule und an der Universität bekommt man leider eine ganze Menge Käse aufgetischt und Praktiken gelehrt, die mitunter seit Jahrzehnten obsolet sind, zum Beispiel der Mythos, dass guter Code möglichst viele Kommentare enthält.

Allerdings ist das alles ein anderes Thema. Erstmal gucken, ob's mit dem Read-Befehl endlich tut. Augenzwinkern

air
31.03.2013, 19:29 Auf diesen Beitrag antworten »
InformaTiger

Sagen wir so, es funktioniert halb...
ich muss jetzt noch mehrere Varianten testen und kann erst dann genaueres Bescheid geben.

Aber es ist schon mal ein Ansatz zur Lösung des Problems. smile

Lg
InformaTiger
31.03.2013, 20:41 Auf diesen Beitrag antworten »
InformaTiger

Hurra,
es funktioniert.... Prost

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

while (aKeineUebereinstimmung)
          {
                  aDatenSet.Read();
                  aAnmeldedaten[0] = aDatenSet.GetString(0);
                  aAnmeldedaten[1] = aDatenSet.GetString(1);
                  if (mBenutzername == aAnmeldedaten[0])
                  {
                      aKeineUebereinstimmung = false;
                  }
            }


Dieser Code hat es in sich! großes Grinsen

Tanzen

Danke, Airblader.
Danke, Karlito.
Danke, eulerscheZahl.

Lg
InformaTiger
31.03.2013, 20:59 Auf diesen Beitrag antworten »
Airblader

Super! Daumen hoch

code: 
1:
2:
3:
4:

if (mBenutzername == aAnmeldedaten[0])
{
    aKeineUebereinstimmung = false;
}


könnte man übrigens noch verkürzen zu

code: 
1:

aKeineUebereinstimmung = (mBenutzername != aAnmeldeDaten[0]);


Wobei da auch die Meinungen auseinandergehen, ob sowas guter Stil ist. Schadet aber nicht, wenn man solche Dinge ab und an mal sieht. Augenzwinkern

air
31.03.2013, 21:16 Auf diesen Beitrag antworten »
InformaTiger

Wenn ich das so machen würde, könnte es leicht sein dass ich den Überblick verliere. geschockt

Danke, nochmal für deine Hilfe.

Lg
InformaTiger
 
Neue Frage »
Antworten »


Verwandte Themen

Die Beliebtesten »
Die Größten »
Die Neuesten »


Informatiker Board » Themengebiete » Sonstige Fragen » SQL Abfrage

 
© 2005-2019 bei Informatikerboard.de | Archiv, alt, neueste | Kontakt | Jugendschutz | Nutzungsbedingungen