Discussion :

[L8O8L]

Salut à tous,

Je dois faire un analyseur avec yacc et j'ai un probleme.
Je dois créer un nouveau dossier en générant son nom automatiquement mais je n'y arrive pas. A mon avis j'écris des adresses au lieu d'écrire les chaines mais je n'y arrive plus. Ca fait longtemps que je n'ai pas fait de C...
Pouvez-vous m'aider?

Voici mon 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
 
// Ici c'est ou j'appel mes fonctions dans le programme principal
char nomScene[11];
genereNomSceneDefaut(nomScene);
printf("%s",nomScene);
miseEnPlaceScene(nomScene);
 
// Ca c'est ma fonction pour créer un dossier avec un nom passé en parametre
void miseEnPlaceScene(char *nomScene){
		//Creation du dossier
                if(!(mkdir(nomScene,0777))){
		              /* RESTE DU CODE PAS DE PROBLEME ICI :D 
                     EN FAIT JE CREE DES FICHIERS*/
                }
}
 
// Là c'est la fonction avec laquelle je veux generer mon nom de fichier
void genereNomSceneDefaut(char *nomChaine){
  int i = 1;
  nomChaine = malloc(11 * sizeof(char));
  strcpy(nomChaine,"newScene");
  while(mkdir(nomChaine,0777) == EEXIST){
    i++;
    sprintf(nomChaine,"%s%d","newScene",i);
  }
}

Bon y'a ptet des erreurs autres paske il me créer un dossier newScene (surement dans la fonction genereNomSceneDefaut ) et en plus il me crée un dossier avec un nom tout chelou >> ���x�.��. Celui-ci se crée surmenent dans la fonction miseEnPlaceScene car le code d'après (créations de fichiers) est réalisé pour ce dossier.


J'espère que je suis assez clair et que vous pourrez m'aider.
Pour toutes questions n'hésitez pas...

MERCI ENORMEMENT A TOUS!!!!

[souviron34]

ton erreur vient d'ici :

Code :

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

1
2
 
void genereNomSceneDefaut(char *nomChaine){

tu fais une alloc, mais tu n'as passe que le pointeur, pas l'ADRESSE du pointeur.. Donc comme l'alloc te retournes une adresse, tu ne peux pas la faire remonter...

Ton interface doit donc etre :

Code :

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

1
2
void genereNomSceneDefaut(char **nomChaine){

[L8O8L]

AHHH ok!!!

MERCI
MERCI
MERCI
MERCI
MERCI
MERCI
MERCI
MERCI
MERCI
MERCI
MERCI

En gros c'est un pointeur sur le tableau (qui est lui aussi un pointeur??) C'est pour ca que je doit mettre ** et pas *
Chui sur que c'est une des bases en plus... :'( J'ai pas fini de galerer moi :p

Bon maintenant j'ai un truc qui marche...

Et si j'auvais voulu retourner une chaine de caractere j'aurais du faire comment??? MERCI D'AVANCE
(promis apres j'met mon ptit "resolu")

[L8O8L]

Bon allez en fait je vais chercher, je ne vais pas pourrir le forum plus longtemps...

A ciao tt le monde et encore merci

[souviron34]

Et si j'auvais voulu retourner une chaine de caractere j'aurais du faire comment???

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char *genereNomSceneDefaut(void){
  int i = 1;
  char *nomChaine = NULL ;
 
  if ( (nomChaine = malloc(11)) != NULL )
   {
      strcpy(nomChaine,"newScene");
      while(mkdir(nomChaine,0777) == EEXIST)
         i++;
      sprintf(nomChaine,"%s%d","newScene",i);
  }
  return nomChaine ;
}

[L8O8L]

J'avais eu un warning avec ca, je crois. Comme quoi je retournais une variable locale... enfin j'avais du mal faire
Paske j'avais testé avant,quand même.

Mais merci

[diogene]

J'avais eu un warning avec ca, je crois. Comme quoi je retournais une variable locale... enfin j'avais du mal faire
Paske j'avais testé avant,quand même.

Mais merci

Cen'est pas le cas dans le code présenté par souviron34.

NomChaine est bien une variable locale dont il retourne la valeur. Ce n'est pas interdit. Ce qui l'est est de retourner l'adresse d'une variable locale puisque celle-ci est détruite en sortie de fonction et qu'on se retouve avec l'adresse de quelque chose qui n'existe plus. Ici, on a l'adresse du bloc alloué par malloc et ce bloc existe encore en sortie de la fonction.
Le code suivant est donc pervers et interdit :

Code :

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

1
2
3
4
5
char *genereNomSceneDefaut(void){
  char nomChaine[11]  ;
....
    return nomChaine ;
}

mais celui-ci, non :

Code :

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

1
2
3
4
5
char *genereNomSceneDefaut(void){
  char * nomChaine = malloc(11) ;
....
    return nomChaine ;
}