Also erstmal benutze doch bitte kein ocmd wenn du auch keine Parameter benutzt, wäre nämlich sinnfrei.
Ich denke du bist generell bisschen durcheinander geraten in deinen Zeilen. Allein bei /quitjob die Meldung "Du hast bereits einen Job" ist seltsam 
Ich würde den Stock erstmal ändern:
stock HatPlayerAnNebenJob(playerid)
{
if(GetPVarInt(playerid,"Jobstatus")>=1)return 1;
return 0;
}
Dann würde ich Quitjob erstmal in das ändern:
if(!strcmp(cmdtext,"/quitjob"))
{
if(!HatPlayerAnNebenJob(playerid)return SendClientMessage(playerid,DUNKELROT,"Du bist bereits arbeitslos.");
if(!IsPlayerInRangeOfPoint(playerid,2.0,358.4476,180.6003,1008.3828))return SendClientMessage(playerid,DUNKELROT,"Du bist nicht am tresen bei /quitjob");
{
ShowPlayerDialog(playerid,DIALOG_QUITJOB,DIALOG_STYLE_MSGBOX,"Job Kündigen","Willst du deinen Job Wirklich Kündigen ?","Ja","Nein");
}
return 1;
}
Und bei /jobs:
if(!strcmp(cmdtext,"/quitjob"))
{
if(HatPlayerAnNebenJob(playerid)return SendClientMessage(playerid,DUNKELROT,"Du hast bereits einen Job!");
if(!IsPlayerInRangeOfPoint(playerid,2.0,358.3224,184.6619,1008.3828))return SendClientMessage(playerid,DUNKELROT,"Du bist nicht am /jobs Tresen");
{
ShowPlayerDialog(playerid,DIALOG_JOBS,DIALOG_STYLE_LIST,"Job Liste","Taxifahrer\nMüllman","Auswählen","Schließen");
}
return 1;
}
Wäre einfacher denk ich