
WeaponObject(wid) {
if(wid == 1) return 331;
else if(wid == 2) return 332;
else if(wid == 3) return 333;
else if(wid == 5) return 334;
else if(wid == 6) return 335;
else if(wid == 7) return 336;
else if(wid == 10) return 321;
else if(wid == 11) return 322;
else if(wid == 12) return 323;
else if(wid == 13) return 324;
else if(wid == 14) return 325;
else if(wid == 15) return 326;
else if(wid == 23) return 347;
else if(wid == 24) return 348;
else if(wid == 25) return 349;
else if(wid == 26) return 350;
else if(wid == 27) return 351;
else if(wid == 28) return 352;
else if(wid == 29) return 353;
else if(wid == 30) return 355;
else if(wid == 31) return 356;
else if(wid == 32) return 372;
else if(wid == 33) return 357;
else if(wid == 4) return 335;
else if(wid == 34) return 358;
else if(wid == 41) return 365;
else if(wid == 42) return 366;
else if(wid == 43) return 367;
return 0;
}
|
Ok it`s nice, I really like it, but do some optimisation, first thing, reduce statement if with loop switch, second thing is instead of using "function", use stock.
|
|
Hello. It's look good but you can do some optimization to your code.
Code:
WeaponObject(wid) {
if(wid == 1) return 331;
else if(wid == 2) return 332;
else if(wid == 3) return 333;
else if(wid == 5) return 334;
else if(wid == 6) return 335;
else if(wid == 7) return 336;
else if(wid == 10) return 321;
else if(wid == 11) return 322;
else if(wid == 12) return 323;
else if(wid == 13) return 324;
else if(wid == 14) return 325;
else if(wid == 15) return 326;
else if(wid == 23) return 347;
else if(wid == 24) return 348;
else if(wid == 25) return 349;
else if(wid == 26) return 350;
else if(wid == 27) return 351;
else if(wid == 28) return 352;
else if(wid == 29) return 353;
else if(wid == 30) return 355;
else if(wid == 31) return 356;
else if(wid == 32) return 372;
else if(wid == 33) return 357;
else if(wid == 4) return 335;
else if(wid == 34) return 358;
else if(wid == 41) return 365;
else if(wid == 42) return 366;
else if(wid == 43) return 367;
return 0;
}
|