CODEWITHSUNDEEP
c#asp.net
user979331
user979331

Reputation: 11961

ASP.NET adding data to class in loop

I have this code here:

for (int i = 0; i < reader.FieldCount; i++)
{
    RedBlue item = new RedBlue();

    if (reader.GetName(i).ToString().Contains("BID"))
    {
        item.baselinefinish = reader.GetValue(i).ToString();
    }
    if (reader.GetName(i).ToString().Contains("AID"))
    {
        item.actualenddate = reader.GetValue(i).ToString();
    }

    redBlue.Add(item);
}

What I am trying to do loop through data and add it to a class, but my problem is in my class I have two strings and I want to populate each string like this (first string gets the first item in the loop, the second string get the second item in the loop, and keep going like that, so instead of each one in the loop, for every two items in the loop add them to the string and continue on....I really hope this makes sense. Anyone know how I would accomplish this?

Currently what is happening, is it will add one of the strings to the class and then add the second string to a new class.

Upvotes: 1

Views: 669

Answers (3)

Fsittius
Fsittius

Reputation: 160

Not sure if I follow your idea, but you can use even or odd index number to represent the first and second item on the loop.

Something like that:

for (int i = 0; i < reader.FieldCount; i++)
{
    RedBlue item = null;
    //if it's even index number
    if(i % 2 == 0){
      item = new RedBlue();
      redBlue.Add(item);

      if (reader.GetName(i).ToString().Contains("BID"))
         item.baselinefinish = reader.GetValue(i).ToString();

    }else{

      if (reader.GetName(i).ToString().Contains("AID"))
        item.actualenddate = reader.GetValue(i).ToString();
    }    
}

Upvotes: 0

Niko
Niko

Reputation: 444

Not sure I follow, but right now you are creating a new object at every iteration and therefore each object will have only either string.

If your data is as you say, containing BID every second and AID every second element, as a poor mans solution, you could simply increment the i after adding the first string.

for (int i = 0; i < reader.FieldCount; i++)
{
    RedBlue item = new RedBlue();

    if (reader.GetName(i).ToString().Contains("BID"))
    {
        item.baselinefinish = reader.GetValue(i).ToString();
        i++;
    }
    if (reader.GetName(i).ToString().Contains("AID"))
    {
        item.actualenddate = reader.GetValue(i).ToString();
    }

    redBlue.Add(item);
}

Or maybe I'm missing something?

Upvotes: 0

Pac0
Pac0

Reputation: 23174

You can use Automapper and do something like this :

(adapted from what I remember of this framework, the docs here and your example)

// Configure AutoMapper
Mapper.Initialize(cfg =>
  cfg.CreateMap<YourReaderClass, RedBlue>()
    .ForMember(dest => dest.baselinefinish , opt => opt.MapFrom(src => src.BID))
    .ForMember(dest => dest.actualenddate , opt => opt.MapFrom(src => src.AID))

// Perform mapping
RedBlue item = Mapper.Map<YourReaderClass, RedBlue>(reader);

You do the configuration once somewhere and then you can perform as many mapping you want. Of course, you have to manually indicate which field is mapped to which field with as many ForMember as you need.

EDIT

Actually, you could of course still do it without 3rd party, as you were thinking. To solve the specific problem with your method :

Currently what is happening, is it will add one of the strings to the class and then add the second string to a new class.

(by the way, you mean instance of your class (object), not class )

Of course this happening, because you are creating new objects each time you iterate your loop !

If you do it like this, it should work :

// instantiate your object once, before the loop :
RedBlue item = new RedBlue();

for (int i = 0; i < reader.FieldCount; i++)
{
    if (reader.GetName(i).ToString().Contains("BID"))
    {
        item.baselinefinish = reader.GetValue(i).ToString();
    }
    if (reader.GetName(i).ToString().Contains("AID"))
    {
        item.actualenddate = reader.GetValue(i).ToString();
    }
}

// now you have one object named 'item' which should be what you want.

Upvotes: 1

Related Questions