How to remove the hard coded path for image

Question

0.00/5 (No votes)

See more:

Here I am trying to copy and save the image from picture box.
I am having five picture box.

but I am Hardcoded the "image im = PictureBox1.Image;"
and image name,

this way i can copy and save the image from picturebox1.
How to copy the image from other picture box.

C#

private void MouseClick()
            {     
                PictureBox1.MouseClick += form_MouseClick;
              
                PictureBox2.MouseClick += form_MouseClick;
    
                PictureBox3.MouseClick += form_MouseClick;
    
                PictureBox4.MouseClick += form_MouseClick;

                PictureBox5.MouseClick += form_MouseClick;
            }
    		
    	private void frmRightClick_Disposed(object sender, EventArgs e)
            {
                folderBrowser.ShowDialog();
                string filepath = folderBrowser.SelectedPath;
                Image im = PictureBox1.Image;
                var obj = new Random();
                SaveImage(im, filepath + "\\" + obj.Next() + ".png");
            }
    		
    		private void SaveImage(Image im, string destPath)
            {
                im.Save(destPath, System.Drawing.Imaging.ImageFormat.Png);
            }

What I have tried:

I tried using the sender

C#

private void frmRightClick_Disposed(object sender, EventArgs e)
            {
                folderBrowser.ShowDialog();
                string filepath = folderBrowser.SelectedPath;
                Image im = PictureBox1.Image;
                var obj = new Random();
                SaveImage(im, filepath + "\\" + obj.Next() + ".png");
            }

Posted 17-Apr-17 5:55am

Member 12277263

Updated 17-Apr-17 5:59am

CHill60

v2

Add a Solution

Comments

CHill60 17-Apr-17 11:58am

If you can copy an image from pictureBox1 why can't you do it from one of the other pictureBoxes?

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 · Answer 1 · 2017-04-17T05:59:00

Solution 1

The sender parameter of the event handler is there for just that reason:

private void form_MouseClick(object sender, EventArgs e)
    {
    PictureBox pb = sender as PictureBox;
    if (pb != null)
        {
        folderBrowser.ShowDialog();
        string filepath = folderBrowser.SelectedPath;
        Image im = pb.Image;
        var obj = new Random();
        SaveImage(im, filepath + "\\" + obj.Next() + ".png");
        }
    }

Posted 17-Apr-17 5:59am

OriginalGriff

Comments

Member 12277263 17-Apr-17 12:13pm

Thanks, I tried that way, I am getting the pb null all the time

OriginalGriff 17-Apr-17 12:25pm

Then you aren't clicking on the PictureBox!
Use the debugger to find out what type sender is ... that should help you work out what is going on.

Member 12277263 17-Apr-17 14:59pm

thanks
Its working

Member 12277263 17-Apr-17 15:01pm

Here I am saving the Image name with random number, is there any alternatives

OriginalGriff 17-Apr-17 15:33pm

Loads! What are you trying to do?