cpJobCreator

Wichtiger Hinweis: Bitte ändert nicht manuell die Schriftfarbe auf schwarz sondern belasst es bei der Standardeinstellung. Somit tragt ihr dazu bei dass euer Text auch bei Verwendung unseren dunklen Forenstils noch lesbar ist!

Tipp: Ihr wollt längere Codeausschnitte oder Logfiles bereitstellen? Benutzt unseren eigenen PasteBin-Dienst Link

  • Beitrag von Artus ()

    Dieser Beitrag wurde gelöscht, Informationen über den Löschvorgang sind nicht verfügbar.

    Nicht schlecht, können Anfänger bestimmt gebrauchen
    9/10

    Hab den Code mal überflogen.


    for(new i = 1; i <= MAX_JOBS; i++)
    Das machst du ganz oft und ist leider ganz oft falsch.
    1) Warum nutzt du den Index 0 nicht?
    2) Mit <= ist ein Index out of Bounce vorprogrammiert.
    MAX_JOBS hast du auf 25 festgelegt. Das heißt, du kannst als Index 0 bis MAX_JOBS-1 verwenden. Das ist 0 bis 24.
    In der Schleifer geht es aber bis <= , somit bis 25. 25 ist aber ein nicht erlaubter Indexwert.
    Gleiches gilt auch für
    for(new o=1; o <= MAX_CPS; o++)
    bei zB jobloeschen. Genau genommen hast du durch den Fehler immer einen weniger als durch MAX_* angegeben.
    MAX_JOBS 25 -> Man kann aber nur 24 nutzen
    MAX_CPS 25 -> Man kann aber nur 24 nutzen
    Der Fehler wirkt sich bestimmt noch an ein paar Stellen im Code aus.


    Normalerweise sollte es so aussehen:
    for(new o; o < MAX_CPS ; o++) { }
    for(new i; i < MAX_JOBS ; i++) { }


    for(new i=1; i <= 5; i++) // z.B. Zeile 599
    Wieso nimmst du 5 und nicht MAX_CPS ?

    Großes Dankeschön an dich :) Habe ja erst wieder angefangen. Bin alle Schleifen nochmal durchgegangen. Der Index 0 wird jetzt genutzt, sodass jetzt auch wirklich 25 Jobs & Checkpoint nutzbar sind.


    ich weiß nicht, wie ich auf die 5 gekommen bin. ^^ Wurde aber gefixt.


    Sollten dir weitere Optimierungen auffallen bitte sofort posten. :)


    Werde das Script nochmal durchgehen und dann einfügen.