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; } |