[Luis-]

Can anyone tell me whats wrong with this?

pawn Код:

forward OnPlayerPickUpDynamicPickup(playerid, pickupid);
public OnPlayerPickUpDynamicPickup(playerid, pickupid)
{
    switch(pickupid)
    {
        case LowClassCar:
        {
            GameTextForPlayer(playerid, "~g~Low Class Car Dealership~n~~r~/buycar", 6000, 3);
        }
        case VIPCar:
        {
            GameTextForPlayer(playerid, "~r~VIP Dealership~n~~r~/buycar", 6000, 3);
        }
        case CourierInfo:
        {
            ShowPlayerDialog(playerid, 1905, DIALOG_STYLE_MSGBOX, "Courier Help", "{FFFFFF}When doing this job, you're basically delivering supplies to the city of San Andreas. A red marker will appear on your map after you type {00B3FF}/work{FFFFFF}. We hope you find some interesting ideas for this job!", "Okay", "");
        }
        case HeliPad1LS:
        {
            SetPlayerPos(playerid, 1771.7668,-2425.3403,26.2578);
        }
        case HeliPad1LSE:
        {
            SetPlayerPos(playerid, 1771.3595,-2432.6167,13.5547);
        }
    }
    return 1;
}

I am getting these errors:

Код:

C:\Users\BooNii3\Scripts\Project Avaition 0.3d\gamemodes\projectaviation.pwn(5159) : error 008: must be a constant expression; assumed zero
C:\Users\BooNii3\Scripts\Project Avaition 0.3d\gamemodes\projectaviation.pwn(5163) : error 008: must be a constant expression; assumed zero
C:\Users\BooNii3\Scripts\Project Avaition 0.3d\gamemodes\projectaviation.pwn(5167) : error 008: must be a constant expression; assumed zero
C:\Users\BooNii3\Scripts\Project Avaition 0.3d\gamemodes\projectaviation.pwn(5171) : error 008: must be a constant expression; assumed zero
C:\Users\BooNii3\Scripts\Project Avaition 0.3d\gamemodes\projectaviation.pwn(5175) : error 008: must be a constant expression; assumed zero

[sciman001]

Try putting LowClassCar and the other cases into " " ex:
"LowClassCar"
If that doesnt work, Sorry, but idk.

[Luis-]

That's obviously not going to work.

[Flyfishes]

If the variables like 'LowClassCar' was created in another callback and not global it won't work. Cretae them outside any callback and change their value inside a callback if you need to.

[MadeMan]

You can't use variables with switch. You have to use else if

[Ash.]

Quote:

Originally Posted by MadeMan You can't use variables with switch. You have to use else if

This^

You have to use static integers for switch statements, you should use if statements, or you could use the ternary (triadic) operator.

[Scenario]

Are all of your cases defines?

[Phanto90]

Quote:

must be a constant expression; assumed zero

You can't use variables.
You need a code that should not change during execution, so i suggest a macro #define PICKUPID idpickup or simply an enumeration if you know pickupid
Example:

Код:

enum{
pickupinfo1,
pickupinfo2,
pickuphouse1,
pickuphouse2
}

My suggestion is to use if statements because pickup are very dynamic (at least in some gamemodes...)
If you doesn't have a dynamic gamemode that creates and destroyes pickup in game then you can use macros or enumeration.

Best Regards
-P