Hallo @Schlaubi
ist es richtig, dass ist der Befehl wie er aussah wie er noch ging?
Nun was hast du geändert dass er auf 0 steht?
Zudem wo hast du diese Code Teile hergenommen
aus dem Public sekunde?
Du musst abfragen ob das Auto existiert
das oben ins Skript
native IsValidVehicle(vehicleid);
for(new i=1; i==sizeof(tank); i++)
{
if(!IsValidVehicle(i))continue;
new vModel = GetVehicleModel(i);
tank[i]=VehicleName[vModel-400][carinfo_tankvolumen];
}
Falls das Fahrzeug nicht existiert ist vModel quasi 0 und du greifst auf einen negativen index des Arrays zu.
ist MAX_VEHICLES ein vorgegebener Wert, oder ändert der sich auf die tatsächliche Fahrzeug Anzahl ?
MAX_VEHICLES ist ein Define bedeutet es ist ein konstanter wert und beträgt 2000, aber dies kannst du in der a_samp.inc nachlesen
// Limits and internal constants
#define MAX_PLAYER_NAME (24)
#define MAX_PLAYERS (500)
#define MAX_VEHICLES (2000)
#define INVALID_PLAYER_ID (0xFFFF)
#define INVALID_VEHICLE_ID (0xFFFF)
Beim kompilieren des Skriptes ist es so das die defines gegen diese Wert getauscht werden, dies kennt man unter dem Namen Pre-Processor
für weiteres siehe auch
IPrototypeI:
Der Inhalt der Schleife würde bei dir nie ausgeführt werden.
for(new i=1; i<sizeof(tank); i++)
{
if(!IsValidVehicle(i))continue;
new vModel = GetVehicleModel(i);
tank[i]=VehicleName[vModel-400][carinfo_tankvolumen];
}
So.
Es können 1999 Fahrzeuge erstellt werden, von 1 bis MAX_VEHICLES-1 (=1999).
IPrototypeI:
Der Inhalt der Schleife würde bei dir nie ausgeführt werden.
Danke für die Verbesserung, dass habe ich bei ihm übersehen.
for(new i=1; i<sizeof(tank); i++)
Heißt:
i=1
i=2
i=3
i=4
i=5
...
i=1997
i=1998
i=1999
ENDE.
Da sizeof(tank) 2000 ist, und 2000 nicht kleiner als 2000 ist, wird 2000 nicht mehr für i genommen.
Nein, wenn du i==sizeof(tank) machst, dann würde er gar nichts machen, da beim ersten Durchlauf i = 1 ist, und das ist nicht sizeof(tank), sprich die Scheife wird beendet.
Wenn, dann musst du i<=sizeof(tank) machen, allerdings wird dir das beim letzten Durchlauf (i=2000) das Array sprengen, da ein Array[2000] keinen Index 2000 (i) hat, sondern maximal den Index 1999 (0 bis 1999 = 2000).
Nein, du machst die Schleife ja so:
for(new i=1; i<sizeof(tank); i++)
Und da 2000 nicht mehr kleiner als 2000 (sizeof) ist, wird i = 2000 nicht vorkommen.
Anfangen tut es bei 1, das hast du ja so angegeben (new i = 1).
Wenn du es so machst, dann wird die Schleife nie ausgeführt.
Warum?
i = 1
=> Prüfe i == 2000 -> Nein (da i = 1)
=> Beende Schleife
Die Bedingung ist immer die Bedingung für den nächsten Durchlauf. Ist die falsch, wird sofort beendet.
