Discussion :

[zicoadis]

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 :

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
 
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();
}

[Médinoc]

Bonjour,

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...

[Médinoc]

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

Code :

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

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...

[cmoibal]

Pas de langage SMS SVP !!!!

Code :

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

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 :

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

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 :

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

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 !!!

[dot-_-net]

Bonjour,

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

[Médinoc]

L'explication est bien simple:

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
#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".

[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

[Médinoc]

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

[cmoibal]

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 :

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

1
2
3
4
5
6
7
8
9
 
int test(test, test)
int test;
int test;
{
 
....
 
}

[Médinoc]

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

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
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.

[Médinoc]

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...

[cmoibal]

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é ... !!!!

[Thierry Chappuis]

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

[souviron34]

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 )