caesar wrote:I got to admit that Get_A_Fix's method of removing the bot from the nick list is much better than having a isbotnick check inside a foreach loop.
Not for me, this removes the bots nick from the channel list. The bot is in the channel. This is incorrect. This is lying. My method highlights the bots nick. My method is accurate and honest.
caesar wrote:Code: Select all
} elseif {[string equal -nocase $nick $n]} {
Shouldn't
$n be a
$t?
Yes, i fixed that miniscule mistake.
caesar wrote:Also, elseif statements should be better suited instead of plain if statements in the proc:mode function, or at least append that mode to mode variable if you wish to check if the user has two or multiple modes set, or leave it as it is if you wish to display only the highest one.
No, those MUST be left as individual if statements. A user can have MORE than one mode set. This is how it is intended. Your method is flawed. It is not accurate nor honest. It is lying, which I won't do.
---
Now, don't get me wrong. There are several ways to skin a cat. But when a poacher approaches you, and tells you that there are better ways to skin your cat and he can show you how. It appears condescending and low.
For example:
Shouldn't $n be a $t? ![Smile :)](./images/smilies/icon_smile.gif)
<-- do you notice the smiley? It's there as a "neener neener" for no other reason.
This is something equated with those who can't feel good about themselves, without making others feel bad. In this case, nit-picking and mis-understanding concepts has me feeling like some forum members always "just talk" and sit back on their laurels. They never provide any code of their own, just rip apart other peoples. Judge not, lest ye be judged. Keep this in mind. This is whats happening right now..
It doesn't matter what you like, or how you want things. The original posters question is the focus of this thread. Not trying to fit your whims..