Souci de conception dans mon appli

Version imprimable

2 pièce(s) jointe(s)

Salut !

J'ai un souci avec mon appli; je ne parviens pas à coder ce dont j'ai besoin.

Je vous explique le rôle de l'appli et le passage périlleux :

L'appli doit lire une liste de fichiers mp3 dans un répertoire x, en extraire les artistes et les titres. Ensuite, ça remplit une série de textbox avec les artistes et titres mentionnés :

Pièce jointe 459280

(Pour la suite de l'histoire : Ensuite, via le bouton 4 ("Trier et zipper" ci-dessus (géré dans la Form6)), l'appli copie les fichiers vers un sous-dossier de x, enlève les tags, et renomme les fichiers en 01, 02, 03,... .mp3, zippe le tout et crée un fichier playlist.txt. Mais tout ça fonctionne correctement !)

Là, où ça coince, c'est que le user a la possibilité de renommer les artistes et les titres via les textbox. Et en conséquence, les noms de fichiers mp3 dans les dossiers x et sous-dossier de x correspondants doivent être renommés également.
Et autre complication en vue, le user peut modifier l'ordre des extraits; image ci-dessous :

Pièce jointe 459292

Je poste ci-dessous, le code du bouton 4 ("Trier et zipper" dans la 1ère image ci-dessus).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 private void button4_Click(object sender, EventArgs e) { int j = 0; String[] files2 = Directory.GetFiles(folder); foreach (String f2 in files2) { if (f2.EndsWith(".mp3")) { String morceau = f2.Substring(f2.LastIndexOf("\\") + 1); for (j = 0; j < _nbrextrait; j++) { if (morceau == tabtb[j, 0].Text + " - " + tabtb[j, 1].Text + ".mp3") { File.Copy(folder + @"\" + morceau, folder + @"\Programme BT\" + tabtb[j, 0].Text + " - " + tabtb[j, 1].Text + ".mp3", true); File.Move(folder + @"\" + morceau, folder + @"\" + tabtb[j, 0].Text + " - " + tabtb[j, 1].Text + ".mp3"); } } } } }
'tabtb' est un tableau à 2 dimensions qui contient les artistes et les titres.
Les lignes 10 et 12 gèrent le changement de l'ordre des extraits; ce code fonctionne.
Et c'est bien dans les lignes 14 et 15 qu'il manque qqch pour que ça fonctionne (comment trouver le bon indice 'j' qui corresponde à 'morceau', bien que 'tabtb[j,0]' ne corresponde plus à 'morceau'). Lors du premier passage tout se passe bien, mais s'il y a plus d'une modification, ça coince ! En gros, la prise en compte de la modification des artistes/titres bugue...
Et je ne vois vraiment pas comment faire pour résoudre ça :(

Merci d'avance pour votre aide. ;)

19/03/2019, 15h00
ebastien
Bonjour,

Je te conseillerais de travailler en objet et non pas avec un tableau. Cela te permettra de mieux séparer les choses.
Un objet Mp3file pourrait comporter le nom de l'artiste, le titre, etc.. et le nom de fichier associé. Tu aurais du coup une liste de Mp3file.
De plus tu peux remplacer tes Texbox par des UserControls comportant 2 TextBox. Un UserControl par Mp3file (Il peut y avoir une propriété dans le UserControl pour stocker le Mp3file). Cela te permettrait de gérer ton Mp3file si une modification est effectuée au niveau des Textbox. Ensuite pour la copie, toutes les infos sont correctes et présentes dans chaque Mp3file.

Pour info, tu as dans la classe System.IO.Path tout ce qu'il faut pour gérer les noms de fichier ce qui évitera l'utilisation de ce genre de chose qui pique un peu les yeux :
Code:

1 2 3 4 5 f2.Substring(f2.LastIndexOf("\\") + 1) // peut être remplacé par Path.GetFileName(f2) // de même : // folder + @"\" + morceau <=> Path.Combine(folder,morceau)
En espérant t'avoir un peu éclairé...
19/03/2019, 16h24
Mike888

Je prends bonne note de tout ce que tu me dis ! ;)

A quoi penses-tu quand tu parles de "UserControls" ? :?
19/03/2019, 16h56
Mike888

En fait, vu que j'ai 'nbrextrait' extraits, il me faudrait 'nbrextrait' usercontrols alors ?

Je ne suis pas habitué aux usercontrols (je ne les ai jamais utilisés pour dire vrai)... :?

Pourrais-tu également expliciter cette phrase :

Citation:

Il peut y avoir une propriété dans le UserControl pour stocker le Mp3file

Merci !
19/03/2019, 17h29
Mike888

Okay, je vois à peu près ce que tu voulais dire; je modifie mon code et je reviens vers vous ! :)

Je suis quand même un peu perdu avec les usercontrols...

J'ai créé le usercontrol qui contient 2 textbox; la classe est donc créée aussi, mais je ne vois pas trop ce que je dois mettre dedans... :?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public partial class UCExtrait : UserControl
    {
        public UCExtrait()
        {
            InitializeComponent();
        }
 
        private void UCExtrait_Load(object sender, EventArgs e)
        {
 
        }
 
       ??????
 
    }

Edit : Je pense avoir saisi; j'ai rajouté ceci :

Code:

1
2
public String Txtb1 { get => textBox1.Text; set => textBox1.Text = value; }
        public String Txtb2 { get => textBox2.Text; set => textBox2.Text = value; }

Bonjour,
Je crains que ça ne finisse en plat de spaghetti ton histoire ;-)
Ce que je te proposais c'était :

- De faire une classe Mp3File contenant uniquement les informations du fichier Mp3 (On ne met pas de UserControl dans une classe métier) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
    public class Mp3File
    {
        public Mp3File(String fileName)
        {
            FileName = fileName;
            ReadTags();
        }
        public String Artiste { get; set; }
        public String Titre { get; set; }
        public String FileName { get; }
// Ajouter d'autres propriétés si besoin !
        private void ReadTags()
        {
            // Méthode qui lira les tags du fichier afin de renseigner Artiste, Titre etc...
        }
// Prévoir une méthode WriteTags() pour modifier les tags et éventuellement RenameFile() pour renommer le fichier si besoin
 
    }

- De créer un UserControl afin de représenter tes fichiers Mp3 :

Code:

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
 
    public partial class UCExtrait : UserControl
    {
        private Mp3File _mp3File;
        public Mp3File File
        {
            get
            {
                return _mp3File;
            }
            set
            {
                _mp3File = value;
                Initialize(value);
            }
        }
 
        public UCExtrait()
        {
            InitializeComponent();
        }
 
        public UCExtrait(Mp3File mp3File)
        {
            InitializeComponent();
            File = mp3File;
        }
 
        private void Initialize(Mp3File file)
        {
            tbxTitre.Text = file.Titre;
            tbxArtiste.Text = file.Artiste;
        }
 
        private void tbxTitre_TextChanged(object sender, EventArgs e)
        {
            _mp3File.Titre = tbxTitre.Text;
        }
 
        private void tbxArtiste_TextChanged(object sender, EventArgs e)
        {
            _mp3File.Artiste = tbxArtiste.Text;
        }
 
    }

- Enfin de construire ton écran depuis ta Form :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 // Si nécessaire, liste des Mp3 :
        List<Mp3File> ListMp3 = new List<Mp3File>();
        private void BuildMp3UC(String folder)
        {
            ListMp3.Clear();
            String[] files2 = Directory.GetFiles(folder);
            int counter = 0;
            foreach (String f2 in files2)
            {
                if (f2.EndsWith(".mp3"))
                {
                    Mp3File file = new Mp3File(f2);
                    ListMp3.Add(file);
                    UCExtrait UCE = new UCExtrait(file);
                    UCE.Left = 30;
                    UCE.Top = 15 + (counter * 20);
                    UCE.Size = new Size(250, 20);
                    counter++;
                    this.Controls.Add(UCE);
                }
            }
        }

Bon c'est juste une trame à travailler. Je n'ai absolument pas testé le code que j'ai écrit et c'est uniquement pour te donner une idée.
Le principe étant de séparer les objets (Mp3File) des IHM (Form et UserControl).

Amuse toi bien ;-)

20/03/2019, 11h00
Mike888

Bon, merci une fois de plus de t'intéresser à mon problème.

Je vais étudier tout cela en détail !

Donc, j'ai mis en place ce que tu m'as recommandé !
J'ai un peu de mal encore à comprendre le rôle du usercontrol... Si je te suis bien, les artistes et les titres vont exister en "double" (une fois dans la classe MP3File et une fois dans le usercontrol)...

Dans mon code, j'ai donc écrit ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
public class MP3File
    {
        private String _Artiste;
        private String _Titre;
        private String _Nomfichier;        
 
        public string Artiste { get => _Artiste; set => _Artiste = value; }
        public string Titre { get => _Titre; set => _Titre = value; }
        public string Nomfichier { get => _Nomfichier; set => _Nomfichier = value; }        
 
        public MP3File(String Nomfichier)
        {
            _Nomfichier = Nomfichier;
        }        
    }

Note : Je ne gère pas les tags puisque le but dans cette appli est de les supprimer (je fais cela plus loin dans le code)...

Code:

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
 
public partial class UCExtrait : UserControl
    {
        private MP3File _mp3File;
 
        public UCExtrait()
        {
            InitializeComponent();
        }
 
        public UCExtrait(MP3File mp3file)
        {
            InitializeComponent();
            _mp3File = mp3file;
        }
 
        private void UCExtrait_Load(object sender, EventArgs e)
        {
 
        }
 
        public MP3File File
        {
            get
            {
                return _mp3File;
            }
            set
            {
                _mp3File = value;
                Initialize(value);
            }
        }
 
        private void Initialize(MP3File file)
        {
            textBox1.Text = file.Artiste;
            textBox2.Text = file.Titre;
        }        
 
        private void textBox1_TextChanged(object sender, EventArgs e)
        {
            _mp3File.Artiste = textBox1.Text;
        }
 
        private void textBox2_TextChanged(object sender, EventArgs e)
        {
            _mp3File.Titre = textBox2.Text;
        }
    }

Note : Dans la ligne 14, j'ai remplacé "File" par "_mp3file"; je pense que tu avais laissé une erreur.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
...
              foreach (String f in files)
                {
                    if (f.EndsWith(".mp3"))
                    {
                        MP3File file1 = new MP3File(f);
                        listmp3.Add(file1);
                        UCExtrait UCE = new UCExtrait(file1);
                        UCE.Left = 30;
                        UCE.Top = 10 + (i * 20);
                        UCE.Size = new Size(550, 20);
                        panel1.Controls.Add(UCE);
                        String morceau = f.Substring(f.LastIndexOf("\\") + 1);
                        file1.Artiste = morceau.Substring(0, morceau.IndexOf(" - "));
                        file1.Titre = morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Substring(0, morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Length - 4);
                        UCE.File.Artiste = morceau.Substring(0, morceau.IndexOf(" - "));
                        UCE.File.Titre = morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Substring(0, morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Length - 4);
                        i++;
                    }
                }

Dans les lignes 17 et 18, est-ce bien de cette façon que je dois affecter les artistes et titres ? J'ai pas l'impression perso, vu le code dans le usercontrol... :?
Pour le moment, écrit comme ça, rien ne s'affiche dans les textbox...
Je continue à chercher !

20/03/2019, 13h47
Mike888

Okay, byebye les 17 et 18 (qui ne font que répéter les 15 et 16); j'ai par contre rajouté ceci :

Code:

UCE.File = file1;

Allez, je m'accroche et je continue ! :mrgreen:

Pour répondre rapidement à tes interrogations :

Un UserControl, c'est comme n'importe quel Control. Tu peux mettre ce que tu veux dedans. Là dans ce cas, il permet de positionner 2 Textbox pour l'affichage et la saisie des informations d'un fichier MP3. L'avantage c'est qu'en utilisant un UserControl, tu ne codes l'affichage et la saisie que de 2 TextBox pour un MP3. Si tu as 10 fichiers MP3, tu multiplies les UserControls mais pas la gestion des affichages puisque chaque UserControl gère son affichage.

Citation:

Note : Je ne gère pas les tags puisque le but dans cette appli est de les supprimer (je fais cela plus loin dans le code)...

Ok, donc si je comprends bien, tu récupères l'artiste et le titre dans le nom du fichier. Alors fais le dans le constructeur de l'objet Mp3File. Cette objet doit gérer tout ce qui concerne le fichier Mp3. Cela ne doit pas être fais à l'exterieur.

Citation:

Note : Dans la ligne 14, j'ai remplacé "File" par "_mp3file"; je pense que tu avais laissé une erreur.

Non ce n'était pas une erreur. File étant une propriété, lorsqu'elle est affectée en ligne 14, le code du setter est exécuté et c'est dans ce code que j'appelais la méthode Initialize() qui mettait à jour les TextBox. Mais pour cela, il faut que les propriétés Artiste et Titre de l'objet Mp3File soit renseignées. (voir ce que j'ai écrit plus haut).

Dans le code que tu as écrit :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 MP3File file1 = new MP3File(f); listmp3.Add(file1); UCExtrait UCE = new UCExtrait(file1); UCE.Left = 30; UCE.Top = 10 + (i * 20); UCE.Size = new Size(550, 20); panel1.Controls.Add(UCE); String morceau = f.Substring(f.LastIndexOf("\\") + 1); file1.Artiste = morceau.Substring(0, morceau.IndexOf(" - ")); file1.Titre = morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Substring(0, morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Length - 4); UCE.File.Artiste = morceau.Substring(0, morceau.IndexOf(" - ")); UCE.File.Titre = morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Substring(0, morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Length - 4); i++;
Tu affectes 2 fois l'artiste et le titre au MP3File (file1 et UCE.File pointent sur le même objet). Et en plus ce n'est pas à ce niveau qu'il faut les renseigner mais au sein de la classe Mp3File. L'idée c'est que je dois pouvoir créer un objet Mp3File n'importe où dans le code sans avoir à me préoccuper de savoir comment l'Artiste ou le Titre est renseigné. Ce n'est pas le problème du créateur de l'objet mais celui de l'objet lui-même. Il faut l'encapsuler. De ce fait, tu n'auras pas de duplication de code et l'objet Mp3File devient une boite noire. De plus si tu veux ajouter un jour le nom de l'album, il n'y aura qu'une ligne de code à ajouter dans ton objet. Sinon, il faudra que tu analyses l'intégralité de ton application pour savoir où ajouter le nom de l'album. En fait c'est un peu le principe de base de la POO.

J'espère avoir été à peu près clair ;-)

Note au passage :

morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Substring(0, morceau.Substring(morceau.IndexOf(" - ")).Substring(3).Length - 4); :calim2:

Il est construit comment le nom de fichier ? Tu n'as pas plus simple ? Une expression régulière peut-être ?

Well, j'ai mis tout mon code à jour et ai modifié certaines choses maintenant que je vois plus clair...

J'ai néanmoins encore une question après avoir trié aléatoirement ma liste d'extraits ('listmp3') :

Comment mettre les textbox à jour ? Ca se fait dans la classe UCExtrait.
Code:

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 public partial class UCExtrait : UserControl { private MP3File _mp3File; public UCExtrait() { InitializeComponent(); } public UCExtrait(MP3File mp3file) { InitializeComponent(); File = mp3file; } private void UCExtrait_Load(object sender, EventArgs e) { } public MP3File File { get { return _mp3File; } set { _mp3File = value; Initialize(value); } } private void Initialize(MP3File file) { textBox1.Text = file.Artiste; textBox2.Text = file.Titre; } private void textBox1_TextChanged(object sender, EventArgs e) { _mp3File.Artiste = textBox1.Text; } private void textBox2_TextChanged(object sender, EventArgs e) { _mp3File.Titre = textBox2.Text; } }
C'est donc via la property "public MP3File File" (qui appelle la méthode "Initialize") que ça se passe; mais comment l'appeler depuis ailleurs ?
J'ai bien tenté ceci :

Code:

UCExtrait UCE = new UCExtrait(m); // m est une instance de MP3File

Mais les textbox ne changent pas... :?

A part cela, je n'ai pas d'autres questions pour le moment :D

EDIT 1 :

Je crois que j'ai capté ce qui cloche : l'appli fonctionne bien mais elle met à jour les mêmes textbox liées aux objets (forcément, on a l'impression que rien ne change...).

EDIT 2 :

En fait, il faut recharger le panel avec les usercontrols ! Mais comment supprimer chaque usercontrol ajouté au panel (comment y accéder ?) ?

Code:

panel1.Controls.Remove();

Que mettre comme paramètre à Remove() ?

EDIT 3 :

Je pensais avoir trouvé la solution avec ceci :
Code:

1 2 3 4 foreach (Control ucext in panel1.Controls.OfType<UCExtrait>()) { panel1.Controls.Remove(ucext); }
Mais il m'enlève seulement un usercontrol sur deux !
Je pige plus rien là ! :aie:

21/03/2019, 09h26
Mike888

Bon, j'ai par facilité simplement vidé le panel et l'ai re-rempli avec les controls !

Eh bien, je crois que je vois enfin le bout du tunnel.
Ebastien, je te dois une fière chandelle; sans toi, je n'y serais jamais parvenu ! Je suis admiratif de ta capacité à t'approprier un code qui n'est pas le tien et à le rendre plus élégant et surtout plus pratique !

1000 mercis à toi ! :applo: