Beiträge von Jeffry

[DT]Midoman:
Ja, mit einem Trick:

Oben im Code:
new Float:MarkerPos[100][3]; //100 Map Icons ist ja das Limit

stock SetPlayerMapIcon_Ex(playerid, iconid, Float:x, Float:y, Float:z, markertype, color, style)
{
MapIconPos[iconid][0] = x;
MapIconPos[iconid][1] = y;
MapIconPos[iconid][2] = z;
return SetPlayerMapIcon(playerid, iconid, x, y, z, markertype, color, style);
}

Position wird dann so abgefragt:
printf("Map Icon Nummer 15 ist an der Position: %f, %f, %f.", MapIconPos[15][0], MapIconPos[15][1], MapIconPos[15][2]);

Man könnte die _Ex Funktion auch hooken, darauf habe ich aber mal verzichtet, das lohnt sich wirklich nicht.

Zitat von NicoWiss

Ehm das kann man auch ohne Timer regeln gib doch den Neulingen direkt den reocurcenfrssenden Müll
So easy ohne timner statt return 1; bei dem ersten if deine Sache hineinschreiben (ban(); kick())

Gegen einen Timer ist nichts einzuwenden, der ist im Gegensatz zu deiner Variante sogar besser. Warum?

Folgendes Szenario:

• Cheater kommt in den Server, bekommt meinetwegen 100€ Startgeld, ok. Funktion wird aufgerufen, passt alles.
• Cheater cheated sich 100.000€
• Nichts passiert, die Funktion wurde ja noch nicht aufgerufen.
• Cheater verlässt den Server, sein Geld wird nun gespeichert (100.100€), kein Ban, die Funktion wurde noch nicht wieder aufgerufen.
• Cheater kommt wieder in den Server und erhält nun völlig legal die 100.100€ vom Server.

Daher: Timer, und zwar einen der alle paar Sekunden abfragt, so wird die Chance minimiert, dass bei einem Server-Absturz oder Neustart ein Cheater sich durchmogelt. Und bei OnPlayerDisconnect unbedingt auch überprüfen.

Zitat von [COE]CodeX

Warum funktioniert der folgende Code nicht?
Aber ich hätte trotzdem gerne eine Antwort^^

Ganz einfach. Wie dir die Error-Ausgabe schon sagt, ein Array muss mit einer konstanten Größe deklariert werden. Du weist dem Array eine variable Größe zu (len), daher weiß der Kompiler nicht, wie viel Platz er reservieren muss. Arrays müssen immer konstant deklariert werden, das heißt mit "#define" oder einer Zahl, wobei du bei define eben ein Wort diese Zahl ist und darstellt.

Zitat von xMr.Lix

ich suche ein Editor,womit man zb eine kleine Sirene also Undercover Sirene ans auto machen kann.

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

CreateVehicle(522,x,y,z,rott,0,0,0);
PutPlayerInVehicle(playerid,522,0);
muss so sein:

new id = CreateVehicle(522,x,y,z,rott,0,0,0);
PutPlayerInVehicle(playerid,id,0);

stock GiveMoney(playerid,value)
{
SetPVarInt(playerid,"Money",GetPVarInt(playerid,"Money")+value);
Spieler[playerid][pMoney]+=value;
return value;
}

Es ist += und nicht =+.

http://wiki.sa-mp.com/wiki/GetPlayerAnimationIndex
Hiermit. Du setzt es unter OnPlayerUpdate um herauszufinden welche Animations ID du brauchst. Dann kannst du es wieder entfernen, und abfragen.

printf("Index: %d", GetPlayerAnimationIndex(playerid)); //Nicht vergessen wieder weg zu machen

if(GetPlayerAnimationIndex(playerid) == 1234) //Irgendwas...
(1234 ist ein Beispiel)

new id = CreateVehicle(model, X, Y, Z, A, -1, -1, -1);
PutPlayerInVehicle(playerid, id, 0); //0 = Fahrer

Zitat von heyhooo

Also das heißt auf gut Deutsch ich mülle dann meinen Server mit Textdraws zu... oder?

Ja, da image2textdraw eben für jeden Pixel ein Textdraw erstellt. Daher solltest bzw. kannst du ja auch nur kleine Bilder nehmen.
Du bekommst ja auch eine Warnung wenn du ein Bild mit mehr als 2048 Pixel konvertieren willst.

Enums so bekanntermaßen schneller als PVars, das spielt aber kaum eine Rolle bei dem was du machen willst. Daher nimm das, was dir am liebsten ist.

Zitat von Maximilian

Nunja, was möchtest du noch mit der Variable wenn du gar nicht mehr Online bist?

Ein neuer Spieler mit der selben ID wird ja sicher bald den Server betreten, in dem Fall wäre die Variable angenommen vom Spieler der das Spiel verlassen hat noch "12345", gibst du dem neuen Spieler dann sein Geld, so hat die Variable 12345 mehr an Wert, was nach deinem System mit != zu einem Ban führt.

Variablen IMMER bei OnPlayerDisconnect resetten, sofern sie für den Spieler bestimmt sind, sie resetten sich NIE von alleine (außer du startest den Server neu).

Zitat von Blu3scReeN

Jeffry: Wieso heißt sein stock denn nicht GetGiveMoney(playerid, value), denn sein stock gibt etwas zurück :o

CreateVehicle gibt dir ja auch was zurück (Vehicle ID) und heißt deswegen nicht "GetCreateVehicle", oder?
Wie er die Funktion nennt, und was er zurück gibt ist irrelevant, solange er durchblickt. Er kann es auch "IchMagSchokodonuts" nennen.
Siehe dazu: http://forum.sa-mp.com/showthread.php?t=255246

reoarni:
WENN, dann so:
Spieler[playerid][pMoney]+=value;

Zitat von Maximilian

Wenn der Spieler den Server verlässt wird die Variable im normalfall sowieso resetet.

Nö. Nur wenn du es auch angibst. Von selber geht nichts.
In dem Fall bei OnPlayerDisconnect: pMoney[playerid] = 0;

Hab ich dich richtig verstanden, dass du 348 commands hast, in deiner .pwn Datei. Diese willst du nun in einer Liste (o.ä.) ausgegeben haben?
Das geht eigentlich ganz einfach, wenn du dich mit filefunctions auskennst:

• .pwn in /scriptfiles/ tun.
• Per Script die datei auslesen (io_read) und nach dem ZCMD definer (CMD:) suchen (strfind).
• Wird der gefunden, den string splitten, einmal am : und einmal am ( und schon kannst du den Command ausgeben, z.B. in einer Textdatei.

Wobei 348 commands ja auch nicht die Welt ist, die kannst du in 3-4 Stunden kurz selbst raustippen.

EDIT: Hami: Oder das, wenn es tut. Wäre noch besser.

Das stock an sich ist schon falsch, der Compiler wird da sicher rummäckern.

stock GiveMoney(playerid,value)
{
SetPVarInt(playerid,"Money",GetPVarInt(playerid,"Money")+value);
Spieler[playerid][pMoney]+value; //Das hier hat keinen Sinn, und auch keinen Effekt.
return value;
}

Blu3scReeN: Die Toleranzgrenze sollte dann über 36.000.000 sein, da ich bei einem Einsatz von 1.000.000 im Casino auf die 0 beim Roulette 36.000.000 gewinnen kann. Zugegebenermaßen etwas unrealistisch, aber es kann doch gut sein, dass der Spieler ein paar Hunderttausend gewinnt.

reoarni: Du kannst auch ein ganz auf Server Variablen basierendes Geld System machen, dabei würden allerdings sämtliche genannten Funktionen (Casino, Ammu Nation...) ihren Sinn verlieren, da man kostenlos sich deren bedienen kann. Je nach GM wäre das sinnvoll, oder auch nicht.

Zitat von Maximilian

So sieht ein Anti Money Hack aus

Also da muss ich doch mal einlenken. So sieht ein Anti Money Hack hoffentlich bei niemanden auch nur annähernd aus. Warum? Erklärungen folgen:

• pMoney[playerid] = money; <= Falsch. Alleine der Aufruf deiner GiveUserMoney Funktion würde den Spieler schon bannen.
• Es fehlen sämtliche Variable-Resets, wenn der Spieler den Server verlässt.
• SetTimer("CheckMoneyHack", true), <= Alleine hier sind schon 2 Fehler. Zum einen Fehlt die Abfragezeit, also das Intervall, zum anderen gehört da ein Semikolon hin, und kein Komma.
• Du beachtest keinen Lag. Spieler mit sehr hoher Ping können gebannt werden, wenn die Variable bereits hoch gesetzt wurde, der Befehl aber noch nicht beim Spieler Client ankam. Das ist tatsächlich der Fall, hatte das bei meinem Anti Money Hack System.
• Außerdem fehlt dir jeglicher Ausschluss von Desynchronisation. Was ist das? Geh mal in einen Server, und dann lade eine größere Datei hoch, je nach Internetverbindung bist du danach im Server nicht mehr synchronisiert, was folglich zu einem Ban führt, sobald sich dein Geld per Aufruf ändert.
• Es fehlen sämtliche Ausschlüsse, dazu zählen:
  
  
  • Snack Maschinen
  • Ammu Nation
  • Stunt Bonus
  • Tune Garagen
  • Pay'n'Spray
  • Casino
  • (...)

  
  Da du abfragst ob GetPlayerMoney nicht der Variable entspricht, wirst du, sofern der Timer richtig gesetzt wird (siehe (1)) sofort gebannt, solltest du eine der obigen Funktionen nutzen. Das ist sehr schlecht.
  

Es gibt noch weitere Kriterien die unbedingt beachtet werden sollten, um falsche Bans auszuschließen. Ich habe über 1 Jahr lang mein Anti Money Hack immer wieder verbessert und mit diesen Kriterien ausgestattet. Also so was ist nicht in 5 Minuten gemacht.

Zitat von reoarni

Wie denn? xD Wenn man im Script trotzdem GivePlayerMoney verwendet wie im GF

Hooking ist das Stichwort.
Siehe: http://forum.sa-mp.com/showthread.php?t=187195

@NicoWiss: Danke für das Kompliment
|-|auke: Deine Veränderung der Funktion ändert nichts, aber der Hinweis auf die 0 terminierenden Zeichen war Gold Wert!
@BlackAce: Das würde allerdings nicht den gewünschten Effekt bringen. Erst durch das Ersetzen von sizeof() mit 128 wird der 129. Character nicht mehr beschrieben, und bildet somit das Ende.

new string[478], salt[129], key[129], File:saltfile;
randomString(salt, 128);
printf("%s (%i)",salt, strlen(salt)); //Länge ist 128
randomString(key, 128);
printf("%s (%i)",key,strlen(key)); //Länge ist 256????!

Man könnte es (als Alternative) auch so machen:
new tmp[259];
format(tmp, 259, "%s\r\n%s", salt, key);
format(string, 100, "%s.txt", SpielerName(playerid));
saltfile = fopen(string, io_write);
if(saltfile)
{
fwrite(saltfile, tmp);
fclose(saltfile);
}

Das ist aber bei Weitem nicht so "schön" wie die erste Variante.

Beide Varianten wurden kompiliert und getestet.

Hast du einen vierdimensionalen Array gemacht? Also ich hatte die Arrays kompiliert, hat getan.
Zeig mal den Code wie du ihn jetzt hast.

stock IsVehicleNearPlayer(playerid, vehicleid, Float:range) //©Jeffry
{
new Float:x, Float:y, Float:z;
GetPlayerPos(playerid,x,y,z);
new Float:fDistance = GetVehicleDistanceFromPoint(vehicle, x,y,z);
if(fDistance > range) return 0;
else return 1;
}

Nur mal kurz zusammengetippt. Sollte aber funktionieren.

Ich erklär dir das mal kurz, warum das so ist, sonst weißt du es beim nächsten Mal wieder nicht.^^

new Float:DtcParkPlatz[7][4][4] = {
new Float:DtcSonstiges[7][5][4] = {
Dies wären die richtigen Angaben (bzw. so wie Kaliber es gemacht hat, wobei ich das da besser finde, es ist übersichtlicher und verhindert Fehler in großen Arrays, meine Meinung).

Nun, warum?

DtcSonstiges:
[7] bedeutet, dass wir 7 Abteile haben
[5] gibt nun an, dass jeder Abteil 5 Zeilen hat
[4] sagt schließlich, dass jede Zeile 4 Parameter hat.

Diese Zahlen müssen immer so stimmen, sonst bekommst du den Error 052.

Zitat

error 052: multi-dimensional arrays must be fully initialized

Das heißt übersetzt: Multidimensionale Arrays (in deinem Fall: 3 Dimensionen) müssen vollständig eingeleitet werden. => Es darf keine "Lücke" vorhanden sein, es muss alles gefüllt sein. Es müssen 7 Abteile vorhanden sein, jedes diese Abteile muss genau 5 Zeilen haben, wovon jede wiederum genau 4 Parameter hat.

new Float:DtcSonstiges[AUTOHAUSER][5][7] = {

zu:
new Float:DtcSonstiges[AUTOHAUSER][5][4] = {

Falls noch weitere Fehler sind, bzw der Fehler noch besteht, poste bitte das gesamte Array.