conversion d'une chaine de caractere en une matrice

mon programe consiste a placer un vecteur de caractere ds une matrice, mais le probleme c ke jarrive pa a afficher ma matrice :s svp veuillez tester ce code et repondez moi merci davance

Code:

---

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

void main() { char ph[200],phent[10][20]; int i=0,j=0,k=0; clrscr(); printf("veuillez saisir la phrase d'entre:\n"); gets(ph); printf("\nvotre phrase d'entre est:%s",ph); do {if(ph[i]==' ') { k++; j=0;phent[k][j]=ph[i+1]; } else{ phent[k][j]=ph[i]; j++;   } i++; }while(ph[i]!='\0'); printf("%s",phent);     getch(); }

---

Bonjour,

1. :tagcode:
2. La fonction main() retourne un int. Toujours.
3. gets() est un bug. La première règle de gets(), c'est "Ne pas utiliser gets()". La seconde règle de gets(), c'est...
4. Quel est précisément le rapport entre la phrase et la matrice ? Tu cherches à séparer les mots ?
5. Tes noms de variable sont insuffisamment expressifs.
6. Fichiers d'en-tête manquants.
7. clrscr() et getch() ne sont pas standard, et ce sont des fioritures. Tu fignoleras quand ton code marchera...

Ce if() engendre une répétition de la lettre dans ta matrice.

Code:

---

1 2 3 4 5 6 7 8

if(ph[i]==' ') { k++; j=0; phent[k][j]=ph[i+1]; } else { phent[k][j]=ph[i]; j++; }

---

De plus, tu ne peux afficher ta matrice avec %s, puisque les différentes lignes ne sont pas terminées par le caractère nul nécessaire...

Pas de langage SMS SVP !!!!

Code:

---

1 2	printf("%s",phent);

---

comment tu veut afficher la matrice avec la fonction printf(); ?
phent pointe sur phen[0][0] ( si c'est vrais ), alors la matrice ne sera pas afficher en totalité ....

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

do {if(ph[i]==' ') { k++; j=0;phent[k][j]=ph[i+1]; } else{ phent[k][j]=ph[i]; j++;   } i++; }while(ph[i]!='\0');

---

d'après le code tu veut avoir chaque mot dans une ligne de la matrice ....
voici la correction que je peut les mentionner :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

int k = j = 0; for(i=0, ph[i]!='\0', i++) { if(ph[i]==' ') { k++; j=0; } else{ phent[k][j]=ph[i]; j++; } }

---

j'ai pas tester le code, alors je sais pas si ca marche ou non, mais d'après moi, c'est comme ça qu'il faut faire !!!

Citation:

---

Envoyé par Médinoc

Bonjour,

1. :tagcode:
2. La fonction main() retourne un int. Toujours.
3. gets() est un bug. La première règle de gets(), c'est "Ne pas utiliser gets()". La seconde règle de gets(), c'est...
4. Quel est précisément le rapport entre la phease et la matrice ? Tu cherches à séparer les mots ?
5. Tes noms de variable sont insuffisamment expressifs.
6. Fichiers d'en-tête manquants.
7. clrscr() et getch() ne sont pas standard, et ce sont des fioritures. Tu fignoleras quand ton code marchera...

---

Moi aussi je suis d’accord que la fonction gets() ; est un beug
Mais j’ignore pour quoi elle l’est.
Ou bien pour quoi tous le monde nous déconseille de l’utiliser

L'explication est bien simple:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

#include <stdio.h>   void Bug(void) { char bufofl[10];   printf("Taper une chaîne : "); fflush(stdout);   gets(bufofl); printf("Vous aves tape :%s\n", bufofl); }   int main(void) { puts("Avant..."); Bug(); puts("Apres!"); return 0; }

---

Compile ce code en désactivant les optimisations pour être sûr que la fonction ne sera pas inlinée, lance le prog et tape: "abcdefghijklmnopqrstuvwxyz". En théorie, le programme devrait planter avant d'avoir pu afficher "après".

Mais ça fait 2 ans que je travaille avec la fonction fgets j’ai jamais eu de problème ‘beug’ en plus c’est le prof qui nous a apprit cette fonction.
Avec ma petite expérience je pense qu’on peut utiliser la fonction gets sans aucun problème

Avec fgets(), aucun problème.
Avec gets(), as-tu testé mon code ?

Citation:

---

Envoyé par dot-_-net

Mais ça fait 2 ans que je travaille avec la fonction fgets j’ai jamais eu de problème ‘beug’ en plus c’est le prof qui nous a apprit cette fonction.
Avec ma petite expérience je pense qu’on peut utiliser la fonction gets sans aucun problème

---

tu sais quoi, parce que gets() c'est une fonction des norme C89 et C99 trés vieux, elle n'est pas POSIX (pas portable sur n'importa qu'elle système) C'est comme la déclaration des fonctions :
avant on utilise :

Code:

---

1 2 3 4 5 6 7 8 9

int test(test, test) int test; int test; {   ....   }

---

Tiens, j'ai amélioré le code, maintenant il met forcément en évidence le bug même si on l'optimise!

Code:

---

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 32 33

#include <stdio.h>   struct ofl { int unEntier; char buf[10]; int unAutreEntier; };   void Bug2(struct ofl *pofl) { printf("Taper une chaine : "); fflush(stdout);   gets(pofl->buf); printf("Vous aves tape :%s\n", pofl->buf);   printf("unEntier = %d, unAutreEntier = %d\n", pofl->unEntier, pofl->unAutreEntier); }   void Bug(void) { struct ofl o = { 42, "", 42 }; Bug2(&o); }   int main(void) { puts("Avant..."); Bug(); puts("Apres!"); return 0; }

---

Sur une architecture little-endian comme le x86, il suffit de taper une chaîne de 10 caractères ou plus.
Sur une architecture big-endian, il faut taper au moins 11 caractères pour montrer le bug.

Citation:

---

Envoyé par cmoibal

tu sais quoi, parce que gets() c'est une fonction des norme C89 et C99 trés vieux

---

Ne dis pas de bêtises avec les normes. La norme C99 est la dernière norme du C en date, et gets() en fait toujours partie pour de sombres raisons de compatibilité antérieure...

Citation:

---

Envoyé par Médinoc

Ne dis pas de bêtises avec les normes. La norme C99 est la dernière norme du C en date, et gets() en fait toujours partie pour de sombres raisons de compatibilité antérieure...

---

désolé, mais c'est ce que j'ai pensé ... !!!!

Citation:

---

Envoyé par cmoibal

tu sais quoi, parce que gets() c'est une fonction des norme C89 et C99 trés vieux, elle n'est pas POSIX (pas portable sur n'importa qu'elle système) C'est comme la déclaration des fonctions :

---

En principe, POSIX.1 englobe C99. gets() est donc POSIX. C'est pour des raisons de sécurité qu'il est déconseillé de l'utiliser. Rien à voir avec la norme...

Thierry

Citation:

---

Envoyé par cmoibal

désolé, mais c'est ce que j'ai pensé ... !!!!

---

quand on poste ici (surtout un conseil) on ne PENSE pas (qudn on parle de normes) on se renseigne ;)

Surtout en etant debutant.....

Et meme quand on ne l'est pas (ca m'arrive a moi aussi :oops:)