[SAMMELTHREAD] Kleine Scripting Fragen

Marschl · 14. Mai 2021

Zitat von Kaliber

Weil du da ein else in der Schleife hast.

Kannst so Funktionen auch wunderbar auslagern:

C

ocmd:fdelveh(playerid, params[])
{
    if(!pInfo[playerid][pLoggedIn]) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
    if(pInfo[playerid][pTeam] == 0) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
    if(pInfo[playerid][pTeam] < 5) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
    if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht in einem Vehicle als Fahrer.");
    
    new i = GetFrakCarIndex(playerid);
    if(i == -1) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist in keinem Fraktionsvehicle.");
    
    new msg[248];
    
    DestroyVehicle(fVehInfo[i][fvehID]);
    fVehInfo[i][fvehInUse] = false;
    mysql_format(Handle, msg, sizeof(msg), "DELETE FROM `fraktion_vehicle` WHERE `VehID` = '%d'", i);
    mysql_pquery(Handle, msg);
    format(msg, sizeof(msg), "* Fehler"HTML_WEISS": Du hast ein Fraktionsvehicle der Fraktion %s gelöscht.", fInfo[fVehInfo[i][fvehFraktion]-1][fName]);
    SendClientMessage(playerid, COLOR_GELB, msg);
    return 1;
}

stock GetFrakCarIndex(playerid)
{
    for(new i,v=GetPlayerVehicleID(playerid); i<MAX_FRAK_VEHICLE; i++)
    {
        if(v == fVehInfo[i][fvehID]) return i;
    }
    return -1;
}

Alles anzeigen

kussi danke ich geh mal schlafen haha

Dr. Frauenarzt · 20. Mai 2021

Code

ocmd@3:rc,rac,respawncars(playerid, params[])
{
    if(!IsLoggedIn(playerid))return 1;
    if(Spieler[playerid][pAdmin] < 1)return SendClientMessage(playerid, COLOR_WHITE, ERROR_ADMIN);
    for(new i;i<MAX_VEHICLES;i++) {
        if(IsValidVehicle(i)) {
            if(GetVehicleDriver(i) == INVALID_PLAYER_ID) {
                SetVehicleToRespawn(i);
            }
        }
    }
    SendClientMessageToAll(COLOR_WHITE, "{F70D0D}Es wurden alle Fahrzeuge respawnt!");
    return 1;
}

Alles anzeigen

Moin, Kurze frage es wird kein Fahrzeug neu gespawnt...

Habe auch schon IsValidVehicle ausgeklammert das es eventuell drann liegt aber leider erfolglos..

Kaliber · 20. Mai 2021

Zitat von Dr. Frauenarzt

Moin, Kurze frage es wird kein Fahrzeug neu gespawnt...

Wie sieht n deine Funktion GetVehicleDriver aus?

Dr. Frauenarzt · 20. Mai 2021

Zitat von Kaliber

Wie sieht n deine Funktion GetVehicleDriver aus?

Code

stock GetVehicleDriver(vehicleid)
{
    for(new i = 0; i < GetMaxPlayers(); i++) {
        if(IsPlayerConnected(i) && GetPlayerState(i) == PLAYER_STATE_DRIVER && GetPlayerVehicleID(i) == vehicleid) {
            return i;
        }
    }
    return -1;
}

/E:

Fehler gefunden -1 mit INVALID_PLAYER_ID ausgetauscht

Jameso · 26. Mai 2021

Morgen!

ich suche schon seit längerem ein Plugin bzw. eine Methode, um Texte farbig in der Konsole anzuzeigen.

Mit der Zeit verliert man extrem den Überblick zumindest bei uns.

Ist so etwas möglich? Die Alten Hasen haben bestimmt sofort nh Antwort

Canna · 26. Mai 2021

Das ist soweit ich weiss in SAMP nicht möglich.

Kaliber · 26. Mai 2021

Zitat von Jameso

Ist so etwas möglich? Die Alten Hasen haben bestimmt sofort nh Antwort

Ja, sowas ist möglich, allerdings nur mit Plugin. In C++ kannst du das einfach mit ANSI Codes realisieren.

Damals hatte das wer im englischen Forum gemacht, allerdings nur für Windows:

Kannst du einfach so nutzen:

C

printc(TXT_LIGHTGREEN,"Hallo, das ist ein Test!");

Kannst sogar die Background-Color ändern.

Im Include selbst sind alle Farben gelistet.

Ich lads mal hier hoch.

Viel Spaß

Dr. Frauenarzt · 1. Juni 2021

Spoiler anzeigen

Whice · 1. Juni 2021

Moin, was ich mir gut Vorstellen kann, dass evtl. einer der beiden Strings leer ist.

Sollte dies so sein gibt die Funktion auch 0 zurück. (https://team.sa-mp.com/wiki/Strcmp_DE.html)

Dr. Frauenarzt · 2. Juni 2021

Das ding ist das soll nur erschein wen ich ein Teilhaber aber einer Tankstelle, das bin ich ja nicht wird aber trotzdem angezeigt im chat.

Whice · 2. Juni 2021

Zitat von Dr. Frauenarzt

das bin ich ja nicht wird aber trotzdem angezeigt im chat.

Jo, das Problem wird wohl (so wie ich das sehe) das oben beschriebene sein.

Eine Lösung wäre zusätzlich abzufragen, ob die Variable tTeilhaber gefüllt ist.

Code

for(new i=1;i<MAX_TANKEN;i++) {
    if(Tanke[i][tCreated] == 1) {
        if(strlen(Tanke[i][tTeilhaber]) > 0) {
            if(strcmp(Tanke[i][tTeilhaber], GetName(playerid), true) == 0) {
                new TankeKasse = Tanke[i][tKasse];
                TankeMoney = floatround((TankeKasse/100)*5);
                format(string, sizeof(string), ""COLOR_HEX_WHITE"{FFFFFF}Tankstellen-Teilhaber {33AA33}%d$", TankeMoney);
                SendClientMessage(playerid, COLOR_WHITE, string);
                Spieler[playerid][pBank] += TankeMoney;
                break;
            }
          }
    }
}

Alles anzeigen

Dr. Frauenarzt · 3. Juni 2021

Code

        if(Spieler[playerid][pBank] >= 500000 || Spieler[playerid][pHouseRent] != 0 && Spieler[playerid][pHouseKey] != 0)
        {
            format(string, sizeof(string), ""COLOR_HEX_WHITE"Lohn: "COLOR_HEX_GREEN"%d$ "COLOR_HEX_WHITE"| Lohnsteuer: "COLOR_HEX_RED"-%d$ {FFFFFF}| KFZ-Steuer: {AA3333}-%d$ {FFFFFF}| Mietpreis: {AA3333}-%d$", (Lohn+steuerGeld), steuerGeld, SteuerKFZ, MietPreis);
            SendClientMessage(playerid, COLOR_WHITE, string);

            format(string, sizeof(string), "{FFFFFF}Kirchensteuer: {AA3333}-%d${FFFFFF} | {FFFFFF}Solidaritätszuschlag: {AA3333}-%d$ {FFFFFF}| Reichensteuer: {AA3333}-%d$", KirchenSteuer, SolidSteuer, ReichenSteuer);
            SendClientMessage(playerid, COLOR_WHITE, string);
            
            Spieler[playerid][pBank] -= floatround(ReichenSteuer*fsteuern[pReichensteuer]);
            Others[Staatskasse] += ReichenSteuer;
        } else {
              format(string, sizeof(string), ""COLOR_HEX_WHITE"Lohn: "COLOR_HEX_GREEN"%d$ "COLOR_HEX_WHITE"| Lohnsteuer: "COLOR_HEX_RED"-%d$ {FFFFFF}| KFZ-Steuer: {AA3333}-%d$", (Lohn+steuerGeld), steuerGeld, SteuerKFZ);
            SendClientMessage(playerid, COLOR_WHITE, string);

            format(string, sizeof(string), "{FFFFFF}Kirchensteuer: {AA3333}-%d${FFFFFF} | {FFFFFF}Solidaritätszuschlag {AA3333}-%d$", KirchenSteuer, SolidSteuer);
            SendClientMessage(playerid, COLOR_WHITE, string);
        }

Alles anzeigen

Moin, Kurze Frage wen ich weniger als 5mio habe und nicht eingemietet bin kommt keine Nachricht?

habe ich mehr als 5mio und eingemietet bin krieg ich eine Nachricht weiß einer was ich falsch mache bin so lost gerade

Whice · 3. Juni 2021

Moin,

Zitat von Dr. Frauenarzt

Moin, Kurze Frage wen ich weniger als 5mio habe und nicht eingemietet bin kommt keine Nachricht?
habe ich mehr als 5mio und eingemietet bin krieg ich eine Nachricht weiß einer was ich falsch mache bin so lost gerade

Was ist denn dein Ziel, also unter welchen Bedingungen soll eine Nachricht kommen?

Weiterhin schreibst du von 5 Mio, fragst aber nach 500k ab.

Wäre gut noch ein paar Infos zu bekommen

Dr. Frauenarzt · 3. Juni 2021

Ja sehe gerade hat der Spieler 5mio soll die Reichensteuer dazu kommen, ist der Spieler eingemietet kommt der Mietpreis dazu.

Mogly1 · 3. Juni 2021

Wenn du das einzelnd machen willst.

Code

if(Spieler[playerid][pBank] >= 5000000)
{
    //steuern dazu geben oder so
    Spieler[playerid][pBank] -= floatround(ReichenSteuer*fsteuern[pReichensteuer]);
    Others[Staatskasse] += ReichenSteuer;
}
if(Spieler[playerid][pHouseRent] != 0)
{
    //Wenn er eingemitet ist, miete abziehen
    Spieler[playerid][pBank] -= MietPreis;
} 
if(Spieler[playerid][pHouseKey] != 0)
{
    //ka. glaube wenn er nen hause hat dann halt mietpreise dazu geben oder stromkosten abziehen :D:D
}
//und am ende dann erst den Text anzeigenlassen evt.
format(string, sizeof(string), ""COLOR_HEX_WHITE"Lohn: "COLOR_HEX_GREEN"%d$ "COLOR_HEX_WHITE"| Lohnsteuer: "COLOR_HEX_RED"-%d$ {FFFFFF}| KFZ-Steuer: {AA3333}-%d$", (Lohn+steuerGeld), steuerGeld, SteuerKFZ);....

Alles anzeigen

Whice · 3. Juni 2021

Wenn dir das so zu unübersichtlich ist, hilft es mir meist mit Kommentaren zu arbeiten (Kommentare sind immer gut )

Da du hier ja zwei Bedingungen (Reichensteuerpflichig und Mietzahlungspflichtig) hast, müsstest du ja auch beide Prüfen

Das kannst du dir noch ordentlich anordnen und dann gelingt es meistens auch direkt:

Falls du selbst nicht weiter kommst

Dr. Frauenarzt · 6. Juni 2021

Jo, Danke dir

Habe noch ne frage wen ich fixveh 5000 eingebe repairt er trotzdem nur 1000.

schreibt er nur fixveh soll er 1k repairen gibt er /fixveh Zahl ein soll das Auto mit der Zahl repairen

Code

ocmd@2:arepair,fixveh(playerid, params[])
{
    if(!IsLoggedIn(playerid))return 1;
    new Float:repair, vehicleid = GetPlayerVehicleID(playerid);
    if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_WHITE, ""COLOR_HEX_RED"Du befindest dich in keinem Fahrzeug.");
    if(Spieler[playerid][pAdmin] < 3)return SendClientMessage(playerid, COLOR_WHITE, ERROR_ADMIN);
    if(sscanf(params, "f", repair)) {
        RepairVehicle(vehicleid);
        SetVehicleHealth(vehicleid, 1000.0);
        return 1;
    }
    if(repair < 0 || repair > 10000)return SendClientMessage(playerid, COLOR_WHITE, ""COLOR_HEX_RED"Der Zustand darf eine Zahl zwischen 0-10000 haben.");
    SetVehicleHealth(vehicleid, repair);
    RepairVehicle(vehicleid);
    return 1;
}
//=============================================================================//
//=============================================================================//

Alles anzeigen

undbyebye · 6. Juni 2021

Zitat von Dr. Frauenarzt

Jo, Danke dir

Habe noch ne frage wen ich fixveh 5000 eingebe repairt er trotzdem nur 1000.

schreibt er nur fixveh soll er 1k repairen gibt er /fixveh Zahl ein soll das Auto mit der Zahl repairen

Code

ocmd@2:arepair,fixveh(playerid, params[])
{
    if(!IsLoggedIn(playerid))return 1;
    new Float:repair, vehicleid = GetPlayerVehicleID(playerid);
    if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_WHITE, ""COLOR_HEX_RED"Du befindest dich in keinem Fahrzeug.");
    if(Spieler[playerid][pAdmin] < 3)return SendClientMessage(playerid, COLOR_WHITE, ERROR_ADMIN);
    if(sscanf(params, "f", repair)) {
        RepairVehicle(vehicleid);
        SetVehicleHealth(vehicleid, 1000.0);
        return 1;
    }
    if(repair < 0 || repair > 10000)return SendClientMessage(playerid, COLOR_WHITE, ""COLOR_HEX_RED"Der Zustand darf eine Zahl zwischen 0-10000 haben.");
    SetVehicleHealth(vehicleid, repair);
    RepairVehicle(vehicleid);
    return 1;
}
//=============================================================================//
//=============================================================================//

Alles anzeigen

Dann musst du auch SetVehicleHealth(vehicleid, 1000.0); zu SetVehicleHealth(vehicleid, 5000.0); machen.

Dr. Frauenarzt · 6. Juni 2021

schon klar aber man kann ja beides nutzen einmal so: /arepair ohne was dahinter = 1000

Schriebe ich aber /arepair 5000 sollte er die 5.000 HP setzen das ist leider nicht der fall

**JiU** · 6. Juni 2021

Dann setz den Parameter ein anstatt 1000.0