schon wieder was faslch...

[MM]Marco · 11. Oktober 2011

hi ich bins mal wiedaaa,

ich hab gerade meinen jobbefehl erstellt, aber als ich compilieren wollte ist er abgestürzt...
ich zeig euch am besten mal den code:
if (strcmp("/job", cmdtext, true, 10) == 0)
{
if(GetPVarInt(playerid, "Job") >= 1)
{
SendClientMessage(playerid,COLOR_ROT,"Du hast bereits einen Job! Kündige erst deinen aktuellen Job, bevor du einen neuen anfängst");
return 1;
}

else if(GetPVarInt(playerid, "Job") == 0)
{
else if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
SetPVarInt(playerid, "Job", 1),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Trucker angenommen."); //Trucker
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 2),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Abschleppdienst angenommen.");//adac
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, -51.6429,-1.0805,3.1172))
{
SetPVarInt(playerid, "Job", 3),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Farmer angenommen.");//farmer
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 4);),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");//reinigungsdienst
return 1;
}

if(isPlayerInFrakt(playerid,9)|isPlayerInFrakt(playerid,10)|isPlayerInFrakt(playerid,12))
//if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 5),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Prostituierte angenommen.");//prostituierte
{
else return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Mitglied einer Mafia/Gang!");
}
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 6),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Drogenkurier angenommen.");//drogenkurier
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 589.9217,871.1524,-42.4973))
{
SetPVarInt(playerid, "Job", 7), SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzarbeiter angenommen.");//erzarbeiter
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 816.8825,856.6313,12.7891))
{
SetPVarInt(playerid, "Job", 8),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzlieferant angenommen.");//erzarbeiter
return 1;
}

}
}
}
return 1;
}

würde mch freuen wenn jemand sich die zeit nehmen könnte und sich das mal ansieht

MFG
marco

cHaOsFaTz · 11. Oktober 2011

probiere das mal so, da waren 2-3 Klammer falsch. Hoffe das es geht, wenn nicht poste den Fehler nochmal hier

if (strcmp("/job", cmdtext, true, 10) == 0)
{
if(GetPVarInt(playerid, "Job") >= 1)
{
SendClientMessage(playerid,COLOR_ROT,"Du hast bereits einen Job! Kündige erst deinen aktuellen Job, bevor du einen neuen anfängst");
return 1;
}

else if(GetPVarInt(playerid, "Job") == 0)
{
else if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
SetPVarInt(playerid, "Job", 1),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Trucker angenommen."); //Trucker
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 2),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Abschleppdienst angenommen.");//adac
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, -51.6429,-1.0805,3.1172))
{
SetPVarInt(playerid, "Job", 3),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Farmer angenommen.");//farmer
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 4);),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");//reinigungsdienst
return 1;
}

if(isPlayerInFrakt(playerid,9)|isPlayerInFrakt(playerid,10)|isPlayerInFrakt(playerid,12))
//if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 5),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Prostituierte angenommen.");//prostituierte
{
else return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Mitglied einer Mafia/Gang!");
}
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 6),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Drogenkurier angenommen.");//drogenkurier
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 589.9217,871.1524,-42.4973))
{
SetPVarInt(playerid, "Job", 7), SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzarbeiter angenommen.");//erzarbeiter
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 816.8825,856.6313,12.7891))
{
SetPVarInt(playerid, "Job", 8),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzlieferant angenommen.");//erzarbeiter
return 1;
}

return 1;
}

Tony_Vegas · 11. Oktober 2011

Mach mal die returns usw in eine Zeile mit tab

[MM]Marco · 11. Oktober 2011

ähm.. die sind in einer zeile, das ist nur hier verutscht, außerdem hab ich auch tabsize 0 drin^^

also ich habs jetzt so, war noch ein fehler mit einem else zuviel drin:
if (strcmp("/job", cmdtext, true, 10) == 0)
{
if(GetPVarInt(playerid, "Job") >= 1)
{
SendClientMessage(playerid,COLOR_ROT,"Du hast bereits einen Job! Kündige erst deinen aktuellen Job, bevor du einen neuen anfängst");
return 1;
}

else if(GetPVarInt(playerid, "Job") == 0)
{
if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
SetPVarInt(playerid, "Job", 1),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Trucker angenommen."); //Trucker
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 2),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Abschleppdienst angenommen.");//adac
return 1;
}

else if(IsPlayerInRangeOfPoint(playerid,2, -51.6429,-1.0805,3.1172))
{
SetPVarInt(playerid, "Job", 3),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Farmer angenommen.");//farmer
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 4);),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");//reinigungsdienst
return 1;
}

if(isPlayerInFrakt(playerid,9)|isPlayerInFrakt(playerid,10)|isPlayerInFrakt(playerid,12))
//if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 5),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Prostituierte angenommen.");//prostituierte
{
else return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Mitglied einer Mafia/Gang!");
}
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 6),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Drogenkurier angenommen.");//drogenkurier
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 589.9217,871.1524,-42.4973))
{
SetPVarInt(playerid, "Job", 7), SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzarbeiter angenommen.");//erzarbeiter
return 1;
}

if(IsPlayerInRangeOfPoint(playerid,2, 816.8825,856.6313,12.7891))
{
SetPVarInt(playerid, "Job", 8),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Erzlieferant angenommen.");//erzarbeiter
return 1;
}

return 1;
}

hier die error codes:

Zitat

C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4060) : error 029: invalid expression, assumed zero
C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4069) : error 029: invalid expression, assumed zero
C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4069) : warning 215: expression has no effect
C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4069) : error 001: expected token: ";", but found "return"
C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4071) : warning 225: unreachable code
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

3 Errors.

Alles anzeigen

Tony_Vegas · 11. Oktober 2011

Ja sry hab ich auch bemerkt xD

[MM]Marco · 11. Oktober 2011

ist ja kein problem^^

wolltest nur helfen

Akino Kiritani · 11. Oktober 2011

Mach aus

if(isPlayerInFrakt(playerid,9)|isPlayerInFrakt(playerid,10)|isPlayerInFrakt(playerid,12))
//if(IsPlayerInRangeOfPoint(playerid,2, 2446.3088,-2115.9871,13.5469))
{
SetPVarInt(playerid, "Job", 5),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Prostituierte angenommen.");//prostituierte
{
else return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Mitglied einer Mafia/Gang!");
}
return 1;
}
das
if(isPlayerInFrakt(playerid,9)||isPlayerInFrakt(playerid,10)||isPlayerInFrakt(playerid,12))
{
SetPVarInt(playerid, "Job", 5),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Prostituierte angenommen.");//prostituierte
}
else return SendClientMessage(playerid,COLOR_ROT,"Du bist kein Mitglied einer Mafia/Gang!");

[MM]Marco · 11. Oktober 2011

danke,

jetzt hab ich aber noch ein error:

Zitat

C:\Users\Marco\Desktop\Marcos Server\gamemodes\MarcosServerTM.pwn(4060) : error 029: invalid expression, assumed zero
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

hier der zeilen abschnitt, ich poste nur den abschnitt, weil ich ja oben den ganzen befehl schon stehen hab:^^

if(IsPlayerInRangeOfPoint(playerid,2, 1494.6995,-1281.4718,14.5787))
{
SetPVarInt(playerid, "Job", 4);),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");//reinigungsdienst <-zeile 4060
return 1;
}

cHaOsFaTz · 11. Oktober 2011

das ist falsch

SetPVarInt(playerid, "Job", 4);),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");

so ist es richtig

SetPVarInt(playerid, "Job", 4),SendClientMessage(playerid, COLOR_GRÜN1, "Du hast den Job als Reinigungsdienst angenommen.");

du hattest dazwischen noch hinter der 4) das " ; ) " ööhmm hat nix da zu suchen

Akino Kiritani · 11. Oktober 2011

Such
SetPVarInt(playerid, "Job", 4);),...

[MM]Marco · 11. Oktober 2011

ok,

ich habs verbessert, jetzt bekomme ich auch keine errors, aber wenn ich an der eine der positionen stehe und /job eingebe passiert gar nichts

PaddyBoy · 11. Oktober 2011

guter tipp erstelle lieber die pickups mit z.b.
new JobPickup[ANZAHL DER PICKUPS]; im script oben und bei den ganzen job sachen

OnGamemodeInit
JobPickup = CreatePickup(COODS 'USW);

public OnPlayerPickUpPickup(playerid, pickupid)
{
if(JobPickup == PICKUPID)
{
blablabla was hald passieren soll
}

falls dus net verstehst mach ich dir dann nen beispiel aber ist zumindest ressoucensparender als isplayerinrange usw usw ;)^^

MfG

[MM]Marco · 11. Oktober 2011

also ich versteh schon was du meinst, aber iwie gilt das bei mir dann für alle pickups^^

vllt schreib ich das ja auch falsch

kannst du mir vllt einfach mal zeigen wie man das schreibt?

[MM]Marco · 11. Oktober 2011

eine frage hätte ich aber noch: <BR><BR><BR><BR>wie mache ich es jetzt so, das man den befehl /job eingeben muss? oder muss ich einfach nur isplayerinrangeofpoint auswechseln mit der pickupnumer?

PaddyBoy · 12. Oktober 2011

mach ich dir morgen/heut mittag grad net gut gelaunt und sry für die späte meldung^^

breadfish.de 11. März 2022