How do I apply the correct format when trying to assign a pictureBox an image?

Question

1.00/5 (1 vote)

See more:

Hello community,

I am using a variable called "progressCounter" in a game of hangman to run through a string[] to fetch words for the user to spell. progressCounter will start at 0, and increase++ until 4 (which will load the 5th and final word). I wish to apply a picture to a pictureBox representing the current score (in the form of apples, where each image will have 1 apple more than the last). progressCounter will provide an output at 5 to inform the user that they have passed all 5 spellings, and will then close the program to avoid crashing. progressCounter state 0 will not have an image.

My question is "what is the correct method of writing a filepath reference for the pictureBox to use the images on each of the following IF statements?" I have provided the function below, with an example of what I tried to construct:

C#

public void imageLoader()
       {
           if (progressCounter == 1)
           {
               pbxclue.Image = Image.FromFile(c:\\college/Y2/Visual_Programming/VS01/VP01/VP01/resources/pictures/1.jpg);
           }
           if (progressCounter == 2)
           {

           }
           if (progressCounter == 3)
           {

           }
           if (progressCounter == 4)
           {

           }
           if (progressCounter == 5)
           {

           }
       }

Any help is much appreciated, and as I am somewhat a novice at C#, please be kind.

Posted 17-Nov-15 7:35am

F_8

Updated 17-Nov-15 7:37am

v2

Add a Solution

Comments

Richard MacCutchan 17-Nov-15 13:49pm

Why do you need the if statements? Just use a StringBuilder to create the path using the counter value as the filename.

Sergey Alexandrovich Kryukov 17-Nov-15 14:55pm

And then the code using StringBuilder will use "if" internally, right? :-)
Nobody needs such paths and hard-coded 1, 2, 3 "options". Please see Solution 1.
—SA

2 solutions

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)

BillWoodruff · Answer 1 · 2015-11-17T13:33:00

Assuming this is a Windows Forms project:

1. use an ImageList Component

a. Drag an 'ImageList Component onto your Form from the 'Components hive of the ToolBox

b. at design-time add your images to the ImageList's 'Images collection; or, add them at run-time as needed.

2. you can then set the current image using the 'progressCounter directly ... since the progressCounter is #0 based:

pbxclue.Image = imageList1[progresscounter];

Sergey Alexandrovich Kryukov · Answer 2 · 2015-11-17T07:48:00

None. There are no situations when hard-coded path names can be useful. In all production-level applications, path names are calculated during runtime based on environment, content of some configuration file, user input, and the like. If you hard-code file paths, nothing can guarantee the applications to work on another machine, or simply next time.

If your images are constant, they belong to resources. Some recommendations: create *.resx resource and add images via "Add existing file". Better don't edit image files in Visual Studio, provide them as source files in your project. If you add the file from some outside directory, remember that it will be copied to the some project sub-directory, so make sure you are not creating duplicates. The file is added to the project and referenced in the project file and resource file as well. Then the auto-generated file will be created, it has all the declarations (static class with static properties), so everything is ready to use. Don't read from resource, just use the declarations in that auto-generated file.

Let me note that your code design is unacceptably bad (bad for maintenance). Again, the problem is hard-coded 1, 2, 3… and the sequence of "if". For working maintainable code, you would need to develop something much more regular, but it depends on what you want to achieve.

—SA

How do I apply the correct format when trying to assign a pictureBox an image?

2 solutions

Solution 2

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us