Unter welchen Umständen kann diese Funktion eine AV auslösen

[Pseudemys Nelsoni]

Siehe Topic.

Hier die Funktion:

Delphi-Quellcode:

function TUsers.UserByNickname(const Nickname: string): TUser;
var
  I: Integer;
begin
  Result := nil;
  for I := FUserList.Count-1 downto 0 do
    if CompareText(TUser(FUserList.Items[I]).FNickname, Nickname) = 0 then
      Result := TUser(FUserList.Items[I]);
end;

Ich bekomme hier manchmal eine AV, weiss nur nicht woran das liegt. In FUserList sind, wenn vorhanden, NUR Objekte der Klasse TUser enthalten. Theoretisch dürfte hier doch kein Fehler auftreten oder?

[edit=Luckie]Zitat-Tags durch Delphi-Tags ersetzt. Mfg, Luckie[/edit]

Edit: Danke, wollts gerade auch machen

[Luckie]

ich sehe da nichts. Aber warum machst du weiter, wenn du den User mit den passenden Nicknamen gefunden hast?

Delphi-Quellcode:

function TUsers.UserByNickname(const Nickname: string): TUser; 
var 
  I: Integer; 
begin 
  Result := nil; 
  for I := FUserList.Count-1 downto 0 do 
  begin
    if CompareText(TUser(FUserList.Items[I]).FNickname, Nickname) = 0 then 
    begin
      Result := TUser(FUserList.Items[I]); 
      break;  // <== !!!
    end;
  end;
end;

[Pseudemys Nelsoni]

Danke, habs nun drinnen, hatte es draussen gelassen, da ich denke, das jeder User gleichschnell gefunden werden sollte, anstatt das der letzte User am schnellsten und dementsprechend der erste User am langsamsten gefunden wird.

[Luckie]

Zitat von Pseudemys Nelsoni:

Danke, habs nun drinnen, hatte es draussen gelassen, da ich denke, das jeder User gleichschnell gefunden werden sollte

Gleichberechtigung für alle oder nennt sich das 'demokratisches Programmieren'?

[SirThornberry]

das ganze kann zu einer AV führen wenn FUserList.Items[I] nicht vom Typ TUser ist oder eben nil.
Es kann allerdings auch sein das du an einer ganz anderen stelle im programm speicher überschreibst und sich das erst an dieser stelle auswirkt. desweiteren kann FUserList auch nil sein bzw. schon zerstört (mit Free).

[Pseudemys Nelsoni]

Genau so nennt es sich, Luckie

@SirThornberry: Wie gesagt, ich adde dort 100% nur Objekte der Klasse TUser und "FUserList.Items[I]" kann nicht nil sein, da die Schleife sonst nicht ausgeführt werden würde.

[Luckie]

Zitat von Pseudemys Nelsoni:

"FUserList.Items[I]" kann nicht nil sein, da die Schleife sonst nicht ausgeführt werden würde.

Stimmt, da würde es vorher schon crashen. Ein Abfrage vor der Schleife
if Assigned(FUserLsit) then wäre bestimmt sinnvoll.

[GuenterS]

Zitat von Pseudemys Nelsoni:

Genau so nennt es sich, Luckie

@SirThornberry: Wie gesagt, ich adde dort 100% nur Objekte der Klasse TUser und "FUserList.Items[I]" kann nicht nil sein, da die Schleife sonst nicht ausgeführt werden würde.

Wie kommst Du da drauf?

Wenn FUserlist nil ist hast deine erste AV, weil Du da ja schon auf das Porperty Count zugreifst.

[derpepe]

Zitat von Pseudemys Nelsoni:

Genau so nennt es sich, Luckie

Man braucht ja auch Raum für spätere Optimierungen!

[Pseudemys Nelsoni]

@Luckie: Das Objekt "FUserList" ist die ganze "Lebenszeit" des Programms verfügbar, d.h ich rufe Create und auch Destroy in der Hauptklasse auf. Also FUserList ist auf jedenfall existent. Theoretisch dürfte es also keine Probleme machen oder?

MfG und danke für die Antworten.