AutohausSystem Errors ;(

SchafiSAMP · 30. April 2012

Guten Tag,

Ich habe das Autohaus Script von Kaliber.
Nun habe ich ein 2 Autohaus eingefügt, wo man 2 Räder kaufen kann.

Dann kommen diese Errors:
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(90) : error 032: array index out of bounds (variable "radhaus")
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(91) : error 032: array index out of bounds (variable "radhaus")
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(218) : error 032: array index out of bounds (variable "radhaus")
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(224) : error 032: array index out of bounds (variable "radhaus")
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(296) : error 021: symbol already defined: "string"

Ich hoffe ihr könnt mir helfen

Kaliber · 30. April 2012

Poste mal die Zeilen pls

mfg.

SchafiSAMP · 30. April 2012

Zeile 90 - 91
radhaus[0] = AddStaticVehicleEx(461,2137.2698,-1128.6577,25.1739,97.0748,0,0,-1); // Motorrad
radhaus[1] = AddStaticVehicleEx(462,2137.3391,-1131.5660,25.2841,91.5664,0,0,-1); // Motorrad
radhaus[2] = AddStaticVehicleEx(463,2137.8147,-1134.6567,25.2437,89.5464,0,0,-1); // Motorrad
radhaus[3] = AddStaticVehicleEx(468,2137.7708,-1137.6604,25.2947,87.9942,0,0,-1); // Motorrad
radhaus[4] = AddStaticVehicleEx(509,2137.4614,-1139.9176,24.8905,91.3523,0,0,-1); // Motorrad
radhaus[5] = AddStaticVehicleEx(510,2137.3643,-1142.1135,24.7462,89.9158,0,0,-1); // Motorrad
radhaus[6] = AddStaticVehicleEx(586,2118.4263,-1129.7975,24.8021,267.9649,0,0,-1); // Motorrad
radhaus[7] = AddStaticVehicleEx(522,2118.4214,-1127.4648,24.8766,266.4201,0,0,-1); // Motorrad
radhaus[8] = AddStaticVehicleEx(521,2118.5430,-1125.3544,24.9063,259.1242,0,0,-1); // Motorrad
Error 218

else if(GetPlayerVehicleID(playerid) == radhaus[7])
{
preis[playerid] = 30000;
TogglePlayerControllable(playerid, 0);
ShowPlayerDialog(playerid,Dialog12,DIALOG_STYLE_MSGBOX,"{FF0000}2 Rad Shop","{FFFFFF}Willst du dieses Fahrzeug für {FF0000}30.000${FFFFFF} kaufen ?","Kaufen","Abbrechen");
}

Error 224

else if(GetPlayerVehicleID(playerid) == radhaus[8])
{
preis[playerid] = 12000;
TogglePlayerControllable(playerid, 0);
ShowPlayerDialog(playerid,Dialog12,DIALOG_STYLE_MSGBOX,"{FF0000}2 Rad Shop","{FFFFFF}Willst du dieses Fahrzeug für {FF0000}12.000${FFFFFF} kaufen ?","Kaufen","Abbrechen");
}
}
return 1;
}

Error 296

new string[256], str[256], sts[128], name[MAX_PLAYER_NAME];
if(dialogid == Dialog13 || dialogid == Dialog14 || dialogid == Dialog15 || dialogid == Dialog16 || dialogid == Dialog17 || dialogid == Dialog18 || dialogid == Dialog19 || dialogid == Dialog20)
{
if(response == 1)
{
if(PlayerInfo[playerid][pKey] > 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt schon ein Auto !");
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
if(GetPlayerMoney(playerid) >= preis[playerid])
{
GivePlayerMoney(playerid, -preis[playerid]);
PlayerInfo[playerid][pPreis] = preis[playerid];
format(string, sizeof(string),"769.4449, -1327.1703, 13.3326, 268.7346, 0, 0, %d, %d",GetVehicleModel(GetPlayerVehicleID(playerid)),preis[playerid]);
SaveVehicles(string, playerid);
PlayerInfo[playerid][pKey] = CreateVehicle(GetVehicleModel(GetPlayerVehicleID(playerid)), 769.4449,-1327.1703,13.3326,268.7346,0,0,-1);
GetPlayerName(playerid, name, sizeof(name));
format(sts, sizeof(sts),"%s",name);
SetVehicleNumberPlate(PlayerInfo[playerid][pKey], sts);
SetVehicleToRespawn(PlayerInfo[playerid][pKey]);
SetPlayerPos(playerid, 769.7058,-1324.5464,13.3906);
SetCameraBehindPlayer(playerid);
TogglePlayerControllable(playerid, 1);
format(str, sizeof(str),"Auto erfolgreich für %d$ gekauft !",preis[playerid]);
SendClientMessage(playerid, COLOR_LIGHTBLUE, str);
return 1;
}
else
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt zu wenig Geld für dieses Auto !");
return 1;
}
}
if(response == 0)
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
}
return 1;
}

Kaliber · 30. April 2012

Wie hast du radhaus deklariert ?

(new radhaus[?])

mfg.

SchafiSAMP · 30. April 2012

new radhaus[7];

namespace · 30. April 2012

Der Array "radhaus" ist zu klein.
"string" gibt es schonmal - benenn die Variable anders.

SchafiSAMP · 30. April 2012

Wie zu klein?

Marius_DePrimo · 30. April 2012

new radhaus[9];

0-8 = 9

SchafiSAMP · 30. April 2012

Okay jez habe ich nur noch den Error mit dem string wenn ich den ändere gehts nicht?

Marius_DePrimo · 30. April 2012

lösch dein new string

SchafiSAMP · 30. April 2012

Kannst du mir das mal im Beispiel zeigen?. Ich bin noch neu in sachen Scripten

namespace · 30. April 2012

Bennen einfach die Variable "string" und alle anderen Vorkommen dieser Variable in "stringWhatEver" um.

Marius_DePrimo · 30. April 2012

new str[256], sts[128], name[MAX_PLAYER_NAME];
if(dialogid == Dialog13 || dialogid == Dialog14 || dialogid == Dialog15 || dialogid == Dialog16 || dialogid == Dialog17 || dialogid == Dialog18 || dialogid == Dialog19 || dialogid == Dialog20)
{
if(response == 1)
{
if(PlayerInfo[playerid][pKey] > 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt schon ein Auto !");
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
if(GetPlayerMoney(playerid) >= preis[playerid])
{
GivePlayerMoney(playerid, -preis[playerid]);
PlayerInfo[playerid][pPreis] = preis[playerid];
format(string, sizeof(string),"769.4449, -1327.1703, 13.3326, 268.7346, 0, 0, %d, %d",GetVehicleModel(GetPlayerVehicleID(playerid)),preis[playerid]);
SaveVehicles(string, playerid);
PlayerInfo[playerid][pKey] = CreateVehicle(GetVehicleModel(GetPlayerVehicleID(playerid)), 769.4449,-1327.1703,13.3326,268.7346,0,0,-1);
GetPlayerName(playerid, name, sizeof(name));
format(sts, sizeof(sts),"%s",name);
SetVehicleNumberPlate(PlayerInfo[playerid][pKey], sts);
SetVehicleToRespawn(PlayerInfo[playerid][pKey]);
SetPlayerPos(playerid, 769.7058,-1324.5464,13.3906);
SetCameraBehindPlayer(playerid);
TogglePlayerControllable(playerid, 1);
format(str, sizeof(str),"Auto erfolgreich für %d$ gekauft !",preis[playerid]);
SendClientMessage(playerid, COLOR_LIGHTBLUE, str);
return 1;
}
else
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt zu wenig Geld für dieses Auto !");
return 1;
}
}
if(response == 0)
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
}
return 1;
}

SchafiSAMP · 30. April 2012

Denn kommen diese 3 Warnings:
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(296) : warning 204: symbol is assigned a value that is never used: "stsds"
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(296) : warning 204: symbol is assigned a value that is never used: "strds"
C:\Users\Alex\Desktop\Selfmade!\gamemodes\Autohaussystem.pwn(296) : warning 204: symbol is assigned a value that is never used: "stringds"

Marius_DePrimo · 30. April 2012

mach ma so

Und mach nicht nur C&P

if(dialogid == Dialog13 || dialogid == Dialog14 || dialogid == Dialog15 || dialogid == Dialog16 || dialogid == Dialog17 || dialogid == Dialog18 || dialogid == Dialog19 || dialogid == Dialog20)
{
if(response == 1)
{
if(PlayerInfo[playerid][pKey] > 0)
{
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt schon ein Auto !");
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
if(GetPlayerMoney(playerid) >= preis[playerid])
{
GivePlayerMoney(playerid, -preis[playerid]);
PlayerInfo[playerid][pPreis] = preis[playerid];
format(string, sizeof(string),"769.4449, -1327.1703, 13.3326, 268.7346, 0, 0, %d, %d",GetVehicleModel(GetPlayerVehicleID(playerid)),preis[playerid]);
SaveVehicles(string, playerid);
PlayerInfo[playerid][pKey] = CreateVehicle(GetVehicleModel(GetPlayerVehicleID(playerid)), 769.4449,-1327.1703,13.3326,268.7346,0,0,-1);
GetPlayerName(playerid, name, sizeof(name));
format(sts, sizeof(sts),"%s",name);
SetVehicleNumberPlate(PlayerInfo[playerid][pKey], sts);
SetVehicleToRespawn(PlayerInfo[playerid][pKey]);
SetPlayerPos(playerid, 769.7058,-1324.5464,13.3906);
SetCameraBehindPlayer(playerid);
TogglePlayerControllable(playerid, 1);
format(str, sizeof(str),"Auto erfolgreich für %d$ gekauft !",preis[playerid]);
SendClientMessage(playerid, COLOR_LIGHTBLUE, str);
return 1;
}
else
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
SendClientMessage(playerid, COLOR_LIGHTRED,"Du besitzt zu wenig Geld für dieses Auto !");
return 1;
}
}
if(response == 0)
{
RemovePlayerFromVehicle(playerid);
TogglePlayerControllable(playerid, 1);
return 1;
}
}
return 1;
}

namespace · 30. April 2012

Marius, dein Code ist fehlerhaft.
Die Variablen werden logischerweiße auch im Code verwendet.
Da du in deinem bereitgestellten C&P Code aber nur die Deklaration umbenannt hast, wird das Script nun die bereits vorherfestgelegten Variablen benutzen (weswegen vorhin ja der "already defined" Error aufgetreten ist).
Also solltest du wenigstens auch alle Vorkommen der neu-benannten Variable dementsprechend ändern.

Marius_DePrimo · 30. April 2012

Zitat von namespace

Marius, dein Code ist fehlerhaft.
Die Variablen werden logischerweiße auch im Code verwendet.
Da du in deinem bereitgestellten C&P Code aber nur die Deklaration umbenannt hast, wird das Script nun die bereits vorherfestgelegten Variablen benutzen (weswegen vorhin ja der "already defined" Error aufgetreten ist).
Also solltest du wenigstens auch alle Vorkommen der neu-benannten Variable dementsprechend ändern.

Ich weiß schon was ich da tu...

Er hat den code erweitert daher brauch er den string nicht nochmal definieren daher meinte ich das auch mit C&P.....
und deine variante den string einfach umzubenennen ist auch schwachsinn, da er den einmal in den public hat.

es reicht einer führ mehrere nicht 2oder4 für 2 Funktionen

namespace · 30. April 2012

Okay, aber was wenn er den alten Inhalt der string-Variable noch einmal verwenden will?
Dann steht da schon der neu-formatierte Text drin.
Also macht es durchaus Sinn, für mehrere Verwendungszwecke auch mehrere Variablen anzulegen, als immer eine zu überschreiben.

breadfish.de 11. März 2022