/repair (ocmd)

wo ist jetzt genau dein Problem?

Mahc das in dein Script rein
stock Spielername(playerid)
{
new xn[MAX_PLAYER_NAME];
GetPlayerName(playerid,xn,sizeof xn);
return xn;
}

Und dann
if(dialogid==DIALOG_ME)
{
if(reponse==0)
{
format(string,sizeof(string),"Dein Auftrag wurde von %s Abgelehnt",Spielername(pID));
SendClientMessage(playerid,Rot,string);
return 1;
}
}
return 1;
Dann sollte es klappen

Das bringt rein garnix, ausser dass der Name angezeigt ist.
PId ist bei dialogresponse nicht definiert.
Save die playerid des mechanikers im oberen Teil,
Und frag Sie dann bei dialogrespobse wieder ab.

Zb mit setpvarint(pId, "mechanikerid", playerid);
Und unten:
pId = getpvarint(playerid, "mechanikerid");

Oder man macht es einfacher:
Ganz oben:
new Spielerid[MAX_PLAYERS], Gegnerid[MAX_PLAYERS];

ocmd:repair(playerid,params[])
{
new pID;
if(!SpielerInfo[playerid][Job] == 1))return SendClientMessage(playerid,Rot,"Du bist kein Mechaniker.");
if(sscanf(params,"u",pID)) return SendClientMessage(playerid,Hellblau,"/repair [ID]");
Spielerid[playerid] = playerid;
Gegnerid[playerid] = pID;
ShowPlayerDialog(pID,DIALOG_ME,DIALOG_STYLE_MSGBOX,"Mechaniker - Repair","Dir wird eine Reperatur für 90$ angeboten.","Annehmen","Ablehnen");
return 1;
}

if(dialogid==DIALOG_ME)
{
if(reponse==0)
{
format(string,sizeof(string),"Dein Auftrag wurde von %s Abgelehnt",Gegnerid[playerid]);
SendClientMessage(Spielerid[playerid],Rot,string);
return 1;
}
}
Das ganze ist ungetestet sieht aber akzeptabel aus.

ach kommschon, dann vertausch einfah die beiden playerid's, das wird doch nicht so schwer sein ?
im zweiten teil hat der die "playerid", der im ersten die pId hatte, und andersrum...
geht nicht anders...^^

Hier bitte schön, mein letzter Post.
repair mit annahme

mfg. =)

Zitat

if(dialogid==DIALOG_ME)
{
if(reponse==0)
{
format(string,sizeof(string),"Dein Auftrag wurde von %s Abgelehnt",Gegnerid[playerid]);
SendClientMessage(Spielerid[playerid],Rot,string);
return 1;
}
}

Alles anzeigen

Das is doch irwie a schmarrn oder ? ;D
Spielerid[playerid]=playerid; bewirkt mal wieder sehr viel....

nimm das hier.. auch wenn dann noch viel unnützes zeug im oberen teil steht...
aber mit c&p erhält man eben nie etwas besseres

if(dialogid==DIALOG_ME)
{
	    if(reponse==0)
	    {
	        format(string,sizeof(string),"Dein Auftrag wurde von %s Abgelehnt",playerid);
	        SendClientMessage(Gegnerid[playerid],Rot,string);
                return 1;
            }
		if(response==1){
                       new pId=Gegnerid[playerid];
			if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage((pId,Rot,"Der Spieler ist in keinem Fahrzeug.");
	     	RepairVehicle(GetPlayerVehicleID(playerid));
	     	SendClientMessage(pId,Grün,"Deine Anfrage wurde Angenommen. Du erhälst 90$.");
	     	GivePlayerMoney(playerid,-90);
	     	GivePlayerMoney(pId,90);
	     	return 1;
		}
}

ja tut er doch....
maaaan
bei ondialogresponse(playerid, ...)
ist die playerid, die id von der normalen Person, nicht die von dem Mechaniker...

dann kriegste eben den letzten fetzen auch noch gemacht

ocmd:repair(playerid,params[])
{
new pID;
if(!SpielerInfo[playerid][Job] == 1))return SendClientMessage(playerid,Rot,"Du bist kein Mechaniker.");
if(sscanf(params,"u",pID)) return SendClientMessage(playerid,Hellblau,"/repair [ID]");
Gegnerid[pID] = playerid;
ShowPlayerDialog(pID,DIALOG_ME,DIALOG_STYLE_MSGBOX,"Mechaniker - Repair","Dir wird eine Reperatur für 90$ angeboten.","Annehmen","Ablehnen");
return 1;
}

sollte eigentlich schon funktionieren.
pID muss eben immer gleich geschrieben werden,
nicht einmal pId und einmal pID...

Hey ich weiß nicht ob du das mit PVarInt machst aber hier so würde ich es machen

SetPVarInt(pID, "Mechaniker", playerid);
Musst du im Befehl des Mechanikers einschreiben.

Und im Dialog dann:
SendClientMessage(GetPVarInt(playerid, "Mechaniker"),Rot,string); Da nimmt er die ID von dem Mechaniker wo oben im Befehl gespeichert wurde. Das bringt schonmal
das es an den Mechaniker gesendet wird, im String wird aber der richtige Name nicht rausgelesen. Habe ich ehrlich gesagt gerade null bock alles zu testen, also ich verspreche nichts.