[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 28. Mai 2015

Zitat von Jeffry

Wäre der String gelöscht, wäre das "u" nicht mehr da.

...zum 3.x ja ich denke jetzt hat es jeder verstanden, da aber das EOS verschoben wird, existiert das "u" zwar im Speicher, aber gehört nicht direkt mehr zu dem String

...es war nur eine Ergänzung, dass man so strings für die Funktionen quasi "leeren" kann...

Akino Kiritani · 28. Mai 2015

Kaliber: benutzt du das irgendwo in deinen Scripten ? O.o

Jeffry · 28. Mai 2015

Zitat von Kaliber

...es war nur eine Ergänzung, dass man so strings für die Funktionen quasi "leeren" kann...

Die Variable ist auch nicht "quasi" leer, sie ist gar nicht leer. Man nutzt nur einen Fehler, wie BlackAce sagte. Jetzt stell dir mal vor (die Vorstellung ist bei SA-MP zwar etwas absurd, aber generell) du hast in der Variable sensible Daten, von mir aus eine Kontonummer. Du "leerst" nun die Variable nach deiner Methode, es folgt etwas Code und dann übergibst du den Wert der Variable (wohlgemerkt den neuen) an eine andere Variable und nehmen wir mal an das läuft über eine Schnittstelle in ein anderes System, vielleicht extern. Derjenige Programmierer bekommt dadurch dann möglicherweise Zugriff auf die Kontonummer, wenn du Pech hast, sie steht ja schließlich (wie das "u") noch in der Variable drin. Ich denke ich muss das nicht weiterführen, es ist klar auf was das hinauslaufen kann, wenn man sich solcher vermeintlich nützlichen Dinge betätigt.

Außerdem wäre eine direkte Zuweisung (x = "Noob";) wesentlich performanter und hat das gleiche Ergebnis (mal von obiger Hypothese abgesehen, da das Problem dort auch existiert).
Willst du das Problem wirklich umgehen und den Speicher wirksam leeren (sodass kein sinnvoller Zugriff mehr darauf besteht), musst du ihn entweder überschreiben oder memcpy verwenden (was übrigens gleich schnell ist aber nur eine Zeile Code benötigt).

Eddy · 29. Mai 2015

Wieso bekomme ich keine Nachricht, das ich in keinem Fahrzeug sitze?
Und wieso bekomme ich beim LichtAn nur die Nachricht, SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in einem Fahrzeug,Boot,Flugzeug, welches kein Licht hat!");

stock StartMotor(playerid)
{
if(GetPlayerVehicleSeat(playerid) == 0)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um den Motor zu starten!");
new car = GetPlayerVehicleID(playerid),m = GetVehicleModel(car);
if(m == 509 || m == 481 || m == 510) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt auf einem Fahrrad, das hat keinen Motor!");
else if (Motor[car] == false)
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,VEHICLE_PARAMS_ON,lights,alarm,doors,bonnet,boot,objective);
Motor[car] = true;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {36D61A}an!");
return 1;
}
else
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[car] = false;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {E83A3D}aus!");
return 1;
}
}
return 1;
}

stock LichtAn(playerid)
{
if(GetPlayerVehicleSeat(playerid) == 0)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um das Licht einzuschalten!");
new car = GetPlayerVehicleID(playerid),m = GetVehicleModel(car);
if(m == 417 || 425 || m == 447 || m == 464 || m == 465 || m == 469 || m == 487 || m == 488 || m == 497 || m == 501 || m == 548 || m == 563 || m == 430 || m == 446 || m == 452 || m == 453 || m == 454 || m == 472 || m == 473 || m == 484 || m == 493 || m == 595 ||
m == 460 || m == 476 || m == 511 || m == 512 || m == 513 || m == 519 || m == 520 || m == 553 || m == 577 || m == 592 || m == 593) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in einem Fahrzeug,Boot,Flugzeug, welches kein Licht hat!");
if (Licht[car] == false)
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,engine,VEHICLE_PARAMS_ON,alarm,doors,bonnet,boot,objective);
Licht[car] = true;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {36D61A}an!");
return 1;
}
else
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,engine,VEHICLE_PARAMS_OFF,alarm,doors,bonnet,boot,objective);
Licht[car] = false;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {E83A3D}aus!");
return 1;
}
}
return 1;
}

xGreekz7x · 29. Mai 2015

Zitat von Eddy

Wieso bekomme ich keine Nachricht, das ich in keinem Fahrzeug sitze?
Und wieso bekomme ich beim LichtAn nur die Nachricht, SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in einem Fahrzeug,Boot,Flugzeug, welches kein Licht hat!");

stock StartMotor(playerid)
{
if(GetPlayerVehicleSeat(playerid) == 0)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um den Motor zu starten!");
new car = GetPlayerVehicleID(playerid),m = GetVehicleModel(car);
if(m == 509 || m == 481 || m == 510) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt auf einem Fahrrad, das hat keinen Motor!");
else if (Motor[car] == false)
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,VEHICLE_PARAMS_ON,lights,alarm,doors,bonnet,boot,objective);
Motor[car] = true;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {36D61A}an!");
return 1;
}
else
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Motor[car] = false;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {E83A3D}aus!");
return 1;
}
}
return 1;
}

stock LichtAn(playerid)
{
if(GetPlayerVehicleSeat(playerid) == 0)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um das Licht einzuschalten!");
new car = GetPlayerVehicleID(playerid),m = GetVehicleModel(car);
if(m == 417 || 425 || m == 447 || m == 464 || m == 465 || m == 469 || m == 487 || m == 488 || m == 497 || m == 501 || m == 548 || m == 563 || m == 430 || m == 446 || m == 452 || m == 453 || m == 454 || m == 472 || m == 473 || m == 484 || m == 493 || m == 595 ||
m == 460 || m == 476 || m == 511 || m == 512 || m == 513 || m == 519 || m == 520 || m == 553 || m == 577 || m == 592 || m == 593) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in einem Fahrzeug,Boot,Flugzeug, welches kein Licht hat!");
if (Licht[car] == false)
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,engine,VEHICLE_PARAMS_ON,alarm,doors,bonnet,boot,objective);
Licht[car] = true;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {36D61A}an!");
return 1;
}
else
{
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(car,engine,VEHICLE_PARAMS_OFF,alarm,doors,bonnet,boot,objective);
Licht[car] = false;
SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {E83A3D}aus!");
return 1;
}
}
return 1;
}

Alles anzeigen

Probier es mal so:

stock StartMotor(playerid)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um den Motor zu starten!");
if(GetPlayerVehicleSeat(playerid) == 0)
{

new car = GetPlayerVehicleID(playerid),
m = GetVehicleModel(car);

if(m == 509 || m == 481 || m == 510) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt auf einem Fahrrad, das hat keinen Motor!");
GetVehicleParamsEx(car,engine,lights,alarm,doors,bonnet,boot,objective);
Motor[car] = !Motor[car];
SetVehicleParamsEx(car, Motor[car],lights,alarm,doors,bonnet,boot,objective);

if (Motor[car] == true)
return SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {36D61A}an!");
else
return SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen den Zündschlüssel, und schalten den Motor {E83A3D}aus!");

}
return 1;
}

stock LichtAn(playerid)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in keinem Fahrzeug, um das Licht einzuschalten!");
if(GetPlayerVehicleSeat(playerid) == 0)
{
new car = GetPlayerVehicleID(playerid),
m = GetVehicleModel(car);

if(m == 417 || 425 || m == 447 || m == 464 || m == 465 || m == 469 || m == 487 || m == 488 || m == 497 || m == 501 || m == 548 || m == 563 || m == 430 || m == 446 || m == 452 || m == 453 || m == 454 || m == 472 || m == 473 || m == 484 || m == 493 || m == 595 ||
m == 460 || m == 476 || m == 511 || m == 512 || m == 513 || m == 519 || m == 520 || m == 553 || m == 577 || m == 592 || m == 593) return SendClientMessage(playerid, 0x9B0000FF, "Du sitzt in einem Fahrzeug,Boot,Flugzeug, welches kein Licht hat!");

GetVehicleParamsEx(car, engine, lights, alarm, doors, bonnet, boot, objective);
Licht[car] = !Licht[car];
SetVehicleParamsEx(car, engine, Licht[car], alarm, doors, bonnet, boot, objective);

if (Licht[car] == true)
return SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {36D61A}an!");
else
return SendClientMessage(playerid, 0x4B94FFFF, "{1AADD6}Gedanke: {F2F2F2}Sie drehen am Schalter, und schalten das Licht {E83A3D}aus!");

}
return 1;
}

Aber du müsstest uns auch noch sagen, welches Fahrzeug du gefahren bist als das mit dem Licht kam.

Kaliber · 29. Mai 2015

Zitat von Jeffry

Außerdem wäre eine direkte Zuweisung (x = "Noob";) wesentlich performanter

Achwas...aber man kann halt nicht immer direkt den Wert der Variable wissen und wenn schon was drinnen steht...ist es schneller einfach das NULL-Byte zu verschieben als format zu verwenden..Bei lokalen Variablen spielt das eh keine Rolle, da die nach verlassen des jeweiligen Scopes gecleart werden...

Zitat von Jeffry

memcpy verwenden (was übrigens gleich schnell ist aber nur eine Zeile Code benötigt).

memcpy ist eine alternative zu strcat, aber die "löscht" dir auch nicht den string, sondern kopiert nur einen anderen String quasi darein.

Jeffry · 29. Mai 2015

Wie oft willst du das noch sagen? Das hab ich im ersten Post bereits dargestellt, daher verstehe ich nicht warum du so ein Drama darum machst. Natürlich ist das so, aber es ist nicht die beste Möglichkeit, daher ist sie, vor allem hinsichtlich dem was xMichael wissen wollte, nutzlos.

Zitat von Kaliber

memcpy ist eine alternative zu strcat, aber die "löscht" dir auch nicht den string, sondern kopiert nur einen anderen String quasi darein.

Wenn man es richtig macht ist der Rest dahinter auch weg, bzw nicht mehr als das erkennbar was es mal war.
new x[] = "BlaBlub";
memcpy(x, "Noob", 0, 32);

Exqoolz · 29. Mai 2015

Sobald ich ein Fraktionsfahrzeug erstelle, crashe ich, also man kann keine Befehle mehr eingeben - nichts, aber Server ist noch an!
Die Erstellmeldung kommt nicht.

Weiß jemand evtl. warum?

http://pastebin.com/q4hgLjHF

#edit: nach kurzer Zeit Server komplett down.

fvckbitchesgetmoney · 29. Mai 2015

Poste mal den log.

Exqoolz · 29. Mai 2015

Zitat von Adrenalinx

Poste mal den log.

Fehler doch nicht gefunden. Im Serverlog ist nichts.

MySQL-Log:

Code

[14:42:02] [DEBUG] mysql_format - connection: 1, len: 256, format: "INSERT INTO `script_cmdlog` (cmd, name) VALUES ('%s', '%s')"
[14:42:02] [DEBUG] mysql_tquery - connection: 1, query: "INSERT INTO `script_cmdlog` (cmd, name) VALUES ('/addfcar', 'Exq", callback: "(null)", format: "(null)"
[14:42:02] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:02] [DEBUG] mysql_tquery - scheduling query "INSERT INTO `script_cmdlog` (cmd, name) VALUES ('/addfcar', 'Exqool')"..
[14:42:02] [DEBUG] CMySQLQuery::Execute[()] - starting query execution
[14:42:02] [DEBUG] CMySQLQuery::Execute[()] - query was successful
[14:42:02] [DEBUG] CMySQLQuery::Execute[()] - no callback specified, skipping result saving
[14:42:02] [DEBUG] CMySQLQuery::Execute[()] - data being passed to ProcessCallbacks()
[14:42:02] [DEBUG] CMySQLQuery::~CMySQLQuery() - deconstructor called
[14:42:05] [DEBUG] mysql_format - connection: 1, len: 512, format: "INSERT INTO `script_fvehicles` (`vehFraktion`, `modelid`, `vehPosX`, `vehPosY`, `vehPosZ`, `vehPosA`, `vehInterior`, `vehWorld`,..."
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "INSERT INTO `script_fvehicles` (`vehFraktion`, `modelid`, `vehPo", callback: "insertDB", format: "d"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:05] [DEBUG] mysql_tquery - scheduling query "INSERT INTO `script_fvehicles` (`vehFraktion`, `modelid`, `vehPosX`, `vehPosY`, `vehPosZ`, `vehPosA`, `vehInterior`, `vehWorld`, `vehHealth`, `vehNumberPlate`, `vehPaintjob`, `vehPreis`, `vehColor1`, `vehColor2`, `vehNeonID`, `vehFuel`) VALUES ('1', '596', '1528.869263', '-1679.037598', '5.890625', '243.600937', '0', '0', '1000.000000', '{B22222}LSPD 72', '-1', '0', '0', '1', '0', '1115815936')"..
[14:42:05] [DEBUG] CMySQLQuery::Execute[insertDB(d)] - starting query execution
[14:42:05] [DEBUG] CMySQLQuery::Execute[insertDB(d)] - query was successful
[14:42:05] [DEBUG] CMySQLResult::CMySQLResult() - constructor called
[14:42:05] [DEBUG] CMySQLQuery::Execute[insertDB(d)] - data being passed to ProcessCallbacks()
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "SELECT * FROM `script_fvehicles`", callback: "LoadFVehicles", format: "(null)"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:05] [DEBUG] mysql_tquery - scheduling query "SELECT * FROM `script_fvehicles`"..
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "SELECT * FROM `script_fvehicles`", callback: "LoadFVehicles", format: "(null)"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:05] [DEBUG] mysql_tquery - scheduling query "SELECT * FROM `script_fvehicles`"..
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "SELECT * FROM `script_fvehicles`", callback: "LoadFVehicles", format: "(null)"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:05] [DEBUG] mysql_tquery - scheduling query "SELECT * FROM `script_fvehicles`"..
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "SELECT * FROM `script_fvehicles`", callback: "LoadFVehicles", format: "(null)"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called
[14:42:05] [DEBUG] mysql_tquery - scheduling query "SELECT * FROM `script_fvehicles`"..
[14:42:05] [DEBUG] mysql_tquery - connection: 1, query: "SELECT * FROM `script_fvehicles`", callback: "LoadFVehicles", format: "(null)"
[14:42:05] [DEBUG] CMySQLQuery::CMySQLQuery() - constructor called

Alles anzeigen

Kaliber · 29. Mai 2015

Zitat von Exqoolz

Im Serverlog ist nichts.

Dann verwende mal das crashdetect Plugin

tayson87 · 29. Mai 2015

Zitat von tayson87

Nach einem gangfightkill sollte man gespawnt werden aber klappt irgendwie nicht

//Gangwarsystem
if(IsPlayerConnected(killerid) && killerid != INVALID_PLAYER_ID)
{
if(IsAFightFaction(GetPlayerFaction(playerid)))
{
new TempZone = GetPlayerFightzone(playerid);
if(GangwarZones[TempZone][GW_AttackFaction] != -255 && GangwarZones[TempZone][GW_FightTime])
{
if(GangwarZones[TempZone][GW_CtfOfOwner][0] == playerid)
{
new fsName[MAX_PLAYER_NAME];
GetPlayerName(playerid,fsName,MAX_PLAYER_NAME);
new fsText[128];
format(fsText,sizeof(fsText),"Gangfight: %s hat die Flagge verloren! (Gestorben)",fsName);
for(new i;i<GetMaxPlayers();i++)
{
if(IsPlayerConnected(i))
{
if(GetPlayerFaction(i) == GangwarZones[TempZone][GW_OwnerFaction] || GetPlayerFaction(i) == GangwarZones[TempZone][GW_AttackFaction])
{
PlayerPlaySound(i,1058,0.0,0.0,0.0);
SendClientMessage(i,GW_COLOR_ORANGE,fsText);
RemovePlayerAttachedObject(i,0);
}
}
}
DestroyPickup(GangwarZones[TempZone][GW_FlagOfOwner]);
GangwarZones[TempZone][GW_FlagOfOwner] = CreatePickup(2993,23,GangwarZones[TempZone][GW_CtfOfOwnerx],GangwarZones[TempZone][GW_CtfOfOwnery],GangwarZones[TempZone][GW_CtfOfOwnerz],-1);
GangwarZones[TempZone][GW_CtfOfOwner][0] = -255;
GangwarZones[TempZone][GW_CtfOfOwner][1] = 0;
format(string,sizeof(string),"%s\nAttacker Flagge\nGebiet der Fraktion '%s'\n/startwar",GangwarZones[TempZone][GW_Zone_Name],GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
Update3DTextLabelText(WarLabel[TempZone][0],0x009A1EB2,string);
}
else if(GangwarZones[TempZone][GW_CtfOfAttack][0] == playerid)
{
new fsName[MAX_PLAYER_NAME];
GetPlayerName(playerid,fsName,MAX_PLAYER_NAME);
new fsText[128];
format(fsText,sizeof(fsText),"Gangfight: %s hat die Flagge verloren! (Gestorben)",fsName);
for(new i;i<GetMaxPlayers();i++)
{
if(IsPlayerConnected(i))
{
if(GetPlayerFaction(i) == GangwarZones[TempZone][GW_OwnerFaction] || GetPlayerFaction(i) == GangwarZones[TempZone][GW_AttackFaction])
{
PlayerPlaySound(i,1058,0.0,0.0,0.0);
SendClientMessage(i,GW_COLOR_ORANGE,fsText);
RemovePlayerAttachedObject(i,0);
}
}
}
DestroyPickup(GangwarZones[TempZone][GW_FlagOfAttack]);
GangwarZones[TempZone][GW_FlagOfAttack] = CreatePickup(2993,23,GangwarZones[TempZone][GW_CtfOfAttackx],GangwarZones[TempZone][GW_CtfOfAttacky],GangwarZones[TempZone][GW_CtfOfAttackz],-1);
GangwarZones[TempZone][GW_CtfOfAttack][0] = -255;
GangwarZones[TempZone][GW_CtfOfAttack][1] = 0;
format(string,sizeof(string),"%s\nDeffender Flagge\nGebiet der Fraktion '%s'\n/startwar",GangwarZones[TempZone][GW_Zone_Name],GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
Update3DTextLabelText(WarLabel[TempZone][1],0x009A1EB2,string);
}
}
}
if(IsAFightFaction(GetPlayerFaction(killerid)))
{
new TempZone = GetPlayerFightzone(killerid);
if(TempZone != -255)
{
if(GangwarZones[TempZone][GW_FightTime] && GangwarZones[TempZone][GW_AttackFaction] != -255)
{
if(GangwarZones[TempZone][GW_OwnerFaction] == GetPlayerFaction(killerid) || GangwarZones[TempZone][GW_AttackFaction] == GetPlayerFaction(killerid))
{
if(IsAFightFaction(GetPlayerFaction(playerid)))
{
new fsText[64];
if(GangwarZones[TempZone][GW_AttackFaction] == GetPlayerFaction(killerid) && GangwarZones[TempZone][GW_OwnerFaction] == GetPlayerFaction(playerid))
{
PlayerPlaySound(killerid,1084,0.0,0.0,0.0);
GameTextForPlayer(killerid,"~g~GANGWARKILL",6000,5);
GameTextForPlayer(playerid,"~r~GANGWARKILL",6000,5);
GangwarZones[TempZone][GW_PointsOfAttack]++;
Spieler[killerid][pGW_TotalKills]++;
if(GangwarZones[TempZone][GW_PointsOfOwner] > GangwarZones[TempZone][GW_PointsOfAttack])
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
else if(GangwarZones[TempZone][GW_PointsOfAttack] > GangwarZones[TempZone][GW_PointsOfOwner])
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
else
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
}
else if(GangwarZones[TempZone][GW_OwnerFaction] == GetPlayerFaction(killerid) && GangwarZones[TempZone][GW_AttackFaction] == GetPlayerFaction(playerid))
{
PlayerPlaySound(killerid,1084,0.0,0.0,0.0);
GangwarZones[TempZone][GW_PointsOfOwner]++;
if(GangwarZones[TempZone][GW_PointsOfOwner] > GangwarZones[TempZone][GW_PointsOfAttack])
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
else if(GangwarZones[TempZone][GW_PointsOfAttack] > GangwarZones[TempZone][GW_PointsOfOwner])
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
else
{
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_OwnerFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_WinFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfOwner]);
TextDrawSetString(GangwarZones[TempZone][GW_WinPoints],fsText);
format(fsText,sizeof(fsText),"%s",GetFactionNameOfFaction(GangwarZones[TempZone][GW_AttackFaction]));
TextDrawSetString(GangwarZones[TempZone][GW_LoseFaction],fsText);
format(fsText,sizeof(fsText),"[%02d]",GangwarZones[TempZone][GW_PointsOfAttack]);
TextDrawSetString(GangwarZones[TempZone][GW_LosePoints],fsText);
}
}
}
}
}
}
}
SpawnPlayerEx();
}

Alles anzeigen

Kennt wer eine Lösung warum das mit dem Spawnen nicht klappt ?

Akino Kiritani · 29. Mai 2015

Zitat von Kaliber

ist es schneller einfach das NULL-Byte zu verschieben als format zu verwenden

Auf so eine verquere Denkweise und mit ähnlichem Code ist damals Heartbleed entstanden.
Beim nächsten ähnlichen Problem denk ich dann an dich

Exqoolz · 29. Mai 2015

CMD:treasury(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pAdmin] < 6)return SendClientMessage(playerid, COLOR_GREY, "** Du besitzt nicht die benötigten Rechte.");
new enter[128], wert, string[128];
if(sscanf(params,"s",enter))
{
SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
return 1;
}
if(strcmp(enter, "balance", true) == 0)
{
format(string, sizeof(string), "* In der Staatskasse sind %i$.", Others[Staatskasse]);
SendClientMessage(playerid, COLOR_AM, string);
return 1;
}
if(strcmp(enter, "deposit", true) == 0)
{
if(sscanf(params, "si", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat %i$ in die Staatskasse eingezahlt.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] += wert;
GiveMoney(playerid, -wert);
return 1;
}
if(strcmp(enter, "withdraw", true) == 0)
{
if(sscanf(params, "si", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat %i$ von der Staatskasse abgehoben.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] -= wert;
GiveMoney(playerid, wert);
return 1;
}
if(strcmp(enter, "set", true) == 0)
{
if(sscanf(params, "si", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat den Kontostand der Staatskasse auf %i$ gesetzt.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] = wert;
return 1;
}
return 1;
}

Nur /treasury balance geht

Jeffry · 29. Mai 2015

Nutzt du das sscanf Plugin oder die stock Funktion?

Exqoolz · 29. Mai 2015

D

Zitat von Jeffry

Nutzt du das sscanf Plugin oder die stock Funktion?

Das sscanf2 Plugin nutze ich.

Jeffry · 29. Mai 2015

Dann musst du die Größe angeben.
s -> s[32] in sscanf.

Spoiler anzeigen

Exqoolz · 29. Mai 2015

K

Zitat von Jeffry

Dann musst du die Größe angeben.
s -> s[32] in sscanf.

Spoiler anzeigen

CMD:treasury(playerid,params[])
{
if(!IsLoggedIn(playerid))return 1;
if(Spieler[playerid][pAdmin] < 6)return SendClientMessage(playerid, COLOR_GREY, "** Du besitzt nicht die benötigten Rechte.");
new enter[32], wert, string[128];
if(sscanf(params,"s[32]",enter))
{
SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
return 1;
}
if(strcmp(enter, "balance", true) == 0)
{
format(string, sizeof(string), "* In der Staatskasse sind %i$.", Others[Staatskasse]);
SendClientMessage(playerid, COLOR_AM, string);
return 1;
}
if(strcmp(enter, "deposit", true) == 0)
{
if(sscanf(params, "s[32]i", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat %i$ in die Staatskasse eingezahlt.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] += wert;
GiveMoney(playerid, -wert);
return 1;
}
if(strcmp(enter, "withdraw", true) == 0)
{
if(sscanf(params, "s[32]i", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat %i$ von der Staatskasse abgehoben.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] -= wert;
GiveMoney(playerid, wert);
return 1;
}
if(strcmp(enter, "set", true) == 0)
{
if(sscanf(params, "s[32]i", enter, wert))return SendClientMessage(playerid, COLOR_GREY, "** Benutze /treasury [balance/deposit/withdraw/set] [value]");
format(string, sizeof(string), "* %s %s hat den Kontostand der Staatskasse auf %i$ gesetzt.", GetPlayerAdminName(playerid), GetName(playerid), wert);
SendAdminMessage(COLOR_AM, string);
Others[Staatskasse] = wert;
return 1;
}
return 1;
}

Alles anzeigen

Klappt leider nicht

Jeffry · 29. Mai 2015

Bin mir grade nicht sicher ob ich das wieder auf sscanf schieben kann.
Versuch mal folgendes. Ändere das erste
if(sscanf(params,"s[32]",enter))
zu:
if(sscanf(params,"s[32]I(0)",enter,wert))

Beitrag von Exqoolz (29. Mai 2015)