[Neuro Toxin]

Everytime I see IsValidClient I instantly think bad code.

[ThatKidWhoGames]

Originally Posted by Neuro Toxin Everytime I see IsValidClient I instantly think bad code.

In a loop (or a situation where you only need to use IsClientInGame), I agree.

[Vaggelis]

Originally Posted by Neuro Toxin Everytime I see IsValidClient I instantly think bad code.

You deserve a cookie

[Ilusion9]

Originally Posted by Vaggelis Whats the point posting that code? Edison1318 got what he wanted

your question is so dumb
i offered him another solution for the problem, support for health > 100.

[Vaggelis]

Originally Posted by Ilusion9 your question is so dumb i offered him another solution for the problem, support for health > 100.

Dumb? If you posted that code only to support health > 100 propably you are not able to think that you can set min to a higher value like 99999, also you have a variable named size for no reason.

[Psyk0tik]

Originally Posted by Vaggelis Dumb? If you posted that code only to support health > 100 propably you are not able to think that you can set min to a higher value like 99999, also you have a variable named size for no reason.

It sounds like a dumb question because there's nothing wrong with posting an alternative to a solution. It also acts as a visual representation of what he was saying to Shane about not needing IsClientConnected when IsClientInGame is already being used. Plus his code looks more optimized than the one Shane provided. I'm not hating on Shane, its just an observation.

[Vaggelis]

Originally Posted by Crasher_3637 It sounds like a dumb question because there's nothing wrong with posting an alternative to a solution. It also acts as a visual representation of what he was saying to Shane about not needing IsClientConnected when IsClientInGame is already being used. Plus his code looks more optimized than the one Shane provided. I'm not hating on Shane, its just an observation.

Shane could see my code, also how can you say its an alternative code? the only thing he did, was to
add extra statements to support "health > 100" while you can do that thing with my code

[Ilusion9]

Originally Posted by Vaggelis Shane could see my code, also how can you say its an alternative code? the only thing he did, was to add extra statements to support "health > 100" while you can do that thing with my code

you should start to learn some algorithms and see why is better to initialise the minimum with the first element.
that size = 0 was for multiple clients with the same minimum health, but i was too lazy to do that.

this is the scripting forum, anyone can post ideas or code. if you ask something here, you should be able to understand, to optimize and to correct someone else's code.

[Vaggelis]

Originally Posted by Ilusion9 you should start to learn some algorithms and see why is better to initialise the minimum with the first element. that size = 0 was for multiple clients with the same minimum health, but i was too lazy to do that. this is the scripting forum, anyone can post ideas or code. if you ask something here, you should be able to understand, to optimize and to correct someone else's code.

You said to me learn some algorithms? Nice joke. Its not a must to always initialize the minimum from the first element if you know the maximum clever boy, also i dont see any optimization in your code, only the opposite.