Proplem mit if else Struktur

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
  • hallo ich habe eine lspd frak gescriptet und wollte die pickups festlegen das nur die lspd leute die aufnehmen können hat auch super funktioniert nur wenn jemand nicht in der lspd ist steht bei jedem pick up in rot "du bist nicht in der LSPD" ich habe mit der if else struktur dann für 1 pick up erfolgreich festlegen können und es erscheint auch nicht mehr bei jedem anderen pick up der satz aber jetzt ist es eben nur für 1 pick up könnt ihr mir helfen?? hier die zeile mit meinem proplem


    public OnPlayerPickUpPickup(playerid, pickupid)
    {
    if(pickupid==lspdwaffen)
    if(isPlayerInFrakt(playerid,1))
    {
    GivePlayerWeapon(playerid,25,100);
    GivePlayerWeapon(playerid,29,100);
    GivePlayerWeapon(playerid,22,100);
    return 1;
    }


    if(pickupid==lspdheal)
    if(isPlayerInFrakt(playerid,1))
    {
    SetPlayerHealth(playerid,100);
    return 1;
    }

    if(pickupid==lspdarmor)
    if(isPlayerInFrakt(playerid,1))
    {
    SetPlayerArmour(playerid, 100);
    return 1;
    }
    else
    SendClientMessage(playerid,ROT,"Du bist nicht in der LSPD.");
    return 0;
    }


    thanks schon mal für help ^^

  • Du hast das nur für den letzten gemacht :D
    Setzt das Else einfach mal hinter jedes Pickup

  • ja das hatte ich mir auch schon gedacht :D aber wenn ich das mach dann passiert das:


    C:\Users\Kiwi\Desktop\Server\gamemodes\Realife.pwn(474) : warning 225: unreachable code
    C:\Users\Kiwi\Desktop\Server\gamemodes\Realife.pwn(484) : warning 225: unreachable code

  • Dann stimmt das was nichz :D
    Aber letztendlich sinds nur Warnings

  • public OnPlayerPickUpPickup(playerid, pickupid)
    {
    if(pickupid==lspdwaffen)
    {
    if(isPlayerInFrakt(playerid,1))
    {
    GivePlayerWeapon(playerid,25,100);
    GivePlayerWeapon(playerid,29,100);
    GivePlayerWeapon(playerid,22,100);
    }
    else
    {
    SendClientMessage(playerid,ROT,"Du bist nicht in der LSPD.");
    }
    return 1;
    }
    else if(pickupid==lspdheal)
    {
    if(isPlayerInFrakt(playerid,1))
    {
    SetPlayerHealth(playerid,100);
    }
    else
    {
    SendClientMessage(playerid,ROT,"Du bist nicht in der LSPD.");
    }
    return 1;
    }
    else if(pickupid==lspdarmor)
    {
    if(isPlayerInFrakt(playerid,1))
    {
    SetPlayerArmour(playerid, 100);
    }
    else
    {
    SendClientMessage(playerid,ROT,"Du bist nicht in der LSPD.");
    }
    return 1;
    }
    }


    //edit: leicht verbessert xD

    //tipp: Die Meldung ''Du bist nicht in der LSPD'' klingt blöd...mach da was anderes hin ;) z.B. ''Du bist kein Beamter'' oder so..

    2 Mal editiert, zuletzt von Shine ()

  • ok danke an alle die mir jetzt so schnell geantwortet haben :D
    shine seine verbesserung is super funktioniert alles prima ich hab nur return 0; nach jedem else gesetzt
    und ja is ja auch nur zum testen bisher shine ^^

  • do.de - Domain-Offensive - Domains für alle und zu super Preisen