Binco by Jacke2009

Vielen Dank

Da solltest du einiges editieren:

• if (strcmp("/info", cmdtext, true, 10) == 0)
  {
  GivePlayerMoney(playerid,1);
  }
  Was hat das GivePlayerMoney da zu suchen...raus damit
  

  
  

• Du solltest Abfragen bei deinem Dialog ob der Spieler überhaupt 50$ besitzt, sonst kann das tief in den Minus gehen...
• 20x Skin 1..Skin2 geschrieben ist auch nicht das gelbe vom Ei einfach mal sowas gemacht:
  //Oben im Script
  static const string_dialog[128];
  //Unter OnGameModeInit
  new buffer[10];
  for(new i=1; i!=20; i++) format(buffer, sizeof buffer,"Skin %d\n",i),strins(string_dialog, buffer, 0, 128);
  

  
  

  //Und dann einfach bei deinem Dialog so einsetzen
  ShowPlayerDialog(playerid,dSkinshop,DIALOG_STYLE_LIST,"Bincoladen",string_dialog,"Bestätigen","Abbrechen");

  
  

• Ich rate dir zur Nutzung von switch und case, da dieses um einiges schneller ist als eine if-Abfrage
• Achte ein wenig auf Zeilensparen:
  Vorher (Dein Code):
  if(dialogid == dInfo)
  {
  if(response == 1)
  {
  SendClientMessage(playerid,Grün,"(c)Jacke2009");
  }
  if(response == 0)
  {
  SendClientMessage(playerid,Grün,"(c)Jacke2009");
  }
  return 1;
  }
  Optimierter Code:
  if(dialogid == dInfo) {
  if(response == 1)SendClientMessage(playerid,Grün,"(c)Jacke2009");
  else SendClientMessage(playerid,Grün,"(c)Jacke2009");
  }

Nicht sehr viel Arbeit und auch nicht so schön

mfg.

Zitat von Jacke2009

Problem ist, ich weiss nicht wie ich das abfrage mit dem Geld Kann mir dort jemand helfen? Dann kommt das heut Abend noch dazu.

if(GetPlayerMoney(playerid) < 50) return SendClienMessage(playerid, 0xFF0000FF,"Du besitzt zu wenig Geld für diesen Skin");

mfg.

new result = GetPlayerMoney(playerid);if(result < 50)return SendClientMessage(playerid,0xFFFFFFFF,"Du hast nicht genügend Geld!");

//edit: Oder direkt so wie Kaliber | Warst etwas schneller

if(GetPlayerMoney(playerid) >= 50) // er hat $50 oder mehr...

Allerdings fehlerhaft.

Dies
if(GetPlayerMoney(playerid) < 50)
mit diesem
if(GetPlayerMoney(playerid) >= 50)
ersetzen

Zitat von k4biX

Allerdings fehlerhaft

Genau, direkt falsch reingeschrieben

und wenn du dabei bist, solltest du dir direkt nochmal die Basics anschauen und evtl noch mehr meiner Vorschläge realisieren

mfg.

Das ganze könnte man auch in viel weniger Zeilen und eleganter lösen
http://pastebin.com/ncnsbyfE

frag mal ab wie viel money der noch hat ob weniger als > 50 ;D

Zitat von Jacke2009

•Auf Anfrage mach ich auch noch Fraktionsskins rein. (Bezahlung)

Das ist ja lächerlich, biete den Leuten doch einfach die Möglichkeit ihre skin ids einfach einzutragen.