Steamy2010 schrieb:

Leider funzt das nicht wie gewünscht.

For Each funktioniert nicht.
Mach eien For-Schleife, die rückwärts die Items durchläuft, so dass beim Löschen die Collection da, wo noch zugegriffen werden muss, nicht umgebaut wird.

Versuche das einmal mit der SelectedRows-Auflistung des DGV.
msdn.microsoft.com/de-de/libra…lectedrows(v=vs.110).aspx
Wichtig: man löscht aus einer Auflistung immer von hinten beginnend!

Steamy2010 schrieb:

Fehler

Wenn, dann so (ungetestet):

wichtig auch, ob es sich um ein gebundenes DGV handelt, denn dann sollte man aus der BindingSource löschen statt ins DGV zu grabschen

Die SelectedRows-Auflistung representiert doch alle ausgewählten Zeilen im DGV, da brauchst du die Selected-Eigenschaft nicht extra abrufen...

Ups, hast recht - diesen Member gibt's da gar nicht Sorry - obiges war nur aus dem Kopf...
Da habe ich damals doch die Bindingsource dazu hergenommen...
oder bei ungebundenen DGV's...

Da der Thread noch nicht all zu alt ist möchte ich ein Problem vorstellen und mich anschließen:

Ich möchte in meinem DGV ebenfalls mehrere Zeilen löschen können.
Das DGV ist gebunden an Typdataset, also soll in der BS gelöscht werden.

Zusätzlich soll jedoch geprüft werden ob Childrows da sind da ich keine Löschweitergabe im Dataset will (sonst sind mal eben bei unsachgemäßer Bedienung sehr viele Datensätze weg!).
Fehler:

Ein Ausnahmefehler des Typs "System.ArgumentOutOfRangeException" ist in mscorlib.dll aufgetreten.

Zusätzliche Informationen: Der Index lag außerhalb des Bereichs. Er muss nicht negativ und kleiner als die Auflistung sein.

Hier der Code:


Fällt Euch was ein?

edit: sehe grad da steckt noch ein Remote current drin was natürlich Mist ist... Stammt von vor der änderung auf mehrere Zeilen löschen

vlt. hat dein DGV Hinzufügen aktiviert, also eine ZufügeZeile.
Diese Zeile ist eine zuviel für die BindingSource, denn da ist ja kein Datensatz drin.

Aber mach doch wie vorgeschlagen, über die SelectedRows-Auflistung. Da erhälst du zuverlässig nur Indizees, die auch da sind.

Übrigens würde ich empfehlen, den MVB-Namespace - GeneralImport zu entfernen.

Und übrigens du brauchst niemals mit True zu vergleichen: Boolean, Vergleiche und bedingte Verzweigungen

ErfinderDesRades schrieb:

vlt. hat dein DGV Hinzufügen aktiviert, also eine ZufügeZeile.
Diese Zeile ist eine zuviel für die BindingSource, denn da ist ja kein Datensatz drin.

Aber mach doch wie vorgeschlagen, über die SelectedRows-Auflistung. Da erhälst du zuverlässig nur Indizees, die auch da sind.

also die Bearbeitung des DGV ist über den Designer komplett gesperrt, habe hierfür Detail VIew.

Die Selected Rows Auflistung habe ich doch im Direct Cast drin? Funktioniert aber trotzdem nicht.

uih - ja, so ist das natürlich ganz falsch.
Weil die äussere Schleife zählt noch immer alle DGV-Rows durch, und natürlich gibts mehr DGV-Rows als es SelectedRows gibt.

Du dir die SelectedRows erstmal in eine Array umkopieren, denn möglicherweise verhält sich eine SelectedRowCollection noch komischer, wenn eine Row davon gelöscht wird (vlt. geht dann die Selection gleich verloren.)
Und dann das Array rückwärts durchlaufen - nicht die DGV-Rows.

so nun ist es spät, ich habe jedoch folgendes zusammen gebracht was scheinbar auch funktioniert.(mir als anfänger ist aber ehrlich gesagt im augenblick nicht mehr klar ob ich alle eventualitäten beachtet habe).

kritik erwünscht, hoffe ich habe es richtig verstanden:

Bitte jetzt nicht erschrecken wgDas ist einfach eine Methode, die aus DgvTblCatEinrichtung die SelectedRows abruft, von denen die Indicees, und dann absteigend sortiert.
Wenn du Linq nicht magst, schreib dir selbst eine kleine Methode, die das kann - die könnte etwa eine List(Of Integer) returnen.

Der wichtige Punkt ist, dass ich nirgends DGV.Rows benötige.
Weil ich hab ja die DGV.SelectedRows - an die muss ich mich halten.
Und da brauch ich auch nicht If DgvTblCatEinrichtung.Rows(i).Selected = True zu prüfen - ich weiß ja, dass die SelectedRows selected sind.
Und übrigens du brauchst niemals mit True zu vergleichen: Boolean, Vergleiche und bedingte Verzweigungen

vielen dank für die ausführliche antwort.

ich werde es entsprechend ändern.

aber eine frage (für mich als anfänger wichtig): mein code, der mit sicherheit umständlicher ist, hat der fehler? ein bischen stolz wäre ich ja schon wenns vom prinzip her richtig ist...

naja - ist halt extrem krude.
index ist ein Array, so groß wie alle Zeilen.
Dann gehst du das DGV durch, und schreibst in index überall -1 rein, ausser wenn die Row selected ist - dann schreibst du den RowIndex nach index (könntest ebensogut auch 99 reinschreiben oder whatever).
Somit hast du also eine eigene (sehr komisch strukturierte) .SelectedIndex-Auflistung erstellt, wohl weil die .SelectedRows-Auflistung des DGVs dir nicht gefällt
Und dann gehst du das index durch, und immer wenner nichtnegativ ist, holst du dir vom Index den Index, damit holst du die DGV-Row, und davon den Index. Mit letzterem Index holst du die DataRow:

Einfacher wäre, du würdest die DataRow gleich mit y holen, da steht nämlich auch nix anneres drin - alsomüsste auch gehen.

Fehler, die zum Absturz führen, kann ich nicht erkennen, aber was da vorgeht ist so verschroben, dasses mir eher wie durch Herumprobieren entstanden aussieht.

Eine Zielstrebige Vorgehensweise - auch ohne .SelectedRows-Auflistung - hätte eine List(Of Integer) angelegt, und in der ersten Schleife befüllt, und in der 2. Schleife rückwärts durchlaufend ausgewertet.

das hat mir sehr weiter geholfen, vielen dank.


ich habe nun folgendes problemchen: wenn in einer Zeile keine Bezeichnung eingetragen ist, möchte ich gerne (durch die übernahme vorhandener Datasets) die Felder der Spalten mit einem Standardwert belegen.

Wenn ich nun prüfen will:



kommt trotzdem die fehlermeldung:

Zusätzliche Informationen: Der Wert für Spalte Bezeichnung in Tabelle tblEinrichtung ist DBNull.

Edit: Und noch eins häng ich gleich mit ran:

Bei einer späteren Linq Abfrage über die Tabelle schmeists

​Auf gelöschte Zeileninformationen kann nicht über die Zeile zugegriffen werden.

Ich habe dahre nach dem löschen DS.Acceptchanges angehängt.

Ist das die korrekte Art?
Wieso sind die removte Rows noch da? Damit ich sie wieder zurück holen kann?

Es heisst und ist eine generierte, typisierte Methode.
Wo kommt das IsDBNull her - ist das nicht aus dem Deppen-Namespace? Visual Studio - Empfohlene Einstellungen

---

die Removten Zeilen bleiben drin, damit die Änderungsverfolgung die Löschung auch an die DB übermittlen kann. Um mit Linq nicht auf eine gelöschte Zeile zu treten muss man den RowState der Datarow abfragen.

Von AcceptChanges besser die Finger lassen - das löscht die Änderungsverfolgung (ja, und dann sind die gelöschten Zeilen auch aussem Dataset richtig weg.)
Ausserdem, wenn du dich darauf stützen willst, darfst du das nie vergessen.

Denkbar wäre auch, mit DatarowCollection.Remove/.RemoveAt zu removen - das lässt die Zeile ganz still verschwinden, als wäre sie nie da gewesen.