Discussion :

[bobbyy]

Bonjour.
Je dois faire une rosace en C.
Pour cela j'utilise la minilibx:

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
47
#include "mlx.h"
#include <stdlib.h>
#include "my.h"
#define PI 3.14159
#include <math.h>
 
int	gere_key(int keycode, void *param)
{
  param = 0;
  if (keycode == 65307)
    exit(1);
  return (0);
}
 
int	option1()
{
  void		*mlx_ptr;
  void		*win_ptr;
  int		x;
  int		y;
  int		clr;
  int		nb;
  double	theta;
  int		r;
  int		angle;
 
  nb = 1000;
  mlx_ptr = mlx_init();
  win_ptr = mlx_new_window(mlx_ptr, 500, 500, "option1");
  angle = 0;
  clr = 0xFF0000;
  while (nb--)
    {      
      theta = PI * angle / 180;
      r = cos(3.5 * theta);
      x = 250 + r * cos(theta) * 1000;
      y = 250 + r * sin(theta) * 1000;
      r = r * 1000;
      mlx_pixel_put(mlx_ptr, win_ptr, x, y, clr);
      angle++;
      if (angle == 361)
	angle = 0;
    }
  mlx_key_hook(win_ptr, gere_key, 0);
  mlx_loop(mlx_ptr);
  return (0);
}

theta sert a convertir l'angle en radian.

Seulement au lieu d'avoir une rosace j'ai juste un point au milieu de ma fenetre et je ne comprend pas pourquoi.

[Invité(e)]

Bonjour,

Ce genre de calculs doivent être réalisés sur des types à virgule flottante : float ou double.

en effet :

Code :

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

r = cos(3.5 * theta);

si r est entier, cos étant entre -1 et 1, r ne pourra prendre que des valeurs entière : -1, 0 ou 1, mais quasiment tout le temps 0.

De plus les constantes peuvent aussi être typées, écrire

Code :

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

200

n'est pas la même chose qu'écrire

Code :

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

200.

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
int	option1()
{
  void		*mlx_ptr;
  void		*win_ptr;
  double	x;
  double	y;
  int		clr;
  int		nb;
  double	theta;
  double	r;
  double	angle;
 
  nb = 1000;
  mlx_ptr = mlx_init();
  win_ptr = mlx_new_window(mlx_ptr, 500, 500, "option1");
  angle = 0.;
  clr = 0xFF0000;
  while (nb--)
    {      
      theta = PI * angle / 180.;
      r = cos(3.5 * theta);
      x = 250. + r * cos(theta) * 1000.;
      y = 250. + r * sin(theta) * 1000.;
      r = r * 1000.;
      mlx_pixel_put(mlx_ptr, win_ptr, x, y, clr);
      angle += 1.;
      if (angle > 360.)
	angle = 0.;
    }
  mlx_key_hook(win_ptr, gere_key, 0);
  mlx_loop(mlx_ptr);
  return (0);
}

[bobbyy]

Merci bien.

Ca marche nickel

(je n'ai pas trouve le bouton regle)

[Médinoc]

Et maintenant, un peu de programmation défensive pour consolider le tout:

• Limitations de portée des variables
• Variables déclarées const.

Code C :

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
int	option1()
{
  void		*mlx_ptr;
  void		*win_ptr;
  int		nb;
  int const clr = 0xFF0000;
  double	angle;
 
  nb = 1000;
  mlx_ptr = mlx_init();
  win_ptr = mlx_new_window(mlx_ptr, 500, 500, "option1");
  angle = 0.;
  while (nb--)
    {      
      double const theta = PI * angle / 180.;
      double const r = cos(3.5 * theta);
      double const x = 250. + r * cos(theta) * 1000.;
      double const y = 250. + r * sin(theta) * 1000.;
      /*r = r * 1000.; cette ligne est inutile, puisque r n'est plus lu avant sa prochaine modif*/
      mlx_pixel_put(mlx_ptr, win_ptr, x, y, clr);
      angle += 1.;
      if (angle > 360.)
	angle = 0.;
    }
  mlx_key_hook(win_ptr, gere_key, 0);
  mlx_loop(mlx_ptr);
  return (0);
}

PS: mlx_loop() fait le nettoyage elle-même, ou bien manque-t-il un appel à une fonction genre mlx_clean() à la fin?

[Invité(e)]

Merci bien.

Ca marche nickel

(je n'ai pas trouve le bouton regle)

C'est le bouton "résolu" en dessous du denier post