au weia. manchmal tut sowas weh.

Was sagst du? Bahnhof?

Na ja ist ja möglich, dass sich mal was ändert, dann bist du schneller beim Anpassen

Ich denke dieser Code ist historisch gewachsen - grins

Das kommt hier drauf, was genau Du damit bezwecken wolltest. Ausserdem sieht man den zusammenhang nicht. Da ist ja bestimmt noch merh Code drumrum, denke ich aml. Wie würdest Du denn es jetzt machen?

Schaut euch den Code doch mal an, in dem fall hätte es die Zeile auch getan!

würdest du
iPSegmentIndex = index;
machen, dann könnte iPSegmentIndex auch Werte ausserhalb des Bereiches [0,3] annehmen. Vielleicht ist das aber gerade nicht gewollt.
Natürlich könnte man das durch if(index >= 0 && index <=4) eleganter lösen, dass geht aber nur solange es sich um einen nicht gesplitteten Bereich handelt.

Will jetzt den Coder nicht unbedingt verteidigen, weil wirklich schön ist der Code echt nicht, aber gleich abzugehen ohne die Umstände zu kennen halte ich für etwas unangebracht.

ultrapaine:
Das kannst du so ohne Zusammenhang nicht sagen,

ich würde
if (index >= 0 && index <= 3) {
IPSegmentIndex = index;
}
schreiben..

ich fänds mal interessant wie viele verstehen über was ihr diskutiert...

Im übrigen finde ich eine Switch-Case garnet sooo schlimm, selbst wenn es so "unnötig" wie hier ist.
Es ist (finde ich) einfach übersichtlicher, als sowas:

Wie es mit der Performance aussieht weiß ich allerdings nicht...

Der Code ist aber ausgesprochen blödsinnig. Eine "inrange" Prüfung kann man auch gescheiter machen als alle Werte als case-Fall zu schreiben.

Es ist tatsächlich historisch gewachsen. Die switch-case Abfrage sah vorher auch anders aus. index wird nur einen Wert von 0-3 haben und später einfach verwendet. iPSegmentIndex ist einfach unnötig.

Es sollte ursprünglich ja aber was passieren, wenn es eben 2 ist und zwar genau zwei und nicht zwischen 0 und 3.

Da fragt man sich eher wieso du überhaupt in index speicherst und nicht gleich in iPSegmentIndex...

Och, da sehen manche gewachsenen Routinen von mir noch viel, viel wilder aus.
Insofern: gräme dich nicht, es könnte noch viel schlimmer sein. Mir fällt da grad ne sehr wirre Variante mit ineinander verschachtelten Schleifen ein...

Ich sehe hier kein Problem. Eventuell sind Deine Gedankengänge damals nicht davon ausgegangen dass der IPSegemtIndex in jedem Fall dem Index entspricht. Und wie bereits geschrieben werden so nur valide IPSegmentIndex-Werte zugelassen.
Zum Problem wird es nur wenn dieser Aufruf messbar ist bzw. spürbar mehr Zeit benötigt als die einfache Zuweisung und eine eventuelle Werte-Überprüfung vorab. Das dürfte aber nur der Fall sein wenn er mehrere hundert male pro Sekunde ausgeführt wird. Und auch da muss man individuell testen. Das ist nicht in jeder Sprache gleich.