BruNews
Messages postés21040Date d'inscriptionjeudi 23 janvier 2003StatutModérateurDernière intervention21 août 2019
-
23 août 2010 à 17:04
cs_ghuysmans99
Messages postés3983Date d'inscriptionjeudi 14 juillet 2005StatutMembreDernière intervention30 juin 2013
-
23 août 2010 à 22:08
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
cs_ghuysmans99
Messages postés3983Date d'inscriptionjeudi 14 juillet 2005StatutMembreDernière intervention30 juin 201316 23 août 2010 à 22:08
En effet ça m'étonnerait qu'il l'améliore, ou il y passera des jours (vu qu'il ne maitrise déjà pas les bases du langage).
BruNews
Messages postés21040Date d'inscriptionjeudi 23 janvier 2003StatutModérateurDernière intervention21 août 2019 23 août 2010 à 21:47
NE RESTERA QUE QUELQUES JOURS EN TANT QU'EXEMPLE DE CE QU'IL NE FAUT JAMAIS FAIRE.
cs_ghuysmans99
Messages postés3983Date d'inscriptionjeudi 14 juillet 2005StatutMembreDernière intervention30 juin 201316 23 août 2010 à 19:11
On ne t'a jamais dit qu'il fallait placer le code dans un fichier .c et les en-têtes dans un .h ? Ton programme ne se compile même pas ! int SALLE(char etage) ne retourne aucune valeur. Pourquoi donc déclarer cette fonction comme retournant un int ? Une fois qu'on a ajouté un return 0; à la fin de cette fonction, VC++ dit que le stack a été corrompu dans la fonction REMOVE_FILE (code cité par BruNews). C'est vraiment un miracle si ça fonctionne chez toi (et ça ne fonctionne alors qu'avec MinGW). De plus, ton code n'est même pas factorisé*. Et on se demande pourquoi t'as inclus iostream, alors que ton code n'est qu'une poubelle portant l'extension cpp qui ne fait nul part appel aux flux et à la POO. Pour conclure, le code tel qu'il est là n'est vraiment pas un exemple à suivre ...
(*) à la ligne 167 de fonctions.h, je vois pas l'utilité d'un switch, vu que dans les deux cas c'était exactement la même chose (et ce n'est pas un cas isolé) :
system("cls");
id[0]='R';
id[1]=salle;
printf("\t\t\t\t\20 %s \21\n\n",id);
R_W(choix);
BruNews
Messages postés21040Date d'inscriptionjeudi 23 janvier 2003StatutModérateurDernière intervention21 août 2019 23 août 2010 à 17:04
J'ouvre fonctions.h, le 1er fichier que je vois dans la liste.
On y trouve:
void REMOVE_FILE(char fille[])
{
char a[]="Salles_Classe/",b[]=".txt";
strcat(a,fille);
strcat(a,b);
remove(a);
}
J'adore le chainage sur chaines fixes.
Il était vraiment temps que l'année se termine...
23 août 2010 à 22:08
23 août 2010 à 21:47
23 août 2010 à 19:11
(*) à la ligne 167 de fonctions.h, je vois pas l'utilité d'un switch, vu que dans les deux cas c'était exactement la même chose (et ce n'est pas un cas isolé) :
system("cls");
id[0]='R';
id[1]=salle;
printf("\t\t\t\t\20 %s \21\n\n",id);
R_W(choix);
23 août 2010 à 17:04
On y trouve:
void REMOVE_FILE(char fille[])
{
char a[]="Salles_Classe/",b[]=".txt";
strcat(a,fille);
strcat(a,b);
remove(a);
}
J'adore le chainage sur chaines fixes.
Il était vraiment temps que l'année se termine...