Discussion :

[Sanguko]

N'y a-t-il pas un probleme avec le bout de code suivant ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
   ...
            void *tmp = realloc (nom, size);
 
            if (tmp != NULL)
            {
               nom = tmp;
 
               strcpy (nom, argv[i]);
               strcat (nom, tmp);
    ...

Mise a part le fait que nom va être copié dans lui même apres strcat, je ne suis pas sure que ca passe à l'execution. Le deuxieme argument de strcat (ici tmp) est const, et va être modifié (puisque c'est aussi le premier argument).

[Emmanuel Delahaye]

N'y a-t-il pas un probleme avec le bout de code suivant ?

Effectivement, c'est très faux. Je suis entrain de réécrire tout ça...

#44 mis à jour.

[Trap D]

Regarde bien ce que j'ai fait :

Code :

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

1
2
3
4
5
tmp = realloc(nom, 1 + strlen(argv[i]));
if(tmp!=NULL)
{
	nom = tmp;
	strcpy(nom,argv[i]);

Dans le realloc, je me contente d'allouer la bonne taille pour recopier argv[i] c'est tout, ça ne plante pas, mais comme je ne sais pas ce que tu veux faire exactement, à toi de réallouer la bonne taille pour ton propos. Si tu veux concaténer des arguements, à toi de calculer la bonne longueur et de faire la bonne recopie (le data contenu dans nom avant le realloc est recopié à l'adresse de tmp si le realloc réussi.

Code :

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

1
2
3
4
5
6
7
8
            void *tmp = realloc (nom, size);
 
            if (tmp != NULL)
            {
               nom = tmp;
 
               strcpy (nom, argv[i]);
               strcat (nom, tmp);

Je n'ai jamais écrit celà.

P.S.
Attention, j'ai l'impression qu'on est en train de travailler tous sur des versions différentes, mon code est correct mais ne fait pas ce que le P.O. recherche, je ne suis pas dans sa tête.

[Flophx]

Je dissipe les doutes de tout le monde:

- Le code de Trap D fonctionne bien et permet effectivement de lire les différents caractères, sans les concaténer, donc à moi de faire le nécessaire...

- Le code d'Emmanuel fonctionne, mais finalement mon idée de passer les paramètres en sens inverse à pour conséquence (évidente) d'avoir un nom inversé. Je suis en train de faire le nécessaire pour retrouver le bon ordre et les espaces dans le nom

Je me retrouve donc avec deux versions de code, il me reste qu'à choisir!

Une simple question Emmanuel: à quoi sert le assert?

P.S
P.O = Programmateur O????

[scriptoff]

assert(expression)
equivalent a :

Code :

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

1
2
3
4
if(!expression){
{
puts("assertion  failed  in  file  foo.c,   function do_bar(), line 1287");}
}

[Sanguko]

Pour assert : voir ici

P.O : Poster Originel

Je crois qu'il y a également un petit souci dans ce que tu prevois de faire.

Si j'ai bien compris, la ligne de commande de ton programme est du genre :

Code :

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

./video /mnt/My U drive

(peu importe les autres parametres, c'est un chemin qui m'interesse)
Ton programme recoit alors dans argv
argv[0] video
argv[1] /mnt/My
argv[2] U
argv[3] drive
Tu veux reconstruire à partir de argv[1], argv[2], et argv[3] le chemin complet, i.e. :
/mnt/My U drive
Pour le faire tu vas ajouter un espace entre argv[1], argv[2] et argv[3]

Le probleme, c'est que la ligne de commande suivante

Code :

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

./video /mnt/My   U drive

(avec plusieurs espaces) te donne exactement les memes contenus dans argv
Et le chemin reconstruit sera alors
/mnt/My U drive

[Flophx]

D'abord merci pour Assert!
j'ai simplement rajouté un:

Code :

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

1
2
3
 
strcat(nom, " ");
strcat(nom, argv[i]);

Cela me reconstitue bien ceci:

Code :

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

1
2
 
/mnt/My U3 drive

Je rajoute ensuite "manuellement" (sprintf) un

Code :

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

1
2
 
/video.yuv

et là en revanche, cela devient;

Code :

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

1
2
 
/mnt/My U3 drive       /video.yuv

avec un losange noir devantle "/".

J'en suis à résoudre ce problème!

Mais je suis d'accord que le problème que tu mentionnes rique de m'enquiquinner..

[Emmanuel Delahaye]

à quoi sert le assert?

A vérifier le fonctionnement de code (debug uniquement)

http://emmanuel-delahaye.developpez....tes.htm#assert

P.S
P.O = Programmateur O????

Posteur Originel (Original Poster). Celui qui a initié la discussion.