[SAMMELTHREAD] Kleine Scripting Fragen

**Andosius** · 4. Januar 2020

Erstmal musst du bedenken, dass der eingegebene Text immer ein String ist. Du kannst ihn jedoch umwandeln und überprüfen, ob das Ergebnis eine Zahl ist.

Versuche es doch so:

Code

#if !defined isnull
    #define isnull(%1) ((!(%1[0])) || ((%1[0] == '\1') && (!(%1[1]))))
#endif
    if(dialogid == DIALOG_BANKPIN)
    {
      if(response == 1)
      {
        //Verwende hier lieber isnull, in diesem Fall ist die Funktion dafür viel besser geeignet. :)
         if(!isnull(inputtext))
         {
            SendClientMessage(playerid,COLOR_RED,"Du musst etwas eingeben!");
            return 1;
         }
         new Pin = PlayerInfo[playerid][Bankpin];
         if(/* hier musst du entweder bei Zahlen gleichsetzen oder bei String strcmp verwenden, siehe Bsp */)
         if(!strcmp(PlayerInfo[playerid][Bankpin], inputtext))
        if(PlayerInfo[playerid][Bankpin] != strval(inputtext))
         {
             SendClientMessage(playerid,COLOR_RED,"Falsche Pin");
             return 1;
         }
         ShowPlayerDialog(playerid,87,DIALOG_STYLE_LIST,"Bank","Einzahlen\nAbheben\nKontostand\nÜberweisung\nHandyguthaben aufladen\nBankpin beantragen","Absenden", "Abbrechen");
         SendClientMessage(playerid,COLOR_GREEN,"Erfolgreich");
         return 1;
       }
    }

Alles anzeigen

Kaliber · 4. Januar 2020

Zitat von Rocky

Da musst du die Funktion zeigen .... und nicht den DIalog

Wäre ich gemein, würde ich sagen, bleib du mal lieber im Smalltalk

Zitat von Andosius

Erstmal musst du bedenken, dass der eingegebene Text immer ein String ist. Du kannst ihn jedoch umwandeln und überprüfen, ob das Ergebnis eine Zahl ist.

Technisch gesehen hast du recht, gerade weil der Pin mit 0 starten kann, sollte er als String gehandhabt werden.

Allerdings wird er hier als Zahl verwendet, somit kannst du kein strcmp nutzen.

C

if(dialogid == DIALOG_BANKPIN)
{
    if(!response) return 1;
    if(!inputtext[0]) return SendClientMessage(playerid,COLOR_RED,"Du musst etwas eingeben!");
    new Pin = PlayerInfo[playerid][Bankpin];
    if(!Pin) return SendClientMessage(playerid,COLOR_RED,"Noch keine Pin vorhanden!");
    if(Pin != strval(inputtext)) return SendClientMessage(playerid,COLOR_RED,"Falsche Pin!");
    ShowPlayerDialog(playerid,87,DIALOG_STYLE_LIST,"Bank","Einzahlen\nAbheben\nKontostand\nÜberweisung\nHandyguthaben aufladen\nBankpin beantragen","Absenden", "Abbrechen");
    SendClientMessage(playerid,COLOR_GREEN,"Erfolgreich");
    return 1;
}

Alles anzeigen

So kannst du das als Integer nutzen, macht halt aber nur Sinn, wenn du beim Erstellen der Pin verhinderst, dass sie mit 0 startet.

Ansonsten, wie einen String behandeln und strcmp zum Vergleichen nutzen

Dr. Frauenarzt · 4. Januar 2020

Kaliber

Code

ocmd@2:su,suspect(playerid,params[])
{
    if(!IsLoggedIn(playerid))return 1;
    new pID, wanteds, string[128], reason[64];
    if(!(Spieler[playerid][pFraktion] == 2 || Spieler[playerid][pFraktion] == 3 || Spieler[playerid][pFraktion] == 4))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
    if(sscanf(params, "uis[64]", pID, wanteds, reason))return SendClientMessage(playerid, COLOR_WHITE, "{247BA6}* Benutze: {00CC00}(/su)spect [SpielerID/Name] [Wanteds] [Grund]");
    if(!IsPlayerConnected(pID))return SendClientMessage(playerid, COLOR_WHITE, ERROR_OFFLINE);
    if(IsPlayerNPC(pID))return 1;
    if(Spieler[pID][pFraktion] == 1 || Spieler[pID][pFraktion] == 2 || Spieler[pID][pFraktion] == 3 || Spieler[pID][pFraktion] == 4 || Spieler[pID][pFraktion] == 5 || Spieler[pID][pFraktion] == 6)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Der Spieler ist ein Beamter.");
    if(Spieler[pID][pJailed] == 1)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Der Spieler ist im Gefängnis.");
    if(wanteds < 1 || wanteds > 10)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Du kannst maximal 10 Wanteds gleichzeitig vergeben.");
    if(Spieler[pID][pWanteds] >= 500)return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Der Spieler hat 500 Wanteds.");
    OnWantedChange(pID, Spieler[pID][pWanteds], (Spieler[pID][pWanteds]+wanteds));
    Spieler[pID][pWanteds] += wanteds;

    format(string, sizeof(string), "{AA3333}Du hast ein Verbrechen begangen. (%s) Reporter: %s", reason, GetName(playerid));
    SendClientMessage(pID, COLOR_WHITE, string);

    format(string, sizeof(string), "{FFFF00}Dein aktuelles Wanted Level: %d", Spieler[pID][pWanteds]);
    SendClientMessage(pID, COLOR_WHITE, string);

    format(string, sizeof(string), "{8D8DFF}[WANTED] Verdächtiger: %s, Zeuge: %s, Verbrechen: %s", GetName(pID), GetName(playerid), reason);
    SendFraktionMessage(2, COLOR_WHITE, string);

    format(string, sizeof(string), "{8D8DFF}[WANTED] Verdächtiger: %s, Zeuge: %s, Verbrechen: %s", GetName(pID), GetName(playerid), reason);
    SendFraktionMessage(3, COLOR_WHITE, string);

    format(string, sizeof(string), "{8D8DFF}[WANTED] Verdächtiger: %s, Zeuge: %s, Verbrechen: %s", GetName(pID), GetName(playerid), reason);
    SendFraktionMessage(4, COLOR_WHITE, string);

    UpdateWantedTD(pID);
    format(Spieler[pID][pWantedReason], 128, reason);
    return 1;
}

Alles anzeigen

Kaliber · 4. Januar 2020

Tausch mal Zeile 31 und 32

Dr. Frauenarzt · 4. Januar 2020

Code

                if(weaponid == 34 && bodypart == 9)
            {
                if(Spieler[playerid][pTot] == 0) Headshot(playerid, issuerid);
            }

Code

stock Headshot(hitid, playerid)
{
     new Float:x, Float:y, Float:z, Float:FrontX, Float:FrontY, Float:FrontZ, Float:CamX, Float:CamY, Float:CamZ;
       GetPlayerCameraFrontVector(playerid, FrontX, FrontY, FrontZ);
    GetPlayerCameraPos(playerid, CamX, CamY, CamZ);
    for(new Float:i=0.0;i<100;i=i+0.5)
    {
          x = FrontX * i + CamX;
         y = FrontY * i + CamY;
         z = FrontZ * i + CamZ;
        if(hitid != playerid)return 1;
           if(GetPlayerSpecialAction(hitid) == SPECIAL_ACTION_DUCK)
           {
            if(IsPlayerInRangeOfPoint(hitid, 0.4, x, y, z))
            {
                   GameTextForPlayer(playerid,"~r~~h~~h~] Headshot! ]", 5000, 5);
                SetPlayerHealth(hitid, 0.0);
            }
        }
        else
        {
              if(IsPlayerInRangeOfPoint(hitid, 0.4, x, y, z-0.7))
              {
                   GameTextForPlayer(playerid,"~r~~h~~h~] Headshot! ]", 5000, 5);
                SetPlayerHealth(hitid, 0.0);
            }
        }
    }
    return 1;
}

Alles anzeigen

Kurze frage kann mir mal einer sagen warum der Headshot nicht funktioniert schieße auf den Kopf mit der WaffenID 34

Kaliber · 4. Januar 2020

Zitat von Dr. Frauenarzt

Spieler[playerid][pTot] == 0

Ist diese Variable denn 0?

Dr. Frauenarzt · 4. Januar 2020

Positiv steht auf 0 ja wird resettet beim spawnen wen er aus den Krankenhaus kommt auch alles habe ich schon überprüft

Kaliber · 4. Januar 2020

Was soll der Code denn da genau bezwecken?

Zitat von Dr. Frauenarzt

if(hitid != playerid)return 1;

Das hier z.B. macht keinen Sinn, mach das mal raus.

In diesem Fall kan hitid niemals playerid sein

Dr. Frauenarzt · 4. Januar 2020

Trotzdem geht das Headshot System irgendwie nicht und er ist nicht tot

Kaliber · 4. Januar 2020

Ich habe dir 2 Fragen gestellt

Du machst da eine Berechnung...die du eigentlich nicht tun brauchst..

C

if(weaponid == 34 && bodypart == 9 && !Spieler[playerid][pTot])
{
    GameTextForPlayer(issuerid,"~r~~h~~h~] Headshot! ]", 5000, 5);
    SetPlayerHealth(playerid, 0.0);               
}

Das sollte es eigentlich tun

Dr. Frauenarzt · 4. Januar 2020

Entschuldigung Meister habe ich überlesen tut mir leid ich bedanke mich vielmals für deine Hilfe!

-> Wen ich den Spieler Wanteds gebe ist es schlau mit ein Timer OnPlayerStreamIn Updaten zu lassen? Weil wen ich jemand Wanteds gebe bekommt er nicht sofort den Label sondern erst wen ich weiter weg war...

Kaliber · 5. Januar 2020

Zitat von Dr. Frauenarzt

-> Wen ich den Spieler Wanteds gebe ist es schlau mit ein Timer OnPlayerStreamIn Updaten zu lassen? Weil wen ich jemand Wanteds gebe bekommt er nicht sofort den Label sondern erst wen ich weiter weg war...

Du nutzt wahrscheinlich n Streamer für das Label...

Kannst manuell einfach 1x für die Spieler updaten: https://github.com/samp-incogn…(Updates)#Streamer_Update

Dann brauchst du keinen Timer und es sollte sofort zu sehen sein

**GangstaSunny** · 5. Januar 2020

Merke: Timers sind Bad (vor allem in SAMP)

Wenn zu verhindern, dann verhindere die Nutzung.

Dr. Frauenarzt · 5. Januar 2020

Das Heißt Streamer_UpdateEx

muss ich bei OnPlayerStreamIn erst die Position auslesen also so:

Code

new Float:PoS[3];
GetPlayerPos(playerid, PoS[0], PoS[1], PoS[2]);


Streamer_UpdateEx(playerid, PoS[0], PoS[1], PoS[2]);

Oder geht das einfacher?

Kaliber · 5. Januar 2020

Zitat von Dr. Frauenarzt

muss ich bei OnPlayerStreamIn erst die Position auslesen also so:

Da musst du gar nix machen.

Da wo du das Label setzt, einfach die Funktion aufrufen:

C

UpdateStreamer(playerid, STREAMER_TYPE_3D_TEXT_LABEL);

//Und dann die Funktion:
stock UpdateStreamer(playerid, _type)
{
    new Float:x,Float:y,Float:z;
    GetPlayerPos(playerid,x,y,z);

    for(new i=GetPlayerPoolSize(); i!=-1; i--)
    {
        if(IsPlayerConnected(i) && !IsPlayerNPC(i) && IsPlayerInRangeOfPoint(i,50.0,x,y,z))
        {
            Streamer_UpdateEx(i, x,y,z, .type = _type, .freezeplayer=0)
        }
    }
}

Alles anzeigen

So sollte das klappen.

Natürlich musst du für alle den Streamer Updaten, die sich da in der Nähe befinden.

Dr. Frauenarzt · 5. Januar 2020

Code

ocmd:gebieteinfo(playerid)
{
    new belstr1[16], belstr2[16];
    if(!IsGang(Spieler[playerid][pFraktion]))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
    for(new i=1;i<sizeof(GangFight);i++)
    {
        switch(GangFight[i][warBelohnungType])
        {
            case 0:
            {
                format(belstr1,16, "Geld");
                format(belstr2, 16, "%d$", GangFight[i][warBelohnungVar]);
            }
            case 1:
            {
                format(belstr1, 16, "Materialien");
                format(belstr2, 16, "%d", GangFight[i][warBelohnungVar]);
            }
        }
        strdel(DialogString, 0, sizeof(DialogString));
        format(DialogString, sizeof(DialogString), "Gebiet: %s, Besitzer: %s, Belohnung: %s [%s]\n", GangFight[i][warZoneName], GetFraktionName(GangFight[i][warOwner]), belstr1, belstr2);
        ShowPlayerDialog(playerid, DIALOG_4ALL_SONSTIGES, DIALOG_STYLE_MSGBOX, ""#SERVER_FARBE""SERVER_HOST"  {FFFFFF}| GANGFIGHT", DialogString, "OK","");
    }
    return 1;
}

Alles anzeigen

Wen ich das eingebe kommt nur ein GF gebiet obwohl ich mehrere im script definiert habe?

Kaliber · 5. Januar 2020

Nutze keinen globalen Dialog-String.

Das nicht gut.

C

ocmd:gebieteinfo(playerid)
{
    if(!IsGang(Spieler[playerid][pFraktion]))return SendClientMessage(playerid, COLOR_WHITE, ERROR_FRAKTION);
    new const type[][] = {"Geld","Materialien"}, v[][] = {"$]","]"};
    new string[64*sizeof(GangFight)];
    for(new i; i<sizeof(GangFight); i++)
    {
        format(string,sizeof(string),"%sGebiet: %s, Besitzer: %s, Belohnung: %s [%d%s\n",
        string,GangFight[i][warZoneName],
        GetFraktionName(GangFight[i][warOwner]),
        type[GangFight[i][warBelohnungType]],
        GangFight[i][warBelohnungVar],
        v[GangFight[i][warBelohnungType]);
      }
      ShowPlayerDialog(playerid, DIALOG_4ALL_SONSTIGES, DIALOG_STYLE_MSGBOX, ""#SERVER_FARBE""SERVER_HOST"  {FFFFFF}| GANGFIGHT", string, "OK","");
    return 1;
}

Alles anzeigen

Dr. Frauenarzt · 5. Januar 2020

Code

        if(Spieler[forplayerid][pFraktion] == 2 || Spieler[forplayerid][pFraktion] == 3 || Spieler[forplayerid][pFraktion] == 4)
        {
            if(Spieler[playerid][pWanteds] > 0){
                for(new a=0;a<MAX_WANTEDLABEL;a++)
                {
                    if(Spieler[forplayerid][pWantedLabelCreated][a] == 0){
                        format(string, sizeof(string), "{FFFF00}%s (ID: %d)\nWanteds: %d\nGrund: %s", GetName(playerid), playerid, Spieler[playerid][pWanteds], Spieler[playerid][pWantedReason]);
                        Spieler[forplayerid][pWantedLabel][a] = CreateDynamic3DTextLabel(string, COLOR_YELLOW, 0.0, 0.0, 0.0, 10.0, playerid, INVALID_VEHICLE_ID, 0, GetPlayerVirtualWorld(playerid), GetPlayerInterior(playerid), forplayerid);
                        Spieler[forplayerid][pWantedLabelID][a] = playerid;
                        Spieler[forplayerid][pWantedLabelCreated][a] = 1;
                        Streamer_SetFloatData(STREAMER_TYPE_3D_TEXT_LABEL, Spieler[forplayerid][pWantedLabel][a], E_STREAMER_ATTACH_OFFSET_Z, 0.7);
                        UpdateStreamer(playerid, STREAMER_TYPE_3D_TEXT_LABEL);
                        return 1;
                    }
                }
            }
        }

Alles anzeigen

So ist das richtig korrekt oder soll ich den vorher updaten lassen?

Kaliber · 5. Januar 2020

Zitat von Dr. Frauenarzt

So ist das richtig korrekt oder soll ich den vorher updaten lassen?

Sollte, probiers halt einfach mal aus

Dr. Frauenarzt · 5. Januar 2020

Habs eben Probiert mit ein kollegen klappt leider nicht muss in ein Interior und dann wieder raus das der Label da ist etwas komisch habe mir überlegt das ganze einfach nicht dynamic zu machen