Re: Image Crop Issue - C# Discussion Boards

Programm3r8-Sep-09 23:14

8-Sep-09 23:14

Hi all,

I am trying to crop an image based on a transparent panel that I have placed on the image. The transparent panel is placed on the image control based on certain coordinates (selected by a user based on mouse events). This works fine.

Now I want to try and crop the piece where the transparent panel is located on the image, but I keep receiving an exception: OutOfMemoryException => rect is outside of the source bitmap bounds.

But why is this happening? (note: The image Bounds value of X and Y is 0 and the transparent panel width and height is in bounds with the height and width of the image) The transparent panel is located within the picturebox, shouldn't it be able to get the correct position Confused | :confused:

??
Here is the code ...

...
                TransparentPanel tmpPnl = e.ClickedItem.Tag as TransparentPanel;
                Rectangle rec = tmpPnl.Bounds;
                Bitmap bmpImage = new Bitmap(base.Image);
                Bitmap bmpCrop = bmpImage.Clone(rec, bmpImage.PixelFormat); // exception occurs here
...

FYI

That "out of memory" exception appears when you attempt to crop a piece of the image outside itself.

Eg: If the image is 800×600 and you go:

Bitmap b = CropBitmap(myPic, 700, 500, 200, 200);

That’ll try select from 700×500 to 900×700, which does not actually exist.

Many thanks in advance
Kind regards,

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

modified on Wednesday, September 9, 2009 6:01 AM

Re: Image Crop Issue

Programm3r8-Sep-09 23:55

Programm3r

8-Sep-09 23:55

Ok, so the new method doesn't cause an exception.

But the new cropped image's position is a bit out. It crops more to the right. What am I doing wrong?

public static Bitmap CropBitmap(Bitmap srcBitmap, int cropX, int cropY, int cropWidth, int cropHeight)
{
    // Create the new bitmap and associated graphics object
    Bitmap bmp = new Bitmap(cropWidth, cropHeight);

    Graphics g = Graphics.FromImage(bmp);
    // Draw the specified section of the source bitmap to the new one
    g.DrawImage(srcBitmap, new Rectangle(0, 0, cropWidth, cropHeight), cropX, cropY, cropWidth, cropHeight, GraphicsUnit.Pixel);
    // Clean up
    g.Dispose();

    // Return the bitmap
    return bmp;
}

Thanks again.
Kind regards,

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

Re: Image Crop Issue

carlecomm21-Sep-09 21:49

carlecomm

21-Sep-09 21:49

there is an error 'bmpImage.Clone(rec, bmpImage.PixelFormat);'.
i think it might be 'bmpImage.size'<'rec.size',
so bmpImage's memory < rec's memory ,and it caused the error .

modified 27-May-14 5:34am.

Re: Image Crop Issue

Rob Smiley8-Sep-09 23:56

Rob Smiley

8-Sep-09 23:56

Hi, can you get the Clone to work if you hard-code the rectangle to say {0, 0, 50, 50}? Try putting a trace on tmpPnl.Bounds too, so you can see what's being passed to Clone - unless tmpPnl is a member of the pictureBox.Controls collection, it's bounds are likely to be relative to the pictureBox.Parent control. Also, wrap up bmpImage var in a using() block to ensure you don't leak memory.

Perhaps you could post the code that creates tmpPnl if this doesn't help

"An eye for an eye only ends up making the whole world blind"

Re: Image Crop Issue

Programm3r9-Sep-09 0:14

Programm3r

9-Sep-09 0:14

Hi,

Thanks for the reply.

Rob Smiley wrote:
can you get the Clone to work if you hard-code the rectangle to say {0, 0, 50, 50}?

Yes, it takes the top left-hand corner and computes the correct image result based on the width and the height.

Rob Smiley wrote:
unless tmpPnl is a member of the pictureBox.Controls collection, it's bounds are likely to be relative to the pictureBox.Parent control

The transparent panel is part of the control collection of the image, so it should be in bounds. I think Confused | :confused:

Values of rec1 and rec2 are the same..

TransparentPanel tmpPnl = e.ClickedItem.Tag as TransparentPanel;

Rectangle rec1 = base.Controls[tmpPnl.Name].Bounds;
Rectangle rec2 = tmpPnl.Bounds;

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

Re: Image Crop Issue

Rob Smiley9-Sep-09 0:36

Rob Smiley

9-Sep-09 0:36

ok, it sounds like the panel bounds are relative to the pictureBox... hmm. does the image take up the full space in the pictureBox? also have you set a sizeMode on it? e.g. an image smaller than the control area & a size mode of centre or stretch could allow you to have the rectangle appear over the image on screen, but give bounds that are outside the image area. Set sizeMode to normal (so image appears in top-left corner) & see what happens

"An eye for an eye only ends up making the whole world blind"

Re: Image Crop Issue

Programm3r9-Sep-09 1:35

Programm3r

9-Sep-09 1:35

Thanks again for the reply.

Rob Smiley wrote:
does the image take up the full space in the pictureBox?

Yes, I made the image to that it stretches the image.

Rob Smiley wrote:
Set sizeMode to normal (so image appears in top-left corner) & see what happens

OK, when I change the size mode on the image to normal; all seems to be working. It crops the correct piece relevant to the transparent panel.

When I change the size mode stretch Image, that's when the issue occurs with the cropped image.

How can I accommodate for the stretched image, I must make use of the stretched size mode, otherwise it will cause problems with an image bigger that the pictureBox control?

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

Re: Image Crop Issue

Luc Pattyn9-Sep-09 1:47

Luc Pattyn

9-Sep-09 1:47

to discover which part of the image you want, you have to apply to your cropping (i.e.mouse) coordinates the same transformation that are being applied to the entire image; so you have to mimick the stretching if you PictureBox is set to stretch mode.

That is one of the many problems a PictureBox causes. I almost never use that class, I rather use a simple Panel, and create a Paint handler for it; that way I choose the transformations I want (if any) and I can apply the same transformations to mouse operations. Also, I can then keep the aspect ratio, something a PictureBox isn't really capable of.

Conclusion:
- apply transformations to all coordinates you get (from mouse handlers)
- if possible, get rid of PictureBox

Smile | :)

Luc Pattyn

Have a look at my entry for the lean-and-mean competition; please provide comments, feedback, discussion, and don’t forget to vote for it! Thank you.

Re: Image Crop Issue

Programm3r9-Sep-09 1:54

Programm3r

9-Sep-09 1:54

Thanks for the reply.

Luc Pattyn wrote:
I rather use a simple Panel, and create a Paint handler for it

Hmmm ... I'll give it a shot

Kind regards,

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

Re: Image Crop Issue

Rob Smiley9-Sep-09 2:22

Rob Smiley

9-Sep-09 2:22

Hi, you need to do as Luc suggests & scale the rectangle of your cropping area. This should do the trick:

float scaleX = (float)bmpImage.Width / (float)this.ClientSize.Width;
float scaleY = (float)bmpImage.Height / (float)this.ClientSize.Height;

rec = new Rectangle((int)(rec.X * scaleX), (int)(rec.Y * scaleY), (int)(rec.Width * scaleX), (int)(rec.Height * scaleY));

"An eye for an eye only ends up making the whole world blind"

Re: Image Crop Issue

Programm3r9-Sep-09 2:54

Programm3r

9-Sep-09 2:54

Genius!!! Code works like a charm!! Cool | :cool:

Thanks.

The only programmers that are better C# programmers, are those who look like this -> Green Alien | [Alien]

Programm3r
My Blog: ^_^

Problem Sending Message from SERVER to CLIENT

muhammad_umair8-Sep-09 23:09

muhammad_umair

8-Sep-09 23:09

Hi, im making a Window Application in C# using Socket Programming. I have developed a Server & a Client. Both are working fine but the problem which im gettin is that when ever i send any message from CLIENT, its send perfectly and receives on SERVER but whenever i try to send any message from SERVER it doesn't send to Client, since at the beginning when a connection is built, server sends the message to client that "Connection Established" and received at Client perfectly,but later on server does not send any message to client!!! Could anyone please help me out ???????
Regards
Umair

Re: Problem Sending Message from SERVER to CLIENT

monstale8-Sep-09 23:45

monstale

8-Sep-09 23:45

Hi,

could you please send code, how client receives messages? Would help us, to help you. Wink | ;-)

bye

Re: Problem Sending Message from SERVER to CLIENT

muhammad_umair9-Sep-09 0:19

muhammad_umair

9-Sep-09 0:19

//this is the code at client side for receiving data

NetworkStream networkStream = new NetworkStream(server);
string input = textBoxUser.Text + ": " + textBoxWrite.Text;
ASCIIEncoding encoding = new ASCIIEncoding();
byte[] inputByte = encoding.GetBytes(input);
if (networkStream.CanWrite)
{
networkStream.Write(inputByte, 0, inputByte.Length);
textBoxShow.Text = textBoxShow.Text +
Environment.NewLine + input;
textBoxWrite.Text = "";
networkStream.Flush();
}

Re: Problem Sending Message from SERVER to CLIENT

monstale9-Sep-09 0:52

monstale

9-Sep-09 0:52

looks like a sending code. You used NetworkStream.Write(). That might be the problem. Smile | :)

Re: Problem Sending Message from SERVER to CLIENT

muhammad_umair9-Sep-09 1:43

muhammad_umair

9-Sep-09 1:43

but i have also used server.send(AnyMessage);
this isn't working as well..

Re: Problem Sending Message from SERVER to CLIENT

monstale9-Sep-09 2:30

monstale

9-Sep-09 2:30

Okay, maybe this a problem in understanding:
you wrote code for receiving data at client side, right?
You used the method .Write(). This method writes bytes to the stream. I think in a receiving method you should use e.g. Read(byte[], int, int), or am I wrong? Wink | ;-)

Re: Problem Sending Message from SERVER to CLIENT

muhammad_umair9-Sep-09 2:44

muhammad_umair

9-Sep-09 2:44

the problem has been resolved.. the message was not sending for the second time to client because i was using IF loop in client side that would take data from server just once, now on the client side i have replaced if by WHILE(true) and it receives the data until a break is performed send by server in the form of Disconnect..newys thank you guys Smile | :)

Re: Problem Sending Message from SERVER to CLIENT

J4amieC9-Sep-09 2:54

J4amieC

9-Sep-09 2:54

muhammad_umair wrote:
newys

Syntax error. 'newys' does not exist in the english language.

MouseWheel Event

Lyon Sun8-Sep-09 21:52

Lyon Sun

8-Sep-09 21:52

Hi all,

I am using Visual Studio 2005 now. But why can't I get this MouseWheel Event of a panel working on? Code for that is as follows:

void t_MouseWheel(object sender, MouseEventArgs e)
        {
            // Update the drawing based upon the mouse wheel scrolling.

            int numberOfTextLinesToMove = e.Delta * SystemInformation.MouseWheelScrollLines / 120;
            int numberOfPixelsToMove = numberOfTextLinesToMove * fontSize;

            if (numberOfPixelsToMove != 0)
            {
                System.Drawing.Drawing2D.Matrix translateMatrix = new System.Drawing.Drawing2D.Matrix();
                translateMatrix.Translate(0, numberOfPixelsToMove);
                mousePath.Transform(translateMatrix);
            }
            t.Invalidate();
        }

t indicates a TableLayoutPanel.
And then I added this line: t.MouseWheel += new MouseEventHandler(t_MouseWheel); after InitialComponent();.
Is there anything wrong here?
Please help me... Thanks!

BR!

Re: MouseWheel Event

OriginalGriff8-Sep-09 22:17

OriginalGriff

8-Sep-09 22:17

Probably because it doesn't have the focus. Do you have any controls on your panel? If not, and you have controls outside the panel, they they will get the focus and your panel will not. I would suggest that you move the MouseWheel event handler to the form, and check if the mouse pointer is over the panel for your processing.

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones

Re: MouseWheel Event

Lyon Sun8-Sep-09 23:09

Lyon Sun

8-Sep-09 23:09

Hi,
Thanks for your reply. I have some labels on this TableLayoutPanel, and I tried to make this panel get focus by the following code:

void t_MouseEnter(object sender, EventArgs e)
        {
            t.Focus();
        }

But still it doesn't work. I check on the designer window, it's actually doing something cuz the panel is a little bit shaking, however, the panel doesn't scroll up and down. Since I am making a user control, there is a seperated scrollbar beside this panel. Well, that's another thing. Please give some advices, thank you!

Sincerely

Re: MouseWheel Event

Christian Graus8-Sep-09 23:12

Christian Graus

8-Sep-09 23:12

Leapsword wrote:
it's actually doing something cuz the panel is a little bit shaking,

OK. So, you didn't set any breakpoints to see if your event fired then ? Sounds like it's firing and your code does not work.

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

Re: MouseWheel Event

Lyon Sun8-Sep-09 23:29

Lyon Sun

8-Sep-09 23:29

Christian Graus wrote:
So, you didn't set any breakpoints to see if your event fired then ?

Well, of course I did. Code works well, but I don't know why it can't scroll, Frown | :(

. Anyway, thank you for reply me.

BR!

Re: MouseWheel Event

OriginalGriff8-Sep-09 23:43

OriginalGriff

8-Sep-09 23:43

The panel itself cannot recieve the focus, because the user cannot "do anything" with it. As a result, it does not get MouseWheel events unless there is a control on the panel which can interact with the user - such as a text box. In your case, what I would do is hook into the panel.Parent.MouseWheel event and use the MouseEventArgs to check if the mouse pointer is over the panel. This should work fine in a custom control (but I haven't tested it).

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.