Was...machst du denn da genau? 
cInfo[i][besitzer] Das ist ja ein String...aber playerInfo[playerid][pMySQLID] ist doch ein Integer?! Und du willst hier die Besitzer vergleichen..was du aber machst..ist den Besitzer mit deiner MySQLID zu vergleichen?! Oder steht da der Name von dem..Besitzer drinnen? 
Wenn ja, wäre das dann so richtig:
if(!strcmp(cInfo[i][besitzer], playerInfo[playerid][pMySQLID]) && cInfo[i][slot] == s)
Da wäre noch die 2190:
Das gibt dir Zahlen von 1-6:
new zahl = random(6)+1;
Was meinst du mit zufällig entscheiden lassen, wer von den Spielern gewonnen hat?!
Ja kannste auch nutzen 
Sieht dann so aus:
ocmd:contract(playerid, params[])
{
// Wenn der Spieler in der Fraktion 18 ist brauchen wir auch nicht Prüfen was er eingegeben hat da er es sowieso nicht darf.
if(PlayerInfo[playerid][pMember] == 18) return SCM(playerid, COLOR_GRAU, "Du kannst keinen Auftrag in deiner eigenen Agentur abgeben!");
new pID,kopfgeld;
if(sscanf(params,"ud",pID,kopfgeld)) return SCM(playerid, COLOR_RED, "{33CCFF}FEHLER:{FFFFFF} /contract [playerid] [kopfgeld]");
if(!IsPlayerConnected(pID)) return SCM(playerid, COLOR_GRAU, "Ein Spieler mit dieser ID ist nicht online!");
// Wir prüfen hier schon ob der Spieler der selbe ist der den Command eingegeben hat dergleiche ist wie der Zielspieler
// Unabhängig davon ob der Spieler der den Command eingegeben hat mehr oder weniger als 10k bzw 100k eingegeben hat.
if(giveplayerid == playerid) return SCM(playerid, COLOR_GRAU, "Du kannst auf dich keinen Auftrag setzen!");
if(kopfgeld < 10000 || kopfgeld > 100000) return SCM(playerid, COLOR_GRAU, "Nicht weniger als €10.000 oder mehr als €100.000!");
if (PlayerMoney[playerid] < kopfgeld) return SCM(playerid, COLOR_GRAD1, "Sie besitzen zu wenig Geld für diese Summe!");
PlayerMoney[playerid] = SafeGivePlayerMoney(playerid, -kopfgeld);
PlayerInfo[pID][pKopfgeld]+=kopfgeld;
new string[128];
format(string, sizeof(string), "%s hat ein Kopfgeld auf %s ausgesetzt, Lohn €%d.",GetName(playerid), GetName(pID), kopfgeld);
SendFamilyMessage(18, COLOR_YELLOW, string);
format(string, sizeof(string), "* Du hast ein kopfgeld auf %s ausgesetzt, Lohn €%d.",GetName(playerid), GetName(pID));
SCM(playerid, COLOR_LIGHTBLUE, string);
PlayerPlaySound(playerid, 1052, 0.0, 0.0, 0.0);
return 1;
}
//Und die GetName Funktion:
stock GetName(playerid)
{
static n[MAX_PLAYER_NAME];
return GetPlayerName(playerid,n,MAX_PLAYER_NAME),n;
}Nichts für ungut, aber nutze einen CMD-Prozessor wie ocmd,zcmd,ycmd,rcmd...
Und mach dir das Leben einfach & performant 
1. Heißt das, dass man immer static statt new nehmen soll?
Nein...da du ja nicht immer Speicherplatz reservieren willst...
Sondern nur dann, wenn es entweder um große Strings geht..die du oft nutzt...
oder kleine Strings die du sehr oft nutzt...
wie lasse ich jede Linie ausführen?
Was genau meinst du jetzt damit....
Unter OnDialogResponse...prüfst du über das listitem welche Route er ausgewählt hat..und lässt ihn darüber dann einen Checkpoint anzeigen...
Und einen Countdown kannst du z.B. so realisieren:
//Countdown-Starten
SetTimerEx(!"@countdown", 999, 0, !"i", playerid);
//Das hier als forward/public ansehen
@countdown(i);@countdown(i)
{
static x;
if(++x == 10)
{
GameTextForPlayer(i, "10~n~GoGoGo", 1000, 5);
//Hier dann machen..was passieren soll...
return 1;
}
new tmp[4];
valstr(tmp,x);
GameTextForPlayer(i, tmp, 1000, 5);
return SetTimerEx(!"@countdown", 999, 0, !"i", i);
}static oder new?
static, dadurch werden nicht permanent 2048 Cells (8192 Bytes = 8 KiloByte) pro Aufruf angefordert, sondern reserviert.
Das spart eine Menge Zeit
Wobei du eigentlich versuchen solltest alles zu packen.
Du musst doch nicht unbedingt format verwenden?
Verstehe nicht so genau...wieso du darauf zurückgreifen musst 
Denn wenn du z.B. einen Namen adden willst geht das auch so:
new packed_name[MAX_PLAYER_NAME char];
strpack(packed_name, GetName(playerid));
//DialogString = !""; //Wichtig das dient als reset (aber da wir darunter schon was setzen, brauchen wir das nicht)
DialogString = !"Willkommen ";
strcat(DiaogString, packed_name);
strcat(DialogString, !" :)\nSchön, dass sie zu uns gefunden haben!");
Ist zwar ein bisschen umständlicher...aber du büßt nur minimale Performanz ein..und sparst dir 6 KiloByte von 8
Und den verwende ich dann jetzt, richtig?
Einfach als ganz normalen String...
format(DialogString,sizeof(DialogString),"Haaaaaaaaaaaaaaalo\nHaaaaalo %s", GetName(playerid));
ShowPlayerDialog(playerid,0,0,"Caption", DialogString, "ok", "");
Wie gesagt, kannst das char weglassen, dann geht das klar
Also nur im strcat benutzen?
...das wäre am sichersten oder eine direkte zuweisung
Wie new test[] = !"Hi";
Du solltest das nicht global nutzen!
Da kann es zu Fehlern kommen...zwar meint man immer, Pawn ist in der VM Single-Threaded was auch so ist, allerdings können Callbacks sogut wie zeitgleich für Spieler ausgeführt werden.
Besser wäre zu versuchen, Strings zu packen oder einen großen statischen Speicherbereich im Callback zu allozieren.
//Ganz oben im OnDialogResponse
static DialogString[2048 char]; //Mit char wäre es am schönsten ^^ geht aber auch ohne xD
Nein format kannst du da nicht nutzen 
[900 char] bedeutet, dass eine Cell (wo du hier 900 erstellst) nicht 4 Bytes groß ist, sondern nur 1 Byte
Wenn du format nutzen musst, dann musste normale Cells verwenden
Als erstes rate ich dir von dcmd ab...nutze ocmd/zcmd/ycmd/rcmd..iwas anderes außer dcmd
Als nächstes schreib den code mal so (return..immer klein):
dcmd_setfraktion(playerid,params[])
{
if(!IstSpielerAdmin(playerid, 5)) return SendClientMessage(playerid,ROT,"Du bist kein Admin!");
new pid, frakid;
if(sscanf(params,"ud",pid,frakid)) return SendClientMessage(playerid,GELB,"Verwendung : /setfraktion [ID][Fraktions ID]");
if(!IsPlayerConnected(pid)) return SendClientMessage(playerid,ROT,"Ungültiger Spieler");
if(IstSpielerInFraktion(pid,frakid) == frakid) return SendClientMessage(playerid,ROT,"Der Spieler ist bereits in der Fraktion");
sSpieler[pid][Fraktion] = frakid;
SendClientMessage(pid,BLAU,"Du bist nun in der Fraktion");
SendClientMessage(playerid,BLAU,"Du hast den Spieler in die Fraktion gepackt");
return 1;
}Jaa es heißt nicht /n sondern \n
Die Anderen Klappen kaum hab ich einen Langen Text spinnt es rum?
Jaa das liegt am Compiler 
Du musst des zum Beispiel so machen:
new packer[900 char];
strcat(packer,!"Artikel 4: Fragen und Meldungen\n\nEs gibt zwei Methoden, um Fragen bzw. Anliegen an die Administration zu senden.");
strcat(packer,!"\n\nCheater,Buguser,sinnloses Deathmatsch und Dinge in die Richtung können direkt\n");
strcat(packer,!"per /a <Text> in den Admin-Chat gesendet werden,admit diese schnell bearbeitet werden.");
strcat(packer,!"\n\nBei anderen Anliegen wie z.B. bei Fragen wende dich bitte mit /sup <Frage>");
strcat(packer,!"\nan einen Communityhelfer,Suppoter oder Administrator.\n\n");
strcat(packer,!"Die Ticketanzahl ist aus scriptteschnischen Gründen limitiert, es kommt selten bis nie vor, dass du warten musst.");
strcat(packer,!"\nBitte gedulde dich, wenn du in dem Moment kein Ticket erstellen kannst, versuche es eventuell später noch einmal");
strcat(packer,!"\nund schreibe es nicht in den /a-Chat, da es sein kann, das du dafür gemutet wirst.\n\n");
strcat(packer,!"Formuliere dein Anliegen bitte direkt und nicht so: Frage, Hilfe und Sonstiges.\n");
strcat(packer,!"Das hilft unserem Team nicht, dir schnell zu helfen. Und dass sollte dir ja auch wichtig sein.");
ShowPlayerDialog(playerid, DIALOG_HELP4, DIALOG_STYLE_MSGBOX, !"Hilfe", packer, !"Schließen", !"");Das char packed den string, dadurch sparst du einfach Speicher
Dieses ! sagt einfach nur, dass der String gepackt werden soll
Kannst du auch ideal so bei Texten im Dialog verwenden
...ja dann prüf was passiert
Mach n print-Debug und schau..was aufgerufen wird..und so
Doch...das kannst du da einfach drunter klatchen..schau dir meinen code an..da unter die if-Abfrage kommt das alles mit dem IsPlayerConnected
In dem Fall musste kein if(listitem) checken, da wir die ID über den inputtext filtern und der je nach listitem angepasst ist
dini hat nichts mit PVars zu tun ._.
Das eine ist eine Speicherung, das andere ist quasi eine Speicherung im RAM und nicht im Stack wie bei einer Variable
Du machst quasi folgendes:
SetPVarInt(playerid, "lohn_id", pID);
Und dann öffnest du einen neuen Dialog und dort setzt du dann den lohn so:
new pID = GetPVarInt(playerid, "lohn_id"), lohn = strval(inputtext);
DeletePVar(playerid, "lohn_id"); //Wichtig! Hier Ram reinigen
//Und nun haste die pID = Der Spieler wo der Lohn gesetzt werden soll...lohn = der Betrag vom Lohn :)