Discussion :

[Blodsvept]

Bonjour à tous,

J'ai un programme qui doit écrire dans un fichier texte selon ce que l'on rentre dans les textbox d'une IHM.
Voici le code :

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
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.IO;

namespace GénérationCDR
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            dateTimePickerD.Format = DateTimePickerFormat.Custom;
            dateTimePickerD.CustomFormat = "yyyyMMdd";
            dateTimePickerH.Format = DateTimePickerFormat.Custom;
            dateTimePickerH.CustomFormat = "HHmmss";
            
        }

        private void buttonQuitter_Click(object sender, EventArgs e)
        {
            Environment.Exit(0);
        }

        private void buttonGene_Click(object sender, EventArgs e)
        {
            StreamWriter Fichier = new StreamWriter("C:\\Users\\______\\test.txt", true);        
            //Fichier.WriteLine(nblignes);
            ClassGenerer obj1 = new ClassGenerer(Convert.ToInt32(textBoxNbCDR.Text), Convert.ToInt32(textBoxDuree.Text), Convert.ToInt32(textBoxNum1.Text), Convert.ToInt32(textBoxNum2.Text), Convert.ToInt32(dateTimePickerD.Text), Convert.ToInt32(dateTimePickerH.Text));
            Fichier.WriteLine(obj1.getnbligne + obj1.getduréeappel + obj1.getnumappelant + obj1.getnumappelé + obj1.getdateappel + obj1.getheureappel);
            MessageBox.Show("Le fichier a bien été généré !", "Génération");
            Fichier.Close();
        
        }
    }
}

Et le code de la classe :

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace GénérationCDR
{
    class ClassGenerer
    {
        //Attributs
        private int c_nblignes, c_duréeappel, c_numappelant, c_numappelé;
        private int c_dateappel, c_heureappel;
        //private string c_cdrid;
 
        //Constructeur
        public ClassGenerer(int nblignes, int duréeappel, int numappelant, int numappelé, int dateappel, int heureappel)
        {
            //c_cdrid = cdrid;
            c_nblignes = nblignes;
            c_duréeappel = duréeappel;
            c_numappelant = numappelant;
            c_numappelé = numappelé;
            c_dateappel = dateappel;
            c_heureappel = heureappel;
        }
 
        //Accesseurs
        public int getnbligne(int unNbLignes)
        {
            return c_nblignes;
        }
 
        public int getduréeappel(int uneDuréeAppel)
        {
            return c_duréeappel;
        }
 
        public int getnumappelant(int unNumAppelant)
        {
            return c_numappelant;
        }
 
        public int getnumappelé(int unNumAppelé)
        {
            return c_numappelé;
        }
 
        public int getdateappel(int uneDateAppel)
        {
            return c_dateappel;
        }
 
        public int getheureappel(int uneheureappel)
        {
            return c_heureappel;
        }
    }
}

La ligne en rouge pose problème, le message d'erreur étant : Erreur 1 L'opérateur '+' ne peut pas être appliqué aux opérandes de type 'groupe de méthodes' et 'groupe de méthodes' C:\Users\____\Form1.cs 37 31 ProjGénérationCDR

Je suis sûr que c'est une erreur absolument débile mais bon !

Merci d'avance !

[François M.]

Bonjour

Tu appelles comme méthodes comme si c'était des propriétés (donc sans () après le nom) .
Ca ne peut pas marcher.

Accessoirement, on ne voir pas l’intérêt d'une méthode qui se contente de retourner son paramètre : on a un peu de mal à comprendre ce que tu as voulu faire.

Quand on voit :

Code :

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

1
2
3
4
public int getheureappel(int uneheureappel)
        {
            return c_heureappel;
        }

on se demande à quoi ça peut servir :
- le paramètre ne sert à rien.
- une méthode qui retourne juste une variable membre, sans traitement d'aucune sorte, il est préférable de la remplacer par une propriété (le cas contraire est la signature de l'ex dev Java qui a mal digéré sa conversion et qui continue à utiliser des "getter" et des "setter").

Je pense que tu voulais faire ça :

Code :

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

1
2
3
4
5
6
7
public int heureappel
{
  get
     {
           return c_heureappel;
     }
}

[Blodsvept]

Bonjour,

En fait, je veux écrire une ligne dans un fichier .txt en fonction des valeurs des textbox, c'est ce que j'ai essayé de faire...
Et oui ce sont des méthodes, j'ai oublié les () en effet...

[François M.]

Bonjour,

En fait, je veux écrire une ligne dans un fichier .txt en fonction des valeurs des textbox, c'est ce que j'ai essayé de faire...
Et oui ce sont des méthodes, j'ai oublié les () en effet...

J'ai édité mon message; relis le. Ce sont des méthodes mais qui n'ont pas de sens.

[Blodsvept]

Oui en effet ça n'a pas de sens...
J'ai essayé ton code, visual studio m'affiche plusieurs erreurs dessus.

Ce que j'ai voulu faire dans ces deux lignes :

Code :

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

1
2
ClassGenerer obj1 = new ClassGenerer(Convert.ToInt32(textBoxDuree.Text), Convert.ToInt32(textBoxNum1.Text), Convert.ToInt32(textBoxNum2.Text), Convert.ToInt32(dateTimePickerD.Text), Convert.ToInt32(dateTimePickerH.Text));
            Fichier.WriteLine(obj1.getduréeappel + obj1.getnumappelant + obj1.getnumappelé + obj1.getdateappel + obj1.getheureappel);

C'était de récupérer les valeurs de mes textbox et de les écrire dans "Fichier" mais je dois avouer que je me suis perdu en route...

[François M.]

Ce que j'ai voulu faire dans ces deux lignes :

Code :

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

1
2
ClassGenerer obj1 = new ClassGenerer(Convert.ToInt32(textBoxDuree.Text), Convert.ToInt32(textBoxNum1.Text), Convert.ToInt32(textBoxNum2.Text), Convert.ToInt32(dateTimePickerD.Text), Convert.ToInt32(dateTimePickerH.Text));
            Fichier.WriteLine(obj1.getduréeappel + obj1.getnumappelant + obj1.getnumappelé + obj1.getdateappel + obj1.getheureappel);

C'était de récupérer les valeurs de mes textbox et de les écrire dans "Fichier" mais je dois avouer que je me suis perdu en route...

Non dans le fichier tu écris la somme des valeurs, car tu les additionnes (si tu appelles les méthodes correctement - méthodes qui devraient être remplacée par des propriétés), je doute fort que ce soit ce que tu veux faire.

[Blodsvept]

C'est ce que je veux faire mais je n'y arrive pas c'est ça le truc...

[François M.]

C'est ce que je veux faire mais je n'y arrive pas c'est ça le truc...

Dans quel format veux tu écrire dans ton fichier ?
Une valeur par ligne ? les valeur concaténée avec une séparation ?

Merci de préciser ce que tu attends dans le fichier, on est pas devin.

[Blodsvept]

Oui, les valeurs concaténées avec ";" comme séparateur entre chaque donnée !

[Blodsvept]

C'est bon j'ai fini par comprendre !

Code :

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

 Fichier.WriteLine(obj1.getduréeappel() + "; " + obj1.getnumappelant() + "; " + obj1.getnumappelé() + "; " + obj1.getdateappel() + "; " + obj1.getheureappel() + ";");

et

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
public int getduréeappel()
        {
            return c_duréeappel;
        }
 
        public int getnumappelant()
        {
            return c_numappelant;
        }
 
        public int getnumappelé()
        {
            return c_numappelé;
        }
 
        public int getdateappel()
        {
            return c_dateappel;
        }
 
        public int getheureappel()
        {
            return c_heureappel;
        }

Merci à toi François pour l'aide !