Discussion :

[Thyyb]

Bonjour,

Alors je ne comprends pas trop ce qui m'arrive. Déjà voici mon code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
For i = 0 To NbIndex - 1
            gridview_tbl.Rows.RemoveAt(tabsup(i))
        Next

Travaille sous visual studio 2012 (vb.net) et access 2013 (mdb).

Mon NbIndex vaut 22, donc j'ai un tableau qui va de 0 à 21.
Ma boucle va donc de 0 à 21. Et ma boucle s'arrête pour i=1 me disant "L'index de ligne fourni est hors limites." ... Mon tableau à une taille de 22 je ne comprends pas pourquoi il dépasse à 1.
Des idées de pourquoi de comment ?

[callo]

Apparemment ton datagridview ne comporte qu'une seule ligne, raison pour laquelle cette exception est levée quand tu tentes d'aller au delà.

[Thyyb]

J'ai déjà vérifié et si j'enlève ma boucle, mon datagridview comporte presque une trentaine (il me semble) de lignes.
C'est pour ça que je trouve ça bizarre, j'ai un tableau qui à 22 éléments et un datagridview qui en à au moins 22 ... Je ne vois pas d'où vient mon soucis

[callo]

Je viens de revoir ton code. C'est quoi ton tabsup(i)? Au lieu de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
For i = 0 To NbIndex - 1
  gridview_tbl.Rows.RemoveAt(tabsup(i))
Next

est-ce qu'il ne faut pas plutot mettre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
For i = 0 To NbIndex - 1
  gridview_tbl.Rows.RemoveAt(i)
Next

?

[ebastien]

Bonjour,

Je viens de revoir ton code. C'est quoi ton tabsup(i)?

Effectivement...

Lorsque tu supprimes la ligne d'indice 0, la ligne d'indice 1 passe à l'indice 0 ...
Pour supprimer des éléments d'une collection dans une boucle, soit tu boucles à l'envers :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
For i = NbIndex - 1 to 0 step -1
  gridview_tbl.Rows.RemoveAt(i)
Next

Soit tu supprimes systématiquement le premier élément :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
For i = 0 To NbIndex - 1
  gridview_tbl.Rows.RemoveAt(0)
Next

Je ne peux pas tester mais ça devrait fonctionner.
eb.

[Thyyb]

Je n'y avait pas pensé. Mais ce que vous me dites ça va supprimer toutes les lignes de mon datagridview en fait non ?

Parce que moi je veux supprimer les doublons dont je range le numéro de ligne dans mon tabsup. En supprimant les lignes avec l'indice contenu dans tabsup, je supprime les doublons de mon datagridview ... Je sais pas si c'est clair :S

[Thyyb]

Sinon je viens de penser, vu que dans mon tabsup je n'ai que mes doublons si je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
For i = 0 To NbIndex - 1
            gridview_tbl.Rows.RemoveAt(tabsup(0))
Next

Ca devrait supprimer touts mes doublons ?

[ebastien]

Sinon je viens de penser, vu que dans mon tabsup je n'ai que mes doublons si je fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
For i = 0 To NbIndex - 1
            gridview_tbl.Rows.RemoveAt(tabsup(0))
Next

Ca devrait supprimer touts mes doublons ?

Non ! Ca supprimera NbIndex fois la ligne correspondant à TabSup(0).
Si tu as TabSup(0) = 5, ça supprimera NbIndex fois la 6ème ligne sachant qu'à chaque suppression, toutes tes lignes se décalent !

Je ne sais pas comment tu contruits ton TabSup mais il y a plusieurs solutions :

Au lieu de stocker les indices en double, tu stockes directement dans une liste les DataGridViewRow à supprimer. Pour la suppression, tu fais

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
For Each aRow in TabSup
  gridview_tbl.Rows.Remove(aRow)
Next

Ou bien tu peux aussi trier ta liste des indices par ordre descendant afin de supprimer la collection en commençant par la fin.

[Thyyb]

Donc même si je fais comme ça

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
Array.Sort(tabsup)
Array.Reverse(tabsup)
For i = 0 To NbIndex - 1
       gridview_tbl.Rows.RemoveAt(tabsup(0))
Next

C'est pas bon c'est ça ? En fait je pose la question mais j'ai déjà essayé et ça ne fonctionne pas xD

[Uranne-jimmy]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
Array.Sort(tabsup)
Array.Reverse(tabsup)
For i = 0 To NbIndex - 1
       gridview_tbl.Rows.RemoveAt(tabsup(i))
Next

C'est la subtilité, remet bien le i à la place du 0.

[Thyyb]

Mais si je remet le i, je ne supprime qu'une ligne sur 2 non ? Vu que mon i va s'incrémenter et au contraire mon indice va "décrémenter" .

Sinon je viens d'entre apercevoir une lueur que je n'avais pas encore remarqué ...
Dans mon tabsup, j'ai ceci
20
20
19
19
18
18
.
.
10
9
8
7
J'ai aussi des doublons dans mon tabsup, donc quand je supprime la ligne indicé 20, ensuite il veut encore la supprimé mais elle n'existe plus. C'est ça mon problème je pense non

[ebastien]

Mais si je remet le i, je ne supprime qu'une ligne sur 2 non ? Vu que mon i va s'incrémenter et au contraire mon indice va "décrémenter" .

Tu confonds l'indice de ton tableau et l'indice de ta collection de rows. Tu ne supprimes pas les élément de ton tableau donc ...

J'ai aussi des doublons dans mon tabsup

C'est un problème en effet... Il faudrait savoir comment tu construits ton tableau car je pense que la méthode n'est pas au top !

[Thyyb]

Dans mon tableau je rangeais l'indice des lignes de mon dgv à supprimer donc en faisant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
dgv.Rows.RemoveAt(tabsup(0))

par exemple, je supprime la ligne dont l'indice est contenu à l'endroit 0 de mon tabsup de mon dgv non ?

Merci à tous, mon patron vient de me dire :
"Bon ba tu mets un ComboBox, dans le ComboBox tu mets le nom des champs et l'utilisateur fera sa recherche dans le champ qu'il choisi"
Ca fait 2 jours que je me casse la tête à faire une recherche dans touts les champs et la plus besoin. J'ai juste à faire une petite requête sql qui est toute facile... xD