probleme de codage

**lolo81** · 29/12/2007, 19h19

Ben moi impossible de compiler j'ai les erreurs :
error C2059: erreur de syntaxe : 'namespace'
error C2238: jetons inattendus avant ';'
fatal error C1075: fin de fichier rencontrée avant que l'élément accolade '{' gauche .

pas grave je finirai bien par trouver , cela dis j'en suis encore au
switch(code)
{

case CODE_i456:
action
break;
Alors c'est encore difficile de comprendre ton codage mais je compte bien y arriver.
@+

**lolo81** · 29/12/2007, 20h14

Ouf c bon plus de message d'erreur , mais je n'arrive pas a faire afficher la recherche et le résultat dans les texBox1 et2

**Albenejean** · 29/12/2007, 20h18

Donne ton code...
PS: utilise les balises [CODE] quand tu compose ton message pour insérer le code (le # au dessus de la zone où tu compose ton texte)

**lolo81** · 29/12/2007, 20h37

Voila mais j'ai enlevé mes modifs car rien ne fonctionne !
c'est sympa de m'aider mais j'aimerai progresser aussi alors si tu peus mettre les //commentaires ce serais super .

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
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
#pragma once
#include <iostream>
#include <map>
#include <string>
 
namespace codedefaut {
 
	using namespace System;
	using namespace System::ComponentModel;
	using namespace System::Collections;
	using namespace System::Windows::Forms;
	using namespace System::Data;
	using namespace System::Drawing;
 
	/// <summary>
	/// Summary for Form1
	///
	/// WARNING: If you change the name of this class, you will need to change the
	///          'Resource File Name' property for the managed resource compiler tool
	///          associated with all .resx files this class depends on.  Otherwise,
	///          the designers will not be able to interact properly with localized
	///          resources associated with this form.
	/// </summary>
	public ref class Form1 : public System::Windows::Forms::Form
	{
	public:
		Form1(void)
		{
			InitializeComponent();
			//
			//TODO: Add the constructor code here
			//
		}
 
	protected:
		/// <summary>
		/// Clean up any resources being used.
		/// </summary>
		~Form1()
		{
			if (components)
			{
				delete components;
			}
		}
	private: System::Windows::Forms::Button^  button1;
	protected: 
	private: System::Windows::Forms::TextBox^  textBox1;
	private: System::Windows::Forms::TextBox^  textBox2;
 
	private:
		/// <summary>
		/// Required designer variable.
		/// </summary>
		System::ComponentModel::Container ^components;
 
#pragma region Windows Form Designer generated code
		/// <summary>
		/// Required method for Designer support - do not modify
		/// the contents of this method with the code editor.
		/// </summary>
		void InitializeComponent(void)
		{
			this->button1 = (gcnew System::Windows::Forms::Button());
			this->textBox1 = (gcnew System::Windows::Forms::TextBox());
			this->textBox2 = (gcnew System::Windows::Forms::TextBox());
			this->SuspendLayout();
			// 
			// button1
			// 
			this->button1->Location = System::Drawing::Point(105, 116);
			this->button1->Name = L"button1";
			this->button1->Size = System::Drawing::Size(75, 23);
			this->button1->TabIndex = 0;
			this->button1->Text = L"Valider";
			this->button1->UseVisualStyleBackColor = true;
			this->button1->Click += gcnew System::EventHandler(this, &Form1::button1_Click);
			// 
			// textBox1
			// 
			this->textBox1->Location = System::Drawing::Point(92, 56);
			this->textBox1->Name = L"textBox1";
			this->textBox1->Size = System::Drawing::Size(100, 20);
			this->textBox1->TabIndex = 1;
			// 
			// textBox2
			// 
			this->textBox2->Location = System::Drawing::Point(29, 179);
			this->textBox2->Multiline = true;
			this->textBox2->Name = L"textBox2";
			this->textBox2->Size = System::Drawing::Size(239, 43);
			this->textBox2->TabIndex = 2;
			// 
			// Form1
			// 
			this->AutoScaleDimensions = System::Drawing::SizeF(6, 13);
			this->AutoScaleMode = System::Windows::Forms::AutoScaleMode::Font;
			this->ClientSize = System::Drawing::Size(292, 266);
			this->Controls->Add(this->textBox2);
			this->Controls->Add(this->textBox1);
			this->Controls->Add(this->button1);
			this->Name = L"Form1";
			this->Text = L"Form1";
			this->ResumeLayout(false);
			this->PerformLayout();
 
		}
#pragma endregion
	private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) {
			 }
	typedef map<string, string> TAction;
    TAction mapAction;
 
    //Création de la map
    mapAction.insert(TAction::value_type("i456", "contrôler le serrage de l'ampoule"));
    mapAction.insert(TAction::value_type("i457", "contrôler l'alimentation de l'ampoule"));
    mapAction.insert(TAction::value_type("i458", "contrôler le bouton"));
 
    //Exemple de recherche
    std::string a = "i456";
 
    if( mapAction.find(a) != mapAction.end() )
    {
        cout<<mapAction[a]<<std::endl;
    }
    else
    {
        cout<<"Cette action n'est pas reconnue\n";
    }
	};
}

**Albenejean** · 29/12/2007, 21h22

Hmmm ce n'est pas du C++ à proprement parler (et je n'utilise pas ce language), mais quelque chose comme ce qui suit devrais marcher sans trop d'effort de ta part:

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
 
private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) 
{
  // On définit un type (comme les int, float et autres qui s'appelle TAction
  typedef map<string, string> TAction;
 
  //Création de la map
  //  la map est en fait un tableau assiocioatif 
  //     on associe au premier texte le deuxieme text
  TAction mapAction;
  mapAction.insert(TAction::value_type("i456", "contrôler le serrage de l'ampoule"));
  mapAction.insert(TAction::value_type("i457", "contrôler l'alimentation de l'ampoule"));
  mapAction.insert(TAction::value_type("i458", "contrôler le bouton"));
 
  //on recherche maintenant
  // je suppose que textBox1 contien ton code de commande
  // et que tu affiche le résultat dans textBox2
  if( mapAction.find(textBox1->Text) != mapAction.end() )
  {
      // on a trouvé quelque chose (ie la fonction find s'est **arrété avant la fin de notre map)
      textBox2->Text = mapAction[textBox1->Text];
  }
  else
  {
      TextBox2->Text = "Cette action n'est pas reconnue";
  }
}

Attention toutefois, comme je ne connais pas le language, j'ai allègrement mélangé les std::string et les chaines de caractères de ton language. Il faut peut-être remplacé les string par le type qu'utilise ce language.

Enfin je te conseille de nouveau de commencer par des exemples simples, car il y a des fautes dans ton code qui ne devrait pas être là. par exemple :

Code :

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

1
2
3
4
private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) {
			 }
...
	};

Sinon (et c'est la solution la plus simple), tu reprend ton ancien code et tu remplace les "else if" par des simples "if" (cette solution est à mon avis la plus adapté dans un premier temps).

Rien ne t'empêche ensuite de te mettre à quelque chose de plus propre.
Et commence par des exemples simples dans ce language.

**lolo81** · 29/12/2007, 21h41

sans trop d'effort !! dis pas ça a ma femme car j'y suis tous les jours en train de lire sur mon ecran mais bon ce sont les tuto qui ne sont pas tres clair parfois et il faut bien comencer !!!

Code :

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

1
2
3
4
 
 // je suppose que textBox1 contien ton code de commande
  // et que tu affiche le résultat dans textBox2
  if( mapAction.find(textBox1->Text) != mapAction.end() )

Ayant jusqu'à présent utilisé les if /else ou if je n'est pas eu besoin de coder la textBox.Donc j'étais loin d'y arriver avec un switch !

avant de coder en else if j'ai essayé avec if mais le résultat est pire il accepte moins de conditions environ 10 maxi .
Dans mon cas je compte en mettre peut être 200 voire 400 codes donc les if c'est impossible j'ai essayé .
j'ai pensé a mettre un lien vers un fichier base mais je n'ai rien trouvé dans la jungle du codage c++.
En tout cas je te remercie pour aide j'y est passé tout mes congés ainsi que presque tous les soir et ce depuis quelques mois.

**Albenejean** · 29/12/2007, 21h54

Dans mon cas je compte en mettre peut être 200 voire 400 codes donc les if c'est impossible j'ai essayé .

Il n'y a pas de probleme pour faire ça.

if(code == "i456")
{
....
}
if(code=="i457")
{
...
}
...

L'erreur devais venir d'ailleurs.

**lolo81** · 29/12/2007, 21h59

t'est sur !!! ?
je re-teste ...

**lolo81** · 29/12/2007, 22h58

J'ai essayé avec les accolades ça marche pas sans ça fonctionne si c'est pas trop long ,
voila comment c codé avec les acolades

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
 
	 String^ a;
		 a="";
        a=textBox1->Text;			
 
        if(a == L"O006") 
			 {textBox2->Text = L"message1";}		 
		if(a == L"O007") 
			{textBox2->Text = L"message2"; }                 
         if(a == L"O008") 
			{textBox2->Text = L"message4";}
          else
			{textBox2->Text = L"Erreur code";}
}
	};
}

**Albenejean** · 29/12/2007, 23h14

Tu dois écrire la fonction dans un fichier source (.cpp) et pas dans un fichier header (.h). Le .h doit contenir la déclaration de la fonction et le .cpp doit contenir le code de la fonction.

De plus tes dernières lignes

Code :

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

1
2
3
4
5
6
		if(a == L"O007") 
			{textBox2->Text = L"message2"; }                 
         if(a == L"O008") 
			{textBox2->Text = L"message4";}
          else
			{textBox2->Text = L"Erreur code";}

ne sont pas correctes.

En effet si a="O007", alors if(a==L"O007") est vrai et donc message2 est donc bien mis dans le texte de texBox2. Par contre if(a==LO008) est faux et dans ce cas ce qui est après le else est executé et donc le texte de texBox2 va donc être Erreur code à la fin.

Il faut donc enlever le

Code :

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

1
2
else
{textBox2->Text = L"Erreur code";}

Ou bien faire quelque chose du genre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        String^ a = texBox1->Text;
        bool Result = false;	
 
        if(a == L"O006") 
			 {textBox2->Text = L"message1";
                           Result = true;}		 
        if(a == L"O007") 
			{textBox2->Text = L"message2";
                           Result = true; }                 
        if(a == L"O008") 
			{textBox2->Text = L"message4";
                           Result = true;}
        if(Result == false)
			{textBox2->Text = L"Erreur code";}

Note les quelques modifications que j'ai apportés.

**Médinoc** · 29/12/2007, 23h15

Il faut utiliser else if et non if tout seul:

Code C++/CLI :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
	String^ a;
	a="";
	a=textBox1->Text;			
 
	if(a == L"O006") 
		{textBox2->Text = L"message1";}		 
	else if(a == L"O007") 
		{textBox2->Text = L"message2"; }                 
	else if(a == L"O008") 
		{textBox2->Text = L"message4";}
	else
		{textBox2->Text = L"Erreur code";}
	}
};
}

**lolo81** · 29/12/2007, 23h26

Envoyé par Médinoc

Il faut utiliser else if et non if tout seul:

Code C++/CLI :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
	String^ a;
	a="";
	a=textBox1->Text;			
 
	if(a == L"O006") 
		{textBox2->Text = L"message1";}		 
	else if(a == L"O007") 
		{textBox2->Text = L"message2"; }                 
	else if(a == L"O008") 
		{textBox2->Text = L"message4";}
	else
		{textBox2->Text = L"Erreur code";}
	}
};
}

je l'ai fais hier soir mais au bout de plusieurs code (env 30)ça bugue ..

**Albenejean** · 29/12/2007, 23h29

Tu dois écrire la fonction dans un fichier source (.cpp) et pas dans un fichier header (.h).

Je pense que tout viens de là. Si effectivement tu à écris cette fonction dans le header (.h), tu dois le mettre dans un .cpp.

Tu pourras ensuite utiliser les if .. else if.

**Médinoc** · 29/12/2007, 23h31

Non, ce n'est pas vraiment important généralement.
Visual place par défaut le code des fonctions dans la classe, mais normalement ça n'empêce pas de marcher (même si ça reste une mauvaise pratique).

**lolo81** · 29/12/2007, 23h41

J'ai bien compris ton explication sur 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
 
        String^ a = texBox1->Text;
        bool Result = false;	
 
        if(a == L"O006") 
			 {textBox2->Text = L"message1";
                           Result = true;}		 
        if(a == L"O007") 
			{textBox2->Text = L"message2";
                           Result = true; }                 
        if(a == L"O008") 
			{textBox2->Text = L"message4";
                           Result = true;}
        if(Result == false)
			{textBox2->Text = L"Erreur code";}

je l'ai essayé et c sans succès même sur 2 code.
Et pour écrire la fonction dans le .ccp je suis pas capable et en plus je ne saurais pas faire le lien avec le bouton Valider !

**lolo81** · 30/12/2007, 00h35

Voila le seul message d'erreur avec avec plus de 100 codes :fatal error C1061: limite du compilateur : blocs imbriqués trop profondément
avec ce 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
 
	  String^ a;
		a="";
        a=textBox1->Text;
          if(a == L"O006") 
			{textBox2->Text = L"Message0";}
	      else if(a == L"O007") 
			{textBox2->Text = L"Message 1";}
          else if(a == L"O008") 
			{textBox2->Text = L"Message 2";}
	      else if(a == L"O009") 
			{textBox2->Text = L"Message 3";}
etc..........100 fois	
 
		  else
			{textBox2->Text = L"Erreur code";}
	}
	private: System::Void textBox1_TextChanged(System::Object^ sender, System::EventArgs^ e) {
	}
};
}

**Médinoc** · 30/12/2007, 00h52

OK, le compilo doit traiter la chaine de else if de la mauvaise façon.

Enfin, si la seule chose qui change c'est le message, tu peux utiliser une map (ou plutôt un System::Collections::Generic:ictionary< String^, String^ > et le remplir...

**lolo81** · 30/12/2007, 01h00

ok bon ça ira pour ce soir je vais mettre de la glace sur la tête et au doigts et je verrai çà demain !!

Merci en tous cas pour votre patience et votre aide !!
Bonne fin de soirée.

**lolo81** · 30/12/2007, 23h16

Envoyé par

Enfin, si la seule chose qui change c'est le message, tu peux utiliser une map (ou plutôt un [B

System::Collections::Generic:

ictionary< String^, String^ >[/B] et le remplir...

Bonsoir,
ou puis-je trouver un exemple fais avec VS c++

**Médinoc** · 30/12/2007, 23h28

Ça donne un truc de ce genre:

Code :

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

1
2
3
4
5
6
7
8
9
10
using namespace System::Collections::Generic;
 
void UneFonction(void)
{
	Dictionary< String^, String^ > dict = gcnew Dictionary< String^, String^ >();
	dict->Add(L"O007", L"Message1");
	//etc.
 
	textBox2->Text = dict[a];
}

Bien sûr, tu peux mettre le code d'initialisation du dictionnaire dans le constructeur, comme ça tu ne le fais qu'une fois mais tu le consulte chaque fois que nécessaire...

probleme de codage

C++

Discussions similaires

Partager

Partager