[C#] Problème de déclenchement

Voilà j'ai fait un petit prog pour calculer les moyennes :

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
		private void button1_Click(object sender, System.EventArgs e)
		{
			nbreNotes=Int32.Parse(txtNbreNotes.Text);
			notes=new double[nbreNotes];
			i=0;
		}
 
		private void btnValidSaisie_Click(object sender, System.EventArgs e)
		{
 
			if(i<nbreNotes)
			{
				notes[i]=Double.Parse(txtNoteI.Text);
				lstNotes.Items.Add("Note n°"+(i+1).ToString()+" : "+notes[i].ToString());
				somme+=notes[i];
				i++;
			}
			else
			{
				MessageBox.Show("Fin de saisie !","Information saisie",MessageBoxButtons.OK,MessageBoxIcon.Exclamation);
				moyenne=calcul();
				label1.Text+=moyenne.ToString();			
			}
 
		}
 
		private void btnCalcul_Click(object sender, System.EventArgs e)
		{
 
		}
 
		private double calcul()
		{
			return (moyenne=somme/nbreNotes);
		}

Mais le problème est qu'il me faut si j'ai deux notes cliquer trois fois sur saisir note ...

Avez vous une idée pour que mon if soit respecté ?

Merci beaucoup

salut,
tu n'as pas un problème d'initialisation avec ton "i"?
PS : ça n'as rien à voir mais c'est normal que tu mettes en paramètre un object sender (et un System.EventArgs e) à chaque fois et que tu t'en serves jamais?

Citation:

Envoyé par trotters213

PS : ça n'as rien à voir mais c'est normal que tu mettes en paramètre un object sender (et un System.EventArgs e) à chaque fois et que tu t'en serves jamais?

Biensur que c'est normal ... Ce code est généré par le concepteur.

je suis débutant dans le domaine donc je demande juste, ça m'étonné juste de voir ça si on s'en sert pas. :oops:

C'est vrai que ca peut surprendre au départ .... mais c'est comme si tu utilises une fonction qui te renvoie pleins de valeurs mais que tu sers seulement de l'une d'entre elles. :)

j'avais pensé utiliser un delegate mais je ne sais où le placer ou le déclarer :(

j'ai réécrit le tout de la sorte :

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
using System;
using System.Drawing;
using System.Collections;
using System.ComponentModel;
using System.Windows.Forms;
using System.Data;
using System.Globalization;
 
namespace MoyenneDimitri
{
	/// <summary>
	/// Description résumée de Form1.
	/// </summary>
	public class Form1 : System.Windows.Forms.Form
	{
		private System.Windows.Forms.Button button1;
		private System.Windows.Forms.TextBox textBox1;
		private System.Windows.Forms.Label label1;
		private System.Windows.Forms.ListBox lstNotes;
		private System.Windows.Forms.Button button2;
		/// <summary>
		/// Variable nécessaire au concepteur.
		/// </summary>
		private System.ComponentModel.Container components = null;
 
		public Form1()
		{
			//
			// Requis pour la prise en charge du Concepteur Windows Forms
			//
			InitializeComponent();
 
			//
			// TODO : ajoutez le code du constructeur après l'appel à InitializeComponent
			//
			nbreNotes=0;
			somme=0;
		}
 
		/// <summary>
		/// Nettoyage des ressources utilisées.
		/// </summary>
		protected override void Dispose( bool disposing )
		{
			if( disposing )
			{
				if (components != null) 
				{
					components.Dispose();
				}
			}
			base.Dispose( disposing );
		}
 
		#region Code généré par le Concepteur Windows Form
		/// <summary>
		/// Méthode requise pour la prise en charge du concepteur - ne modifiez pas
		/// le contenu de cette méthode avec l'éditeur de code.
		/// </summary>
		private void InitializeComponent()
		{
			this.lstNotes = new System.Windows.Forms.ListBox();
			this.button1 = new System.Windows.Forms.Button();
			this.textBox1 = new System.Windows.Forms.TextBox();
			this.label1 = new System.Windows.Forms.Label();
			this.button2 = new System.Windows.Forms.Button();
			this.SuspendLayout();
			// 
			// lstNotes
			// 
			this.lstNotes.Location = new System.Drawing.Point(32, 24);
			this.lstNotes.Name = "lstNotes";
			this.lstNotes.Size = new System.Drawing.Size(96, 407);
			this.lstNotes.TabIndex = 0;
			// 
			// button1
			// 
			this.button1.Location = new System.Drawing.Point(184, 96);
			this.button1.Name = "button1";
			this.button1.Size = new System.Drawing.Size(112, 23);
			this.button1.TabIndex = 1;
			this.button1.Text = "Ajouter la note";
			this.button1.Click += new System.EventHandler(this.button1_Click);
			// 
			// textBox1
			// 
			this.textBox1.Location = new System.Drawing.Point(184, 48);
			this.textBox1.Name = "textBox1";
			this.textBox1.TabIndex = 2;
			this.textBox1.Text = "";
			// 
			// label1
			// 
			this.label1.Location = new System.Drawing.Point(40, 464);
			this.label1.Name = "label1";
			this.label1.Size = new System.Drawing.Size(272, 23);
			this.label1.TabIndex = 3;
			// 
			// button2
			// 
			this.button2.Location = new System.Drawing.Point(192, 192);
			this.button2.Name = "button2";
			this.button2.Size = new System.Drawing.Size(112, 23);
			this.button2.TabIndex = 4;
			this.button2.Text = "Fin de saisie";
			this.button2.Click += new System.EventHandler(this.button2_Click);
			// 
			// Form1
			// 
			this.AutoScaleBaseSize = new System.Drawing.Size(5, 13);
			this.ClientSize = new System.Drawing.Size(336, 614);
			this.Controls.Add(this.button2);
			this.Controls.Add(this.label1);
			this.Controls.Add(this.textBox1);
			this.Controls.Add(this.button1);
			this.Controls.Add(this.lstNotes);
			this.Name = "Form1";
			this.Text = "Form1";
			this.ResumeLayout(false);
 
		}
		#endregion
 
		/// <summary>
		/// Point d'entrée principal de l'application.
		/// </summary>
		[STAThread]
		static void Main() 
		{
			Application.Run(new Form1());
		}
 
		/// <summary>
		/// //Nombre de notes
		/// </summary>
		public int nbreNotes;
 
		private void button1_Click(object sender, System.EventArgs e)
		{
			lstNotes.Items.Add(textBox1.Text);
			textBox1.Text="";
		}
 
		private void button2_Click(object sender, System.EventArgs e)
		{
			nbreNotes=lstNotes.Items.Count;
			label1.Text=nbreNotes.ToString();
 
			button1.Enabled=!button1.Enabled;
 
			NumberFormatInfo nfi = new CultureInfo( "fr-FR", false ).NumberFormat;
			nfi.NumberDecimalSeparator = ".";
 
			double[] notes=new double[nbreNotes];
			for(int i=0;i<nbreNotes;i++)
			{
				notes[i]=double.Parse(lstNotes.Items[i].ToString(),nfi);
				somme+=notes[i];
			}
			double moyenne=0;
			moyenne=CalculMoyenne();
			label1.Text="Nombre de notes : "+nbreNotes.ToString()+" - Moyenne : "+moyenne.ToString();
			button2.Enabled=!button2.Enabled;
		}
 
		/// <summary>
		/// //Somme
		/// </summary>
		public double somme;
 
		/// <summary>
		/// //Calcul de la moyenne
		/// </summary>
		public double CalculMoyenne()
		{	
			return(somme/nbreNotes);
		}
	}
}

Ca ne me plait guère mais ca fonctionne