Discussion :

[stallaf]

Bonjour à tous,
Je débute et apprend seul, j'ai donc quelques difficultés...
Pour le code posté ci-dessous je n'arrive pas à boucler afin d'arriver à 1.
Pourriez-vous m'éclairer ?

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
28
29
30
31
32
33
34
35
36
 
#!bin/sh
 
# Programme qui, tant qu'un nombre est différent de 1, le divise
# par 2  s'il est pair et le multiplie par 3 et lui ajoute 1 s'il
# est impair.
 
case $# in
    0) echo "Fournissez un nombre s'il vous plait :"
    read nombre
    nombre2=$[ nombre % 2 ]
    while [ $nombre > 1 ]
        do
     	   if [ $nombre2 = 0 ]
            	then
                        nombre3=$[ nombre / 2 ]
                        echo "$nombre est pair, j'obtiens donc $nombre3"
                else
                   	nombre3=$[ (nombre * 3) + 1 ]
                        echo "$nombre est impair, j'obtiens donc $nombre3"
           fi
	done;;
  1)  nombre4=$[ $1 % 2 ]
   while [ $1 > 1 ]
       do
          if [ $nombre4  = 0 ]
                then
                        nombre4=$[ $1 / 2 ]
                        echo "$1 est pair, j'obtiens donc $nombre4"
                else
                        nombre4=$[ ($1 * 3) + 1 ]
                        echo "$1 est impair, j'obtiens donc $nombre4"
          fi
        done;;
   *) echo "TROP DE PARAMETRES !"
esac

D'avance, merci.

[al1_24]

A la lecture de ton script, je ne comprends pas bien ce qu'il est censé faire...

Ce que je te propose, c'est de commenter chaque ligne du programme en indiquant ce qu'elle fait.
Ensuite tu ne conserves que les commentaires et tu les relis (à haute voix) pour savoir si le programme que tu as écrit fait bien ce que tu voudrais qu'il fasse.

Pour écrire un programme, la technique est d'ailleurs la même : décrire le fonctionnement en langage clair (pseudo-code) et associer ensuite chaque phrase de la description à son équivalent dans le langage de programmation.
C'est un moyen sûr d'arriver au bout de ce que l'on cherche à programmer, de pouvoir se relire quand on reprend le programme pour le faire évoluer ... et de garder de bonnes relations avec ses collègues quand ils doivent travailler sur ton code

[stallaf]

Ce script devrait donner ceci en console :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
$ script2 10
10 est pair, j'obtiens donc 5 
5 est impair, j'obtiens donc 16
16 est pair, j'obtiens donc 8
8 est pair, j'obtiens donc 4
4 est pair, j'obtiens donc 2
2 est pair, j'obtiens donc 1
Je suis parvenu à 1...
$ script 2 
Fournissez un nombre s'il vous plait : 13
13 est impair...

Donc je souhaite faire :

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
 
Si aucun argument
Demander nombre
Le lire
Enregistrer le reste de la division par deux dans nombre2
Tant que nombre est > à 1
	faire
		Si cette variable vaut 0
			alors
				nombre3 vaut nombre divisé / 2
				on affiche un message
# c'est là que je ne perçois pas comment passer au calcul suivant, 
# resultat pair ou impair
			sinon
				nombre4 vaut nombre * 3 + 1
				on affiche un message

Le reste voulant procéder de la même logique mais avec un nombre en argument.

Le scrip boucle de manière infini au niveau du commentaire.

[al1_24]

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
 
Si aucun argument
Demander nombre
Le lire
Tant que nombre est > à 1
    faire
        # A faire dans la boucle
        Enregistrer le reste de la division par deux dans nombre2
        Si cette variable vaut 0
            alors
                nombre3 vaut nombre divisé / 2
                on affiche un message
            sinon
                nombre4 vaut nombre * 3 + 1
                on affiche un message
       finsi
# c'est là qu'il manque quelque chose
    lire le nombre suivant # avant de boucler
Fin tant que

N'oublie pas non plus les fin de structure (finsi, fin tant que, ...) dans ton pseudo-code. C'est aussi important

[stallaf]

Je prend bonne note de ces conseils...

Cependant, après beaucoup de lecture (trop ?) de tutos, faq et autres, je suis toujours bloqué. Je n'arrive pas à visualiser comment finaliser.
J'ai du faire fausse route mais je ne vois pas où.
Montrez-moi la voir SVP

[LLB]

C'est un problème de logique. Quand je lis ton algo, en français, ça me semble normal qu'il y ait une boucle infinie. nombre n'est jamais modifié.

[al1_24]

Je t'ai pourtant donné la réponse....

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
Si aucun argument
Demander nombre
Le lire
Tant que nombre est > à 1
    faire
        # A faire dans la boucle
        Enregistrer le reste de la division par deux dans nombre2
        Si cette variable vaut 0
            alors
                nombre3 vaut nombre divisé / 2
                on affiche un message
            sinon
                nombre4 vaut nombre * 3 + 1
                on affiche un message
       finsi
# c'est là qu'il manque quelque chose
    lire le nombre suivant # avant de boucler
Fin tant que

N'oublie pas non plus les fin de structure (finsi, fin tant que, ...) dans ton pseudo-code. C'est aussi important

[stallaf]

Oopss... je n'avais rien lu prenant les deux premières lignes pour mon propre post.
Je regarde bien sur et reviendrais