Beiträge von Kaliber

Zitat von roOlling

for(new fraks=1;fraks<MAX_FRAKS;fraks++) // new fraks = 1; evtl. das was Kaliber meint?

Nein, das meine ich nicht, das ist aber auch nicht ganz richtig

Muss so aussehen:

for(new fraks=1;fraks<=MAX_FRAKS;fraks++)
Hat aber nichts mit dem Fehler zu tun

mfg.

Keine Ahnung wo du da im Script diesen Fehler gemacht hast

Wahrscheinlich irgentwo ein = 1; oder sowas

mfg.

Zitat von PR3MiiUM

Hm..den delete befehl habe ich auch noch nicht

Einfach sowas machen:

if(!strcmp(cmdtext, "/delveh", true))
{
DestroyVehicle(car[playerid]);
return 1;
}

Zitat von Deather

Schaut mal:
if(strcmp(cmdtext, "/destroyveh", true) == 0 || strcmp(cmdtext,"/dveh",true) == 0)
{
if(SpielerInfo[playerid][Adminlevel] >= 5)
{
new currentveh;
currentveh = GetPlayerVehicleID(playerid);
DestroyVehicle(currentveh);
return 1;
}
return 1;
}

Ist doch nicht schwer zu machen O.o

Alles anzeigen

Nein, so geht das nicht, dann würde er nämlich jedes Auto löschen in dem er drinnen sitzt und nicht nur das, was er mit /veh erstellt hat

mfg.

So:

//oben im Script
new car[MAX_PLAYERS];

//in deinem /veh befehl
car[playerid] = CreateVehicle(...);

//in deinem Delete Befehl
DestroyVehicle(car[playerid]);

mfg.

Zitat von PR3MiiUM

Ja, verschieb es oder map dir eins. Ort kannst du ja entscheiden.

Okay, ich frage nocheinmal: Was hat der Ort mit der Funktionalität des Filterscriptes zu tun

Zitat von Arendium

Das Kommische ist beim Printen in der Lade-Funktion KLAPPT es ja... Wenn du mal mit TeamViewer drüberschauen willst
wäre das Kein Problem... falls das weiterhilft

Ne, sry sitze an der Arbeit, darf das nicht

Ehm...wo funktioniert es denn dann nicht

Den Warning habe ich kassiert, weil ich die Admins getrollt habe, bereue aber nichts

und zu dem Code, mache es so wie ich es gemacht habe

mfg.

Zitat von Arendium

Kaliber: NOCH Manuell da ja erstmal die Lade-Funktion tun muss soll ich es kurz Dynamisch mit einem Testcar machen ?

Nee...ist okay..

aber wenn du die Fraktionsid selber da einträgst würde das doch bedeuten, dass du sie entweder falsch einträgst oder er sie falsch ausliest oder ?

Setzt du die Fraktionsid in der .txt Datei manuell oder lässt du die Dynamisch im Script einlesen ?

mfg.

Hast du auch alle news oben im Script angepasst, die müssten jetzt so aussehen:

new Float:zX[MAX_PLAYERS], Float:zY[MAX_PLAYERS], Float:zZ[MAX_PLAYERS], zID[MAX_PLAYERS];

//achso und bei OnPlayerConnect
zID[playerid] = -1;

mfg.

Zitat von Acrio

SetPlayerCheckpoint(playerid, Float:zX[zID], Float:zY[zID[i]], Float:zZ[zID[i]], 5.0);

Nein, nicht so !!!

So wie ich es gemacht habe, der Compiler weiß doch bereits, dass zX ein Float ist, da wir es so deklariert haben

Zitat von Acrio

zID = playerid;

So:

zID[playerid] = playerid;//sieht zwar etwas komisch aus, ist aber richtig

mfg.

Zitat von KeViN √

/Save sind ganz anders.

Das kann nicht sein, es sei denn du stehst an einer anderen Position

mfg.

Zitat von luca26011

Liegt an dem Hoster pc.
Zu schlechte Upload rate

So ein Quatsch

Du solltest das nicht alles unter OnPlayerUpdate packen !

Erstelle einfach einen Timer oder packe die einzelnen Funktionen in die dafür angefertigten Callbacks !

PS: Nicht der Server laggt sondern die Spieler

mfg.

Zitat von KeViN √

Es gibt eine property.cfg dort müssen die Haus Daten eingetragen werden.

Das ist so aufgebaut:
-2639.734375,168.677398,7.195300,1.200000,-3.400000,999.400024,0,0,0,0,0,0,Zu Verkaufen,Trailer,200000,0,0,2,1,0,0,50,1,2045,542,-1,-1,114,3

Schau doch mal hier: [ SCRIPTING ] [GODFATHER] sbizz/bizz/property.cfg

mfg.

Ja, das ist noch nicht korrekt

So muss das ausschauen:

if(!strcmp(ops, "medic", true))
{
if(pInfo[playerid][pFrak] == FRAK_MEDIC)
{
if(BereitsNotruf[playerid] == 0)
{
for(new i; i < MAX_PLAYERS; i++)
{
if(zID[i] != -1)
{
SetPlayerCheckpoint(playerid, zX[zID[i]], zY[zID[i]], zZ[zID[i]], 5.0);
new zName[24],
zstr[125];
GetPlayerName(zID[i], zName, 24);
format(str, sizeof(str), "Du hast den Notruf von %s angenommen", zName);
SendClientMessage(playerid, COLOR_LIGHTBLUE, str);
GetPlayerName(playerid, name, sizeof(name));
format(zstr, sizeof(zstr), "%s hat deinen Notruf angenommen. Warte an deiner jetzigen Position bis er eintrift.", name);
SendClientMessage(zID[i], COLOR_LIGHTBLUE, zstr);
BereitsNotruf[i] = 1;
zID[i] = -1;
break;
}
}
}
}
}

mfg.

Unfähig selber zu suchen ?

2sek: [ SCRIPTING ] MySQL (Installation, Zugriff, Einstellungen, Login & Register Beispiel)

mfg.

Wenn wir schon mal dabei sind habe ich direkt noch ein paar Tipps

Um ein bisschen Ressourcen zu sparen kann man Variablen anstatt so:

new SkinAuswahl[MAX_PLAYERS];
new Adminlogin[MAX_PLAYERS];
new Float:Gotox[MAX_PLAYERS];
new Float:Gotoy[MAX_PLAYERS];
new Float:Gotoz[MAX_PLAYERS];

besser so deklarieren:

new SkinAuswahl[MAX_PLAYERS],
Adminlogin[MAX_PLAYERS],
Float:Gotox[MAX_PLAYERS],
Float:Gotoy[MAX_PLAYERS],
Float:Gotoz[MAX_PLAYERS];

Leere Callbacks am besten einfach rausschmeißen...

Sachen wie sowas:

if(Adminlogin[playerid] == 0)
{
SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
return 1;
}

Kann man auf sowas kürzen:

if(Adminlogin[playerid] == 0) return SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");

Aber ansonsten sieht das Script eigentlich ganz gut aus, 8.5/10 !

mfg.

//Edit: Rechtschreibung

Würde mich über weitere Feedbacks zum System (nicht über den Ort wo ich die Cars jetzt hingestellt habe) freuen

mfg.

Hätte da ein paar Verbesserungsvorschläge:

Anstatt sowas:

dcmd_cleartext(playerid,params[])
{
#pragma unused params
new String[128];
if(PlayerInfo[playerid][pAdministrator] >= 2)
{
if(Adminlogin[playerid] == 0)
{
SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
return 1;
}
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
SendClientMessageToAll(Weiß," ");
format(String,sizeof(String),"((Administrator: %s hat den Chatlog gecleart))",SpielerName(playerid));
SendClientMessageToAll(Rot,String);
return 1;
}
else
{
SendClientMessage(playerid,Rot,"Du bist kein Administrator!");
}
return 1;
}

einfach sowas:

dcmd_cleartext(playerid,params[])
{
#pragma unused params
new String[128];
if(PlayerInfo[playerid][pAdministrator] >= 2)
{
if(Adminlogin[playerid] == 0)
{
SendClientMessage(playerid,Rot,"Du bist nicht als Administrator Eingeloggt!");
return 1;
}
for(new i; i<24; i++) SendClientMessageToAll(Weiß, " ");
format(String,sizeof(String),"((Administrator: %s hat den Chatlog gecleart))",SpielerName(playerid));
SendClientMessageToAll(Rot,String);
return 1;
}
else
{
SendClientMessage(playerid,Rot,"Du bist kein Administrator!");
}
return 1;
}

und Quini, die mysql_query Systeme von Maddin sind nicht die besten, da er jede Variable einzeln in die Datenbank einträgt...das geht mit einer längeren query viel besser

mfg.

Dann liegt da dein Fehler

mfg.