Komischer Fehler,

Sigi. · 12. November 2013

Hallo Community
Ich habe ein kleines problem hier, unzwar wenn ich ein Befehl eingebe, geht der zwar, aber da kommen immer diese Fehlermeldungen,
Also zum beispiel:

/base aufschließen
dann wird die base auch aufgeschlossen ABER!
Ingame wird das angezeigt, "Du darfst dieses Befehl nicht benutzen"
oder "Du hast nicht den benötigten Rank dafür"
könnte mir villt jemand helfen?...

if(strcmp(cmd,"/base",true) == 0) {
new SpielerName[MAX_PLAYER_NAME],tmp[256];
GetPlayerName(playerid,SpielerName,sizeof(SpielerName));
tmp = strtok(cmdtext,idx);
if(!strlen(tmp)) {
SendClientMessage(playerid,COLOR_GREY,"FEHLER: /base [aufschließen/abschließen]");
return 1;
}
if(strcmp(tmp,"aufschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13)){//Yakuza Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(YAKUZAEINGANGAUF == 0){
if(IsPlayerInRangeOfPoint(playerid, 2,-2624.6577,1412.7295,7.0938)){
format(string, sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGAUF = 1;
YAKUZAEINGANGZU = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
if(SpielerInFraktion(playerid,14)){//LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(LCNEINGANGAUF == 0){
if(IsPlayerInRangeOfPoint(playerid,2,1492.6390,748.6784,11.0234)){
format(string,sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
LCNEINGANGAUF = 1;
LCNEINGANGZU = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
}
else if(strcmp(tmp,"abschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13)){//YAKUZA EINGANG
if(SpielerInfo[playerid][pRank] >= 2){
if(YAKUZAEINGANGZU == 0){
if(IsPlayerInRangeOfPoint(playerid, 2, -2624.6577,1412.7295,7.0938)) {
format(string, sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGZU = 1;
YAKUZAEINGANGAUF = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
if(SpielerInFraktion(playerid,14)){ //LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(LCNEINGANGZU == 0){
if(IsPlayerInRangeOfPoint(playerid,2, 1492.6390,748.6784,11.0234)){
format(string,sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid, COLOR_SPECIALBLUE, string);
LCNEINGANGZU = 1;
LCNEINGANGAUF = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");

heyhooo · 12. November 2013

Also ich sehe in diesem Gewirble leider keinen Fehler...
Ich kann dir aber nur ans Herz legen:
- Arbeite umbedingt an deinem Script Stil.
- Benutze ocmd statt strcmp. (einfacher, übersichtlicher) (oder ycmd....ich empfehle ocmd)

L. G.
heyho

Sigi. · 12. November 2013

Hmm, meinst du wenn ich das jetz mit ocmd mache, das es dann funktionieren wird?,..ich glaube nicht

[c]huky115 · 12. November 2013

nene er meint du kannst übersichtlicher scripten mit ocmd als mit den andren Kram da

.HellYeah · 12. November 2013

Nein, aber OCMD ist einfacher zu Handhaben und einfach besser
Send mir mal bitte einen Screenshot von allen Fehlermeldungen (Gesammelt auf einem Bild)

Jeffry · 12. November 2013

Meine Güte. wer programmiert den so. Da erkennt man ja gar nichts mehr.

Ich hab dir das mal ausgebessert. Wenn ich das richtig verstanden habe, was der Code machen sollte, dann waren da paar Dinge an der falschen Stelle, was bei dem Durcheinander auch kein Wunder ist.

Code:

Spoiler anzeigen

Zwecks ocmd: Es ist völliger Schwachsinn zu sagen, dass man damit übersichtlicher scripten kann. Der Code würde mit ocmd, bis auf die erste Zeile, fast IDENTISCH aussehen!!

Sigi. · 12. November 2013

Zitat von Sigi.

Hallo Community
Ich habe ein kleines problem hier, unzwar wenn ich ein Befehl eingebe, geht der zwar, aber da kommen immer diese Fehlermeldungen,
Also zum beispiel:

/base aufschließen
dann wird die base auch aufgeschlossen ABER!
Ingame wird das angezeigt, "Du darfst dieses Befehl nicht benutzen"
oder "Du hast nicht den benötigten Rank dafür"
könnte mir villt jemand helfen?...

if(strcmp(cmd,"/base",true) == 0) {
new SpielerName[MAX_PLAYER_NAME],tmp[256];
GetPlayerName(playerid,SpielerName,sizeof(SpielerName));
tmp = strtok(cmdtext,idx);
if(!strlen(tmp)) {
SendClientMessage(playerid,COLOR_GREY,"FEHLER: /base [aufschließen/abschließen]");
return 1;
}
if(strcmp(tmp,"aufschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13)){//Yakuza Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(YAKUZAEINGANGAUF == 0){
if(IsPlayerInRangeOfPoint(playerid, 2,-2624.6577,1412.7295,7.0938)){
format(string, sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGAUF = 1;
YAKUZAEINGANGZU = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
if(SpielerInFraktion(playerid,14)){//LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(LCNEINGANGAUF == 0){
if(IsPlayerInRangeOfPoint(playerid,2,1492.6390,748.6784,11.0234)){
format(string,sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
LCNEINGANGAUF = 1;
LCNEINGANGZU = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
}
else if(strcmp(tmp,"abschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13)){//YAKUZA EINGANG
if(SpielerInfo[playerid][pRank] >= 2){
if(YAKUZAEINGANGZU == 0){
if(IsPlayerInRangeOfPoint(playerid, 2, -2624.6577,1412.7295,7.0938)) {
format(string, sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGZU = 1;
YAKUZAEINGANGAUF = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
if(SpielerInFraktion(playerid,14)){ //LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2){
if(LCNEINGANGZU == 0){
if(IsPlayerInRangeOfPoint(playerid,2, 1492.6390,748.6784,11.0234)){
format(string,sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid, COLOR_SPECIALBLUE, string);
LCNEINGANGZU = 1;
LCNEINGANGAUF = 0;
}else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");

Alles anzeigen

Ich mags auch nicht so zu scripten Jeffry aber mir wurde gesagt wenn man es so scriptet das es ressourcerschonend ist ;o

Jeffry · 12. November 2013

Zitat von Sigi.

Ich mags auch nicht so zu scripten Jeffry aber mir wurde gesagt wenn man es so scriptet das es ressourcerschonend ist ;o

Absoluter Müll. Dem Compiler und Server ist es völlig wurscht wie viele Zeilenumbrüche du im Code hast. Du kannst auch 10 zwischen jede Zeile machen, das macht keinen Unterschied.
Der, der dir das erzählt hat, hat dir entweder einen Streich gespielt, oder hat wirklich keine Ahnung.

heyhooo · 12. November 2013

Zitat von Jeffry

Absoluter Müll. Dem Compiler und Server ist es völlig wurscht wie viele Zeilenumbrüche du im Code hast. Du kannst auch 10 zwischen jede Zeile machen, das macht keinen Unterschied.
Der, der dir das erzählt hat, hat dir entweder einen Streich gespielt, oder hat wirklich keine Ahnung.

Ich glaube worauf er hinaus ist, ist ocmd. Da ich dich ja auch mal wegen sscanf gefragt hatte und du glaube ein schlechtes Bild über sscanf hast.

Jeffry · 12. November 2013

Zitat von heyhooo

Ich glaube worauf er hinaus ist, ist ocmd. Da ich dich ja auch mal wegen sscanf gefragt hatte und du glaube ein schlechtes Bild über sscanf hast.

Meines Wissens nach ist OCMD Ressourcenschonender, eigentlich alle (seriösen) Command Prozessoren sind schonender (wobei wir hier von Bruch-Millisekunden reden).

Ja, stimmt. Ich habe ein schlechtes Bild von sscanf und das zurecht. Das hier aufzuführen würde aber zu lange dauern. Es muss jeder für sich selbst entscheiden es zu benutzen oder nicht. Ich rate davon ab.
Aber kommen wir nicht vom Thema ab.

Was er schlichtweg meinte, war, dass dieses gequetschte Zeugs schonender sei, und das stimmt nicht. Und schon zweimal nicht schonender für den, der den Code liest.

Sigi: Du sagst du magst es nicht. Du bist doch der Herr des Scripts, mach es doch so wie du es magst, und nicht so wie es dir einer sagt, der zudem keine Ahnung hat.

Sigi. · 12. November 2013

Zitat von Jeffry

Meine Güte. wer programmiert den so. Da erkennt man ja gar nichts mehr.

Ich hab dir das mal ausgebessert. Wenn ich das richtig verstanden habe, was der Code machen sollte, dann waren da paar Dinge an der falschen Stelle, was bei dem Durcheinander auch kein Wunder ist.

Code:

Spoiler anzeigen

if(strcmp(cmd,"/base",true) == 0)
{
new SpielerName[MAX_PLAYER_NAME],tmp[256];
GetPlayerName(playerid,SpielerName,sizeof(SpielerName));
tmp = strtok(cmdtext,idx);
if(!strlen(tmp))
{
SendClientMessage(playerid,COLOR_GREY,"FEHLER: /base [aufschließen/abschließen]");
return 1;
}
if(strcmp(tmp,"aufschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13))
{
//Yakuza Eingang
if(SpielerInfo[playerid][pRank] >= 2)
{
if(YAKUZAEINGANGAUF == 0)
{
if(IsPlayerInRangeOfPoint(playerid, 2,-2624.6577,1412.7295,7.0938))
{
format(string, sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGAUF = 1;
YAKUZAEINGANGZU = 0;
}
else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}
else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}
else if(SpielerInFraktion(playerid,14))
{
//LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2)
{
if(LCNEINGANGAUF == 0)
{
if(IsPlayerInRangeOfPoint(playerid,2,1492.6390,748.6784,11.0234))
{
format(string,sizeof string,"%s hat die Eingangstür aufgeschlossen.",SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
LCNEINGANGAUF = 1;
LCNEINGANGZU = 0;
}
else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis");
}
else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits aufgeschlossen.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
}
else if(strcmp(tmp,"abschließen",true) == 0)
{
if(SpielerInFraktion(playerid,13))
{
//YAKUZA EINGANG
if(SpielerInfo[playerid][pRank] >= 2)
{
if(YAKUZAEINGANGZU == 0)
{
if(IsPlayerInRangeOfPoint(playerid, 2, -2624.6577,1412.7295,7.0938))
{
format(string, sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid,COLOR_SPECIALBLUE,string);
YAKUZAEINGANGZU = 1;
YAKUZAEINGANGAUF = 0;
}
else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}
else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}
else if(SpielerInFraktion(playerid,14))
{
//LCN Eingang
if(SpielerInfo[playerid][pRank] >= 2)
{
if(LCNEINGANGZU == 0)
{
if(IsPlayerInRangeOfPoint(playerid,2, 1492.6390,748.6784,11.0234))
{
format(string,sizeof string,"%s hat die Eingangstür abgeschlossen.", SpielerName);
SendClientMessage(playerid, COLOR_SPECIALBLUE, string);
LCNEINGANGZU = 1;
LCNEINGANGAUF = 0;
}
else SendClientMessage(playerid,COLOR_GREY,"Du bist nicht an dem Eingang, deiner Fraktionsbasis!");
}
else SendClientMessage(playerid,COLOR_GREY,"Deine Base ist bereits abgeschlossen.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du hast nicht den benötigten Rank dafür.");
}
else SendClientMessage(playerid,COLOR_GREY,"Du darfst dieses Befehl nicht benutzen.");
}
return 1;
}

Zwecks ocmd: Es ist völliger Schwachsinn zu sagen, dass man damit übersichtlicher scripten kann. Der Code würde mit ocmd, bis auf die erste Zeile, fast IDENTISCH aussehen!!

Alles anzeigen

Hei, wenn ich davor "else" mache also vor dem if kommen dann errors...

Jeffry · 12. November 2013

Zitat von Sigi.

Hei, wenn ich davor "else" mache also vor dem if kommen dann errors...

Ja. Das kannst du dir ganz leicht merken, dass das nicht geht:

Wenn (if) x, dann ({ }) y, ansonsten (else) z.
Den Satz kannst du auch nicht umdrehen, und so ist es mit if und else. Erst if, dann else (falls nötig).

Sigi. · 12. November 2013

Hmm meinst du ich muss das so hinschreiben "if else",...oder wie soll ich das jetz verstehen....

Hmm ich komme nicht mehr weiter,...

Jeffry · 13. November 2013

Den Code für dein Problem hab ich dir ja schon gegeben, hoffe du hast das gesehen.

Ein Beispiel:

new playerid = 15;
if(IsPlayerConnected(playerid)) SendClientMessageToAll("ID 15 Connected");
else SendClientMessageToAll("ID 15 Nicht Connected");

Siehe hierzu: http://wiki.sa-mp.com/wiki/Control_Structures

Slash™ · 13. November 2013

Zitat von Jeffry

Meines Wissens nach ist OCMD Ressourcenschonender

Das trifft nur ein, falls du mehrere commands hast, da der cmd direkt aufgerufen wird,
falls du nur 1,2,3,... cmds hast ist strcmp noch schneller

breadfish.de 12. März 2022