Übergabe des Results funktioniert nicht korrekt

Dimadon · Hält's aus hier Beiträge: 11

Hallo Zusammen,

ich habe hier ein Pronblem mit dem Übertrag bzw. dem auslesen meiner Ergebnisse. Meine Datenbankabfrage sieht wie folgt aus:

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:
36:
37:
38:
39:
40:
41:
42:
43:
44:

 public static async Task<IList<String>> GetDocumentsAsync(string ProjectDocument)
        {
            IList<string> documents = new List<string>();
            using (SqlConnection conn = Connection.connection())
                try
                {

                    SqlCommand command = new SqlCommand("SELECT DocumentURL FROM dbo.projectdocuments WHERE Zuordnung= @Document;", conn);
                    conn.Open();
                    command.Parameters.AddWithValue("@Document", ProjectDocument);
                    SqlDataReader reader = command.ExecuteReader();

                    if (reader.HasRows)
                    {
                        while (reader.Read())
                        {
                            Document d = new Document();
                            d.setDocumentURL(reader.GetString(0));
                            d.setZuordnung(reader.GetString(1));
                            documents.Add(d.getDocumentURL());


                            await reader.ReadAsync();
                            {
                                
                                return documents;
                            }
                        }

                    }
                    else
                    {
                        return null;
                    }
                    reader.Close();
                    conn.Close();
                }

                catch (Exception e)
                {

                    
                }
            return null;

Anschließend soll das Ergebnis an diese Methode übertragen werden:

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:

 private static async Task<IList<string>> GetDocument(string ProjectDocument)
        {
            
            IList<string> result;
            result = await DocumentBot_Database.GetDocumentsAsync(ProjectDocument);
            

            if (result == null)
            {
                IList<string> documentFail = new List<string>();
                documentFail.Add("Es konnten mit deiner Eingabe: \"{0}\" leider keine passenden Dokumente gefunden werden.");
                documentFail.Add(ProjectDocument);
         
                return documentFail;
            }
            else
            {
                IList<string> documentResult = new List<string>();
                documentResult.Add("Es wurdem folgende Dokumente für dich gefunden: {1}");
                documentResult.Add(ProjectDocument);

                string finalString = string.Join(",", documentResult, result);
                return documentResult;


            }

        }

Kann mir eventuell hier jemand weiterhelfen? Ich bekommen aktuell leider auf kein Ergebnis.

Moderiert von Christian S.: C#-Tags hinzugefügt
Moderiert von Th69: Topic aus C# - Die Sprache verschoben am Sa 07.01.2017 um 09:58

Ehemaliges Mitglied Erhaltene Danke: 1

- Nachträglich durch die Entwickler-Ecke gelöscht -

Christian S. · Beiträge: 20451 Erhaltene Danke: 2264 Win 10 C# (VS 2019)

Hallo und

!

Nur zum Verständnis, welcher Methode sollte welches Ergebnis liefern und tut dies aber nicht?

Grüße
Christian

_________________
Zwei Worte werden Dir im Leben viele Türen öffnen - "ziehen" und "drücken".

Dimadon · Hält's aus hier Beiträge: 11

Hallo Zusammen,

das Problem liegt im Return meiner Liste documents. Das Programm steigt bei documents.Add(d.getDocumentURL()); aus und ich weiß nicht warum.

Was mir noch aufgefallen ist, das bei setzen von d die URL dem falschen Paramter(Zuordnung) zugeordnet wird....

Christian S. · Beiträge: 20451 Erhaltene Danke: 2264 Win 10 C# (VS 2019)

Was zuerst mal auffällt ist, dass Du im SQL nur eine Spalte selektierst, aber zwei lesen willst. Das kann ja schon mal nicht funktionieren.

Dann stellt sich die Frage, was das Reader.ReadAsync() bewirken soll und ob das return danach richtig ist.

Und wieso baust Du Instanzen der Klasse Document zusammen, wenn Du doch nur Strings rausgeben willst?

_________________
Zwei Worte werden Dir im Leben viele Türen öffnen - "ziehen" und "drücken".

Dimadon · Hält's aus hier Beiträge: 11

Ich möchte prinzipiell alle Einträge aus der Spalte DocumentuRL die als Zuordnung den übergebenen Wert besitzen in eine Liste übergeben und diese anschließend per Return ausgeben.

Den Async.reader habe ich genommen da ich bei einer Async Methode ja await einfügen muss, oder wie lößt man dies? Ich wäre hier echt froh wenn mir hier jemand helfen kann :(

Moderiert von Christian S.: Beiträge zusammengefasst

Eine weitere Frage wäre es natürlich ob man das eleganter Lösen kann ohne eine neue Instanz aufzubauen?

Christian S. · Beiträge: 20451 Erhaltene Danke: 2264 Win 10 C# (VS 2019)

Wenn Du nur die DocumentURLs haben willst, dann packe auch nur die in die Ergebnisliste, dann brauchst Du keine Instanzen der Document-Klasse erstellen.

Irgendwo einen sinnlosen Aufruf einzubauen, nur um eine Methode asynchron deklarieren zu können ist ... nun ja ... sinnlos. Wenn Du keinen sinnvolle Stelle findest, macht die Methode nicht asynchron.

Und es bleibt die Frage, was das return in der Schleife soll.

_________________
Zwei Worte werden Dir im Leben viele Türen öffnen - "ziehen" und "drücken".

Dimadon · Hält's aus hier Beiträge: 11

Danke für den Hinweis:

Wenn ich das jetzt so machen:

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:

public static async Task<IList<String>> GetDocumentsAsync(string ProjectDocument) 
{ 
IList<string> documents = new List<string>(); 
using (SqlConnection conn = Connection.connection()) 
try 
{ 

SqlCommand command = new SqlCommand("SELECT DocumentURL FROM dbo.projectdocuments WHERE Zuordnung= @Document;", conn); 
conn.Open(); 
command.Parameters.AddWithValue("@Document", ProjectDocument); 
SqlDataReader reader = command.ExecuteReader(); 

if (reader.HasRows) 
{ 
while (reader.Read()) 
{ 

documents.Add(reader["DocumentURL"].ToString()); 


await reader.ReadAsync(); 
{ 

return documents; 
} 
} 

} 
else 
{ 
return null; 
} 
reader.Close(); 
conn.Close(); 
}

Bekomme ich laut Debugger keine Ergebnisse?

Ehemaliges Mitglied Erhaltene Danke: 1

- Nachträglich durch die Entwickler-Ecke gelöscht -

Dimadon · Hält's aus hier Beiträge: 11

Beim Debuggen ist mir gerade aufegefall das ich einen System.InvalidCastException Fehler bekommen bei conn.open()

Meine zugehörige Verbindung sieht so aus:

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:

 public class Connection
    {

        private static SqlConnection con = null;

        private static String connString = "url";

        public static SqlConnection connection()
        {
           
            if (con == null)
            {
                try
                {
            
                    con = new SqlConnection(connString);

                }
                catch (Exception e)
                {
                    con = null;
                    return null;
                }

            }


            return con;

        }
    }

Dimadon · Hält's aus hier Beiträge: 11

Vielen Dank Frühlingsrolle..laut Debugger bekomme ich ein Ergebis...Ich weiß aber leider noch nicht ob es mehere sind da meine Ausgabe noch nicht passt...

Ich habe die Übergabe so gelöst nur bin ich mir noch bei der string.Join Lösung nicht ganz klar, wie ich diese an die Return-Anweisung übergebe:

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:

private static async Task<IList<string>> GetDocument(string ProjectDocument)
        {
            
            IList<string> result;
            result = await Document_Database.GetDocumentsAsync(ProjectDocument);
            

            if (result == null)
            {
                IList<string> documentFail = new List<string>();
                documentFail.Add("Es konnten mit deiner Eingabe: \"{0}\" leider keine passenden Dokumente gefunden werden.");
                documentFail.Add(ProjectDocument);
         
                return documentFail;
            }
            else
            {
                IList<string> documentResult = new List<string>();
                documentResult.Add("Es wurden folgende Dokumente für dich gefunden: {1}");
                documentResult.Add(ProjectDocument);

                string finalString = string.Join(",", documentResult, result);
                return documentResult;


            }

        }

Ok Edit:

Wenn ich den return nur auf result stelle bekomme ich eine URL raus. Laut meiner DB müssten aber in diesem Fall zwei Einträge übergeben werden.

Ehemaliges Mitglied Erhaltene Danke: 1

- Nachträglich durch die Entwickler-Ecke gelöscht -

Dimadon · Hält's aus hier Beiträge: 11

Super vielen Dank für deine Hilfe...Der Übertrag funktioniert laut Debugger soweit. Es werden beide Datensätze im result gespeichert. Kannst du mir eventuell noch verraten wie ich die beiden jetzt noch returenen kann?

Derzeit bekomme ich nur den ersten raus:

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:

 private static async Task<IList<string>> GetDocument(string ProjectDocument)
        {
            
            IList<string> result;
            result = await DocumentBot_Database.GetDocumentsAsync(ProjectDocument);
            

            if (result == null)
            {
                IList<string> documentFail = new List<string>();
                documentFail.Add("Es konnten mit deiner Eingabe: \"{0}\" leider keine passenden Dokumente gefunden werden.");
                documentFail.Add(ProjectDocument);
         
                return documentFail;
            }
            else
            {
                IList<string> documentResult = new List<string>();
                documentResult.Add("Es konnten folgende Dokumente für dich gefunden werden: {1}");
                documentResult.Add(ProjectDocument);
                
                return result;

Dimadon · Hält's aus hier Beiträge: 11

Frühlingsrolle...vielen lieben Dank. Habe die Ausgabe mit einem String Join hinbekommen. Danke für deine Hilfe!!

	Mitgliederliste
	Gruppen
	Das Team
	Richtlinien
	Synonyme