foreach with two lists

silasla · May 2nd, 2008, 10:19 AM

the checked list contains strings which can be equaly to the "flaske" propertyId.

Therefore the syste can run throw the list "Flakseliste" and add a "rerservaring" to all "flaske" which have the same ID as the checked items from the checkedListBox.

//makes a reservation
Reservering ny = new Reservering((label12.Text), (dt));

//The reservations are added to the "Flaske"
en_flaske.Add(ny) ;

samjudson · May 2nd, 2008, 10:28 AM

Look, this really isn't that hard - if the user can select more than one Flaske then you need to check the List of Fleske's to see if it contains the current Fleske and then add the reservation:

Code:

private void button2_Click(object sender, EventArgs e)
{
  //makes temporary list
  List<string> temp_flasker = new List<string>();
  foreach (object i in checkedListBox12.CheckedItems) temp_flasker.Add(i.ToString());

  Reservering ny = new Reservering(label12.Text, dt);

  foreach (Flaske en_flaske in Flaskeliste)
  {
    if (en_flaske._flaskeledig(dt) && temp_flasker.Contains(en_flaske.PropertyFlaskeID));
    {
      en_flaske.Add(ny) ;
    }
  }
}

/- Sam Judson : Wrox Technical Editor -/

samjudson · May 2nd, 2008, 10:30 AM

Also, if the flaskeledig() function is a time consuming one then I'd swap the two parts of the 'if' around:

Code:

if (temp_flasker.Contains(en_flaske.PropertyFlaskeID) && en_flaske._flaskeledig(dt))
{
   ...

Actually, come to think of its I've just spotted your problem. You've got a ';' at the end of the line including the if. Get rid of that.

/- Sam Judson : Wrox Technical Editor -/

silasla · May 2nd, 2008, 12:27 PM

okay, thanks a lot for your help..