[HILFE] Probleme mit dem erstellen von Fraktions Fahrzeugen

iiTzFabs · 8. August 2021

Guten Tag,

weiß eventuell jemand woran es liegt das der Befehl nicht geht.

Code

cmd:fv(playerid,params[]) {
    new _varname[18];
    if(!isAdmin(playerid, 1))
        return SendClientMessage(playerid, -1, text_Error""text_NoRights);
    if(sscanf(params,"s[18]",_varname)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete]");
    if(!strcmp(_varname,"create", true)) {
        new _fid,_rang,_fmodel,_color[2];
        if(sscanf(params,"s[18]iiiii",_varname, _fid,_rang,_fmodel,_color[0],_color[1])) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        new Float:_pos[4];
        GetPlayerPos(playerid, _pos[0], _pos[1], _pos[2]);
        GetPlayerFacingAngle(playerid, _pos[3]);
        createFactionVehicle(_fid,_rang, _fmodel, _pos[0], _pos[1], _pos[2], _pos[3], _color[0], _color[1], "Created");
        return SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich erstellt.");
    } else if(!strcmp(_varname,"create", true)) {
        new vid;
        if(sscanf(params,"s[18]i",_varname, vid)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        if(!IsValidVehicle(vid))
            return SendClientMessage(playerid, -1, text_Error"Keine gültige FahrzeugID.");
        for(new i = 0; i < MAX_FRACS_VEHICLES; i++) {
            if(FractionVehicles[i][fracveh_VehID] == vid) {
                new _Query[128];
                format(_Query, 128, "DELETE FROM `"table_faction_vehicles"` WHERE `id`='%i'",FractionVehicles[i][fracveh_dbid]);
                mysql_query(mysql_Con, _Query);
                FractionVehicles[i][fracveh_modelid]=0;
                DestroyVehicle(FractionVehicles[i][fracveh_VehID]);
                SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich gelöscht.");
                return 1;
            }
        }
        return SendClientMessage(playerid, -1, text_Error"Diese Fahrzeug ID ist kein Fraktions Fahrzeug!");
    }
       return 1;
}

Alles anzeigen

undbyebye · 9. August 2021

Versuche es mal so abgeändert habe ich if(!strcmp

Code

cmd:fv(playerid,params[])
{
    new _varname[18];
    if(!isAdmin(playerid, 1))return SendClientMessage(playerid, -1, text_Error""text_NoRights);
    if(sscanf(params,"s[18]",_varname)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete]");
    if(strcmp(_varname,"create", true) == 0)
    {
        new _fid,_rang,_fmodel,_color[2];
        if(sscanf(params,"s[18]iiiii",_varname, _fid,_rang,_fmodel,_color[0],_color[1])) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        new Float:_pos[4];
        GetPlayerPos(playerid, _pos[0], _pos[1], _pos[2]);
        GetPlayerFacingAngle(playerid, _pos[3]);
        createFactionVehicle(_fid,_rang, _fmodel, _pos[0], _pos[1], _pos[2], _pos[3], _color[0], _color[1], "Created");
        return SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich erstellt.");
    }
    if(strcmp(_varname,"remove", true) == 0)
    {
        new vid;
        if(sscanf(params,"s[18]i",_varname, vid)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        if(!IsValidVehicle(vid))return SendClientMessage(playerid, -1, text_Error"Keine gültige FahrzeugID.");
        for(new i = 0; i < MAX_FRACS_VEHICLES; i++)
        {
            if(FractionVehicles[i][fracveh_VehID] == vid)
            {
                new _Query[128];
                format(_Query, 128, "DELETE FROM `"table_faction_vehicles"` WHERE `id`='%i'",FractionVehicles[i][fracveh_dbid]);
                mysql_query(mysql_Con, _Query);
                FractionVehicles[i][fracveh_modelid]=0;
                DestroyVehicle(FractionVehicles[i][fracveh_VehID]);
                SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich gelöscht.");
                return 1;
            }
        }
        return SendClientMessage(playerid, -1, text_Error"Diese Fahrzeug ID ist kein Fraktions Fahrzeug!");
    }
    return 1;
}

Alles anzeigen

iiTzFabs · 9. August 2021

Zitat von Grosim

Versuche es mal so abgeändert habe ich if(!strcmp

Code

cmd:fv(playerid,params[])
{
    new _varname[18];
    if(!isAdmin(playerid, 1))return SendClientMessage(playerid, -1, text_Error""text_NoRights);
    if(sscanf(params,"s[18]",_varname)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete]");
    if(strcmp(_varname,"create", true) == 0)
    {
        new _fid,_rang,_fmodel,_color[2];
        if(sscanf(params,"s[18]iiiii",_varname, _fid,_rang,_fmodel,_color[0],_color[1])) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        new Float:_pos[4];
        GetPlayerPos(playerid, _pos[0], _pos[1], _pos[2]);
        GetPlayerFacingAngle(playerid, _pos[3]);
        createFactionVehicle(_fid,_rang, _fmodel, _pos[0], _pos[1], _pos[2], _pos[3], _color[0], _color[1], "Created");
        return SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich erstellt.");
    }
    if(strcmp(_varname,"remove", true) == 0)
    {
        new vid;
        if(sscanf(params,"s[18]i",_varname, vid)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
        if(!IsValidVehicle(vid))return SendClientMessage(playerid, -1, text_Error"Keine gültige FahrzeugID.");
        for(new i = 0; i < MAX_FRACS_VEHICLES; i++)
        {
            if(FractionVehicles[i][fracveh_VehID] == vid)
            {
                new _Query[128];
                format(_Query, 128, "DELETE FROM `"table_faction_vehicles"` WHERE `id`='%i'",FractionVehicles[i][fracveh_dbid]);
                mysql_query(mysql_Con, _Query);
                FractionVehicles[i][fracveh_modelid]=0;
                DestroyVehicle(FractionVehicles[i][fracveh_VehID]);
                SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich gelöscht.");
                return 1;
            }
        }
        return SendClientMessage(playerid, -1, text_Error"Diese Fahrzeug ID ist kein Fraktions Fahrzeug!");
    }
    return 1;
}

Alles anzeigen

Klappt leider noch immer nocht

undbyebye · 9. August 2021

Was geht den genau nicht, bei dem Befehl ? Create und delete oder passiert garnix ?

iiTzFabs · 9. August 2021

Es wird kein Fahrzeug erstellt und keine Message gesendet("Du hast das Fraktions Fahrzeug erfolgreich erstellt.")

undbyebye · 9. August 2021

Dann setzte das mal ein, unter der Narricht ("Du hast das Fraktions Fahrzeug erfolgreich erstellt.").

Code

print("war hier gewesen");

Und gucke in der Console ob es auftaucht

//edit

Code

cmd:fv(playerid,params[])
{
    new _varname[18];
    if(!isAdmin(playerid, 1))return SendClientMessage(playerid, -1, text_Error""text_NoRights);
    if(sscanf(params,"s[18]i",_varname, vid)) return SendClientMessage(playerid,-1,text_CMDuse"/fv [create/delete] [fraktionid] [rang] [modelid] [color] [color]");
    if(strcmp(_varname,"create", true) == 0)
    {
        new _fid,_rang,_fmodel,_color[2];
        new Float:_pos[4];
        GetPlayerPos(playerid, _pos[0], _pos[1], _pos[2]);
        GetPlayerFacingAngle(playerid, _pos[3]);
        createFactionVehicle(_fid,_rang, _fmodel, _pos[0], _pos[1], _pos[2], _pos[3], _color[0], _color[1], "Created");
        SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich erstellt.");
        return true;
    }
    if(strcmp(_varname,"delete", true) == 0)
    {
        new vid;
        if(!IsValidVehicle(vid))return SendClientMessage(playerid, -1, text_Error"Keine gültige FahrzeugID.");
        for(new i = 0; i < MAX_FRACS_VEHICLES; i++)
        {
            if(FractionVehicles[i][fracveh_VehID] == vid)
            {
                new _Query[128];
                format(_Query, 128, "DELETE FROM `"table_faction_vehicles"` WHERE `id`='%i'",FractionVehicles[i][fracveh_dbid]);
                mysql_query(mysql_Con, _Query);
                FractionVehicles[i][fracveh_modelid]=0;
                DestroyVehicle(FractionVehicles[i][fracveh_VehID]);
                SendClientMessage(playerid, -1, text_Info"Du hast das Fraktions Fahrzeug erfolgreich gelöscht.");
                return true;
            }
        }
        return SendClientMessage(playerid, -1, text_Error"Diese Fahrzeug ID ist kein Fraktions Fahrzeug!");
    }
    return 1;
}

Alles anzeigen

So müsste es gehen wenn ich mich nicht irre

Whice · 9. August 2021

Zitat von Grosim

So müsste es gehen wenn ich mich nicht irre

Auch das wird schwierig, da in Zeile 8 Variablen erstellt werden, die allerdings nicht gefüllt werden (_fid, _rang, _fmodel, _color). Diese müssten, so wie es die Nachricht besagt auch in Zeile 5 abgefragt werden.

Zitat von StaYClean

Es wird kein Fahrzeug erstellt

Dann wäre es sinnvoll, wenn du uns mal die Funktion zeigst, die das Fahrzeug erstellt. (createFactionVehicle)

Weiterhin hast du einige Dopplungen in deinem Code: So liest du beispielsweise in Zeile 5 _varname aus, fragst die in Zeile 6 ab und liest sie nochmals in Zeile 8 aus.

Zusätzlich würde es beispielsweise beim Löschen auch Sinn ergeben fracveh_VehID zurückzusetzen, da du diese zur Abfrage verwendest und sich ggf. dopplen könnte und so Fehler entstehen (DestroyVehicle auf ein falsches Fahrzeug aufrufen)

breadfish.de 12. März 2022