Loop igennem checklistbox ?

Du kan gøre således:


foreach( ListItem item in CheckBoxListRoles.Items )
{
    if( item.Selected )
    {
        //Din kode
    }
}

Ok så tager jeg lige det hele med i det du foreslår :

string ErrorHeader = Resources.Default.ErrorHeadline;
string ErrorMessage = null;
foreach (ListItem item in CheckBoxListRoles.Items)
{
    if (!item.Selected)
    {
        ErrorMessage = "<b>" + ErrorHeader + "</b><br />" + Resources.Membership.ErrorRoles;
    }
}

if (ErrorMessage != null)
{
    LabelError.Text = ErrorMessage;
    LabelError.Visible = true;
}

Som du kan se skal jeg blot smide en fejl besked hvis der ikke er valgt nogen, men eftersom den looper igennem listen vil den jo skrive fejlen ud hvis bare en i listen ikke er valgt ?

/Websam

Så lav den således:

bool anyChecked = false;
foreach (ListItem item in CheckBoxListRoles.Items)
{
    if (item.Selected)
    {
        anyChecked = true;
        break;
    }
}

if( !anyChecked )
{
    LabelError.Text = String.Format( "<b>{0}</b><br />{1}", ErrorHeader, Resources.Membership.ErrorRoles );
    LabelError.Visible = true;
}

eller

private bool IsCheckboxListSelected(CheckBoxList cb)
{
  foreach(ListItem item in cb.Items)
  {
    if (item.Selected)
      return true;
  }           
  return false;
}

Funktion som returnerer true, hvis der er valgt en eller flere og false hvis der ikke er valgt nogle.

if (!this.IsCheckboxListSelected(this.CheckBoxListRoles))
{
  LabelError.Text = ErrorMessage = "<b>" + ErrorHeader + "</b><br />" + Resources.Membership.ErrorRoles;
  LabelError.Visible = true;
}

dj_uncas >> Så er der kun lige en ting tilbage, hvis jeg lader være at vælge nogle i min list kommer den med fejl, hvilket den jo også skal.

Vælger jeg så en i min liste forsvinder fejlen ikke igen hvad kan jeg lige gøre der ?

/Websam

Never mind jeg satte LabelError.Visible = false; og det virker super smid et svar ;o)

Den skulle også meget gerne holde på teksten. Det er ViewState der kommer ind i billedet.

Der er flere måder at ordne det på. At sætte Visible til false er et meget godt bud, at sætte Text til String.Empty også god. Måske er Visible = false bedst da du så også fjerne det ligegyldige HTML <span></span>...

Ja men jeg har ikke viewstate på nogle af mine labels så visible = false blev løsningen ;o)

Har du explicit sat EnableViewState til false på dine Labels? Ellers er der default
ViewState...

Jeg sætter viewstate="false" på alle de controls hvor jeg ikke skal bruge det da det ellers kan blive et problem set med SEO øjne.

/Websam