C# How to get value R.G.B far?

Question

4.00/5 (1 vote)

See more:

private void pictureBox1_MouseMove(object sender, MouseEventArgs e) 
{
        if (pictureBox1.Image != null) 
        { 
 
 
          Bitmap b = new Bitmap(pictureBox1.Image); 
          if (e.X >= 0 && e.X <= b.Width && e.Y >= 0 && e.Y <= b.Height) 
          { 
              Color c = b.GetPixel(e.X, e.Y); 
              mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString() 
                          + " B : " + c.B.ToString(); 
          } 
 
 
            /*Bitmap b = new Bitmap(pictureBox1.Image); 
            Color c = b.GetPixel(e.X, e.Y); 
 
            mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString() 
            + " B : " + c.B.ToString();*/ 
 
 
        } 
        else 
        { 
 
        } 
    }

eX, eYcorresponding values for Width of the crew than the size of the image seems to occur when you just can not figure out how to handle exceptions Mouse Move event put him in a Picture Box

Go to any part of the parameter must be positive and will be less than Width.

The parameter name: x

Tteuneyo that errors

pictuerBox size and ready to bring us a picture to zoom Style

Posted 19-Dec-11 22:08pm

devil1852

Updated 19-Dec-11 22:11pm

RaviRanjanKr

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2011-12-19T22:28:00

Please, please, do not do that!
Every time you move the mouse (and that can be a hundred or more times a second) you are creating a new bitmap the same as the existing one! And you don't dispose of it, so it is hanging around until the Garbage Collector comes along to get rid of it.

I cannot think of a single less efficient way to do that!

Fiurstly, if you create a bitmap, you are responbsible for Disposing it, either explicitly, or by creating it in a using block:

C#

using(Bitmap b = new Bitmap(pictureBox1.Image))
{
   if (e.X >= 0 && e.X <= b.Width && e.Y >= 0 && e.Y <= b.Height)
   {
       Color c = b.GetPixel(e.X, e.Y);
       mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString()
                   + " B : " + c.B.ToString();
   }


     /*Bitmap b = new Bitmap(pictureBox1.Image);
     Color c = b.GetPixel(e.X, e.Y);

     mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString()
     + " B : " + c.B.ToString();*/
 }

But a better idea is not to create a new one at all:

C#

Bitmap b = (Bitmap) pictureBox1.Image;
if (e.X >= 0 && e.X <= b.Width && e.Y >= 0 && e.Y <= b.Height)
{
    Color c = b.GetPixel(e.X, e.Y);
    mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString()
                + " B : " + c.B.ToString();
}


  /*Bitmap b = new Bitmap(pictureBox1.Image);
  Color c = b.GetPixel(e.X, e.Y);

  mouseCOL.Text = "R : " + c.R.ToString() + " G : " + c.G.ToString()
  + " B : " + c.B.ToString();*/

The rest of your problem I don't understand at all - it's probably a language thing, but please try top give us the actual error message, and indicate on which line you are getting it.