C# Index was outside the bounds of the array.

[Kevin (Gast)]

Hallo Gemeinde,

könnt ihr mir kurz auf die Sprünge helfen, warum die Schleife quasi 
endlos läuft, bis sie in den Fehler rennt?

Ich überprüfe doch nach jedem Durchgang, ob das Array Null ist...

int indexx = 0;

do
{
tmpOutPut.WarrantyEndDate.Add(idata.Entitlements[indexx].EndDate);
locSupportDescription += @"[" + idata.Entitlements[indexx].ServiceLevelDescription + @"]";
indexx++;
} while ((idata.Entitlements[indexx].ServiceLevelDescription != null && idata.Entitlements[indexx].EndDate != null) && (idata.Entitlements[indexx].ServiceLevelDescription.Length != 0));

Gruß,
Kevin

[Peter II (Gast)]

Kevin schrieb:
> Ich überprüfe doch nach jedem Durchgang, ob das Array Null ist...

du bekommt aber "Index was outside the bounds of the Array" also ist 
wohl indexx zu gross für das Array.

Da hilft es auch nicht zu prüfen, was aus dem Array rauskommt. Du musst 
vorher prüfen ob indexx gültig ist.

[Markus (Gast)]

Kevin schrieb:
> Ich überprüfe doch nach jedem Durchgang, ob das Array Null ist...

Offensichtlich enthalten alle Array-Elemente immer Daten, die Deine 
While-Bedingung erfüllen.

Grüße
Markus

[Markus (Gast)]

Deine Schleife wird übrigens MINDESTEND EINMAL durchlaufen, auch wenn 
Dein Array überhaupt keine Daten enthält, die Deiner while-Bedingung 
entsprechen oder wenn das Array 0 Elemente haben sollte.

Grüße
Markus

[Peter II (Gast)]

warum keine foreach schleife?

foreach( ServiceLevelDescription sld in idata.Entitlements ) {
   tmpOutPut.WarrantyEndDate.Add(sld.EndDate);
   locSupportDescription += @"[" + sld.ServiceLevelDescription + @"]";

   if ( sld == null || sld.EndDate == null || sld.Length == 0 ) {
      break;
   }
}

[Markus (Gast)]

Peter II schrieb:
> warum keine foreach schleife?
> foreach( ServiceLevelDescription sld in idata.Entitlements ) {
>    tmpOutPut.WarrantyEndDate.Add(sld.EndDate);
>    locSupportDescription += @"[" + sld.ServiceLevelDescription + @"]";
>
>    if ( sld == null || sld.EndDate == null || sld.Length == 0 ) {
>       break;
>    }
> }
Etwas besser wäre

foreach( var entitlement in idata.Entitlements )
{
    tmpOutPut.WarrantyEndDate.Add(entitlement.EndDate);
    locSupportDescription += string.Format("[{0}]", entitlement.ServiceLevelDescription);
 
    if ( entitlement.ServiceLevelDescription == null || entitlement.EndDate == null || entitlement.Length == 0 )
    {
       break;
    }
}

;-)

Grüße
Markus

[Halbbastionenerstürmer (Gast)]

Alternativ vielleicht in dieser Art; dann steht die Bedingung gleich in 
der ersten Zeile.

public void Bla(/* ... */ string formatString = null)
{
  // ...

  idata.Entitlements.TakeWhile(e => e.ServiceLevelDescription != null && e.EndDate != null && e.Length != 0)
  .ForEach(entitlement =>
    {
      tmpOutPut.WarrantyEndDate.Add(entitlement.EndDate);
      locSupportDescription += String.Format(formatString ?? "[{0}]", entitlement.ServiceLevelDescription);
    });
  // ...
}

static class MyExtensions
{
  public static void ForEach<T>(this IEnumerable<T> enumeration, Action<T> action)
  {
    foreach (T item in enumeration) action(item);
  }
}

BTW: Ich verwende ja selbst ständig CamelCase, aber OutPut? ;D

[Kevin (Gast)]

Markus schrieb:
> Peter II schrieb:
>> warum keine foreach schleife?
>> foreach( ServiceLevelDescription sld in idata.Entitlements ) {
>>    tmpOutPut.WarrantyEndDate.Add(sld.EndDate);
>>    locSupportDescription += @"[" + sld.ServiceLevelDescription + @"]";
>>
>>    if ( sld == null || sld.EndDate == null || sld.Length == 0 ) {
>>       break;
>>    }
>> }
> Etwas besser wäre
> foreach( var entitlement in idata.Entitlements )
> {
>     tmpOutPut.WarrantyEndDate.Add(entitlement.EndDate);
>     locSupportDescription += string.Format("[{0}]",
> entitlement.ServiceLevelDescription);
>
>     if ( entitlement.ServiceLevelDescription == null ||
> entitlement.EndDate == null || entitlement.Length == 0 )
>     {
>        break;
>     }
> }
> ;-)
>
> Grüße
> Markus

Ich danke euch, ich werde mich wohl für diese Variante entscheiden.

Markus schrieb:
> Deine Schleife wird übrigens MINDESTEND EINMAL durchlaufen, auch
> wenn
> Dein Array überhaupt keine Daten enthält, die Deiner while-Bedingung
> entsprechen oder wenn das Array 0 Elemente haben sollte.
>
> Grüße
> Markus

Ja, das ist aber kein Problem, da das Array mit dem Index 0 auf jeden 
Fall Daten enthält...

Markus schrieb:
> Kevin schrieb:
>> Ich überprüfe doch nach jedem Durchgang, ob das Array Null ist...
>
> Offensichtlich enthalten alle Array-Elemente immer Daten, die Deine
> While-Bedingung erfüllen.
>
> Grüße
> Markus

Eben das ist eigentlich nicht der Fall und trotzdem stoppt die Schleife 
nur, wenn sie in den Fehler läuft.

[Markus (Gast)]

Kevin schrieb:
> Eben das ist eigentlich nicht der Fall und trotzdem stoppt die Schleife
> nur, wenn sie in den Fehler läuft.

Dann stimmt Deine Loop-Bedingung nicht. Vielleicht solltest Du mal 
Debug-Ausgaben (z.B. mit System.Diagnostics.Debug.Write(), ...) oder 
auch direkt mit dem Debugger prüfen, was da los ist.

Grüße
Markus