Discussion :

[chicabonux]

Bonjour,

Voici un sous-programme permettant d'entrer un temps dans une variable de typre struct temps et un sous programme sortie() permettant de sortir à l'écran un temps stocké dans une variable de type structure temps

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
/*devoir numéro 6 exercice 2A*/
 
#include<stdio.h>
 
	/*définition de la structure pour la représentation du temps*/
	struct t {
		int heures;
		int minutes;
		int secondes;
	};
 
	void entree(void)
	{
		struct t *temps;
		int tps;
 
		do
		{
			tps=1;
			printf("entrer un temps separe par des espaces, svp\n");
			scanf("%d%d%d",&temps->heures,&temps->minutes,&temps->secondes);
 
			if(temps->heures>23||temps->heures<0)
			{
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->minutes>59||temps->minutes>0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->secondes>59||temps->secondes<0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
 
		}while(temps==0);
	}
 
	void sortie(void) 
	{
		struct t *temps;
		printf("%2.2d:%2.2d:%2.2d\n",temps->heures,temps->minutes,temps->secondes);
	}
 
	int main()
	{
		entree();
		sortie();
 
		return 0;
	}

ça me dit que la variable temps n'est pas déclarée à la compilation ?
Je ne comprends pas pourquoi ?

[ram-0000]

Puet être parce que ton compilateur est très fort et qu'il se rend compte que derrière ton pointeur *temps, il n'y a pas d'initialisation et donc que ce pointeur est fou et qu'il refuse de l'utiliser.

[chicabonux]

Je le compile sur visual et sur code blocks et les 2 refusent de l'exécuter

[ram-0000]

et moi, cela me sort 2 warning sur les 2 problèmes que je t'ai signalé plus haut

c:\temp\console.cpp(27) : warning C4700: variable locale 'temps' non initialisée utilisée
c:\temp\console.cpp(51) : warning C4700: variable locale 'temps' non initialisée utilisée

Je suis sous Visual Studio 2005

[chicabonux]

moi aussi je suis sous VS 2005 mais ça me met un run time check failure et je ne peux pas exécuter le programme ?

En fait c'est un exercice que je dois faire et j'ai demandé la correction et je vois qu' il a été réalisé avec le compilateur Eterm . Je ne connais pas ce compilateur et cet exercice ne peut être compilé sur un autre compilateur ?

[droggo]

Bie,

Quel que soit ton compilateur, le fait est que tu utilises - mal, en plus - un pointeur non initialisé.

[chicabonux]

Alors comment utiliser le pointeur et aussi comment l'initialiser ?

[ram-0000]

Pour ne pas trop toucher à ton code, je te propose les modif suivantes

Code :

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

1
2
3
4
void entree(void)
	{
		struct t tempo;
		struct t *temps = &tempo;

et

Code :

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

1
2
3
4
void sortie(void) 
	{
		struct t tempo;
		struct t *temps = &tempo;

C'est pas très beau mais c'est ce qui permet de ne pas trop toucher au reste du code

[chicabonux]

Il faut que je change mon code comme ça ? ça ne fonctionne pas ainsi ça me met des résultats totalement erronés ?

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
/*devoir numéro 6 exercice 2A*/
 
#include<stdio.h>
 
	/*définition de la structure pour la représentation du temps*/
	struct t {
		int heures;
		int minutes;
		int secondes;
	};
 
	void entree(void)
	{
		struct t tempo;
		struct t *temps = &tempo;
 
		int tps;
 
		do
		{
			tps=1;
			printf("entrer un temps separe par des espaces, svp\n");
			scanf("%d%d%d",&temps->heures,&temps->minutes,&temps->secondes);
 
			if(temps->heures>23||temps->heures<0)
			{
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->minutes>59||temps->minutes<0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->secondes>59||temps->secondes<0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
 
		}while(tps==0);
	}
 
	void sortie(void) 
	{
		struct t tempo;
		struct t *temps = &tempo;
 
		printf("%2.2d:%2.2d:%2.2d\n",temps->heures,temps->minutes,temps->secondes);
	}
 
	int main()
	{
		entree();
		sortie();
 
		return 0;
	}

[Obsidian]

Je le compile sur visual et sur code blocks et les 2 refusent de l'exécuter

Donc, ce n'est pas à la compilation que ça bloque, mais à l'exécution. En gros, comme te l'as dit ram-0000, tes pointeurs existent mais pointent vers le vide. Si tu compiles en « mode debug » ou quelque chose d'assimilé dans un environnement de développement, celui-ci est capable de rattrapper l'erreur et de te le dire. En production, ça se traduirait par une segfault.

Il faut que je change mon code comme ça ? ça ne fonctionne pas ainsi ça me met des résultats totalement erronés ?

C'est normal, tes variables « temps » (et désormais « tempo ») sont déclarées en local dans tes deux fonctions « entree » et « sortie ». Si elles portent le même nom, cela reste deux instances complètement distinctes l'une de l'autre.

Ce qu'il faut faire si tu veux les faire communiquer, c'est déclarer une structure « temps » dans ta fonction main() et passer son adresse à chacune de tes fonctions « entree » et « sortie ». Pour cela, il faut que leur paramètre ne soit plus void mais un pointeur vers une instance valide de la structure « temps » (celle-là même que tu déclares dans main).

[chicabonux]

ça serait sans doute une solution (et encore je ne sais pas comment faire ça : )

il faut que leur paramètre ne soit plus void mais un pointeur vers une instance valide de la structure « temps »

Mais dans l'énoncé de mon exercice ça me dit :

fiait simplement pour tester les sous programmes, le programme principal sera le plus simple possible :
-déclarations diverses
-entrée d'un temps du clavier : appel de entree()
-sortie de ce temps à l'écran : appel de sortie()

[Obsidian]

Mais dans l'énoncé de mon exercice ça me dit :

Citation:
fiait simplement pour tester les sous programmes, le programme principal sera le plus simple possible :
-déclarations diverses
-entrée d'un temps du clavier : appel de entree()
-sortie de ce temps à l'écran : appel de sortie()

Renseigne-toi auprès de ton prof.

À priori, j'interpréterais les parenthèses comme étant les signes d'un appel aux fonctions, pas comme le fait que celles-ci ne doivent attendre aucun paramètre. Si c'est le cas, alors il est possible qu'il attende de toi que tu utilises une variable globale et ça, c'est mal.

Au pire, tu peux toujours « tricher » en appelant au sein d'« entree » et « sortie » une troisième fonction qui, elle, te renvoie un pointeur vers une structure déclarée en mémoire. Ça se fait souvent, mais ça ne va pas dans le sens de la simplification de ton programme.

[chicabonux]

Et surtout je ne sais pas comment faire ça

Tu pourrais me montrer ?

[Obsidian]

Et surtout je ne sais pas comment faire ça Tu pourrais me montrer ?

Encore une fois : renseigne-toi d'abord auprès de ton professeur pour savoir s'il veut que tu utilises une globale ou si tu dois utiliser des paramètres. À mon avis, même s'il veut que le programme principal ne contienne que des appels à tes deux fonctions, je penche pour la deuxième option.

Ce serait dommage de rater ton projet si près du but simplement à cause d'une assertion de départ qui n'aurait jamais été vérifiée.

[chicabonux]

J'ai un exercice préléminaire qui ne s'exécute pas non plus !

énoncé :

Ecrire une fonction qui permet d'entrer des caractères du clavier dans un tableau t, y compris espaces et tabulations, ce que ne peut faire l'instruction scanf("%s",t); .L'entrée des caractères se termine par un retour chariot .La fonction sera définie comme suit :

lecture(t)
char *t;
{
...à écrire
}

On prendra le programme suivant pour tester la fonction :

#include<stdio.h>
main()
{
char t[100];
lecture(t);
printf("%s",t);
}

Le corrigé ne s'exécute pas

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
#include<stdio.h>
 
/*fonction permettant la récupération et le stockage d'une phrase*/
/*qui sera par la suite utilisée pour afficher la phrase*/
 
void lecture(t)
char *t;
{
 
 
	while((*t=getchar())!='\n')
		t++;
}
 
int main()
{
	char t[100];
 
	printf("Entrez une phrase (moins de cent caracteres)\n");
	lecture(t);
 
	printf("%s",t);
 
 
	return 0;
}

ça me donne ça comme résultat à la compilation

[img=http://img13.imageshack.us/img13/5928/ex1dev6.th.jpg]

[Invité(e)]

Bonjour,
il faut ajouter une fin de chaine :

Code :

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

1
2
3
4
5
6
7
void lecture(t)
char *t;
{
    while((*t=getchar())!='\n')
       t++;
    *t = '\0';
}

EDIT
On aurai aussi pu initialiser t à sa création :

Code :

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

1
2
3
4
int main()
{
    char t[100] = "";
    ...

/EDIT
au passage, on ne déclare plus des fonction C comme tu le fais. On peut écrire aujourd'hui :

Code :

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

1
2
3
4
5
6
7
8
9
void lecture(char *t)
{
    ...
}
int main(void) 
{
    ...
    return 0;
}

[gl]

Le corrigé ne s'exécute pas

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
#include<stdio.h>
 
/*fonction permettant la récupération et le stockage d'une phrase*/
/*qui sera par la suite utilisée pour afficher la phrase*/
 
void lecture(t)
char *t;
{
 
 
	while((*t=getchar())!='\n')
		t++;
}
 
int main()
{
	char t[100];
 
	printf("Entrez une phrase (moins de cent caracteres)\n");
	lecture(t);
 
	printf("%s",t);
 
 
	return 0;
}

ça me donne ça comme résultat à la compilation

[img=http://img13.imageshack.us/img13/5928/ex1dev6.th.jpg]

Ce qui est normal, nul part dans le code tu ne termines la chaîne de caractères par un '\0'. printf() affiche donc tout ce qu'il trouve en mémoire jusqu'à rencontrer un '\0'.

Ceci devrait résoudre ce problème:

Code :

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

1
2
3
4
5
6
void lecture(char *t)
{
	while((*t=getchar())!='\n')
		t++;
	*t = '\0'
}

Par contre, ton code présente un autre problème grave, il n'y a aucune vérification de la taille du tableau.

[chicabonux]

Merci pour votre aide . Avec la fin de chaîne le programme s'exécute bien .

Je reviens à l'autre exercice

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
/*devoir numéro 6 exercice 2A*/
 
#include<stdio.h>
 
	/*définition de la structure pour la représentation du temps*/
	struct t {
		int heures;
		int minutes;
		int secondes;
	};
 
	void entree(void)
	{
		struct t *temps;
		int tps;
 
		do
		{
			tps=1;
			printf("entrer un temps separe par des espaces, svp\n");
			scanf("%d%d%d",&temps->heures,&temps->minutes,&temps->secondes);
 
			if(temps->heures>23||temps->heures<0)
			{
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->minutes>59||temps->minutes>0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
			else if(temps->secondes>59||temps->secondes<0)
			{		
				printf("temps invalide, recommencer la saisie\n");
				tps=0;
			}
 
		}while(temps==0);
	}
 
	void sortie(void) 
	{
		struct t *temps;
		printf("%2.2d:%2.2d:%2.2d\n",temps->heures,temps->minutes,temps->secondes);
	}
 
	int main()
	{
		entree();
		sortie();
 
		return 0;
	}

Je n'y comprends strictement rien et je ne vois pas comment faire ce qu'on me propose

À priori, j'interpréterais les parenthèses comme étant les signes d'un appel aux fonctions, pas comme le fait que celles-ci ne doivent attendre aucun paramètre. Si c'est le cas, alors il est possible qu'il attende de toi que tu utilises une variable globale et ça, c'est mal.

Au pire, tu peux toujours « tricher » en appelant au sein d'« entree » et « sortie » une troisième fonction qui, elle, te renvoie un pointeur vers une structure déclarée en mémoire. Ça se fait souvent, mais ça ne va pas dans le sens de la simplification de ton programme.

Vous pouvez m'expliquer un petit peu ?

[chicabonux]

En fait je ne sais pas comment initialiser un pointeur ou créer la 3ème fonction ?

[diogene]

Code :

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

1
2
void entree(void)...
void sortie(void)...

Ces deux fonctions agissent sur une grandeur notée struct t *temps. C'est donc une grandeur dont dépend le fonctionnement de ces fonctions. En toute logique, elle devrait être un paramètre de la fonction, pas une variable locale.

Code :

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

1
2
void entree(struct t *temps)...
void sortie(struct t *temps)...

Le main devient alors :

Code :

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

1
2
3
4
5
6
7
int main()
{
  struct t temps; // création d'une variable struct t
  entree(&temps); // passage de son adresse aux fonctions
  sortie(&temps);
  return 0;
}