Discussion :

[hazaki]

Bonjour,
Tout est dans l'intitulé, je voudrais savoir si on peut on déclarer une structure dans un sous programme, car j'ai créé une structure pour gérer une image qui contient sa taille horizontale, sa taille verticale et un tableau contenant les
intensités lumineuses de chacun des pixels.
Le problème est qu'on me demande de faire un sous programme permettant de faire la symétrie verticale d'une image, le résultat doit être stocké dans une autre image et retourné au programme principal.
Donc je dois déclarer une nouvelle structure, mais puis-je le faire dans ma fonction ?
Voilà ce que j'ai fait, dites moi si c'est correct

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
#include <iostream>
using namespace std;
int const Max_X=1000;
int const Max_Y=1000;
 
struct image{
       int tailleX;
       int tailleY;
       int pixel[Max_X][Max_Y];
       };
 
struct image2 symetrie(struct image im)
{
    struct image2{
           int tailleX;
           int tailleY;
           int pixel[Max_X][Max_Y];
           };
 
    struct image2 im2;
    int i,j;
    j=0;
    while(j<Max_X){
                        for(i=0;i<Max_Y;i++){
                                                  im.pixel[i][j]=im2.pixel[Max_X-1-i][j];
                                                  }
                        }
    im2.tailleX=im.tailleX;
    im2.tailleY=im.tailleY;
    return im2;
}

Merci de bien vouloir me répondre

[Mat007]

Salut,

Retire les struct dans l'entête de la fonction :

Code :

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

image symetrie(image im)

Ton type image2 ne sert à rien, utilise aussi image.

Retire aussi le struct dans la déclaration des variables :

Code :

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

image im2;

Tu es supposé faire du C ou du C++ ?

ps : sinon tu peux aussi compiler pour savoir si le code a une chance d'être correct...

MAT.

[hazaki]

Merci, pour ta réponse,
j'ai compilé mais ça marche pas.
Tu peux me dire, si j'ai le droit de déclarer la structure image2 dans la fonction ou pas ?

[Mat007]

j'ai compilé mais ça marche pas.

Quel est le message d'erreur ?

Tu peux me dire, si j'ai le droit de déclarer la structure image2 dans la fonction ou pas ?

Ton type image2 ne sert à rien, utilise aussi image.

MAT.

[hazaki]

Alors il y a un message d'erreur :

conversion from `symetrie(image)::image2' to non-scalar type `image2' requested

Mais le problème c'est que je dois créer une nouvelle image pour stocker l'image symétrique, j'utilise quand même image ?

[hazaki]

Après un peu de réflexion j'ai changé mon code, il n'y a plus de messages d'erreur, dis moi ce que tu en penses

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
#include <iostream>
using namespace std;
int const Max_X=1000;
int const Max_Y=1000;
 
struct image{
       int tailleX;
       int tailleY;
       int pixel[Max_X][Max_Y];
       };
 
 
struct image symetrie(struct image im)
{
    int i,j,a;
    j=0;
    while(j<Max_X){
                        for(i=0;i<Max_Y;i++){
                                                  a=im.pixel[Max_X-1-i][j];
                                                  im.pixel[Max_X-1-i][j]=im.pixel[i][j];
                                                  im.pixel[i][j]=a;
                                                  }
                        }
    return im;
}

[Mat007]

Oui c'est mieux déjà.

Retire le struct dans la signature de la fonction et ça fera sans doute l'affaire :

Code :

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

1
2
3
4
image symetrie(image im)
{
...
}

MAT.

[hazaki]

Mais je comprends pas pourquoi tu enlèves le struct dans l'entête parce que dans mes cours, mes profs le mettaientt toujours

[JolyLoic]

Mais le problème c'est que je dois créer une nouvelle image pour stocker l'image symétrique, j'utilise quand même image ?

J'ai l'impression que tu confonds classe et instance de classe. Tu veux pour retourner ton résultat une nouvelle instance de la classe image, mais pas une nouvelle classe.

Pour ce qui est de la répétition de struct, je crois qu'en C, c'est nécessaire, selon certaines règles je je ne connais pas. Mais en tout cas, en C++, pas besoin.

[hazaki]

Oui, je viens de comprendre la différence, c'est pour ca que j'ai changé mon code.
Ok merci pour la précision, je compile et ça marche donc merci pour l'aide

[Nasky]

C'est en effet plutôt du C si on doit rajouter le "struct". En général, on définit un nouveau type genre :

Code :

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

1
2
3
4
5
6
7
8
typedef struct image{
       int tailleX;
       int tailleY;
       int pixel[Max_X][Max_Y];
       }image;
 
 
image symetrie(image im) {

[koala01]

Salut,

Ceci dit, il est tout à fait légal, en C++, de définir une structure au sein d'une fonction.

Cependant, la connaissance que le compilateur aura de cette structure sera limitée à la portée dans laquelle elle est définie.

Cela signifie que tu peu parfaitement avoir un code proche de

Code :

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

1
2
3
4
5
6
7
8
void foo()
{
    struct MyStruct
    {
        /* le contenu de MyStruct */
    };
    /* du code utilisant MyStruct */
}

Tu pourrais d'ailleurs parfaitement définir MyStruct dans différentes fonctions sans que cela ne pose de problème.

MAIS tu ne peux, quoi qu'il en soit, utiliser MyStruct que dans le corps de la fonction dans laquelle la structure est définie (foo selon l'exemple), c'est à dire, ni comme type de retour, ni comme type d'argument, que ce soit pour foo ou pour une quelconque autre fonction.

Ce genre de pratique est essentiellement utilisé pour créer des foncteurs dont l'utilité se limite à la fonction dans laquelle ils sont définis ou dont on souhaite restreindre l'usage à cette seule fonction:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void bar(std::vector<UnType> /* const */ & tab)
{
    struct functor
    {
        void operator()(UnType /* const */ & t)
        {
            /* un traitement particulier devant être exécuté sur chaque objet
             * de type UnType se trouvant dans tab
             */
        }
    };
    std::for_each(tab.begin(), tab.end(), functor());
}

[Invité]

Code :

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

1
2
 
std::for_each(tab.begin(), tab.end(), functor());

Mais si je ne me trompe pas, avant la nouvelle norme, le code que tu présente était (absurdement ?) illégal.

[guillaume07]

Non tu confonds avec les lambda expression.

[Invité]

Non, je maintiens : une petite recherche sur google avec comme mot-clé "local class as template argument" donne divers résultat dont celui par exemple. Ou encore sur GotW, deuxième réponse. En c++98 ET c++03 une classe locale ne peut pas être utilisée avec des fonction template( et en gros elle ne peut être que très rarement utilisée comme argument d'une fonction, vu qu'on ne peut pas en créer la prenant explicitement comme argument dans un scope exterieur à la fonction)

[JolyLoic]

Effectivement, c'était illégal. Une classe locale ne peut servir de paramètre template.

[guillaume07]

autant pour moi mal compris la question visiblement