OnPlayerStateChange Problem!

Butchey · 17. Juli 2010

Hey..
Es dreht sich um folgenden Codeschnipsel:

public OnPlayerStateChange(playerid, newstate, oldstate)
{
new nvehicleid = GetPlayerVehicleID(playerid);

if(newstate == PLAYER_STATE_DRIVER)
{
if (PlayerInfo[playerid][pFraktion] > 1 && PlayerInfo[playerid][pFraktion] < 1)
{
if (strcmp(fahrzeugdaten[nvehicleid][gewerbeowner],"lspd") == 1)
{
TogglePlayerControllable(playerid, true);
GameTextForPlayer(playerid, "~r~Kein LSPD Officer!", 1000, 1);
return 1;
}
}

Wenn ich in das/ein Auto das dem Gewerbe "lspd" zugeteilt wurde, und ich in der Fraktion 3 bin kann ich trotzdem in das Auto einsteigen.
Wo liegt der Fehler?
an dem laden/speichern der Daten für das Veh liegt es mit sicherheit nicht.

DeathlyPhil · 17. Juli 2010

Wenn du mir die Zeile erklären kannst , dann kann ich vielleicht helfen
if (PlayerInfo[playerid][pFraktion] > 1 && PlayerInfo[playerid][pFraktion] < 1)
Ich verstehe dass nämlich so dass wenn PlayerInfo[playerid][pFraktion] größer als 1 ist UND kleiner als eins ist ... hä?

Optiimal · 17. Juli 2010

weist du eh das dass
if (PlayerInfo[playerid][pFraktion] > 1 && PlayerInfo[playerid][pFraktion] < 1)
wo die 1er sind die carid sein müssen also von >24 bis 67< gehört jetzt polizei
die zahlen sind nur bsp jetzt weis ja nicht wie deine car id´s sind
MFG Mipu

Manuel1948 · 18. Juli 2010

Zitat von ŲľţįMąțė_ŜpęęďFŗēąĶ

weist du eh das dass
if (PlayerInfo[playerid][pFraktion] > 1 && PlayerInfo[playerid][pFraktion] < 1)
wo die 1er sind die carid sein müssen also von >24 bis 67< gehört jetzt polizei
die zahlen sind nur bsp jetzt weis ja nicht wie deine car id´s sind
MFG Mipu

Was laberst du? Da gehören keine CarIDs hin, des soll ja eine Fraktion sein!
Naja aufjeden Fall ist diese Abfrage sinnlos da ein Wert nicht gleich kleiner und größer sein kann als 1.
Machs lieber so:
[pwn
]if (PlayerInfo[playerid][pFraktion]!=1) //wenn der jenige Spieler nicht bei der Fraktion 1 ist (Polizei) wird weitergegangen
{
if (strcmp(fahrzeugdaten[nvehicleid][gewerbeowner],"lspd") == 0) // hier muss 0 stehen, damit er diese Abfrage nur ausführt wenn das car von lspd ist
{ // denn strcmp ergibt 0 wenn die strings gleich sind, und nicht 1
TogglePlayerControllable(playerid, true);
GameTextForPlayer(playerid, "~r~Kein LSPD Officer!", 1000, 1);
return 1;
}
}
[/pwn]

#else · 18. Juli 2010

if (PlayerInfo[playerid][pFraktion] > 1 && PlayerInfo[playerid][pFraktion] < 1)

Völlig überflüssig!
Du brachst nur:
if (PlayerInfo[playerid][pFraktion] == 0)//oder == 1 misr wurst jenachdem...

Templer · 18. Juli 2010

Du butches hör ned auf die anderen, deine Abfrage ist zwar umständlich, jedoch funktioniert sie eh, jedoch wäre folgendes einfacher:

if( PlayerInfo[playerid][pFraktion] != 1)
(Damit ist die Leute in Breadfish auch verstehen )

Zu deinem Problem, der liegt bei strcmp. Bei strcmp wird der Wert 0 (bzw false) wiedergegeben, wenn der Name übereinstimmt und nicht 1, also ändere deine Zeile zu
if (strcmp(fahrzeugdaten[nvehicleid][gewerbeowner],"lspd") == 0)
oder
if (!strcmp(fahrzeugdaten[nvehicleid][gewerbeowner],"lspd"))

Beides ist möglich

breadfish.de 11. März 2022