Discussion :

[altahir007]

Bonjour,

J'ai une structure contenant plein d'info, j'affiche le contenu de mes differents enregistrement de ma structure dans un datagridview. J'aimerai donner la possibilité a l'utilisateur de monter un ligne ou de la descendre en gros d'ordonner la liste de mastructure.

Pour monter, j'ai un code qui ressemble à ça, j'ai pourtant parfois un resultat assez etrange comme si l'une des lignes remontai en derniere position alors que j'inverse uniquement deux lignes du centre ????

Code :

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

1
2
3
4
5
6
7
8
9
 
            savepos = DataGridView1.CurrentCell.RowIndex
            fichier.Mestest.Add(New MaStructureTest) 'on ajoute un test vierge
            fichier.Mestest(fichier.Mestest.Count - 1) = fichier.Mestest(DataGridView1.CurrentCell.RowIndex - 1)
            fichier.Mestest(DataGridView1.CurrentCell.RowIndex - 1) = fichier.Mestest(DataGridView1.CurrentCell.RowIndex)
            fichier.Mestest(DataGridView1.CurrentCell.RowIndex) = fichier.Mestest(fichier.Mestest.Count - 1)
            fichier.Mestest.Remove(fichier.Mestest(fichier.Mestest.Count - 1))
            RefreshTestList() 'fonction pour raffraichir le contenu de mon datagridview
            DataGridView1.CurrentCell = DataGridView1.Rows(savepos - 1).Cells(0)

[asmduty]

Bonjour,

pourrait-on savoir ce que sont "fichier", "savepos" et "Mestest" ?

[altahir007]

fichier c'est une structure incluant un liste de mestests dedans

savepos est un entier déclaré juste avant qui me permet de sauvegarder la position de l'enregistrement selectionné juste avant d'inverser mes lignes. Le but etant à la fin apres avoir rafraichi le datagrigview de le selectionner de nouveau.

mestest est une structure avec plein de paramètres x et y ...

Le but a la fin est de sérialiser dans XML ma structure fichier pour sauvegarder tout ça.

[StringBuilder]

Inspire-toi de cet exemple :

Code :

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

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
 
using System;
using System.Collections.Generic;
 
namespace SwapList
{
    class Program
    {
        static void Main(string[] args)
        {
            List<Test> lst = new List<Test>(10);
 
            for (int i = 0; i < 10; i++)
            {
                lst.Add(new Test("Ligne " + i.ToString()));
            }
 
            Console.WriteLine("Ordre original :");
            foreach (Test t in lst)
            {
                Console.WriteLine(t);
            }
 
            Console.WriteLine("Swap de la ligne 3 et de la ligne 6");
            Swap<Test>(lst, 3, 6);
 
            Console.WriteLine("Ordre après changement :");
            foreach (Test t in lst)
            {
                Console.WriteLine(t);
            }
 
            Console.ReadKey(true);
        }
 
        private static void Swap<T>(List<T> l, int idx1, int idx2)
        {
            T t = l[idx1];
            l.Insert(idx1, l[idx2]);
            l.RemoveAt(idx1 + 1);
            l.RemoveAt(idx2);
            l.Insert(idx2, t);
        }
    }
 
    class Test
    {
        private string _txt;
 
        public Test(string txt)
        {
            _txt = txt;
        }
 
        public override string ToString()
        {
            return _txt;        
        }
    }
}

-> Au "static" près, tu devrais pouvoir reprendre ma méthode Swap.

[asmduty]

Code :

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

1
2
fichier.Mestest(DataGridView1.CurrentCell.RowIndex - 1) = fichier.Mestest(DataGridView1.CurrentCell.RowIndex)
fichier.Mestest(DataGridView1.CurrentCell.RowIndex) = fichier.Mestest(fichier.Mestest.Count - 1)

C'est la logique qui n'est pas bonne

Tu as 4 lignes :

Code :

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

1
2
3
4
1 - A
2 - B
3 - C
4 - D

Tu est positionné sur C
Tu descends B à l'index de C, tu obtiens

Code :

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

1
2
3
4
1 - A
2 - C
3 - B
4 - D

Ensuite tu dis que l'index de C est égal au dernier index

Code :

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

1
2
3
4
1 - A
2 - B
3 - D
4 - C

De plus ton code est un peu brouillon puisque d'un côté tu te base sur le datagridview, et de l'autre sur ton objet.

Si tu cherches juste à inverser la ligne sélectionnée avec celle du dessus, ta première ligne suffit :

Code :

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

fichier.Mestest(DataGridView1.CurrentCell.RowIndex - 1) = fichier.Mestest(DataGridView1.CurrentCell.RowIndex)

[altahir007]

Le soucis est uniquement au niveau de la suppression de mon enregistrement temporaire placé en fin de liste, je n'utilisai pas la bonne fonction :

voici la bonne :

Code :

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

1
2
 
fichier.Mestest.RemoveAt(fichier.Mestest.Count - 1)

Ensuite mon code n'etais pas vraiment propre effectivement j'ai corrigé ça en utilisant uniquement le savepos.

Merci pour votre aide

[StringBuilder]

Je maintiens qu'avec une méthode Swap(liste, index1, index2) ce sera encore plus propre.