TStrings mischen nach Fisher-Yates

[Luckie]

Zitat von shmia:

Sollte der Programmierer fälschlicherweise ein "Nil-Objekt" übergeben, dann ist das Programmierfehler anzusehen; also ein Zustand der "eigentlich" nie auftreten dürfte.
Hier eine Exception zu werfen wäre nicht richtig, da es sich nicht um einen erwarteten Fehler handelt.

Doch genau, das wäre ein Fall für eine Exception, denke ich. Denn es war nicht die Absicht des Programmierer ein NIL.Objekt zu übergeben.

Zitat:

2.) Assert verwenden - man bekommt Dateiname+Zeilennummer; das hilft ungemein bei der Fehlersuche

Das ist schon richtig, hat aber in produktiven Code nichts verloren. Deswegen gibt es ja auch die Option die Assertions per Compilerschalkter abzuschalten. Was machst du dann?

Zitat:

3.) if not Assigned(s) then Exit; Keine schlechte Option; so kann die Procedure keinen Schaden anrichten

Und man sucht sich dumm und dämloich, warum nichts passiert.

Zitat:

4.) Kombination aus 2.) und 3.)

Siehe oben. Teste auf nil und wirf eine Exception.