Nützliche Codeschnipsel

Hier mal ein kleiner Befehl um Verstärkung zu Rufen. Habe ich auf die Schnelle für ein Godfather Script geschrieben im Praktikum.
Wenn ich was falsch gemacht habe, sagt es mir

if(strcmp(cmd,"/vst",true) == 0)
{
GetPlayerName(playerid, sendername, sizeof(sendername));
new Float:X,Float:Y,Float:Z;
GetPlayerName(playerid, sendername, sizeof(sendername));
GetPlayerPos(giveplayerid, X,Y,Z);
{
if (IsACop(playerid) && OnDuty[playerid] ==1) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Cop / Du bist nicht Duty");
{
SendClientMessage(playerid,COLOR_RED, "Alle Einheiten wurden Informiert.");
SendClientMessage(playerid,COLOR_GREEN, "Halte deine Kollegen am laufenden, bis sie eingetroffen sind (/d)");
}
if(PlayerInfo[playerid][pMember] == 1) { format(string, sizeof(string), "** Ich brauche Verstärkung %s: %s, over. **", sendername, playerid); }
{
SetPlayerCheckpoint(playerid, X,Y,Z, 6);
}
}
return 1;
}

Zitat von B l a c k D r a g o n

Hier mal ein kleiner Befehl um Verstärkung zu Rufen. Habe ich auf die Schnelle für ein Godfather Script geschrieben im Praktikum.
Wenn ich was falsch gemacht habe, sagt es mir

if(strcmp(cmd,"/vst",true) == 0)
{
GetPlayerName(playerid, sendername, sizeof(sendername));
new Float:X,Float:Y,Float:Z;
GetPlayerName(playerid, sendername, sizeof(sendername));
GetPlayerPos(giveplayerid, X,Y,Z);
{
if (IsACop(playerid) && OnDuty[playerid] ==1) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Cop / Du bist nicht Duty");
{
SendClientMessage(playerid,COLOR_RED, "Alle Einheiten wurden Informiert.");
SendClientMessage(playerid,COLOR_GREEN, "Halte deine Kollegen am laufenden, bis sie eingetroffen sind (/d)");
}
if(PlayerInfo[playerid][pMember] == 1) { format(string, sizeof(string), "** Ich brauche Verstärkung %s: %s, over. **", sendername, playerid); }
{
SetPlayerCheckpoint(playerid, X,Y,Z, 6);
}
}
return 1;
}

Alles anzeigen

Das ist falsch, total falsch

Das musst du so machen:

if(strcmp(cmd,"/vst",true) == 0)
{
GetPlayerName(playerid, sendername, sizeof(sendername));
new Float:X,Float:Y,Float:Z;
GetPlayerPos(playerid, X,Y,Z);
for(new i; i<100; i++)
{
if (IsACop(i) && OnDuty[i] ==1)
{
SendClientMessage(playerid,COLOR_RED, "Alle Einheiten wurden Informiert.");
SendClientMessage(playerid,COLOR_GREEN, "Halte deine Kollegen am laufenden, bis sie eingetroffen sind (/d)");
format(string, sizeof(string), "** Spieler %s (ID: %i) benötigt Verstärkung, over. **", sendername, playerid);
SendClientMessage(i, COLOR_YELLOW, string);
SetPlayerCheckpoint(i, X,Y,Z, 6);
}
}
return 1;
}

mfg.

Warum ist es falsch :D?
Bei mir hat es funktioniert .. einban frei.

Weildu 2 mal den namen holst -> sinnlos.

if (IsACop(playerid) && OnDuty[playerid] ==1) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Cop / Du bist nicht Duty");
{Ist Falsch und ergibt auch keinen sinn.
Wenn der Spieler ein Cop ist, und OnDuty ist, sendest du die Nachricht das er keiner wäre und beendest durch return die ausführung. Dann kommt die klammer, die niemals ausgeführt wird -> sinnlos.

Dann formatiertst du eine nachricht, die niemals gesendet wird. Eben so sinnlos. Ausserdem verwendest du format falsch. Wenn du eine Zahl (playerid) willst solltest du %f bzw %i nutzen, nicht %s. Das ist für strings. Abgesehen davon macht die nachricht auch keinen wirklichen sinn. "** Ich brauche Verstärkung BlackDragon: 0, over. **" ?

Und zu guter letzt setzt du einen checkpoint für den spieler, an der position des Spielers. Macht auch nicht wirklich viel sinn.

Das von kaliber ist auch aber nicht ganz richtig...
for(new i; i<100; i++)Nur 100? Ist das Realismus, weil der Server diese grenze wohl nie überschreiten wird?^^
SendClientMessage(i,COLOR_RED, "Alle Einheiten wurden Informiert.");
SendClientMessage(i,COLOR_GREEN, "Halte deine Kollegen am laufenden, bis sie eingetroffen sind (/d)");Die nachrichten sollten an den Spieler selbst gehen, nicht an die Kollegen.
Und auch du formatierst den string, verwendest ihn dann allerdings nicht.

Zitat von Maddin

Das von kaliber ist auch aber nicht ganz richtig...

PAWN Quelltext
1
for(new i; i<100; i++)
Nur 100? Ist das Realismus, weil der Server diese grenze wohl nie überschreiten wird?

PAWN Quelltext
1
2
SendClientMessage(i,COLOR_RED, "Alle Einheiten wurden Informiert.");
SendClientMessage(i,COLOR_GREEN, "Halte deine Kollegen am laufenden, bis sie eingetroffen sind (/d)");
Die nachrichten sollten an den Spieler selbst gehen, nicht an die Kollegen.
Und auch du formatierst den string, verwendest ihn dann allerdings nicht.

Alles anzeigen

Ha, shit darauf habe ich gar nicht geachtet und das mit der 100 hast du richtig interpretiert, ich fand 50 wäre zu unfair

mfg.

// Basic MySQL Login/Register
// --------------------------------
// Credits:
// + wife-beater
// + G-sTyLeZzZ (mysql plugin)
// + sa-mp team
// --------------------------------

Filterscript:
Basic Login/Register Filterscript by wife-beater - Pastebin.com

phpMyAdmin SQL Dump:
-- phpMyAdmin SQL Dump -- version 3.3.7deb7 -- http://www.phpmyadmin.net</a> -- - Pastebin.com

MySQL Plugin:
Multiupload.nl - upload your files to multiple file hosting sites!

Ich bin immer offen für Kritik.

stock IsViewInRangeOfPoint(playerid, Float: r, Float: x, Float: y, Float: z)
{
new Float:GP[3], GPCFV[3], Float: Output[3];
const Float: weite = 5.0;
GetPlayerCameraPos(playerid, GP[0], GP[1], GP[2]);
GetPlayerCameraFrontVector(playerid, GPCFV[0], GPCFV[1], GPCFV[2]);
Output[0] = GP[0] + floatmul(GPCFV[0], weite);
Output[1] = GP[1] + floatmul(GPCFV[1], weite);
Output[2] = GP[2] + floatmul(GPCFV[2], weite);
if((Output[0] - x)*(Output[0] - x) + (Output[1] - y)*(Output[1] - y) <= r*r && Output[2] <= z)return 1;
else return 0;
}

stock IsPlayerInRangeOfPoint(playerid, Float: radius, Float:X, Float:Y, Float:Z)
{
new Float:pxx,Float:pxy,Float:pxz;
GetPlayerPos(playerid,pxx,pxy,pxz);
if((pxx - X)*(pxx - X) + (pxy - Y)*(pxy - Y) <= radius*radius && pxz <= Z)return 1;
else return 0;
}

Also die obere ist meine eigenen Funktion mit der es möglich ist zu überprüfen ob der spieler in einen bestimmten punkt rein schaut von einer entfernung von 5.0
längeneinheiten welche ich mir zurecht geschrieben habe für mein feuerwehrsystem welches ich momentan mache ist jedoch noch nicht getestet jedoch geh ich mal
davon aus das diese funktioniert da der IsPlayerInRangeOfPoint stock auch funktioniert

Keine Ahnung ob der Code hier schon drinsteht. Habe zumindest nichts über die Suche gefunden.
Der Codeschnipsel ist ein Array mit allen VehicleModels und den entsprechenden Namen. Erstellt wurde der Code von meinem Bruder noir.

Unnützlich. Index + 400 = ModelID. Der Name reicht aus und bei 606 und 607 sehe ich zwei " ohne \ davor.

Die Function OnVehiclePNS

hier der Code

forward OnVehiclePNS(vehicleid);

public OnVehiclePNS(vehicleid)
{
return 1;
}

public OnVehicleDamageStatusUpdate(vehicleid, playerid)
{
#pragma unused playerid
new panels, doors, lights, tires;
GetVehicleDamageStatus(vehicleid, panels, doors, lights, tires);
if(!panels && !doors && !lights && !tires)CallLocalFunction("OnVehiclePNS", "i", vehicleid);
return 1;
}

habe getestet und es geht

MFG Piet

Zitat von [PA]Piet_Meier

Die Function OnVehiclePNS

hier der Code

forward OnVehiclePNS(vehicleid);

public OnVehiclePNS(vehicleid)
{
return 1;
}

public OnVehicleDamageStatusUpdate(vehicleid, playerid)
{
#pragma unused playerid
new panels, doors, lights, tires;
GetVehicleDamageStatus(vehicleid, panels, doors, lights, tires);
if(!panels && !doors && !lights && !tires)CallLocalFunction("OnVehiclePNS", "i", vehicleid);
return 1;
}

habe getestet und es geht

MFG Piet

Alles anzeigen

Und was genau bewirkt dieser Code?

Er ruft Die Funktion OnVehiclePNS auf also wenn ein Auto Repariert wird
was sehr viele leute schon gesucht haben

Zitat von [PA]Piet_Meier

Er ruft Die Funktion OnVehiclePNS auf also wenn ein Auto Repariert wird
was sehr viele leute schon gesucht haben

Gut, ich dachte schon, das du es unter dem Verkaufen möchtest, dass dies aufgerufen wird, wenn man im PNS fährt.
Klar, es wird dort unteranderem aufgerufen, nur sowas könnte schnell zu bugs führen, wenn man das denkt:)

Nützliche Include

http://forum.sa-mp.com/showthread.php?t=343172

und nützlicher code

for(new i; i< sizeof(Test); i++){
new rand = random(500);
Test[i][test1] = rand;
Test[i][test2] = i;
}
SortDeepArray(Test, test1,.order = SORT_DESC);
for (new i = 0; i < sizeof(Test); i++) {
printf("Test[%d] = {%d, %d}", i, Test[i][test1], Test[i][test2]);
}

enum testinfo{
test1,
test2,
}
new Test[MAX_PLAYERS][testinfo];

Mir war gerade langweilig und habe mal einen Lustigen Code geschrieben, der es möglich machen würde in einer SendClientMessage $variablen zu verwenden wie in PHP

Sähe dann so aus:

stock ScmEx(playerid, farbe, format[])
{
new z = strfind(format, "$", true);
if(z > 0)
{
new x[64], y;
x = _strtok(format, z);
y = (z+1);
z -= strlen(x);
strdel(format, z, y);
strdel(x, 0, 1);
new text[128];
strcat(text, "Gustav", sizeof text);
strins(format, text, z, sizeof text);
SendClientMessage(playerid, farbe, format);
return 1;
}
else return SendClientMessage(playerid, farbe, format);
}

Hier mal ein Beispiel zur Anwendung:

ScmEx(playerid, rot,"Herzlich willkommen $name !");

In meinem Codeschnipsel habe ich eine konstante Variable die den Namen regelt, anstatt $name würde dann "Gustav" da stehen

und da ist der Punkt leider kann man das nicht als Include machen, da man den Wert der Variable herbekommen muss, bzw. sie übergeben muss und das erfordert mehr Zeilen Code als ein einfaches format

Naja, trotzdem viel Spaß damit

mfg.

Warum löst du das nicht über PVars?

Zitat von BlackAce

Warum löst du das nicht über PVars?

Mhm, daran habe ich gar nicht gedacht, aber der Nachteil wäre halt, dass man jede Variable die man abfragen will in pVars packen muss

Die Idee ist schon sehr gut von dir Kaliber nur wurde sowas schon lange umgesetzt und um es so zu sagen auch nicht schlecht und zwar von Ryder und Zeex , beziehungsweise ist die Version von Ryder noch schneller. Sowas ähnliches kann man auch nur mit Variabeln machen anstatt mit #emits nur ist sowas schon relative schwer vor allem
wenn man noch beliebig viele platzhalter verwenden will.

stock SendFormatMessage(const iPlayer, const iColor, const szFormat[], { Float, _ }: ...) {
new
iArgs = (numargs() - 3) << 2
;
if(iArgs) {
static
s_szBuf[144],
s_iAddr1,
s_iAddr2
;
#emit ADDR.PRI szFormat
#emit STOR.PRI s_iAddr1

for(s_iAddr2 = s_iAddr1 + iArgs, iArgs += 12; s_iAddr2 != s_iAddr1; s_iAddr2 -= 4) {
#emit LOAD.PRI s_iAddr2
#emit LOAD.I
#emit PUSH.PRI
}
#emit CONST.PRI s_szBuf

#emit PUSH.S szFormat
#emit PUSH.C 144
#emit PUSH.PRI
#emit PUSH.S iArgs
#emit SYSREQ.C format

#emit LCTRL 4
#emit LOAD.S.ALT iArgs
#emit ADD.C 4
#emit ADD
#emit SCTRL 4

return (iPlayer != -1) ? SendClientMessage(iPlayer, iColor, s_szBuf) : SendClientMessageToAll(iColor, s_szBuf);
}
return (iPlayer != -1) ? SendClientMessage(iPlayer, iColor, szFormat) : SendClientMessageToAll(iColor, szFormat);
}

//edit orginaler link noch hinzu gefügt

http://forum.sa-mp.com/showthread.php?p=1722584#post1722584