Discussion :

[tousa]

bonsoir tout le monde;

je veux ouvrir le tube nommé de chaque utilisateur lorsqu il accède à mon prgm!!
sachant que chacun a un tube "fifo" dans un ss_repertoire à son nom dans un répertoire commun déjà crée.

Code :

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

1
2
3
4
5
6
7
8
9
char *c;
               char k[];
               int ptf;
               c=getenv("USER");
                k=strcat("../",c);
		if(ptf=open(strcat(k,"/fifo"),O_WRONLY))==-1){ 
			perror("erreur d ouverture en ecriture");
			exit(1);}
          		}

mais le strcat ne fonctionne pas!!
parce que apparemment il faut que je les concatène à l aide de cette fonction et si je met directement

Code :

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

ptf=open ("../getenv("USER")/fifo",O_WRONLY)==-1

ça ne fonctionne pas aussi

merci pour votre aide

[ram-0000]

cela métonne que cela compile sans erreur ni warning, ta variable k à une taille inconnue.

Pour bien faire :

Code :

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

1
2
3
char k[1024];
sprintf(k, "../%s/fifo", getenv("USER"));
if(ptf=open(k,O_WRONLY))==-1)

Ce bout de code est bien à améliorer car il n'y a aucun contrôle sur la validité du pointeur retourné par getenv("USER")

[tousa]

ok merciiii j essaye ça et je vous tiendrai au courant

[tousa]

mais une petite question!!
si je remplace mon code par le votre je n ai plus besoin du pointeur *c
vu que moi je l ai mise juste pour lui affecter le getenv("USER")
alors que vous vous l utilisez directement
c est ça?

[ram-0000]

Dans mon code, effectivement, il n'y en a plus besoin mais je n'ai pas fait de contrôle de la validité du getenv().

Ce que tu peux faire :

Code :

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

1
2
3
4
5
6
char *c;
   c = getenv("USER");
   if(c == NULL)
   {
      /* traitement de l'erreur */
   }

[tousa]

ahhh d accord!!
ça marche en tout cas!!
je vous remercie infiniment -->vous me sauvez la vie
bonne journée..

[diogene]

Je ne sais pas où tu en es dans ton code actuel, mais sur ton code initial, il y a deux autres grosses erreurs de principe dans la ligne

Code :

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

k=strcat("../",c);

1- k est un tableau. On ne peut jamais écrire tableau = quelque_chose

2- la première chaine du strcat() doit recevoir le résultat de la concaténation, or une chaine littérale (du genre "----") est non modifiable.

[tousa]

oui oui je me suis rendu compte après aussi
tout va bien maintenant..ça marche..
merci en tout cas
vous m'avez énormément aidé..
Je vous suis reconnaissante ..
bonne soirée..