Discussion :

[prince 1990]

Bonjour tout le monde , dans un exercice il est demandé d'écrire un algorithme qui permet de simuler le fonctionnement d'une horloge digitale j'ai écrit ce qui suit mais je dout qu'il simule vraiment le fonctionnement de cette horloge ,merci beaucoup pour ce qui puisse améliorer mon travail et le corriger :

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
#include <cstdlib>
#include <iostream>
 
using namespace std;
 
int main(int argc, char *argv[])
{int  instant_hour,instant_min,instant_sec;
    int remplir,i,j=0 ,k=0  ;
    printf("donner la seconde  i");
    scanf("%d",&i);
    while (i<60){
    instant_sec=instant_sec+1;
    i=i+1;
    if (i=60)
    {j=1;
    while (j<60)
    {instant_min=instant_min+1;j=j+1;
 
    if(j=60)
    {k=1;
    while(k<60){instant_hour=instant_hour+1;k=k+1;}}
    printf(" %d:%d:%d", instant_hour, instant_min, instant_sec);
    system("PAUSE");
    return EXIT_SUCCESS;
    system("PAUSE");
    return EXIT_SUCCESS;
}

[quentinh]

Plusieurs remarques sur le code tout d'abord :
- Un étrange mélange de C et de C++…
- Code illisible à cause de l'absence d'indentation et d'espacement, et des accolades qui se baladent un peu partout
- Manque cruel de commentaires pour expliquer ce qui se passe
- Les lignes 25 et 26 ne risquent pas d'être exécutées

Je suis pas bien sûr d'avoir compris ce que faisait ton programme, pourrais-tu donner un exemple d'entrée avec la sortie qui doit lui être associée ?

Exemple de code propre (ce n'est pas la seule mise en forme possible, mais celle-ci est lisible) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int main()
{
    while (condition)
    {
        fonction();
        if (condition2)
        {
            fonction2();
            fonction3();
        }
    }
    return 0;
}

[prince 1990]

le programme va en fait jouer le rôle d'une horloge digitale on lui donne l'heure actuelle et puis lui il va se comporter comme l'horloge
merci encore une fois pour ce qui puisse corriger mon travail
remarque: j'utilise DEV++ pour l'exécution du programme

[satenske]

Même remarque que précédemment, soit on code en C, soit en C++, mais là ça fait bizarre d'avoir des using namespace avec des scanf

Alors, pour commencer: ton using namespace est inutile, en C++ il peut être utile si tu te sers de la STL (plutôt que taper std::string, std::stack, std::cin, std::cout, mais là tu ne semble pas t'en servir, au temps pour moi si je n'ai pas bien vu)

ensuite, deux return, avec deux system("pause") c'est complétement inutile, donc tu enlève un pause et un return!

également, il faudra que tu regarde du cotès de devc++ pour utiliser du C et non du C++ (parce que là, c'est du C++ avec une librairie C, donc il faudrait changer le compilateur, mais je ne connais pas assez devc++ pour te dire comment faire)

une simple remarque, scanf c'est assez dangereux à utiliser, mais comme c'est plus simple, on va laisser! ;o

à la place de maVariable = maVariable + 1; utilise maVariable++; (plus propre, plus pratique)
essaye de donner des noms qui veulent dire quelque chose, parce que scanf("%d", &i) je ne comprend pas ce que c'est que i.
Enfin, plutôt que des

Code :

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

1
2
3
4
while(i<truc){ 
    i++;
    // on effectue des calculs
}

utilise cette boucle là:

Code :

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

1
2
3
for(i=0; i < truc; i++){
    //on effectue des calculs
}

Et pour finir revoie l'indentation comme précédemment, une fois que tu as fait ces différentes modifications, repost ton code, et je regarderai l'algorithme en lui même, mais là, c'est assez difficile à lire avec une indentation inexistante.
Tous les conseils que je t'ai donné sont juste pour facilité la relecture du code, avoir quelque chose de propre!

Bon courage!

[kwariz]

Salut,

pour paraphraser satenske, pourrais-tu carrément poster l'algorithme que tu as en tête ?
Car si tu ne sais pas ce que tu veux faire tu ne pourras pas le faire ... en décryptant ton code il m'apparaît que tu ne sais pas trop par où commencer mais tu y es allé

[prince 1990]

Bonjour les amis , aucun de vous ne peut m'aider ??

[quentinh]

On est tous prêts à t'aider, on attend que tu postes ton code corrigé selon les premiers éléments qu'on t'a donné.
La mise en forme est un élément extrèmement important, un peu comme la ponctuation en français. Tu n'imagines pas lire un bouquin sans aucune ponctuation, de la même manière c'est très difficile de lire un code sans mise en forme, comme celui que tu proposes.
Reposte ton code corrigé, et on pourra s'intéresser à l'algorithme qui est pour le moment totalement imbuvable .