Script optimieren

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
  • Hey wie kann ich das hier recoursenfreundlicher schreiben:
    (das ist ein laser system)

    public OnPlayerUpdate(playerid)
    {
    if(GetPlayerWeapon(playerid) == 0)
    {
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);
    return 0;
    }


    if(GetPlayerWeapon(playerid) == 1)
    {
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);
    return 0;
    }
    if(GetPlayerWeapon(playerid) == 2)
    {
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);
    return 0;
    }
    if(GetPlayerWeapon(playerid) == 3)
    {
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);
    return 0;
    }
    if(GetPlayerWeapon(playerid) == 4)
    {
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);
    return 0;
    }


  • Ich denke, dass die Abfragen immer weiter gehen (bzw. alle Waffen-ID's durchlaufen). Liege ich richtig?
    Wenn ja, dann lass einfach alle Abfragen weg, und schreib' nur
    RemovePlayerAttachedObject(playerid, 1);
    RemovePlayerAttachedObject(playerid, 2);


    Ansonsten einfach mehrere Abfragen ineinander verschachteln (if(GetPlayerWeapon(playerid == 0) || GetPlayerWeapon(playerid == 1) || ...))
    Mit freundl. Grüßen
    Padarom

    Ich bin
    .. seit etwa 2007 in der Webentwicklung tätig, seit 2013 professionell
    .. Erfahrener Entwickler in PHP, Swift, Javascript, Typescript und Ruby. Zusätzlich habe ich Erfahrung in Python, Java, C#, C++, Prolog und einigen esoterischen Programmiersprachen
    .. Luftfahrtenthusiast und Segelflieger