Was ist an diesem Code falsch

TeamSpirit · 14. Juli 2010

Hallo ich habe ein code für Auto dieb gemacht und wollte fragen was daran falsch ist

if (strcmp(cmdtext, "/verkaufen", true) ==0 ){

if(Spieler[playerid][Job] ==1)

{

if(IsPlayerInRangeOfPoint(playerid,7, -2323.0105,-1624.4343,483.8835))

{

new vehicle;

vehicle = GetPlayerVehicleID(playerid);

if(vehicle > 0)

{

SendClientMessage(playerid, gelb,"Du hast dieses Auto erfolgreich verkauft und erhälst 1.000");

GivePlayerMoney(playerid, 1000);

}

else

{

SendClientMessage(playerid,rot,"Du bist kein Auto Dieb");

}

else

{

SendClientMessage(playerid,rot,"Du bist nicht an der Richtigen Stelle.");

}
}
}
return 1;
}

der error kommt an dem 2 else

C:\Users\BceX\Desktop\PAWN r8.0\gamemodes\Selfmade.pwn(1495) : error 029: invalid expression, assumed zero

Templer · 14. Juli 2010

Guck mal wie dein Code aufgebaut ist und wie meiner hier unten

if (strcmp(cmdtext, "/verkaufen", true) ==0 )
{
if(Spieler[playerid][Job] != 1) return SendClientMessage(playerid,rot,"Du bist kein Auto Dieb");
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid,rot,"Du sitzt in keinem Fahrzeug!");
if(!IsPlayerInRangeOfPoint(playerid,7, -2323.0105,-1624.4343,483.8835)) return SendClientMessage(playerid,rot,"Du bist nicht an der Richtigen Stelle.");
SendClientMessage(playerid, gelb,"Du hast dieses Auto erfolgreich verkauft und erhälst 1.000");
GivePlayerMoney(playerid, 1000);
return 1;
}

INFO: Deine Abfrage mit if(vehicle > 0) ist sehr sinnlos, denn wenn ein Fahrzeug die ID 0 hat, hat er wohl pech oder? Deswegen nütze lieber IsPlayerInAnyVehicle nebenbei hattest du ein bis zwei Klammern zuviel die eine fehlerhafte Abfrage ermöglichten. Ansonsten passt alles

Schnacke · 14. Juli 2010

PHP

if (strcmp(cmdtext, "/verkaufen", true) ==0 )
{
    if(SpielerInfo[playerid][Job] == 1)
    {
        if(IsPlayerInRangeOfPoint(playerid,7, -2323.0105,-1624.4343,483.8835))
        {
            if(IsPlayerInAnyVehicle(playerid))
            {
                SendClientMessage(playerid, gelb,"Du hast dieses Auto erfolgreich verkauft und erhälst 1.000");
                GivePlayerMoney(playerid, 1000);
            }
            else
            {
                SendClientMessage(playerid,rot,"Du bist kein Auto Dieb");
                return 1;
             }
        }
        else
        {
            SendClientMessage(playerid,rot,"Du bist nicht an der Richtigen Stelle.");
            return 1;
        }
    }
    return 1;
}

Alles anzeigen

TeamSpirit · 14. Juli 2010

danke nun habich auch ne neue frage

Also ich habe es so gemacht

Einbrecher kann FBI base tor öffenen nun möchte ich aber das er ein andere befehl dazu hat es zu öffnen

also net FBi open wie hier

if (strcmp(cmdtext, "/fbiopen", true )==0) // Befehl zum Öffnen
{
if(Spieler[playerid][team] == 3 || Spieler[playerid][Job] == 2)
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein FBi´ler");
}
return 1;
}

Dragonfly · 14. Juli 2010

if (strcmp(cmdtext, "/fbiopen", true )==0) // Befehl zum Öffnen
{
if( Spieler[playerid][pMember] == 2 || Spieler[playerid][pLeader] == 2)
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein FBi´ler");
}
return 1;
}

TeamSpirit · 14. Juli 2010

du hast den selben code hier rein geschrieben bloss mit pLeader pMember

-.-´

ich brauche aber so

/fbiopen
können FBi eingeben
und
/einbrechen die Einbrecher

Dragonfly · 14. Juli 2010

ja pMember und Pleader das die fraktion weiß ja nicht was die einbrecher fürne fraki ind oder was die für ne job id haben

DarthXela · 14. Juli 2010

Mach doch einfach zwei Befehle
if (strcmp(cmdtext, "/fbiopen", true )==0) // Befehl zum Öffnen
{
if(Spieler[playerid][team] == 3) //Nur bei FBI-Agenten
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein FBi-Agent");
}
return 1;
}

Und

if (strcmp(cmdtext, "/einbrechen", true )==0) // Befehl zum Öffnen
{
if(Spieler[playerid][Job] == 2) //Nur bei Einbrechern
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein Einbrecher");
}
return 1;
}

Oder geht das nicht ?

YaBroo · 14. Juli 2010

Hier sind die optimalen Befehle.
Einmal nur fürs FBI und einmal nur für die JOBID2 (Einbrecher)

if (strcmp(cmdtext, "/fbiopen", true )==0) //Befehl
{
if( Spieler[playerid][pMember] == 2 || Spieler[playerid][pLeader] == 2) // Nur fürs FBI
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein FBi´ler");
}
return 1;
}

Und für einbrecher

if (strcmp(cmdtext, "/einbrechen", true )==0) //
{
if(Spieler[playerid][Job] == 2) //Nur bfür die JobID2
{
MoveObjectToStream(FBIGATE,1286.2971191406, -1657.4384765625, 20.675296783447, 2); // Die Kords, wo das tor sein soll wenns offen ist
SetTimer("FBITOR",9000,0);
}
else
{
SendClientMessage(playerid, rot, " Du bist kein Einbrecher");
}
return 1;
}

breadfish.de 11. März 2022