Discussion :

[Armulis]

Salut!

Je rencontre des soucis pour appeler une fonction-membre d'une classe, implémentée comme suit:

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
void MenuColonneCentree::SetContenu(vector<string> argCONTENU)
{   int calc1(argCONTENU.size());
    options=(calc1-(calc1%2))/2;
 
    ligneSelectionnee=0; // Présélection de la première ligne
 
    for (int index(0); index < calc1; index++)
    {   texte[index].setString(argCONTENU[index]);
        posY[index]=(7-options+index)*hauteurTextePrincipal;
    }
 
    etatAnimation=false;
    etatExistence=true;
 
    #ifdef SUIVI
    cout << "MenuColonneCentree, SetContenu OK" << endl;
    #endif
}

L'appel est fait comme ça:

Code :

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

MenuColonneCentree_1.SetContenu({ "", "", "", "", "", "", "", "", "", "LOADING"+points });

ATTENTION! "points" est de type string =D

Mais le compilateur ne l'accepte pas.
L'erreur associée à l'appel est la suivante: "error: no matching function tu call to 'MenuColonneCentre::SetContenu(<brace-enclosed initializer list<)'"

Je suis en train de réécrire mon programme, et le problème vient peut-être d'ailleurs. Si vous pensez que mon appel est correct, alors je verrai ça après avoir terminé la réécriture.

[jo_link_noir]

Cet appel est valide, quelle version du compilo ?

Sinon, concernant le code (y a du mieux par rapport à avant), regroupe les variables texte et posY dans une structure commune.
Quel intérêt aussi de mettre des entrées de menu vide ? Quel est le type de la variable texte et quel supposition y as-tu attaché (là je pense à une taille) ?

[Danny-k]

Suivant les derniers posts j'ai vu que tu utilises codeblocks, donc je suppose que ton compilateur est MinGW sous Windows (ou GCC sous Linux), quoiqu'il en soit il faut que tu rajoutes dans les options de compilation ceci :

Code :

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

-std=c++11

[Armulis]

Suivant les derniers posts j'ai vu que tu utilises codeblocks, donc je suppose que ton compilateur est MinGW sous Windows (ou GCC sous Linux), quoiqu'il en soit il faut que tu rajoutes dans les options de compilation ceci :

Code :

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

-std=c++11

Je me disais justement que le compilateur était peut-être mal configuré pour cette application.

Est-ce que tu peux être un peu plus précis, s'il-te-plaît? En fait j'ai pas vraiment idée de ce que je dois faire pratiquement =)

[ternel]

il faut activer le C++11 à la compilation.

Avec code::blocs, il faut aller cocher la case qui va bien dans les options de compilation.
Normalement le menu est assez explicite.

[foetus]

C'est quand même super précis: Danny-k te demande de rajouter la bascule "std" afin de spécifier à ton IDE quelle version du C++ tu veux utiliser

[Armulis]

Sinon, concernant le code (y a du mieux par rapport à avant), regroupe les variables texte et posY dans une structure commune.

Tu parles d'une "struct" à la C, ou d'autre chose? (et dans ce cas j'ai pas compris)

Quel intérêt aussi de mettre des entrées de menu vide ?

Dans cet exemple, il n'y a qu'une ligne visible, mais là elle est la cinquième d'une colonne de 5 lignes (dont 4 sont vides), ce qui la met plus ou moins bas sur l'écran. C'est un cas particulier; normalement la plupart des lignes sont remplies.

Quel est le type de la variable texte et quel supposition y as-tu attaché (là je pense à une taille) ?

Oui, pardon, tout ça est dans le constructeur de la classe. le type c'est sf::Text, et la taille est donnée par hauteur/13 (hauteur était la hauteur du moniteur, récupérée avec la classe sf::VideoMode). Tout ça fait partie de SFML.

[Armulis]

C'est quand même super précis: Danny-k te demande de rajouter la bascule "std" afin de spécifier à ton IDE quelle version du C++ tu veux utiliser

using namespace std ? C'est ça une bascule?

[foetus]

using namespace std ? C'est ça une bascule?

non de gcc -std=c++11

[Armulis]

Ça ?

[jblecanard]

Oui c'est ça

[Armulis]

Oui c'est ça

Nan! C'est bon, c'est moi qui ai fait de la merde! xD
Problème réglé, merci beaucoup.

Les questions posées à jo_link_noir restent d'actualité (si ça peut m'aider, je prends =D)

[r0d]

Tu parles d'une "struct" à la C

Une struct "tout court" (struct vient de C, mais c'est autant du C que du C++). En général, on utilise struct lorsqu'il s'agit d'un petit objet, mais c'est jsute une sorte de convention implicite, rien d'officiel.
Et c'est bien dans ce sens-là que je comprend la remarque de jo_link_noir. L'idée étant de remplacer 2 tableaux (un pour texte et un pour posY) par 1 seul tableau dont les éléments seraient une structure qui contiendrait ces deux données. Quelque chose comme:

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
struct LocalizedText
{
   sf::Text text;
   float posY; // float ou int, selon ce dont a besoin la SFML
   void Update( const std::string & txt, float pos_y )
   {
      text.setString(txt);
      posY = pos_y;
   }
};
 
class MenuColonneCentree
{
// ton code
 
   std::vector<LocalizedText> localized_text; // <- à la place des 2 tableaux text et posY
};

Ensuite tu fais passer tes paramètres de fonctions par référence constante plutôt que par copie, tu utilises les types qui vont bien, et ça va te donner une fonction qui ressemble à ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void MenuColonneCentree::SetContenu(const vector<string> & argCONTENU)
{   
    size_t calc1 = argCONTENU.size();
    options=(calc1-(calc1%2))/2;
 
    ligneSelectionnee=0; // Présélection de la première ligne
 
    for (int index(0); index < calc1; index++)
        localized_text[index].Update(argCONTENU[index],(7-options+index)*hauteurTextePrincipal);
 
   // etc.
}

C'est une façon plus orientée objet pour faire la même chose. L'approche objet consiste à créer des classes/structures qui résolvent des problèmes. Et il ne faut pas tomber dans le piège de l'approche "donnée": une classe n'a pas pour but de contenir des données, mais de résoudre des problèmes, ou de proposer un service (autre façon de dire la même chose). Contenir des données peut être un service, mais c'est un cas particulier. Dans le code que je te propose ci-dessus, la structure LocalizedText fournit des données, mais également un service (update), qui va s'occuper de mettre à jour ces données.

[Armulis]

Bien compris si "localized_text" est bien la structure dont tu parlais. Je vais revoir ça. Effectuvement, ça pourrait m'aider à éviter de traiter toutes les données une-à-une =D