Thread

That’s one clever piece of code! 😉

That is awesome.  Hopefully one day I will them skillz.

Whats funny with it???I don’t figure it out….Maybe I’m just some kind a stupid…lol

It’s redundant, it serves no purpose.

Whats funny with it???I don’t figure it out….Maybe I’m just some kind a stupid…lol

The funny thing is (assuming it’s found in the form validation library) that it’s longer to write $this->is_numeric($val) than is_numeric($val). And the purpose of the method is probably not going to change (either it returns false on non-numeric values or true on numeric ones).

Somewhat “geeky” to notice but still funny! 😊

Whats funny with it???I don’t figure it out….Maybe I’m just some kind a stupid…lol

The funny thing is (assuming it’s found in the form validation library) that it’s longer to write $this->is_numeric($val) than is_numeric($val). And the purpose of the method is probably not going to change (either it returns false on non-numeric values or true on numeric ones).

Somewhat “geeky” to notice but still funny! 😊

Not only that, the method could have been written more simply like this:

function is_numeric($str)
{
    return is_numeric($str);
}

Have you tried calling that? And what happened?

why not just use the php ‘is_numeric’? why use the helper when the native php is already there?

Sorry to bump an oldish post, but this thread reminded me of some funny code I saw on another site…

public boolean isBooleanFalse(boolean value) {
   boolean response = false;
   if (value == true) {
       response = false;
   } else {
       response = true;
   }
   return response;
}

😊

You know you’re a nerd when you see things like that and find them funny. Needless to say, I laughed when I read that. Haha.

Haaaaaax. Leet skills at camp CI 😉

I can see why they’d do this though. Just to keep everything kinda “uniform” I guess, rather than having some functions prefixed with a class call like $this->is_alpha_numeric() and others that are calls just to native PHP like is_numeric().

Personally I’d do what they did and just keep everything uniform, otherwise it can be confusing for the user deciding which ones to use.

Awesome find though 😊

I believe this snippet is rogue code. It’s not being used anywhere and in fact, the PHP built-in function is_numeric() gets called all over the place.

This is for the Form Validation library.  All rules are basically just callbacks.  So the is_numeric function is required in order to have the is_numeric validation rule.  It’s not that big of deal.

Dan,
Form Validation already suport native php methods that accepts one parameter, so it still makes no sense =)

Any native PHP function that accepts one parameter can be used as a rule, like htmlspecialchars, trim, MD5, etc.