Using a variable for a class instance name C#

Question

I've found a few threads on this for other languages but nothing for C#...

Basically, what I need is to instantiate a class with a different name each time (in order to be able to create links between the classes)

My current code:

foreach (DataRow dataRow in RoutingObjects.Rows)
{
    NodeViewModel node = CreateNode(dataRow.ItemArray[1].ToString(), new Point(globalx, globaly), false, dataRow.ItemArray[0].ToString());
}

This creates a new NodeViewModel for every row in the table. What I really need is for each NodeViewModel to have a different name, but I have no idea how to do this.

I tried using a simple int in the loop but I can't add a variable into the class instance name. The (obviously wrong) code is here:

int i = 0;
foreach (DataRow dataRow in RoutingObjects.Rows)
{
    NodeViewModel node+i = CreateNode(dataRow.ItemArray[1].ToString(), new Point(globalx, globaly), false, dataRow.ItemArray[0].ToString());
    i++;
}

I figured by this point that I should be doing things differently and this was probably a terrible idea, but I'd like some advice! If it is possible I'd like to know - otherwise, some alternatives would be very helpful =)

Thanks!

Tim Schmelter · Accepted Answer · 2016-04-19 12:48:20Z

7

I think you're looking for a collection. You can use a List<NodeViewModel>.

var nodelist = new List<NodeViewModel>();
foreach (DataRow dataRow in RoutingObjects.Rows)
{
    NodeViewModel node = CreateNode(dataRow.ItemArray[1].ToString(), new Point(globalx, globaly), false, dataRow.ItemArray[0].ToString());
    nodelist.Add(node);
}

Now you can access every node in the list via index, for example:

NodeViewModel nvm = nodelist[0];

answered Apr 19, 2016 at 12:48

Tim Schmelter

462k79 gold badges719 silver badges980 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

omicronlyrae Over a year ago

Thank you, that is exactly what I needed!

Timur Lemeshko · Accepted Answer · 2016-04-19 13:30:49Z

0

Tim Schmelter right. And, of course, wou can use Linq for more elegant code:

var nodelist = RoutingObjects.Rows.Select(dataRow=>CreateNode(dataRow.ItemArray[1].ToString(), new Point(globalx, globaly), false, dataRow.ItemArray[0].ToString())

answered Apr 19, 2016 at 13:30

Timur Lemeshko

2,8795 gold badges29 silver badges43 bronze badges

1 Comment

Tim Schmelter Over a year ago

Instead of RoutingObjects.Rows.Select you have to use RoutingObjects.AsEnumerable().Select because a DataRowCollection doesn't implement IEnumerable<DataRow> but only IEnumerable.

Collectives™ on Stack Overflow

Using a variable for a class instance name C#

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related