IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

C Discussion :

correction d'un programme


Sujet :

C

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre confirmé
    Homme Profil pro
    Inscrit en
    Octobre 2011
    Messages
    69
    Détails du profil
    Informations personnelles :
    Sexe : Homme

    Informations forums :
    Inscription : Octobre 2011
    Messages : 69
    Par défaut correction d'un programme
    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;
    }

  2. #2
    Membre éprouvé Avatar de quentinh
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2011
    Messages
    79
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Ardennes (Champagne Ardenne)

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Mars 2011
    Messages : 79
    Par défaut
    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;
    }

  3. #3
    Membre confirmé
    Homme Profil pro
    Inscrit en
    Octobre 2011
    Messages
    69
    Détails du profil
    Informations personnelles :
    Sexe : Homme

    Informations forums :
    Inscription : Octobre 2011
    Messages : 69
    Par défaut
    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

  4. #4
    Membre confirmé Avatar de satenske
    Homme Profil pro
    Développeur Java
    Inscrit en
    Mars 2011
    Messages
    143
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Hauts de Seine (Île de France)

    Informations professionnelles :
    Activité : Développeur Java
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Mars 2011
    Messages : 143
    Par défaut
    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!

  5. #5
    Membre Expert
    Avatar de kwariz
    Homme Profil pro
    Chef de projet en SSII
    Inscrit en
    Octobre 2011
    Messages
    898
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 52
    Localisation : France, Moselle (Lorraine)

    Informations professionnelles :
    Activité : Chef de projet en SSII
    Secteur : Conseil

    Informations forums :
    Inscription : Octobre 2011
    Messages : 898
    Par défaut
    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é

  6. #6
    Membre confirmé
    Homme Profil pro
    Inscrit en
    Octobre 2011
    Messages
    69
    Détails du profil
    Informations personnelles :
    Sexe : Homme

    Informations forums :
    Inscription : Octobre 2011
    Messages : 69
    Par défaut
    Bonjour les amis , aucun de vous ne peut m'aider ??

  7. #7
    Membre éprouvé Avatar de quentinh
    Homme Profil pro
    Étudiant
    Inscrit en
    Mars 2011
    Messages
    79
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 33
    Localisation : France, Ardennes (Champagne Ardenne)

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Mars 2011
    Messages : 79
    Par défaut
    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 .

Discussions similaires

  1. correction d'un programme
    Par ferhat.adel dans le forum CUDA
    Réponses: 6
    Dernier message: 17/12/2012, 17h21
  2. correction d'un programme
    Par karika dans le forum MATLAB
    Réponses: 3
    Dernier message: 28/05/2011, 23h19
  3. correction d'un programme matlab
    Par nadim216 dans le forum MATLAB
    Réponses: 1
    Dernier message: 08/06/2010, 13h03
  4. Correction d'un programme
    Par guipe dans le forum Débuter
    Réponses: 9
    Dernier message: 05/02/2010, 17h25
  5. Correction d'un programme convertisseur de température
    Par nismaty dans le forum Débuter avec Java
    Réponses: 10
    Dernier message: 19/11/2009, 14h11

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo