Overiding ToString() Method

[truezplaya]

I have my collectionOfCars and my car class.

In my car class i have written

public override string ToString()
{
  return String.Format(_carColour + " " + _carMake);
}

I am trying to write these cars to a listView

private void FillCarLists(ListView lvi, CollectionOfCars Cars)
{
  foreach (Car myCar in CollectionOfCars )
  {
    ListViewItem test= new ListViewItem(myCar.ToString());
    lvi.Items.Add(test);
  }
 
}

this returns me
"carApp.Car"

This isn't the desired outcome i gather that the ToString() override isn't working and i have no idea why. AnyOne have any ideas?

i'm sure it's simple but it is just bugging me now

thanks in advanced

Cheers

Truez

[tlhintoq]

TIP: When you are writing your question, there is a button on the tool bar that wraps the [code] tags around your copy/pasted code. It helps a bunch. Its the button with a '#' on it. More on tags. They're cool. Check'em out.

[tlhintoq]

Just for reference... I do something similar in my plugins:

        public override string ToString()
        {
            return string.Format("{0}: {1}", GetTypecharFromEnum(myType), myPath);
        }
 

My only comment is how you are formatting.

return String.Format(_carColour + " " + _carMake);

Really doesn't make use of the String.Format in any way.

You could just as easily

return _carColour + " " + _carMake;

or if you want to use String.Format then:

return String.Format( "{0} {1}", _carColour, _carMake);

I would also suggest that you put a breakpoint on your return line. If you code doesn't break, then you know you are not executing this line.

[truezplaya]

I found the issue, Since i am new to c#.net the namespaces have got jumbled up which was hiding the ToString override

Cheers for the replys

Truez