Discussion :

[rosco-rs]

bonsoir je suis confronter a un petit soucis j'ai pour énoncer de d'afficher toutes les différentes combinaison de deux nombres entre 00 et 99 dans l'ordre croissant

exemple :

$>./a.out | cat -e
00 01, 00 02, 00 03, 00 04, 00 05, ..., 00 99, 01 02, ..., 97 99, 98 99$>

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
37
38
39
40
41
42
43
44
45
46
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1, &c, 1);
}
 
 void ft_print_comb2(void)
 {
     int a;
     int b;
     int c;
     int d;
 
     a = 0;
     b = 0;
     c = 0;
     d = 0;
 
     while (a <= 9)
     {
         while(b <= 8)
         {
             while (c <= 9)
             { 
                 while (d <= 9)
                 {  
                     ft_putchar(48 + a);
                     ft_putchar(48 + b);
                     ft_putchar(' ');
                     ft_putchar(48 + c);
                     ft_putchar(48 + d);
                     ft_putchar(',');
                     d++;
                 }
               c++; 
             }
            b++;
         }
         a++;
     }
 }
 int main ()
 {
     ft_print_comb2();
 }

et je n'affiche que 00 00,00 01,00 02,00 03,00 04,00 05,00 06,00 07,00 08,00 09,$

arrivé a ce moment la je suis censé incrémenter ma variable c mais une fois qu'il lit d++ et qu'il sort de l'accolade le prog s'arrête

Pourriez vous m'aider merci

[Bousk]

- variables non initialisées
- pourquoi ne pas utiliser for ?
- pourquoi ne pas utiliser printf avec le format %02d ?

[rosco-rs]

Voila je les est initialiser

Pour la reste c'est la consigne je peut pas autrement

est pour l'exemple tu voyais bien que sa fonctionnait sans (même si je sais qu'on doit toujours initialiser une Var)

[edgarjacobs]

Hello,

Tes variables ne sont pas initialisées au bon endroit: il faut initialiser b juste avant le while du b, etc

Edit: et ne pas modifier le code initial, mais reposter un code. La réponse de @Bousk perd de son sens....

[rosco-rs]

Voila j'ai fait initialiser les variable avant leur boucle on arrive a un résultat qui se rapproche du final mais ya un soucis a la fin on veut 97 99, 98 99
alors que sur le notre on a 99 98, 99 99

j'ai fait comme ça

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
37
38
39
40
41
42
43
44
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1, &c, 1);
}
 
 void ft_print_comb2(void)
 {
     int a;
     int b;
     int c;
     int d;
 
     while (a <= 9)
     {
         b = 0;
         while(b <= 9)
         {
             c = 0;
             while (c <= 9)
             { 
                 d = 0;
                 while (d <= 9)
                 { 
                     ft_putchar(48 + a);
                     ft_putchar(48 + b);
                     ft_putchar(' ');
                     ft_putchar(48 + c);
                     ft_putchar(48 + d);
                     ft_putchar(',');
                     d++;
                 }
               c++; 
             }
            b++;
         }
         a++;
     }
 }
 int main ()
 {
     ft_print_comb2();
 }

Merci pour les précision déjà

Ps : Dacc

[Bousk]

- a n'est toujours pas initialisé
- une variable non initialisée a une valeur inconnue et aléatoire
- si ça marche c'est uniquement par chance, ça n'indique surtout pas un code bon
- je vois pas pourquoi à la fin tu aurais 97,99 98,99 alors que le reste du temps tu as le deuxième chiffre qui évolue de 0 à 99 pendant que le premier est fixé; tu devrais avoir 97,99 98,00 98,01 ...

Aussi utiliser '0' au lieu de 48 serait plus clair
Et tu pourrait n'utiliser que 2 variables en faisant une division et un %

[dalfab]

Bonjour,

Ici tous les nombres possibles sont énumérés. Sans aucune action pour sauter certains nombres, il ne vont pas disparaître par magie.

A un moment tu as les chiffres a, b, c et d que tu affiches sous la forme de nombres «ab» et «cd». N'y a-t-il pas un test à faire sur a, b, c et d pour ne pas écrire les deux nombres qui correspondrait au nombre «ab» qui serait identique au nombre «cd».

[rosco-rs]

Merci pour t'es judicieuse réponse bousk

dalfab
j'ai pas trop saisis mais ok je vais creuser pour mieux comprendre

bonne soirée

[Sve@r]

Bonjour

dalfab
j'ai pas trop saisis mais ok je vais creuser pour mieux comprendre

Je pense qu'il te dit que tu ne dois pas afficher les nombres comme "1212", "1313", "2121", "4242", etc. bref tout nombre de type "xyxy" avec les deux premiers chiffres identiques aux deux derniers.

De mon côté, au vu de l'énoncé "afficher toutes les différentes combinaison de deux nombres entre 00 et 99 dans l'ordre croissant" je ne pense pas qu'un nombre de type "xyxy" doit être exclu.
Toutefois t'as pensé que ton truc c'était simplement afficher les nombres de 0 à 9999 sur 4 positions ? => for (i=0; i < 10000; i++) printf("%04d\n", i);...

[rosco-rs]

Oui au départ je pensait que c'était sa mais après avoir creuser on ma fait comprendre que je m'y prenait mal mais je voulais vraiment le faire de cette manière

Du coup on ma aider avec les modulo est le résultat donne sa

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
void	ft_print_comb2(void)
{
	int		i;
	int		j;
 
	i = 0;
	while (i <= 98)
	{
		j = i + 1;
		while (j <= 99)
		{
			ft_putchar((i / 10) + 48);
			ft_putchar((i % 10) + 48);
			ft_putchar(' ');
			ft_putchar((j / 10) + 48);
			ft_putchar((j % 10) + 48);
			if (i != 98 || j != 99)
			{
				ft_putchar(',');
				ft_putchar(' ');
			}
			j++;
		}
		i++;
	}
}

Voila !

[Sve@r]

Ok j'ai compris
Tu ne veux pas afficher que les nombres de 0000 jusqu'à 9999 ; tu veux, en partant de x variant de 00 jusqu'à 98, repartir de x+1 jusqu'à 99 et afficher l'ensemble ainsi formé (ex 00 01, 00 02, 00 03, ..., 00 99, 01 02, 01 03, ..., 01 99, 02 03, 02 04, ..., 02 99, 03 04, ..., ..., 97 01, 97 02, ..., 97 99 et enfin 98 99)

Donc déjà on t'a dit de mettre '0' à la place de 48. C'est plus explicite et plus portable.

Ensuite il y a quand-même plus simple

Code c :

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

1
2
3
4
5
6
7
8
9
10
void	ft_print_comb2(void) {
	int i;
	int j;
 
	for (i=0; i < 99; i++) {
		for (j=i+1; j <= 99; j++)
			printf("%02d %02d, ", i, j);
	}
	printf("\n");
}

[rosco-rs]

Oui merci Sve@r mais je n'avais le droit qu'a putchar est la boucle while pour cette éxo et petite question qui peut sembler bête mais
Sur ce code

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
 
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1,&c,1);
}
 
 void ft_putnbr(int nb)
{
    int i;
 
    if (nb > 0 && nb < 2147)
    {
 
        ft_putchar(i / 10 + '0');
    }
}
int main(void)
{
    ft_putnbr(70);
}

Normalement sur la sortie on est censé avoir 7 mais j'ai 0 est je comprend pas pourquoi si quelqu'un pouvais m'éclaire

[foetus]

Je passe par là :

2 questions : Ta variable i dans la procédure ft_putnbr

1. Est-elle initialisée ?
2. À quoi sert-elle ?

En théorie , si tu as 0 c'est parce que tu travailles en mode debug. En mode release tu aurais du random.

2. Rien

[rosco-rs]

C'est le soir

Merci je viens de me rendre compte de ma bêtise
je m'en excuse tellement c béte

[rosco-rs]

je finirais sur une dernière question pour ce soir lol

Sur ce code

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
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1,&c,1);
}
 
 void ft_putnbr(int nb)
{
	unsigned int	nbr;
 
	if (nb < 0)
	{
		ft_putchar('-');
		nbr = nb * -1;
	}
    else 
    nbr = nb;
    if (nbr >= 10)
    {
        ft_putnbr(nbr / 10);
        ft_putchar(nbr % 10 + 48);
    }
}
int main(void)
{
    ft_putnbr(19);
}

Le résultat sera 9

Est sur celui la

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
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1,&c,1);
}
 void ft_putnbr(int nb)
{
	unsigned int	nbr;
 
	if (nb < 0)
	{
		ft_putchar('-');
		nbr = nb * -1; // convertion pour afficher le nombre negatif avec le signe - avant 
	}
    else 
    nbr = nb;
    if (nbr >= 10)
        ft_putnbr(nbr / 10);
        ft_putchar(nbr % 10 + 48);
}
int main(void)
{
    ft_putnbr(19);
}

Le résultat sera 19 comme voulus car la consigne consistait a afficher la totalité des valeur possible dans une variable de type int

Dans le premier code j'ai laisser les accolade car j'ai appris que quand on avait plusieurs instruction on devais les mettre est si il n'y en avait qu'une on pouvait s'en passer alors pourquoi en laissant les accolade dans le premier cas sa ne marche pas est dans le second cas sa marche en les enlevant pourtant on a plus d'une instruction vous me suivez ? ( je parle des accolade dans le dernier if de chaque code )


Est aussi un dernier point svp dans le code ci-dessus c'est a dire celui qui marche si je rentre le chiffre 3 il me l'affiche comme voulu mais en essayant de comprendre comment l'ordinateur va procéder avec le programme je me disais qu'on on regarde les conditions je ne rentre ni dans le premier if (nb < 0) car 3 est au dessus de 0 ni dans le dernier if (nbr >= 10) car 3 est en dessous de 10 alors comment il arrive a afficher le 3 ??

Merci de bien vouloir m'éclairer

[Sve@r]

Sur ce code

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
#include <unistd.h>
 
int ft_putchar(char c)
{
    write(1,&c,1);
}
 
 void ft_putnbr(int nb)
{
	unsigned int	nbr;
 
	if (nb < 0)
	{
		ft_putchar('-');
		nbr = nb * -1;
	}
    else 
    nbr = nb;
    if (nbr >= 10)
    {
        ft_putnbr(nbr / 10);
        ft_putchar(nbr % 10 + 48);
    }
}
int main(void)
{
    ft_putnbr(19);
}

Le résultat sera 9

Alors que tu as demandé "19". Peut-être rajouter un + '0' à l'affichage de nbr/10...

Code c :

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

1
2
3
4
 
    if (nbr >= 10)
        ft_putnbr(nbr / 10);
        ft_putchar(nbr % 10 + 48);

Dans le premier code j'ai laisser les accolade car j'ai appris que quand on avait plusieurs instruction on devais les mettre est si il n'y en avait qu'une on pouvait s'en passer alors pourquoi en laissant les accolade dans le premier cas sa ne marche pas est dans le second cas sa marche en les enlevant pourtant on a plus d'une instruction vous me suivez ? ( je parle des accolade dans le dernier if de chaque code )

Parce que justement dans le premier code tu n'affiches ton nombre que s'il est supérieur ou égal à 10 et sinon tu n'affiches rien. Alors que dans le second, s'il est supérieur ou égal à 10 tu affiches la dizaine (enfin en n'oubliant pas le + '0') et tu affiches les unités dans tous les cas.
Ce n'est pas parce que l'instruction 2 est au même niveau que l'instruction 1 qu'elle sera exécutée avec elle. Le C ce n'est pas comme en Python où ce sont les alignements qui font les blocs. Les blocs sont encapsulés d'accolade et s'il n'y a pas d'accolade; alors le if n'exécute qu'une seule instruction.

Est aussi un dernier point svp dans le code ci-dessus c'est a dire celui qui marche si je rentre le chiffre 3 il me l'affiche comme voulu mais en essayant de comprendre comment l'ordinateur va procéder avec le programme je me disais qu'on on regarde les conditions je ne rentre ni dans le premier if (nb < 0) car 3 est au dessus de 0 ni dans le dernier if (nbr >= 10) car 3 est en dessous de 10 alors comment il arrive a afficher le 3 ??

Même réponse. Le dernier ft_putchar(nbr % 10 + 48) n'est pas dans le if.

[rosco-rs]

Un grand merci !
C'est beaucoup plus clair

Sauf que justement en rajoutant le + '0' dans putnbr comme tu dis je segfault
Mais si je le remplace par putchar donc sa donne

Code :

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

1
2
3
4
5
if (nbr >= 10)
    {
        ft_putchar(nbr / 10 + '0');
        ft_putchar(nbr % 10 + '0');
    }

La effectivement sa marche comme tu dis plus haut