Spawnchange funktioniert nicht

AroX · 23. Oktober 2014

Hallo Community & Scripter

Ich habe ein kleines Problem /spawnchange funktioniert nicht,
aber ich finde einfach keinen fehler, kann mir bitte jemand
helfen?

if(strcmp(cmd, "/spawnchange", true) == 0)
{
if(IsPlayerConnected(playerid))
{
if(PlayerInfo[playerid][pPhousekey] != 255)
{
if(Spectate[playerid] != 255 && PlayerInfo[playerid][pAdmin] < 1)
{
SendClientMessage(playerid, COLOR_RED, "Sie sind im TV.");
return 1;
}
if(SpawnChange[playerid])
{
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihre Fraktion.");
SpawnChange[playerid] = 0;
if(IsAnInstructor(playerid))
{
if(SchoolSpawn[playerid] == 0)
{
SchoolSpawn[playerid] = 1;
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihre Fraktion.");
}
else if(SchoolSpawn[playerid] == 1)
{
SchoolSpawn[playerid] = 0;
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihre Fraktion.");
}
}
}
else
{
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun bei ihrem Wohnplatz.");
SpawnChange[playerid] = 1;
}
}
else if(PlayerInfo[playerid][pPhousekey] == 255 && PlayerInfo[playerid][pLeader] <= 0 || PlayerInfo[playerid][pMember] <= 0 && PlayerInfo[playerid][pPhousekey] == 255)
{
if(PlayerInfo[playerid][pPhousekey] != 255)
{
if(PlayerInfo[playerid][pLeader] == 0 || PlayerInfo[playerid][pMember] == 0)
{
if(PlayerInfo[playerid][pObdachlosenheim] == 1)
{
if(ObdachlosenSpawn[playerid] == 0)
{
ObdachlosenSpawn[playerid] = 1;
SendClientMessage(playerid,rot,"Sie Spawnen nun in ihrem Obdachlosen Heim");
}
else if(ObdachlosenSpawn[playerid] == 1)
{
ObdachlosenSpawn[playerid] = 0;
SendClientMessage(playerid,rot,"Sie Spawnen nicht mehr in ihrem Obdachlosen Heim");
}
}
}
}
}
else if(PlayerInfo[playerid][pPhousekey] == 255)
{
if(IsAnInstructor(playerid))
{
if(SchoolSpawn[playerid] == 0)
{
SchoolSpawn[playerid] = 1;
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihre Fraktion.");
}
else if(SchoolSpawn[playerid] == 1)
{
SchoolSpawn[playerid] = 0;
SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihre Fraktion.");
}
}
else
{
SendClientMessage(playerid, COLOR_RED, "Sie besitzten keine Unterkunft.");
return 1;
}
}
}
return 1;
}
Danke im vorraus
MFG

Jeffry · 23. Oktober 2014

Was heißt es geht nicht? Was passiert wenn du den Befehl eingibst? Was genau gibst du ein? Was sollte eigentlich angezeigt werden bzw. passieren?

AroX · 23. Oktober 2014

Wenn ich /spawnchange eingebe passiert einfach garnicht.
Normalerweise sollte er mir anzeigen wie "Du Spawnst nun in deinem Haus .. "
Aber es passiert einfach nichts

N0SKILL · 23. Oktober 2014

if(SpawnChange[playerid])

Leere Abfrage

Akino Kiritani · 23. Oktober 2014

Zitat von N0SKILL

if(SpawnChange[playerid])

Leere Abfrage

Die Abfrage ist nicht leer

@leader_king:
Deine Abfragen ergeben teilweise absolut keinen Sinn

else if(PlayerInfo[playerid][pPhousekey] == 255 && PlayerInfo[playerid][pLeader] <= 0 || PlayerInfo[playerid][pMember] <= 0 && PlayerInfo[playerid][pPhousekey] == 255)
{
if(PlayerInfo[playerid][pPhousekey] != 255)
wie kann pPhousekey den Wert 255 haben aber auch gleichzeitg nicht haben ?

Die ganze Struktur ist völlig im Eimer und ergibt überhaupt keinen Sinn.

Ich rate dir, mal zu schauen welche Variablen zu für den Spawnchange wirklich brauchst und welche nicht.
Es sieht ganz so aus als weißt du nicht wirklich was du da machst bzw. was du da vor dir hast.

Womöglich hast du sogar noch relevanten Code rausgeworfen.

Versuchs mal so:
if(strcmp(cmd, "/spawnchange", true) == 0)
{
if(SpawnChange[playerid]) SpawnChange[playerid] = 0;
else SpawnChange[playerid] = 1;
if(PlayerInfo[playerid][pLeader] || PlayerInfo[playerid][pMember])
{
if(IsAnInstructor(playerid))
{
if(SchoolSpawn[playerid] == 0) SchoolSpawn[playerid] = 1;
else SchoolSpawn[playerid] = 0;
}
if(SpawnChange[playerid]) SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun an ihrer Fraktion.");
else SendClientMessage(playerid, COLOR_GREEN, "Sie spawnen nun nicht mehr an ihrer Fraktion.");
}
else
{
if(PlayerInfo[playerid][pObdachlosenheim])
{
if(!ObdachlosenSpawn[playerid])
{
ObdachlosenSpawn[playerid] = 1;
SendClientMessage(playerid,rot,"Sie Spawnen nun in ihrem Obdachlosen Heim");
}
else
{
ObdachlosenSpawn[playerid] = 0;
SendClientMessage(playerid,rot,"Sie Spawnen nicht mehr in ihrem Obdachlosen Heim");
}
}
}
return 1;
}
Ich bin kein Freund von C&P aber so können wir ja noch ewig deinen Code korrigieren
PS: Ich gebe keine garantie für Fehlerfreiheit, hab das so ausm Kopf geschrieben.
Errors / Warnings sollte man schnell behoben haben, schließlich weiß ich nicht genau wie es in deinem Script aussieht

AroX · 24. Oktober 2014

Vielen Dank! es funktioniert wunderbar =)

[DT]midomen · 24. Oktober 2014

So
if(PlayerInfo[playerid][pObdachlosenheim]==1)

funktioniert es wunderbar denn er muss ja beim Obdachlosenheim angemeldet sein um dort auch spawnen zu können.

whitetiiger · 24. Oktober 2014

Zitat von [DT]midomen

So
if(PlayerInfo[playerid][pObdachlosenheim]==1)

funktioniert es wunderbar denn er muss ja beim Obdachlosenheim angemeldet sein um dort auch spawnen zu können.

Du weißt schon, dass if(PlayerInfo[playerid][pObdachlosenheim]==1) genau das gleiche ist wie if(PlayerInfo[playerid][pObdachlosenheim])
und das if(PlayerInfo[playerid][pObdachlosenheim]==0) genau das gleiche ist wie if(!PlayerInfo[playerid][pObdachlosenheim]) ist?

[DT]midomen · 24. Oktober 2014

Achso. Dachte if(var[array]) heißt = 0

Akino Kiritani · 24. Oktober 2014

Zitat von [GSR]Kaito-sensei

Du weißt schon, dass if(PlayerInfo[playerid][pObdachlosenheim]==1) genau das gleiche ist wie if(PlayerInfo[playerid][pObdachlosenheim])

Ah pls no.
Es ist nicht das gleich
Das fragt lediglich ab, ob der Wert nicht 0 ist.

whitetiiger · 24. Oktober 2014

seid wann das denn bitte schön?

Mir wurde es vor über 6Jahren so beigebracht

Langform: if(PlayerInfo[playerid][pObdachlosenheim]==1)
Kurzform: if(PlayerInfo[playerid][pObdachlosenheim])
Zurerklährung: PlayerInfo[playerid][pObdachlosenheim] gibt 1 aus 1 == true d.h. eigendlich sieht die abfrage so aus if(true)

und anders rum auch

Langform: if(PlayerInfo[playerid][pObdachlosenheim]==0)
Kurzform: if(!PlayerInfo[playerid][pObdachlosenheim])
Zurerklährung: PlayerInfo[playerid][pObdachlosenheim] gibt 0 aus 0 == false d.h. eigendlich sieht die abfrage so aus if(false)

Dies klappt jedoch nur mit 1=true und 0=false
bei allen anderen funktioniert nur die Langform(eigendlich Logisch oder nicht?)

Jeffry · 24. Oktober 2014

Dann wurde es dir falsch beigebracht.

if(test)
//ist das gleiche wie:
if(test != 0)

if(!test)
//ist das gleiche wie:
if(test == 0)

Das geht mit allen Zahlen.

whitetiiger · 24. Oktober 2014

Jeffry:
dann schreib mal in kurzform eine Abfrage nach dem Wert 255
also nicht so if(Var == 255)

ausserdem kann man es auch so machen

if(!(test != 0)) ist wieder if(true) :D, wie du siehst kann ich auch das kleine Spielchen.

Akino Kiritani · 24. Oktober 2014

@[GSR]Kaito-sensei: Das ist schon seit immer so

Das von dir genannte trifft nur auf bool(ean)s zu.
Nicht auf "normale" Variablen.

Jeffry · 24. Oktober 2014

Wenn du direkt nach 255 abfragen willst, gibt es keine "Kurzform". Die "Kurzform" bezieht sich, wie gesagt, nur auf "ist null" und "ist nicht null".

Zitat von [GSR]Kaito-sensei

if(!(test != 0)) ist wieder if(true) :D, wie du siehst kann ich auch das kleine Spielchen.

Wenn dann:
if(!(test != 0))
//gleich:
if(!test)

Weill: Nicht Nicht null ist das selbe wie null. Entsprechend: !test oder test == 0

Falls du es genau wissen willst:
http://de.wikipedia.org/wiki/De_Morgansche_Gesetze

whitetiiger · 24. Oktober 2014

komischerweise geht meine kurzform in pawn c# php

Schuldigung Jeffry:
hab mich da etwas verschaut.

Jeffry · 24. Oktober 2014

Zitat von [GSR]Kaito-sensei

komischerweise geht meine kurzform in pawn c# php

Klar geht die Kurzform, weil 1 ist ja nicht 0, von daher hast du keine Probleme. Wird deine Variable aber eine 2, dann geht das auch durch. Wenn du mit booleans arbeitest, brauchst du dir darüber keine Sorgen zu machen, logisch.

In boolean ist true das selbe wie !false und false das selbe wie !true.

breadfish.de 12. März 2022