Discussion :

[ralf91]

salut !!
j'ai un probleme avec les couleurs et le GetPixel je crois !! bon ce que je voudrais faire ! j'ai une image noire sur blanc, quand je clique sur quelque chose noir ben un messageBox s'affiche pour dire noire sinon un autre messageBox pour dire blanc voici mon 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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            bmp = new Bitmap(pictureBox1.Width, pictureBox1.Height);
            g = Graphics.FromImage(bmp);
        }
 
        Bitmap img,bmp;
        Graphics g;
 
        Color noire = Color.Black;
 
        private void button1_Click(object sender, EventArgs e)
        {
            openFileDialog1.Title = "Choose a shape...";
            openFileDialog1.DefaultExt = "jpg";
            openFileDialog1.Filter = "Bitmap file (*.bmp)|*.bmp|Images file JPEG (*.jpg)|*.jpg";
            if (openFileDialog1.ShowDialog() == DialogResult.OK)
            {
                img = new Bitmap(openFileDialog1.FileName);
                pictureBox1.Image = img;
                g = Graphics.FromImage(bmp);
                g.DrawImage(pictureBox1.Image, 0, 0);
            }
        }
 
        private void pictureBox1_MouseClick(object sender, MouseEventArgs e)
        {
 
            Color cl = bmp.GetPixel(e.X, e.Y);
 
            if (cl == noire)
            {
                MessageBox.Show("noire");
            }
            else
            {
                MessageBox.Show("blanc");
            }
 
        }
    }
}

le probleme :
il m'affiche tout le temps blanc n'import où je clique !! pourquoi ? meme quand j'utilise img au lieu de bmp c'est le probleme !!

[binoo]

Tu peux faire comme ça :

Code :

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

1
2
3
4
if( bmp.GetPixel(e.X, e.Y).Name == "ffffff")
  MessageBox.Show("blanc");
else
  MessageBox.Show("noir");

ou bien

Code :

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

1
2
3
4
if( bmp.GetPixel(e.X, e.Y).R == 255 && bmp.GetPixel(e.X, e.Y).G == 255 && bmp.GetPixel(e.X, e.Y).B == 255)
  MessageBox.Show("blanc");
else
  MessageBox.Show("noir");

[tomlev]

Non, je pense que la comparaison est correcte... mais pourquoi tu te galères avec des Graphics ? A ta place j'utiliserais seulement la propriété Image du PictureBox, on y verrait plus clair.

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
 
namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            openFileDialog1.Title = "Choose a shape...";
            openFileDialog1.DefaultExt = "jpg";
            openFileDialog1.Filter = "Bitmap file (*.bmp)|*.bmp|Images file JPEG (*.jpg)|*.jpg";
            if (openFileDialog1.ShowDialog() == DialogResult.OK)
            {
                pictureBox1.Image = new Bitmap(openFileDialog1.FileName);
            }
        }
 
        private void pictureBox1_MouseClick(object sender, MouseEventArgs e)
        {
 
            Color cl = pictureBox1.Image.GetPixel(e.X, e.Y);
 
            if (cl == Color.Black)
            {
                MessageBox.Show("noire");
            }
            else
            {
                MessageBox.Show("blanc");
            }
 
        }
    }
}

EDIT : Au fait, si l'image est redimensionnée (PictureBox.SizeMode autre que Normal), ça fausse tout, car l'échelle de la PictureBox n'est plus la même que celle de l'image. Donc il faut transcrire les coordonnées de la PictureBox vers celles de l'image...

[ralf91]

salut !!
la seule solution qui marche c'est d'utiliser les couleur rouge, bleu et vert je ne sais pas pourquoi les autres solutions ne marchent pas ???

[tomlev]

Et en faisant la comparaison comme ça, ça marche pas non plus ?

Code :

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

if (cl.Equals(Color.Black))

[ralf91]

salut !!
j'ai pas essayé ca !! mais je vais l'essayer et je vous répondrez

[Pol63]

Color est une structure, donc le problème ne vient pas de la comparaison !

le graphics n'a dans le code montré aucun intérêt

mets un point d'arrêt sur ton événement pour voir la couleur réelle ...