routine erreur : "Control reaches end ". - Page 1

INT_HANDLER save_int_1;
short keywait(void)
{
SetIntVec (AUTO_INT_1, DUMMY_HANDLER);
for (;
{
for (x=0;x<=6;x++)
{
for (y=0;y<=7;y++)
{
if (_rowread(~((short)(1<<x))) & (1<<y)) goto m;
} } }
m:
SetIntVec (AUTO_INT_1, save_int_1);
if (x==0 && y==0) return 8;
if (x==0 && y==1) return 4;
if (x==0 && y==2) return 2;
if (x==0 && y==3) return 6;
if (x==0 && y==4) return 1;
if (x==6 && y==0) return 264;
if (x==1 && y==0) return 13;
}
cette routine cree une erreur :
"Control reaches end of non-void function"

merci

au leiu du goto t'as essayé de mettre un break, avec un changement de valeur de variable ^pour les boucles for ?

bouah! un goto!! t'as pas honte ?

C'est un warning quand il n'y a pas d'instruction return xyz; à la fin. Tu peux l'ignorer sans problèmes.

>bouah! un goto!! t'as pas honte ?

Le goto en C est recommandé uniquement pour sortir de boucles imbriquées... donc ce qu'il fait est tout a fait adapté.

>boucles imbriquées ??
genre:

while()
{while()
{while()
{goto beurk;
}}}

beurk:
etc...

C mieux que de faire :

while (condition_sortie) {
while (condition_sortie) {
while (condition_sortie) {
if ... condition_sortie=TRUE;
}}}



ou alors dans la condition des for(; il met par ex. (a<=6) || (condition_sortie)

c quand même plus zoli , nan

goto rulez

je me rapelle que sur casio il fallait jamais manipuler les ss prgm comme ça en les imbrikant, ça faisait une erreur....heureusement kil y pas ce bug sur TI

Ce n'est pas la seule horreur qu'il y a dans cette routine

la routine elle mm est laide

heuh merci mais ne vous lachez pas non plus je suis qu'un newbie....

smeet : C'est plus joli mais c'est plus lent (trois tests au lieu d'un). Le goto c'est sale, mais dans le cas d'itérations imbriquées c'est ce qu'il y a de plus efficace.

Que pensez vous de ceci ? (non testé et sans la gestion de l'int-1)

short keywait(void) 
{ 
unsigned short clavier[3][7]={{0,0,0,0,0,6,1},
				{0,1,2,3,4,0,0},
				{8,4,2,6,1,264,13}};
short a=0;

while(!(kbhit()));


for(a=0;a<6;a++)
{
	if(_rowread(~((short)(1<<clavier[0][a])) & (1<<clavier[1][a]) 
	{
		return clavier[2][a];
	}
}
return 0;
}