Using classes and constructors

Question

3.00/5 (3 votes)

See more:

I have a class named OCRChar

C#

class OCRChar
    {
        public string fileName;
        public string binVal;       

        public OCRChar(string iFileName, string iBinVal)
        {
            this.fileName = iFileName;
            this.binVal = iBinVal;          
        }
    }

My Main Program consists of a button that loads some .png files, converts them to binary and I need to store them in memory so that later I can train a neural network with the binary values.

C#

private void btnLoadTrainingData_Click(object sender, EventArgs e)
        {
            string[] filenames = Directory.GetFiles(@"C:\temp\test", "*.png", SearchOption.AllDirectories);

            foreach (string f in filenames)
            {
                Console.WriteLine(getFirstChar(Path.GetFileName(f)));
                img = (Bitmap)Bitmap.FromFile(f);                
                c = new OCRChar(f,convertToBinary(img));               
            }
            //img.Dispose();
        }

How can I list all the items I have stored? maybe using a foreach loop? this is not working

C#

foreach (OCRChar c )
            {
            Console.WriteLine(c.binVal);
            }

Posted 17-Nov-12 2:27am

datt265

Updated 17-Nov-12 2:36am

v2

Add a Solution

Comments

Toli Cuturicu 17-Nov-12 20:34pm

Very poor syntax. This will never compile!

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 · Accepted Answer · 2012-11-17T02:49:00

I think you need to go back a step, and review what a list and so forth is.

"c" is not a list. I can tell this, because you are assigning a single instance of an object to it:

C#

c = new OCRChar(f,convertToBinary(img));

Therefore, it can't be used as an enumerable value in a foreach loop:

C#

foreach (OCRChar c )
            {
            Console.WriteLine(c.binVal);
            }

(Ignoring that the syntax for foreach doesn't look like that).

Step back, and look at what you are trying to do.
Add an emumerable object to your main form, so hold all the OCRChar instances - I would suggest that a List of OCRChar objects would be a good choice:

C#

private List<OCRChar> ocrFiles = new List<OCRChar>();

Now, when you create a new instance, add it to the list:

C#

foreach (string f in filenames)
{
    Console.WriteLine(getFirstChar(Path.GetFileName(f)));
    img = (Bitmap)Bitmap.FromFile(f);
    ocrFiles.Add(new OCRChar(f,convertToBinary(img)));
}

Then you can iterate through them later:

C#

foreach (OCRChar c in ocrFiles)
{
    Console.WriteLine(c.binVal);
}

At the moment, it looks like you are trying to code by guessing and hoping it works - that isn't a viable strategy! I strongly recommend that you go back a stage or two and look at the basics - get a good solid grasp of them first before you go much further. You are venturing into harder and harder territory and I can't see that you are going to do more than confuse and frustrate yourself unless you understand the simple stuff a lot better than you appear to now.