Warning den ich einfach nicht wegbekomme!

Unkn0wnX · 24. Februar 2011

Hallo Liebe Mitglieder.
Ich habe ein Warning im Script, den ich schon seit einer halben Stunde nicht wegbekomme, ich hoffe, ihr könnt mir helfen.
Code:

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 || PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

else ist angeblich falsch eingerückt.

Mit freundlichen Grüßen, Unkn0wnX

Rastla · 24. Februar 2011

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 || PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{//Klammer auf?
}//Klammer zu?
else //dann müsste hier else stehen
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Unkn0wnX · 24. Februar 2011

Ist dein Code nun richtig?

Skeee · 24. Februar 2011

Zitat von Unkn0wnX

Ist dein Code nun richtig?

so wie es aussieht ja teste es doch einfach mal ?

Unkn0wnX · 24. Februar 2011

Das ist der gleiche Code ausser, dass er noch Kommentiert hat mit // lol^^

Rastla · 24. Februar 2011

Nein das soll eine Anregung zum Denken sein, damit du lernst wie du es selbst ausbesserst
Ich zeig dir hier mal die einzelnen Inhalte der Klammern + Die If Bedingungen dazu farblich

Zitat

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Alles anzeigen

Jetzt Klammer 2

Zitat

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 PlayerInfo[playerid][pLeader] == 1
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Alles anzeigen

Jetz Klammer 3

Zitat

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Alles anzeigen

Jetzt Klammer 4

Zitat

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Alles anzeigen

Jetzt Klammer 5

Zitat

public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Alles anzeigen

Mfg,
Firefox

Tillmann-ger · 24. Februar 2011

Diese Warnung ist eigentlich nicht so schlimm, solltest du öfters Probleme damit haben, füge oben im Script folgenden Code ein:
#pragma tabsize 0

Unkn0wnX · 24. Februar 2011

Jo habe ich gemacht, Danke Junkman.
Firefox ich raff das trotzdem nicht, kannst du mir die Lösung nicht sagen?

Rastla · 24. Februar 2011

Lösung wäre das:
public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 || PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX == PosX[playerid] && TempY == PosY[playerid] && TempZ == PosZ[playerid])
{
}
else
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}

Logischer(beides logisch, aber kürzer) wäre das:
public checkparea(playerid)
{
if(PlayerInfo[playerid][pMember] == 18 || PlayerInfo[playerid][pLeader] == 18)
{
if(IsPlayerInRangeOfPoint(playerid, 8.0, 1246.7843,-767.4252,92.0726))
{
new Float:TempX, Float:TempY, Float:TempZ;
GetPlayerPos(playerid, TempX, TempY, TempZ);
if(TempX != PosX[playerid] && TempY != PosY[playerid] && TempZ != PosZ[playerid])
{
GetPlayerPos(playerid, PosX[playerid], PosY[playerid], PosZ[playerid]);
MoveObject(riderstor,1245.698364, -767.397156, 83.123962,4.0);
SetTimer("closegate", 3000, 0);
}
}
}
return 1;
}
if(Variable1 != Variable2) bedeutet wenn Variable1 nicht gleich Variable2 ist

Unkn0wnX · 24. Februar 2011

Habe einfach das kürzere genommen, Dankeschön Firefox.
Mit freundlichen Grüßen, Unkn0wnX.

breadfish.de 11. März 2022