Mass Coding - C# Discussion Boards

10-Jul-05 13:45

I have a ton of picture boxes on a form, I made it so that if you click one it opens an open file dialog box and then opens the picture into the picture box, all seems fine. How do I code so that I can click on any picture box, and call a function that does that without making one for each picture box?

Guffa10-Jul-05 13:53

Guffa

10-Jul-05 13:53

Tie all the click events to the same event handler.

---
b { font-weight: normal; }

Expert Coming10-Jul-05 13:58

10-Jul-05 13:58

That still would be horrilbe since I would have to make all picture boxes call that event. How do I make it not have to do that?

Guffa11-Jul-05 0:44

Guffa

11-Jul-05 0:44

No, it's not horrible. I saw that you solved it that way after all. Smile | :)

---
b { font-weight: normal; }

Different Guffa

Expert Coming11-Jul-05 7:39

11-Jul-05 7:39

I thought that I would have to make a differnet sub for each picture box, not that I could just have all click statements call into a sub.

Luis Alonso Ramos10-Jul-05 13:57

10-Jul-05 13:57

Yes, as it was suggested, set the Click event of every picture box to the same handler. Once in the handler, popup the File Open dialog, and cast the sender (first parameter) to a picture box and set the image.

-- LuisR

Luis Alonso Ramos
Intelectix - Chihuahua, Mexico

Not much here: My CP Blog!

Expert Coming10-Jul-05 13:59

10-Jul-05 13:59

What do you mean sender?

DavidNohejl10-Jul-05 14:06

DavidNohejl

10-Jul-05 14:06

Event handlers generaly have this signature control_event(object sender, EventArgs args). You box sender to type of your control to use, e.g. object stg = (ComboBox)sender.SelectedItem;. If you subscribe more controls events to this handler, this way you will work with *that* control which raised the event.

David

Expert Coming10-Jul-05 14:13

10-Jul-05 14:13

So where do I put that code.

private void map11_Click(object sender, System.EventArgs e)
{
object snd = (ComboBox)cmbLevel.SelectedItem;
ChangePicture(snd);
}

private void cmbLevel_SelectedIndexChanged(object sender, System.EventArgs e)
{
szLevel = cmbLevel.Text;
}

private void ChangePicture(object sender, System.EventArgs e, object snd)
{
// Display an OpenFileDialog so the user can select a Cursor.
openMap.Filter = "GIF Files|*.gif";
openMap.Title = "Select a GIF File";
openMap.Multiselect = true;

// Show the Dialog.
// If the user clicked OK in the dialog and
// a .GIF file was selected, open it.
if (openMap.ShowDialog() == DialogResult.OK)
{
if(openMap.FileName != "")
{
// Assign the images to the picture box according to the selected level.
if (szLevel != "Bottom")
{
"""PICTURE BOX""".Image = Image.FromStream(openMap.OpenFile());
}
if (szLevel != "Top")
{
"""PICTURE BOX""".BackgroundImage = Image.FromStream(openMap.OpenFile());
}
}
}

Luis Alonso Ramos10-Jul-05 14:16

10-Jul-05 14:16

ExpertComing wrote:
// Assign the images to the picture box according to the selected level.
if (szLevel != "Bottom")
{
"""PICTURE BOX""".Image = Image.FromStream(openMap.OpenFile());
}
if (szLevel != "Top")
{
"""PICTURE BOX""".BackgroundImage = Image.FromStream(openMap.OpenFile());
}

Here, you would replace "PICTURE BOX".Image with

((PictureBox) sender).Image</code).<br />
<br />
[EDIT: And in your <code>map11_Click

event handler you would call ChangePicture(sender, e) and remove the third parameter you added.]

I hope this helps!

-- LuisR

Luis Alonso Ramos
Intelectix - Chihuahua, Mexico

Not much here: My CP Blog!

Luis Alonso Ramos10-Jul-05 14:25

10-Jul-05 14:25

ExpertComing wrote:
private void map11_Click(object sender, System.EventArgs e)
{
object snd = (ComboBox)cmbLevel.SelectedItem;
ChangePicture(snd);
}

Change the ChangePicture call to ChangePicture(sender, e);. BTW, what is in the combo box? You would bind every picture box to this handler, to this particular routine is called when the user clicks on *any* picture box.

ExpertComing wrote:
private void ChangePicture(object sender, System.EventArgs e, object snd)
{
// Display an OpenFileDialog so the user can select a Cursor.

Remove the third parameter in the definition so you have:

private void ChangePicture(object sender, System.EventArgs e)

and see my other replies.

-- LuisR

Luis Alonso Ramos
Intelectix - Chihuahua, Mexico

Not much here: My CP Blog!

Luis Alonso Ramos10-Jul-05 14:13

10-Jul-05 14:13

The envent handler has two parameters: Object sender and EventArgs e. The first represents the object that raised the event. In this case, if you have several objects attached to the same event handler, you should use this parameter to distinguish.

Say you have 3 pictures boxes: pictureBox1, pictureBox2 and pictureBox3. You set the event handlers like this:

pictureBox1.Click += new EventHandler(pictureBox_Click);
pictureBox2.Click += new EventHandler(pictureBox_Click);
pictureBox3.Click += new EventHandler(pictureBox_Click);

Then you only have one event handler:

private void pictureBox_Click(object sender, EventArgs e)
{
    // Display file dialog, get the selected file name, and load the image into
    // the image variable.
    OpenFileDialog openDialog = new OpenFileDialog()
    ....
 
    Image image = Image.FromFile(openDialog.FileName);
 
    // Set the image of the picture box that was clicked
    ((PictureBox) sender).Image = image;
}

I hope this explains it better.

-- LuisR

Luis Alonso Ramos
Intelectix - Chihuahua, Mexico

Not much here: My CP Blog!

Expert Coming10-Jul-05 14:21

10-Jul-05 14:21

I got the coding part done before you replied again lol.

But now I get this error.

No overload for method 'ChangePicture' takes '0' arguments

DONE

Expert Coming10-Jul-05 14:26

10-Jul-05 14:26

GOT IT SWEET!

Thanks for the help.

Re: DONE

Luis Alonso Ramos10-Jul-05 14:28