Binco [HILFE]

bL4cK_ · 8. Oktober 2011

Tag

Ich hab mir gestern einen Binco gemacht. Wenn man rein läuft und man /skin eingibt
wird man zum spiegel im Binco geportet. Ich kann aber nur 1 Skin auswählen. Klappen tut alles.
Nur ich bekomm 6 - 7 Warnings. Kann mir evtl. mir einer sagen wie ich es gescheit mache?
damit ich auch mehr skins auswähen kann?

Zeilen:
if(!strcmp("/Buyskin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerPos(playerid,207.6156,-105.9069,1005.1328);
SetPlayerFacingAngle(playerid,179.6174);
SetCameraBehindPlayer(playerid);
SetPlayerInterior(playerid,15);
SendClientMessage(playerid,GRAU,"Vielen Dank für ihren Einkauf");
TogglePlayerControllable(playerid,1);
GivePlayerMoney(playerid, -25);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}
if(!strcmp("/Nextskin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerSkin(playerid,37,207.6294,-100.7798,1005.2578,1.7364);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}
if(!strcmp("/Skin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 4.0,207.6294,-100.7798,1005.2578))
{
SendClientMessage(playerid,WEIß,"Du kannst dir nun einen Skin aussuchen");
SendClientMessage(playerid,WEIß,"Mit /Nextskin kannst du die Skins anschauen");
SendClientMessage(playerid,WEIß,"Mit /Buyskin kannst du dir den Skin für $25 kaufen");
SetPlayerPos(playerid, 218.2416,-98.2081,1005.2578);
SetPlayerFacingAngle(playerid,136.5038);
SetPlayerCameraPos(playerid, 216.2575,-100.0539,1005.2578);
SetPlayerCameraLookAt(playerid, 218.2416,-98.2081,1005.2578);
SetPlayerInterior(playerid,15);
TogglePlayerControllable(playerid, 0);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}

Errors/Warnings
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(538) : warning 202: number of arguments does not match definition
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(538) : warning 202: number of arguments does not match definition
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(538) : warning 202: number of arguments does not match definition
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(538) : warning 202: number of arguments does not match definition
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(1194) : warning 217: loose indentation
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(1201) : warning 203: symbol is never used: "ret_memcpy"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

6 Warnings.

MFG

Kaliber · 8. Oktober 2011

Also

SetPlayerSkin(playerid,37,207.6294,-100.7798,1005.2578,1.7364); //das ist falsch !

das oben muss zu

SetPlayerSkin(playerid,37);

werden.

mfg.

Sereon · 8. Oktober 2011

if(!strcmp("/Buyskin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerPos(playerid,207.6156,-105.9069,1005.1328);
SetPlayerFacingAngle(playerid,179.6174);
SetCameraBehindPlayer(playerid);
SetPlayerInterior(playerid,15);
SendClientMessage(playerid,GRAU,"Vielen Dank für ihren Einkauf");
TogglePlayerControllable(playerid,1);
GivePlayerMoney(playerid, -25);
return 1;
}
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}

1. fehler
2. du hast das defined "ret_memcpy" aber nicht genutzt also löschen

bL4cK_ · 8. Oktober 2011

Danke@ kaliber

jezt hab ich immernoch 2 warnings

C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(1194) : warning 217: loose indentation
C:\Users\bL4cK_OuT\Desktop\Server\gamemodes\Reallife.pwn(1201) : warning 203: symbol is never used: "ret_memcpy"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

2 Warnings.

Kaliber · 8. Oktober 2011

Ändere mal

if(!strcmp("/Buyskin", cmdtext, true))
{

zu

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

mfg.

bL4cK_ · 8. Oktober 2011

hm funktioniert nit.
bekomme 26errors

Kaliber · 8. Oktober 2011

du musst natürlich dann noch eine { setzen

bL4cK_ · 8. Oktober 2011

Ja das weis ich
Hab ich auch gemacht, geht nit

Ramino · 8. Oktober 2011

if(strcmp("/Buyskin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerPos(playerid,207.6156,-105.9069,1005.1328);
SetPlayerFacingAngle(playerid,179.6174);
SetCameraBehindPlayer(playerid);
SetPlayerInterior(playerid,15);
SendClientMessage(playerid,GRAU,"Vielen Dank für ihren Einkauf");
TogglePlayerControllable(playerid,1);
GivePlayerMoney(playerid, -25);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}

Kaliber · 8. Oktober 2011

if(strcmp(cmdtext, "/Buyskin", true) == 0))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerPos(playerid,207.6156,-105.9069,1005.1328);
SetPlayerFacingAngle(playerid,179.6174);
SetCameraBehindPlayer(playerid);
SetPlayerInterior(playerid,15);
SendClientMessage(playerid,GRAU,"Vielen Dank für ihren Einkauf");
TogglePlayerControllable(playerid,1);
GivePlayerMoney(playerid, -25);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}

InternetInk aka Benjamin · 8. Oktober 2011

Falsch @ Ramino

@ Threadersteller

Loose Indentation bedeutet der Abstand zum linken rand stimmt nicht.
Die Leerzeichen löschen und mit der Tab-Taste einrücken

//Edit
@ Kaliber

Beitrag Bearbeiten > Code löschen > Über dem Text Feld über der Grauen Leiste Von "Editor" zu "Quellcode" wechseln > drücken den Code zwischen die Klammern kopieren und erneut absenden

So ist er 1. Im richtigen Format und kein Zitat oder sowas und
2. Er ist eingerückt und deutlich besser lesbar...

Merken, danke...

Ramino · 8. Oktober 2011

Was denn daran falsch. Anscheinend hat der nach dem Code von DJ_Vallah nen Klammerfehler gehabt. Er hat ja selber gesagt er hätte 26Errors.
Das mit dem Einrücken, weiß ich ja, dass das nicht stimmt nur wollt ich , dass er den Klammerfehler los wird?

bL4cK_ · 8. Oktober 2011

Zitat von Kaliber

Also

SetPlayerSkin(playerid,37,207.6294,-100.7798,1005.2578,1.7364); //das ist falsch !

das oben muss zu

SetPlayerSkin(playerid,37);

werden.

mfg.

Alles anzeigen

Danke
Funktioniert zu 50%
Warnings hab ich nur noch 2
jedoch kann ich aber nur 1skin auswählen. obwohl ich mehrere eingefügt habe

Ramino · 8. Oktober 2011

Wo haste denn mehrere eingefügt?

bL4cK_ · 8. Oktober 2011

unter OnPlayerCommandText

if(!strcmp("/Nextskin", cmdtext, true))
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
SetPlayerSkin(playerid,37);
SetPlayerSkin(playerid,40);
SetPlayerSkin(playerid,44);
SetPlayerSkin(playerid,67);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}

InternetInk aka Benjamin · 8. Oktober 2011

Zitat von Ramino (unsichtbar)

Was denn daran falsch.

if(strcmp("/Buyskin", cmdtext, true))
Würde nicht Funktionieren
Entweder müsste es
if(strcmp("/Buyskin", cmdtext, true) == 0)
//oder
if(!strcmp("/Buyskin", cmdtext, true))
//heißen

@ Threadersteller
//Edit
Das macht auch keinen Sinn.
Du Setzt den Skin in dem moment wo du die Funktion SetPlayerSkin aufrufst.
Also setzt du ihn 4 mal hintereinander wobei der letzte bleibt

bL4cK_ · 8. Oktober 2011

ne

4 skins hab ich drin

Ramino · 8. Oktober 2011

Das würde so nicht klappen.

Hier umgebaut:

if(strcmp("/Nextskin", cmdtext, true)==0)
{
if(IsPlayerInRangeOfPoint(playerid, 2.0,218.2416,-98.2081,1005.2578))
{
if(GetPlayerSkin(playerid)==299)
{
SetPlayerSkin(playerid,0);
return 1;
}
SetPlayerSkin(playerid,GetPlayerSkin(playerid)+1);
return 1;
}
else
{
SendClientMessage(playerid,GRAU,"Du bist nicht im Binco");
return 1;
}
}

Einrücken müsstest du selber^^

bL4cK_ · 8. Oktober 2011

@ Internetstink aka Benjamin

kannst du mir nicht die richtigen zeilen sagen?
ich check das nit

Ramino · 8. Oktober 2011

Benjamin.

Oh darauf habe ich garnicht geachtet. Hatte jetzt auf den Klammerfehler geachtet.