[horsemeat]

well the title says it all how do you make this code smaller becuse it won't compile

pawn Код:

if((choicevehiclemid[playerid])==435||(choicevehiclemid[playerid])==441||(choicevehiclemid[playerid])==449||(choicevehiclemid[playerid])==450||(choicevehiclemid[playerid])==464||(choicevehiclemid[playerid])==465||(choicevehiclemid[playerid])==501||(choicevehiclemid[playerid])==537||(choicevehiclemid[playerid])==538||(choicevehiclemid[playerid])==539||(choicevehiclemid[playerid])==564||(choicevehiclemid[playerid])==569||(choicevehiclemid[playerid])==570||(choicevehiclemid[playerid])==584||(choicevehiclemid[playerid])==590||(choicevehiclemid[playerid])==591||(choicevehiclemid[playerid])==594||(choicevehiclemid[playerid])==606||(choicevehiclemid[playerid])==607||(choicevehiclemid[playerid])==608||(choicevehiclemid[playerid])==610||(choicevehiclemid[playerid])==611)

I could use 2 if statement but I like to see if there is an easier way

[iJumbo]

pawn Код:

if(choicevehiclemid[playerid] == 435 || choicevehiclemid[playerid] == 441 ||
choicevehiclemid[playerid] == 449 || choicevehiclemid[playerid] == 450 || 
choicevehiclemid[playerid] == 464 || choicevehiclemid[playerid] == 465 ||
choicevehiclemid[playerid] == 501 || choicevehiclemid[playerid] == 537 || 
choicevehiclemid[playerid] == 538 || choicevehiclemid[playerid] == 539 ||
choicevehiclemid[playerid] == 564 || choicevehiclemid[playerid] == 569 ||
choicevehiclemid[playerid] == 570 || choicevehiclemid[playerid] == 584 ||
choicevehiclemid[playerid] == 590 || choicevehiclemid[playerid] == 591 ||
choicevehiclemid[playerid] == 594 || choicevehiclemid[playerid] == 606 ||
choicevehiclemid[playerid] == 607 || choicevehiclemid[playerid] == 608 ||
choicevehiclemid[playerid] == 610 || choicevehiclemid[playerid] == 611) {


}

[horsemeat]

Quote:

Originally Posted by iJumbo pawn Код: if(choicevehiclemid[playerid] == 435 || choicevehiclemid[playerid] == 441 || choicevehiclemid[playerid] == 449 || choicevehiclemid[playerid] == 450 || choicevehiclemid[playerid] == 464 || choicevehiclemid[playerid] == 465 || choicevehiclemid[playerid] == 501 || choicevehiclemid[playerid] == 537 || choicevehiclemid[playerid] == 538 || choicevehiclemid[playerid] == 539 || choicevehiclemid[playerid] == 564 || choicevehiclemid[playerid] == 569 || choicevehiclemid[playerid] == 570 || choicevehiclemid[playerid] == 584 || choicevehiclemid[playerid] == 590 || choicevehiclemid[playerid] == 591 || choicevehiclemid[playerid] == 594 || choicevehiclemid[playerid] == 606 || choicevehiclemid[playerid] == 607 || choicevehiclemid[playerid] == 608 || choicevehiclemid[playerid] == 610 || choicevehiclemid[playerid] == 611) { }

Thanks I will use this it not exactly what I wanted but this at least fixes the code problem is there any other way of doing this so it is more condense

[MP2]

You'd generally use an array for this.

pawn Код:

stock IsChoiceVehMid(playerid)
{
    stock const gChoiceVehMid[] = {
    435,
    441,
    449,
    450,
    464,
    465,
    501,
    537,
    538,
    539,
    564,
    569,
    570,
    584,
    590,
    591,
    594,
    606,
    607,
    608,
    610,
    611
    };

    for(new i=0; i<sizeof(gChoiceVehMid); i++)
    {
        if(gChoiceVehMid[i] == choicevehiclemid[playerid]) return 1;
    }
    return 0;
}

Not tested, and I wasn't sure what to call the function/array etc. because I have no idea what the hell it is.

Simply replace your code with this:

pawn Код:

if(IsChoiceVehMid(playerid))