Asyncrones Einlesen einer MYSQL Datenbank: ... invalid column ordinal

Moin, ich wollte mal besagtes Einlesen asyncron machen da bekomme ich die Exception:
"You have specified an invalid column ordinal."
Quelle:
bei MySql.Data.MySqlClient.MySqlDataReader.Throw(Exception ex)
bei MySql.Data.MySqlClient.MySqlDataReader.GetFieldValue(Int32 index, Boolean checkNull)
bei MySql.Data.MySqlClient.MySqlDataReader.GetInt32(Int32 i) ...
(funzt im syncronen Modus tadellos)

try
            {
                await Task.Run(() =>
                {
                    using (MySqlConnection connection = new MySqlConnection(ConnString))
                    {
                        connection.Open();

                        queery = "d02d0075.MPrioSK();";

                        using (MySqlCommand command = new MySqlCommand(queery, connection))
                        {
                            using (reader = command.ExecuteReader())
                            {
                                while (reader.Read())
                                {
                                    Application.Current.Dispatcher.Invoke(delegate
                                    {
                                        _listArtikel.Add(new PositionModel()
                                        {
                                            ArtikellisteIndex = reader.GetInt32(0),
                                            ArtikelNummer = reader.GetString(1),
                                            ArtikelName = reader.GetString(2),
                                            BestellMenge = reader.GetInt32(3),
                                            Geliefert = reader.GetInt32(4),
                                            PrioMenge = reader.GetInt32(5),
                                            Datum = reader.GetDateTime(6),
                                        });
                                    });
                                }
                            }
                        }
                    }
                });
            }
            catch (Exception)
            { }

Ich möchte etwas Erfahrung sammeln mit Multitasking, bin da noch recht grün hinter den Ohren und kann mir leider nicht vorstellen wo der Fehler liegt.

Na lies dir doch mal die Fehlermeldung richtig durch. Läuft das SQL-Statement auch in der Konsole nicht bzw. stimmt die Spaltenanzahl noch?
BTW, gibt es auch ein command.ExecuteReaderAsync() und reader.ReadAsync()? Dann könntest du diese verwenden mit await und würdest das Task.Run nicht benötigen.

Läuft das SQL-Statement auch in der Konsole nicht?

SQL Anfrage läuft im syncronen Modus/PHPMyAdmin fehlerfrei.

gibt es auch ein command.ExecuteReaderAsync() und reader.ReadAsync()? Dann könntest du diese verwenden mit await und würdest das Task.Run nicht benötigen.

Ich teste mal "ExecuteReaderAsync()"...

Update: funzt auch nicht...
Stehe wie gesagt in dieser Thematik am Anfang

BTW 2: Wie viele Artikel liest du überhaupt ein? Wenn du bei jedem Objekt eine (eventuelle) Threadsynchronisation durchführst könnte das bei einer großen Anzahl an Objekten sehr teuer werden.

Die Ausgabe ist auf 3 begrenzt (... LIMIT 3;)

Async bei MySQL ist vorhanden:
"using (reader = (MySqlDataReader)await command.ExecuteReaderAsync())
{
while (await reader.ReadAsync())..."

nach dem was ich gelesen habe ist dies jedoch nicht async...
Does MySql Connection methods support async programming?

Sieht auch aus als würdest du in der UI deine Datenbank Verarbeitung einrühren.
Hier scheint also auch eine grundsätzliche Schicktentrennung zu fehlen.

Auch die Verarbeitung mit dem Dispatcher sieht sehr abenteuerlich aus.
Warum liest du nicht in deinem Task.Run() Block die Positionsmodells in eine Liste und lieferst diese zu rück.
Aktuell muss für jedes Item, was du einlist ein Invoke gemacht werden, hier wäre ein Invoke mit der gesamten Liste über AddRagen vermutlich um besser.
Auch kann es durch ein synchrones read vom DataReader aber dein Invoke dann zu Thread Synchronisationsproblemen kommen.
Wenn der Reader im anderen Thread weiterläuft, dann ist es auch kein Wunder dass du diese Meldung bekommst.

Trennen deine UI und deine Datenbank Code durch eine saubere Schichten Trennung nach dem Drei-Schichten Modell.
Dann hast du auch solche Konstrukte nicht.

T-Virus

Auch die Verarbeitung mit dem Dispatcher sieht sehr abenteuerlich aus.

Das habe ich so recherchiert und manches waren Korrekturen von VS2019, Asche auf mein Haupt...

Trennen deine UI und deine Datenbank Code durch eine saubere Schichten Trennung nach dem Drei-Schichten Modell. Dann hast du auch solche Konstrukte nicht.

Das kam in der Umschulung (2.Lj) nicht vor, auch habe ich damit keine praktischen Erfahrungen.
Wenn das die Verwendung von MVVM impliziert, da bin ich auch noch rel. am Anfang.

Nein, es geht um die [Artikel] Drei-Schichten-Architektur.

1. Ich verzichte bei SQL auf async
2. Ja, ich werde mich eingehend mit der 3 Schichten Architektur befassen

Leider weiss ich nicht genau was mit:
"Sieht auch aus als würdest du in der UI deine Datenbank Verarbeitung einrühren."
gemeint ist im Bezug auf den Code (auch ohne async) X(
Ich nahm an GENAU SO wird's gemacht 🙁

Hi,

nein - so wird es genau nicht gemacht.

Wenn du einen Dispatcher in der selben Methode wie eine MySqlConnection verwendest - machst du es im Prinzip schon falsch. (siehe bereits verlinkten Artikel 3-Schichten-Architektur)

Kurz: Ein Dispatcher gehört zur UI - eine Connection zur Datenbank. Beide in der selben Methode zu verwenden zeigt nur, dass du eben nicht getrennt hast wie es sich gehört.

Warum man das so klar sagen kann?
Den Dispatcher verwendet man um eine asynchrone Datenänderung mit dem "UI-Thread" von WPF abzufangen, sodass du UI darüber eben im "richtigen" Thread informiert wird. Wenn du jetzt einen Dispatcher (und auch noch eine Liste, welche der UI bekannt ist - aus der Datenschicht aktualisieren lässt - dann kann das nur falsch sein)

Kann jetzt natürlich sein, dass der Dispatcher komplett unnötig verwendet wurde - aber dann solltest du aufhören einfach Code zu kopieren...

LG

Habe zurückgerudert

try
            {
                using (MySqlConnection connection = new MySqlConnection(ConnString))
                {
                    connection.Open();

                    queery = "d02d0075.MPrioSK();";

                    using (MySqlCommand command = new MySqlCommand(queery, connection))
                    {
                        using (reader = command.ExecuteReader())
                        {
                            while (reader.Read())
                            {
                                    _listArtikel.Add(new PositionModel()
                                    {
                                        ArtikellisteIndex = reader.GetInt32(0),
                                        ArtikelNummer = reader.GetString(1),
                                        ArtikelName = reader.GetString(2),
                                        BestellMenge = reader.GetInt32(3),
                                        Geliefert = reader.GetInt32(4),
                                        PrioMenge = reader.GetInt32(5),
                                        Datum = reader.GetDateTime(6),
                                    });
                            }
                        }
                    }
                }

            }
            catch (Exception)
            { }

Nur wie ich hier die 3-Schichten-Architektur umsetze weiss ich noch nicht genau...
problem mit async ist gelöst da wie T-Virus schrieb, ist MySQL syncron, ergo...

Erzeuge eine Liste von PositionModel ohne den Dispatcher. Das Ding gibst du in die nächsthöhere Schicht raus. Diese baut das in die UI ein.

Ich muss in dieser Thematik (3-Schichten-Modell) in einem etwas überschaubarem Projekt anfangen

Stehe wie gesagt in dieser Thematik am Anfang

Das Doofe ist, dass dies ein Thema ist, dass Du nicht einfach ausprobieren kannst.
Du musst es verstehen.

Wenn man mit einem Try-and-Error Vorgehen an eine Sache wie diese geht, wie Du evtl. andere Themen angegangen bist, dann fällt man bei async/await ganz ganz arg auf die Nase 😉

Ich würde Dir empfehlen ein ORM oder Micro-ORM einzusetzen.

Bei mir würde deine Routine z.B. so aussehen :

foreach (DataObjectsArtikel ar in new DataCollectionArtikel(qry))
{
    ArtikellisteIndex = ar.rowindex;
    ArtikelNummer = ar.nr;
    ArtikelName = ar.name;
    BestellMenge = ar.bestmenge;
    Geliefert = ar.geliefert;
    PrioMenge = ar.priomenge;
    Datum = ar.datum;
}

Alles typsicher und mit richtiger Umwandlung.

Grüße Bernd

Kann mir jemand des besseren Verständnisses Halber beschreiben wie ich diesen Code in die
3 Schichten zerlege? Oder nen neuen Thread aufmachen?

private ObservableCollection<PositionModel> _listArtikel = new ObservableCollection<PositionModel>();

...

private void Prio()
        {
            _listArtikel.Clear();
            try
            {
                using (MySqlConnection connection = new MySqlConnection(ConnString))
                {
                    connection.Open();

                    queery = "d02d0075.MPrioSK();";

                    using (MySqlCommand command = new MySqlCommand(queery, connection))
                    {
                        using (reader = command.ExecuteReader())
                        {
                            while (reader.Read())
                            {
                                _listArtikel.Add(new PositionModel()
                                {
                                    ArtikellisteIndex = reader.GetInt32(0),
                                    ArtikelNummer = reader.GetString(1),
                                    ArtikelName = reader.GetString(2),
                                    BestellMenge = reader.GetInt32(3),
                                    Geliefert = reader.GetInt32(4),
                                    PrioMenge = reader.GetInt32(5),
                                    Datum = reader.GetDateTime(6),
                                });
                            }
                        }
                    }
                }

            }
            catch (Exception)
            { }
}

Du musst es verstehen.

Das Problem...

Ich benötige leider kein abstraktes Theorem sondern ein Beispiel mit Kontext zu etwas
was ich im Code verwende um es zu verstehen.

Das Prinzip verstehe ich schon, ich weiss eben nicht wie ich es realisieren soll.

Google liefert hier leider nur ungenügendes... X(

Im einfachsten Fall hast du ersteinmal nur drei verschiedene Klassen (also je Schicht eine) und nur die UI-Klasse (d.h. bei MVVM die ViewModel-Klasse) benutzt die Logik-Klasse und diese wiederum die Datenzugriffsklasse.

Die Theorie der 3-Schichten Trennung ist mir klar.
Es hapert bei der praktischen Umsetzung und an Beispielen die ich kontextuell nachvollziehen kann.
Falls Fragen auftauchen, für was für eine Art Projekt das gelte, Stichwort: Ockhams Rasiermesser

So habe ich es Verstanden:
3. Schicht
3.1 Datenbankzugriff in einer Methode, Einlesen in eine Liste (Lokale? Variable z.B. List<>, ObservableCollection(OC) oder DataTable)
3.2 Übergabe an Aufrufer (Frage: wie übergebe ich eine OC, eine DataTable - habe ich immer gemieden da ich das Konzept nicht 100%ig durchschaue)

public DataTable DataT()
   {
       DataTable dt = new DataTable(); //oder ObservableCollection
       using(SQL)
        {
          dt = DataReader
        }
       return dt;
   }

Wäre das korrekt, auch bei einer OC?

2. Schicht
   2.1 Bearbeitung der DataTable, OC - Berechnung, etc.
   2.2 evtl. Aufruf einer weiteren Methode mit Rückgabe einer OC, DT

private void Bla()
{
   DataTable dt = DataT();
   dt = Irgendwas;
}

So?

1. Schicht
   1.1 Bedienung und Behandlung der UI und Aufruf der Methoden der 2. Schicht
   1.2 ???

private void Blabla(irgendein KeyDownBlaBlaEventHandler)
{
 if (Textbox.Content != "" || AntwortAufFrageNachDemUniversumUndDemGanzenRest == 42)
   Bla(); 
}

Für sowas und zum Verstehen braucht man nicht fragen: Für was für ein Projekt?
Für das einfachste Projekt das gerade zum Verstehen nötig ist!

Wer es nicht kennt, Ockhams Rasiemesser:

1. Von mehreren hinreichenden möglichen Erklärungen für ein und denselben Sachverhalt ist die einfachste Theorie allen anderen vorzuziehen.
2. Eine Theorie ist einfach, wenn sie möglichst wenige Variablen und Hypothesen enthält und wenn diese in klaren logischen Beziehungen zueinander stehen, aus denen der zu erklärende Sachverhalt logisch folgt - Quelle: Wikipedia

Du solltest dir den Artikel auch mal durchesen. Dort werden eigentlich die meisten deiner Fragen beantwortet. Eine ObservableCollection ist eine Implementierung einer Liste für das DataBinding mit MVVM in WPF, und gehört daher zur View. Wenn du diese jetzt in der Datenzugriffsschicht verwenden willst, hast du das Konzept ganz grundlegend nicht verstanden.

Mit DataTables habe ich noch nie gearbeitet, daher kann ich dazu nichts sagen. Normalerweise würde man dazu Klassen erstellen und verwenden, die die Daten repräsentieren (POCOs).

Ob man die Schichten sauber getrennt hat, merkt man spätestens beim Testen. Wenn du die Funktionalität aus einzelnen Schichten unabhängig von Funktionalität aus den anderen Schichten testen kannst, ist deine Architektur sauber, und sonst halt nicht. Wenn du eine Collection aus WPF benötigst, um deine DAL zu testen, dann eben nicht. Aber wenn du noch nie deinen Code mit Unit-Tests getestet hast, ist das alles nur Theorie.

Und was hat Ockham's Razor damit zu tun? Ockham war ein mittelalterlicher Mönch, der sich mit (wissenschaftlichen) Erklärungsansätzen für unerklärliche Phänomäne beschäftigt hat, nicht mit Entwurfsmustern aus der Software-Entwicklung.

Oh ja, ich vergaß.
Mir ging es weniger um die Anwendung laut (scholastischer) Definition sondern eher um ein vereinfachtes Prinzip. (Mönch hin oder her...) Viele Profi-Entwickler mühen sich in Erklärungen, Sachverhalte in nur aus Ihrer Sicht einfacher Weise darzulegen. Leider trifft dies Nur Aus Ihrer Sicht zu. Für mich ohne jenen langjährigen, professionellen Kontext jedoch ist es dann alles andere als Einfach. Oft finden sich dann Klassen/Methoden/whatever darin, die ich in ihrer Funktion nicht nachvollziehen kann und somit nicht zur Erleuchtung beitragen. Daher so simpel wir möglich und so Komplex wie nötig.

Wenn also 2 oder mehrere mögliche Erklärungen vorhanden sind. ist die Einfache der/den Komplexeren vorzuziehen. Finde ich jedenfalls.

..., hast du das Konzept ganz grundlegend nicht verstanden.

🙁 nach den anderen beiträgen zufolge habe ich das anscheinend nicht.
Und ich dachte, ich hätte. Zumindest Grundlegend.

Wieso? Er hat doch nicht gesagt dass alles in eine Lib gequetscht werden muss, man kann es tun. Wenn du für die einzelnen Schichten jeweils eine Lib verwenden willst dann tu es doch.

Lass gut sein witte, irgendwann machts kein Sinn mehr irgendwas zu erklären 😃