Hilfe beim Motor System

Wo habe ich hier einen Fehler gebaut

ocmd:licht(playerid,params[]);
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return
SendClientMessage(playerid,ROT,"Du bist nicht der Fahrer eines Fahrzeugs.");
new vID=GetPlayerVehicleId(playerid),
tmp_engine,
tmp_lights,
tmp_alarm,
tmp_doors,
tmp_bonnet,
tmp_boot,
tmp_objective;
GetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
if(tmp_lights==1){
tmp_engine = 0;
}else
tmp_engine = 1;
}
SetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
return 1;
}

Zitat von ChainzZ

Du könntest uns ja mal verraten - was genau nicht funktioniert?

Ich möchte Compilen und es stürzt ab. Und das liegt dann ja meist daran das ein großer Fehler in dem Script ist. Wenn ich diese Sektion wegmache kommen nur 2 Errors die aber auch berechtigt sind

Zitat von Rocky

Dann schau mal ganz genau hin. Da fehlt eine {

Ich finds nicht:D

Zitat von iSweety

}else

ocmd:motor(playerid,params[]);
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return
SendClientMessage(playerid,ROT,"Du bist nicht der Fahrer eines Fahrzeugs.");
new vID=GetPlayerVehicleId(playerid),
tmp_engine,
tmp_lights,
tmp_alarm,
tmp_doors,
tmp_bonnet,
tmp_boot,
tmp_objective;
GetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
if(tmp_lights==1){
tmp_engine = 0;
}else{
tmp_engine = 1;
}
SetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
return 1;
}

Wenn ich es so mache Funktioniert es trz. nicht

"

Zitat von iSweety

Hinter der OCMD Abfrage (erste Zeile) st ein Semikolon am Ende. Das gehört da nicht hin

ocmd:motor(playerid,params[])
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER) return SendClientMessage(playerid,ROT,"Du bist nicht der Fahrer eines Fahrzeugs.");
new vID=GetPlayerVehicleId(playerid),
tmp_engine,
tmp_lights,
tmp_alarm,
tmp_doors,
tmp_bonnet,
tmp_boot,
tmp_objective;
GetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
if(tmp_lights==1) {
tmp_engine = 0;
} else {
tmp_engine = 1;
}
SetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
return 1;
}
Ich finde die Abfrage sowieso ein wenig komisch. Du fragst ob das Licht an ist, und steuerst danach den Motor?!

Lg

Alles anzeigen

"

Ja da hab ich gerade bische mist gebaut also so sollte es nun sein:

ocmd:motor(playerid,params[])
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return
SendClientMessage(playerid,ROT,"Du bist nicht der Fahrer eines Fahrzeugs.");
new vID=GetPlayerVehicleId(playerid),
tmp_engine,
tmp_lights,
tmp_alarm,
tmp_doors,
tmp_bonnet,
tmp_boot,
tmp_objective;
GetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
if(tmp_engine==1){
tmp_engine = 0;
}else
tmp_engine = 1;
}
SetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
return 1;
}

Habe aber trotzdem 3 Errors :

C:\Dokumente und Einstellungen\Cedric\Eigene Dateien\Dropbox\samp03x_svr_R2_win32\gamemodes\reallife.pwn(566) : error 017: undefined symbol "GetPlayerVehicleId"
C:\Dokumente und Einstellungen\Cedric\Eigene Dateien\Dropbox\samp03x_svr_R2_win32\gamemodes\reallife.pwn(574) : warning 217: loose indentation
C:\Dokumente und Einstellungen\Cedric\Eigene Dateien\Dropbox\samp03x_svr_R2_win32\gamemodes\reallife.pwn(579) : warning 209: function "ocmd_motor" should return a value
C:\Dokumente und Einstellungen\Cedric\Eigene Dateien\Dropbox\samp03x_svr_R2_win32\gamemodes\reallife.pwn(580) : error 021: symbol already defined: "SetVehicleParamsEx"
C:\Dokumente und Einstellungen\Cedric\Eigene Dateien\Dropbox\samp03x_svr_R2_win32\gamemodes\reallife.pwn(581) : error 010: invalid function or declaration
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

3 Errors.

Zitat von iSweety

Das GetPlayerVehicleID statt GetPlayerVehicleId
Wegen SetVehicleParamsEx
Schau mal in der Zeile nach, oder ist es hier die 19?

Bitte sage einmal welche Zeilen welche Zeilen hier im Forum sind.

Der Loose Identitation Fehler kommt, weil du die Zeilen nicht korrekt einrückst.
Mach das mal, dann sehen wir weiter

Ich empfehle dir zudem hinter jedem Komma oder = Zeichen und auch davor ein Leerzeichen zu machen (Aber nicht vor den Kommata). Das macht das ganze och ein wenig übersichtlicher. Nutz allgemein viel leerzeichen. Es schadet nicht, esseiden du hast ne sehr kleine Festplatte Und schieb das SendClientMessage direkt hinter das return oben.

Lg

Alles anzeigen

ocmd:motor(playerid,params[])
{
if(GetPlayerState(playerid)!=PLAYER_STATE_DRIVER)return SendClientMessage(playerid,ROT,"Du bist nicht der Fahrer eines Fahrzeugs.");

new vID=GetPlayerVehicleID(playerid),
tmp_engine,
tmp_lights,
tmp_alarm,
tmp_doors,
tmp_bonnet,
tmp_boot,
tmp_objective;
//motor an/ausschalten
GetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
if(tmp_engine==1){
tmp_engine = 0;
}else{
tmp_engine = 1;
}
SetVehicleParamsEx(vID, tmp_engine, tmp_lights, tmp_alarm, tmp_doors, tmp_bonnet, tmp_boot, tmp_objective);
return 1;
}

So funktioniert endlich alles:)

Vielen dank für eure Hilfe