Beiträge von Akino Kiritani

  • Schnellere Spieler orientierte Schleifen

    Zitat von Kaliber

    Ehm...nein, ich wollte auf die Schnelligkeit hinaus, aber gut das du meine Gedanken lesen kannst...


    Ich bitte dich - weißt du nicht was du schreibst?


    Zitat von Kaliber

    macht die Schleifen erheblich schneller und ressourcenschonender ;)


    ich markier es nocheinmal fett


    Zitat von Kaliber

    macht die Schleifen erheblich schneller und ressourcenschonender ;)


    Wenn das nicht eindeutig ist - dann weiß ich allerdings auch nicht weiter :thumbdown:


    Zitat von Kaliber

    Ich rede doch gar nicht von den anderen Posts, aber finde es lustig das du auf diese eingehst...

    Interessant - das ist schon der 2. Thread in dem du das so sagst und in einer PN auch - was willst du mehr?

  • Schnellere Spieler orientierte Schleifen

    Zitat von Kaliber

    Omg, aber hauptsache erstmal mich runter machen ;)


    Wie kann man nur so beschränkt sein :pinch:


    Du willst doch ressourcenschonend sein . du bezweckst aber mit deiner "Include" aber das genaue Gegenteil.
    Wo mache ich dich denn bitte nieder - nur weil ich bisher keins deiner "Werke" toll fand - und sie nicht mit > 6 bewertet habe?


    Wenn man sich mal deine Includes anguckt, sind diese ja auch nur Wertzuweisungen mehr nicht.
    Siehe deine Inventarsystem Include - hier ist es doch genauso.

  • Schnellere Spieler orientierte Schleifen

    Zitat von Kaliber

    Ja ist sowas


    Sicherlich nicht - ein Funktionsaufruf dauert länger als eine Variable auszulesen - das ist Grundwissen mein lieber :thumbup:
    Du hast ja static benutzt - deswegen ist es im Script NICHT nutzbar.


    Deswegen sollst du new verwenden und ihr einen vernünftigen Namen geben - sie MAXID - oder MAX_ID - etwas was man auch wirklich zuordnen kann

  • Schnellere Spieler orientierte Schleifen

    Zitat von Kaliber

    Welche Schleife :huh:


    Dann ist der Titel und die erste Zeile in der Include also nur zufällig dahin gekommen?
    Interessant :thumbup:


    Zitat von Kaliber

    Na, h_ID, h = highest ID = playerid ? Finde ich sinvoll...

    Ohne kontext macht das ja auch keinen Sinn - statt static new benutzen - ihr einen vernünftigen Namen geben, so das sie im Script auch verwendbar ist.
    Das hab ich gemeint



    Zitat von Kaliber

    Denk mal über den Code nach, das ist so richtig :rolleyes:

    Das sagt gerade der richtige.
    if(playerid > h_ID) h_ID = playerid;
    Wo ist hier bitte die Logik?


    Spieler mit ID 0 connected h_ID = 0


    for(new i;i<GetID();i++)
    {
    //tue was
    }
    Dieser Code wird absoult nicht ausgeführt.
    Warum weil i 0 ist und GetID 0 zurück gibt -> false da 0 nicht kleiner als 0 ist.


    Sinnvoll wäre es hier h_ID = playerid+1; zu nehmen,
    da sonst einfach ein Slot fehlt



    Zitat von Kaliber

    LoL, das habe ich doch oben schon gesagt, dead meint aber was völlig anderes !

    Dead meinte GENAU das - du hast dich wieder mit "ungefährer Wert" rausgeredet :thumbdown:

  • Schnellere Spieler orientierte Schleifen

    Zitat von Kaliber

    Was würdest du denn Bewerten, den Code oder den sinnvollen Nutzen, der ja für dich irrelevant sein kann, da du es ja nicht nutzen wirst :huh:


    Code 1 Punkt
    (sinnvollen) Nutzen 0 Punkte


    Du hättest 2 Punkte für Nutzen bekommen, wenn du nicht mit static gearbeitet, der Variable einen vernünftigen Namen gegeben und die Funktion GetID rausgelassen hättest.
    Dann wäre es noch sinnvoll und definitv einigermaßen nutzbar.


    Du redest von ressourcen "Schonung", verlangst aber vom Scripter immer wieder eine Funktion in der Schleife aufzurufen, auch wenn die nur einen Wert zurück gibt.
    Ernsthaft was soll der quatsch?
    Die Warscheinlichkeit das ein Spieler in dem momment Connected in der die Schleife gerade läuft, ist gleich 0.


    Du machst wieder genau das, was ich dir letztesmal gesagt habe: Du denkst einfach NICHTS zu Ende.


    Es ist ja schön und gut, wenn du Dinge vereinfachen möchtest und auf ressurcenschonung ziehlst.
    DIe erste Version die du Released hast, war schlimmer als das hier.


    @dead: hat auch vollkommen richtig erkannt, das dieser Code
    if(playerid == h_ID && playerid > 0) h_ID--;
    einfach nicht so funktioniert wie du dir das vorstellst.
    So wie fast alle deine Includes.
    Sei dein Streamer oder deine Limit Include oder dein Inventar "System".


    Beispiel:
    Spieler 1,2,3,6,9 sind auf dem Server
    h_ID steht jetzt auf 9


    ID 9 verlässt den Server.
    Was passiert?
    h_ID steht jetzt auf 8 und das obwohl die nächste höchste ID 6 wäre


    ich find auch diesen Code
    if(playerid > h_ID) h_ID = playerid;
    sehr sinnvoll
    Kein Spieler connected - h_ID ist 0
    Ein Spieler connected - h_ID ist 0
    Noch ein Spieler connected - h_ID ist 1


    Du hast IMMER egal welche Situation, einen "Slot" zuwenig :thumbdown:


    Es tut mir auch nicht wirklich Leid das ich hier kein gutes Haar an der Include hinterlasse,
    ich kann nicht mal etwas positives darüber sagen - was absolut traurig ist.

  • Überprüfen ob Spieler vor der Motorhaube steht - Problem

    Zitat von Martin08

    Dann beweise es ;). Ich kann eigentlich jede Wette eingehen, dass das Addieren nicht funktionieren wird. N bisschen mathematisches sollte man als Programmierer schon mit sich bringen und da ist es doch offensichtlich, dass das so wie oben nicht funktionieren wird.


    Doch das funkoniert, aber NUR wenn die Rotation des Autos auch 0 ist.


    Es gibt ja nicht umsonst die Funktion IsPlayerInFrontOfVehicle die man mit GetVehicleModelInfo kombinieren sollte ;)

  • Inventar System

    Warum machst du bitte nichts ordentlich?


    Dass du static verstanden hast, find ich sogar gut :D
    Nur was mich irgendwie ankotzt ist, das du Systeme schreibst sie päsentierst, aber nicht drüber nachdenkst, sie sinnvoll zu gestalten.


    Beispiel:
    stock GetItem(playerid,slot) {
    if(slot > MAX_SLOTS) return ERROR;
    return Item[playerid][slot];
    }
    Das wird erfolgreich sein immer, egal was kommt, ob ich jetzt über dem "Limit" liege oder nicht.
    Was nützt es, wenn der Rückgabewert "Fehler" ist?
    Weißt du warum kalcor die Funktion GetPlayerName so gestaltet hat wie er es getan hat?


    Nicht weil er keinen anderen Lösungsweg wusste oder weil PAWN da einwenig rummeckert.
    Sondern, weil man damit wunderbar auch prüfen kann, ob der Spieler auch online ist.


    sprich if(GetPlayerName(playerid,name,MAX_PLAYER_NAME)) // wenn online - was dann?


    Das auf dein "Sytsem" zu übertragen wäre überhaupt nicht das Problem.
    Du gibst dann nicht "Fehler" zurück, sondern 0.


    Funktionsheader würde dann so aussehen
    stock GetItem(playerid,slot, name[],len)
    Es ist viel einfacher zu handhaben, als das
    if(!strcmp(GetItem(playerid,25),"Fehler"))


    Man kann es direkt weiterverarbeiten ohne unnötiges Format.
    Auch eins der Gründe warum der Spielername stock - kompletter Müll ist :P


    Dazu ist deine Deklaration, absoulut belustigend :D
    static ...,ERROR[32]="FEHLER",INVALID_SLOT=-1;
    Warum es dem Compiler in Mund schütten, wenn man auch den RAM belasten kann? :rolleyes:


    Item[playerid][s] = "";
    Ein ganz schlimmer Fehler, der dir das "beigebracht" hat - gehört verprügelt.
    Du sorgst hier nur für memory leaks.


    Nutze gleich format oder strdel, dafür sind diese da :thumbup:


    Auch wenn ich mich wiederhole, denk bitte nächstesmal einfach deine Systeme durch,
    nicht das man beim anschauen schon Kopfschmerzen bekommt ;(