Beiträge von StreetFight

    Die Lösung ist doch eigentlich ganz einfach.

    Sofern die Funktion "getPlayerPetByPIDAndSealVnum" nicht vom Kontext des Triggers abhängt.


    unter pc.setqf("pet_vnum", mobVnum)

    pc.setqf("seal_vnum", sealVnum)


    und im login trigger einfach:


    local srcPet = getPlayerPetByPIDAndSealVnum(pc.get_player_id(), pc.getqf("seal_vnum"))

    pet.summon(pc.getqf("pet_vnum"), "'s "..srcPet:getName() , false)

    Bin ich voll deiner Meinung man merkt einfach das viele Leute über m2 programmieren gelernt haben.

    Was ja auch nicht so schlimm ist wenn man danach mal anfängt richtig programmieren zu lernen.

    Es gibt einfach zu viele Leute die den gleichen C++98 bzw. C Stuff vom m2 code abschreiben(std::auto_ptr, c casts, etc.).

    Was ja noch irgendwie verkraftbar wäre wenn wenigstens der Rest passt. Aber es ist eine Hölle irgend ein System von jemanden zu fixxen das voll ist mit Magic Numbers, endlose vielen Verschachtelungen, code kopien, rießig langen Funktionen, schlechten Variablen, Klassen und Funktions-Namen, overused Patterns(singleton) usw.

    Bitte bitte möchte dich unbedingt als meinen Chef haben für dein hammer geiles Projekt.

    *hust*

    JavaScript
    1. if pc.get_skill_level(51) == 30 then
    2. end

    Ich weiß natürlich nicht auswendig was diese Funktion erwartet und was sie zurück gibt :S


    Dir fehlen bei deinem Haufen den du Quest nennst überall ends.
    Was soll den bitte

    JavaScript
    1. if = pc.get_skill_level(51, 30) then begin

    sein ?????
    Jedes when hat ein end und jedes if hat ein then.
    und was soll diese Zuweisung sein ?
    Du solltest dir einen Quester holen weil hier ist alles falsch was man falsch machen kann.
    richtig wäre:

    JavaScript
    1. if pc.get_skill_level(51, 30) then
    2. end

    Einige andere Anmerkungen:
    1)jeder Block sollte eingerückt sein, so wie das hier aussieht will und kann dir keiner helfen.
    2)Man verwendet keine Magic Numbers . (Bitte melden Sie sich an, um diesen Link zu sehen.)
    3)Die ganze Logik hätte man in eine Funktion auslagern sollen und nicht 10 mal kopieren dann hätte deine Quest nur noch 10-15 Zeilen.
    4)Du übergibst Funktionen String-Literale die Numerische Werte erwarten.(z.B pc.remove_item("50512"))
    Falls du trotzdem weiter Lua coden willst solltest du dir zuminderst mal das hier ansehen kostet auch nicht viel:


    Bitte melden Sie sich an, um diesen Link zu sehen.

    das Verdient Respekt da hast du aber mal ein ordentliches Lua-Script erstellt "1815 Zeilen Code" sind schon eine Menge ;)


    #Respekt Risan

    Du weißt aber schon das, dass der größte Müll ever ist mit deutschen Variablennamen und 100000 Code kopien ?

    Hier wurde dann wohl mit hard gecodeden zahlen gearbeitet anstatt sich an der größe des Fensters zu orentieren.

    wie wärs hier mit(habs nicht getestet bin in der Arbeit) :


    Bitte melden Sie sich an, um diesen Link zu sehen. Ich versteh ja was du ihm sagen willst, aber so hilft das keinem
    Bitte melden Sie sich an, um diesen Link zu sehen. Du bist jetzt länger hier im Forum, wie wäre es mal wenn man sich an ratschlägt hält? zu dem syserr gibt es eine Datei (in dem Falle dürfte es die interfaceModule.py sein)


    deinen genauen Fehler kann ekiner vorhersehen..
    hast du das Transmutations System verbaut? oder auch Changelook. Wenn ja mach noch mal das Tutorial, wenn nein musst du die betreffenen Bereich rausnehmen oder auskommentieren.

    Da Steht alles drin ?
    Ich weiß nicht was ihr aus der Fehlermeldung zieht aber ich ziehe daraus genau das was drin steht.
    Es wird wahrscheinlich in class InventoryWindow eine Funktion SetChangeLookWindow benötigt.
    Aus dem Tutorial in uiinventory.InventoryWindow die Funktion rauskopieren und in deinen Code reinkopieren ?

    Kannst ja einfach eine Lua Funktion im Source bauen.
    Außerdem stellt sich ja noch die Frage wie man den Schaden berechnet ? Den Durchschnitsschaden ? mit db und krit ? ohne db und krit ? ohne die ganzen stark gegen xyz oder gegen einen bestimmten mob ? temporärer push wird der mitberechnet ?

    das heißt beide sind müll aber der von great ist noch schlechter deswegen ken benutzen?

    Also muss zugeben ist schon lange her das ich den Code von Kens offlineshop gesehen habe.
    Beim Great Offlineshop isses ja so das der einfach client Pakete entgegen nimmt und strlen drauf aufruft den rest kannst du dir ja denken :D
    Außerdem ist alles voll mit magic numbers, unsicheren operationen, code kopien etc.