Re: 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

File Ops: Windows Service vs. EXE

10-Jul-05 14:28

You're welcome! great it worked!

-- LuisR

Luis Alonso Ramos
Intelectix - Chihuahua, Mexico

Not much here: My CP Blog!

ahfong10-Jul-05 13:24

ahfong

10-Jul-05 13:24

I have to fix a weird problem where a file needs to be moved using File.Move from a local directory to a network drive. The code that I'm trying to fix tries to pass either a network path or a mapped network drive path as a parameter to the Directory.Exists method but it fails. This code runs as a Windows service under XP.

I wrote my own exe to do the same thing but my code works. Why would an EXE work and not the service? Permissions to both drives are for Everyone.

Help. Frown | :(

Dave Kreskowiak10-Jul-05 14:49

Dave Kreskowiak

10-Jul-05 14:49

Because the service in running under the Local System account (by default) which doesn't have rights to anything on the network. That's why the code works when YOU run it. It's running under your account.

You have to go into the Service Manager and change the account the service in running under to one with rights to the network.

RageInTheMachine9532
"...a pungent, ghastly, stinky piece of cheese!" -- The Roaming Gnome

ahfong10-Jul-05 15:46

ahfong

10-Jul-05 15:46

Hmmm... I already changed the service to run under my log on from the services' Properties Log On dialog and also changed the ownership of the mapped network drive to my name as well.

Is there another place where I need to change the log on?

Vasudevan Deepak Kumar10-Jul-05 19:50

Vasudevan Deepak Kumar

10-Jul-05 19:50

Check on the two issues:

(*) The default path that gets formed for destination/source files. I think it would form as %system%, unless you prefix and qualify with AppDomain.CurrentDomain.BaseDirectory to get the proper path.

(*) RunAs of the EXE to run with an account with network access to the target resource.

Deepak Kumar Vasudevan
Personal Web: http://vdeepakkumar.netfirms.com/
I Blog At: http://deepak.blogdrive.com/