Discussion :

[fhoest]

Bonjour,
je voudrais faire une petite pause dans l'exécution d'une boucle for,j'ai bien essayé avec sleep() mais quelque chose cloche et ne passe pas.
Pourriez vous m'éclairer sur ce point.
D'avance merci.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <cstdlib>
#include <iostream>
#include <windows.h>
using namespace std;
void b()
{
    cout<<"descendant"<<endl;
int i;              
  for(i=10;i>0;i=i-1)
sleep(10);
cout<< i <<endl;
}

C'est bon j'ai vu mon erreur en plus je l'avais utiliser pour quelqu'un d'autre.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
#include <cstdlib>
#include <iostream>
#include <windows.h>
using namespace std;
void b()
{
    cout<<"descendant"<<endl;
int i;              
  for(i=10;i>0;Sleep(1000),i=i-1)
  cout<< i <<endl;
}

le S doit être majuscule si cela peut aider quelqu'un.
Merci quand même a ceux qui aurait tenté de m'aider.
A+

[Emmanuel Deloget]

C'est très "pas beau" ce que tu fait là. La bonne technique est d'utiliser des accolades pour créer un bloc.

[fhoest]

Bonjour,
pourriez vous svp me mettre un exemple "beau",pour que je comprenne ce que vous voulez me dire.
Merci,
pour moi le principal pour l'instant n'est pas vraiment la beauté mais la compréhension,mais si je peux également améliorer la beauté pourquoi pas...
A+

[koala01]

Salut,

Ben, simplement, il faut penser au fait qu'un code est beaucoup plus plus souvent lu / analysé qu'il n'est compilé /exécuté.

Il faut donc veiller à rendre le code le plus lisible possible, pour l'humain avant tout.

Pour cela, il y a quelques règles simples à essayer d'appliquer en permanence. En voici une liste non exhaustive

1. essayer d'utiliser des noms (de type définis par l'utilisateur, de fonction ou de variables) aussi explicite que possible quant à l'usage qui est fait
2. mettre les blocs devant s'effectuer de manière plus ou moins atomique entre accolade
3. ne mettre qu'une instruction ou qu'une déclaration de variable par ligne
4. indenter correctement le code en mettant un nombre d'espace suffisant (quatre c'est souvent pas mal )
5. éviter les "sucres syntaxique" tant qu'il n'est pas prouvé qu'ils sont nécessaires
6. respecter des règles de nommage strictes
7. ne déclarer les variables que lorsqu'on en a besoin
8. j'en oublie peut etre

Dans ton cas, un "beau code" ressemblerait à quelque chose proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#include <cstdlib>
#include <iostream>
#include <windows.h>
using namespace std;
void b() // peut être pourrait on renommer b en "affichageDescendant"?
{
    cout<<"descendant"<<endl;
    for(int i=10;i>0;i=i-1)
    {
        Sleep(1000); 
        cout<< i <<endl;
    }
}

NOTALe code n'as pas été corrigé dans le fond, je n'ai changé que la mise en forme pour respecter les règles décrites plus haut

[fhoest]

Bonjour,
Ok pour le code j'essayerai de faire un code un peu plus convenable la prochaine fois,mais je ne garantie pas a coup sur qu'il sera au gout de tous,faut pas oublier que je débute,
tout de même merci pour les conseil et l'exemple.

[koala01]

C'est justement parce que tu débutes qu'il est intéressant de marteler ces conseils: il est très facile de prendre des habitudes, mais très difficile de les perdre! Autant prendre directement les bonnes, pour éviter d'avoir à perdre les mauvaises

[oodini]

il est très facile de prendre des habitudes, mais très difficile de les perdre! Autant prendre directement les bonnes, pour éviter d'avoir à perdre les mauvaises

En ce cas :

Code :

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

for (unsigned int i=10; i>0; i--)

[fhoest]

Bonjour Oodini,
il est vrai qu'il faut tenter de prendre de très bonnes habitude,mais je pense fortement que Koala01 a voulu me conseiller tout simplement et non pas faire le malin avec ces connaissances.
C'est très bien de ta part de t'impliquer et tenter de rendre encore meilleur le code que j'essaie de faire,mais un peu d'explication au sujet du code proposé est pas mal non plus.
Je te souhaite une agréable journée et te remercie pour l'aide que tu apportes.
Au plaisir.

[bertry]

Salut,

Je ne pense pas qu'Oodini ait voulut t'écraser, il a juste apporté sa contribution à la discution...

Il est vrai que l'usage veut, que plutôt que d'écrire

Code :

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

i=i-1;

, on préfère écrire

Code :

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

i--;

( j'ajouterai même que dans une boucle for j'utilise plutôt

Code :

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

--i;

, ça optimise légèrement...( là aussi, c'est une bonne habitude à prendre ) )

L'instruction i-- décrémente la variable i de 1 tout simplement

[koala01]

Je tiens à signaler que j'avais clairement indiqué ne pas avoir corrigé le code

[oxyde356]

En ce cas :

Code :

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

for (unsigned int i=10; i>0; i--)

Moi je te conseille d'éviter de faire ça parce que si par mégarde tu change ton > en >= (parce que le contexte fera que tu veux atteindre 0) tu vas avoir une boucle infinie. Car quand i sera égal à 0 et que la décrémentation va s'exécuter ça ne fera pas -1 mais (2^32-1)-1 soit un nombre bien supérieur à 0.

[fhoest]

Bonjour à tous ,
pour commencer NON je n'ai pas mal pris ce que oodini a voulu me dire.
@ bertry & oodini

C'est très bien de ta part de t'impliquer et tenter de rendre encore meilleur le code que j'essaie de faire.
Je te souhaite une agréable journée et te remercie pour l'aide que tu apportes.

@ Koala01
oui j'avais remarqué et de toute façon la demande n'était pas de revoir le code dans son écriture mais bien dans son style (beauté)

NOTALe code n'as pas été corrigé dans le fond, je n'ai changé que la mise en forme pour respecter les règles décrites plus haut

@ tous,
Je vous remercie encore et je suis TRES SATISFAIT de toutes vos propositions (je ne suis pas ici pour juger mais pour être aider)
Bonne journée à tous.