FUDforum: comp.lang.php » switch with case 0

Home » Imported messages » comp.lang.php » switch with case 0

Show: Today's Messages :: Polls :: Message Navigator

Re: switch with case 0 [message #182593 is a reply to message #182590]

Sat, 17 August 2013 11:16

Thomas Mlynarczyk
Messages: 131
Registered: September 2010

Karma:

Senior Member

J.O. Aho schrieb:

> PHP will make a type casting and try to make the "foo" to a integer

And that's precisely what PHP shouldn't do. Cast an integer to a string?
Yes, no problem, since this can be done without loss of information. But
the other way round, especially if the string is obviously not
representing a numeric value? That's just plain stupid.

PHP should never, never, never try to parse out anything from a string
*unless* explicitly told to do so. And throw an error or return `null`
in situations like `(int) "foo"`.

> PHP don't care so much about data
> types so it internally casts it just to make it easy for those who ain't
> so good at programming.

Easy?!? No it doesn't. On the contrary. A "noob" programmer could write
a piece of code like the OPs first example (switch ("foo") ... case 0
....) and rightfully assume it "to work", because in what strange
parallel universe "foo" could possibly equal 0? Even disregarding the
fact that they are of different types. "foo" == 0? WTF?!?

>> switch ($mode) {
>> case SORT_ASC:
>> case SORT_DESC:
>> case SORT_REGULAR:
>> case SORT_NUMERIC:
>> case SORT_STRING:
>> case SORT_LOCALE_STRING:
>> case SORT_NATURAL:
>> case SORT_FLAG_CASE:
>> break;
>> default:
>> throw new \InvalidArgumentException("invalid sort mode: $mode");
>> }

> Sounds like you are trusting user input

No he isn't, as the above code clearly shows. It would work perfectly
well in any normal programming language (although, personally, I would
have used in_array() instead of switch).

Greetings,
Thomas

--
Ce n'est pas parce qu'ils sont nombreux à avoir tort qu'ils ont raison!
(Coluche)

Report message to a moderator

[Message index]

		switch with case 0 By: Fiver on Sat, 17 August 2013 04:55
		Re: switch with case 0 By: J.O. Aho on Sat, 17 August 2013 06:35
		Re: switch with case 0 By: The Natural Philosoph on Sat, 17 August 2013 07:29
		Re: switch with case 0 By: Luuk on Sat, 17 August 2013 07:51
		Re: switch with case 0 By: Thomas Mlynarczyk on Sat, 17 August 2013 11:16
		Re: switch with case 0 By: Peter H. Coffin on Sat, 17 August 2013 15:56
		Re: switch with case 0 By: The Natural Philosoph on Sat, 17 August 2013 16:45
		Re: switch with case 0 By: Tim Streater on Sat, 17 August 2013 17:58
		Re: switch with case 0 By: The Natural Philosoph on Sat, 17 August 2013 18:39
		Re: switch with case 0 By: Thomas Mlynarczyk on Sat, 17 August 2013 19:36
		Re: switch with case 0 By: Thomas Mlynarczyk on Sat, 17 August 2013 20:14
		Re: switch with case 0 By: Norman Peelman on Sat, 17 August 2013 17:56
		Re: switch with case 0 By: Thomas Mlynarczyk on Sat, 17 August 2013 19:55
		Re: switch with case 0 By: The Natural Philosoph on Sat, 17 August 2013 20:04
		Re: switch with case 0 By: Thomas Mlynarczyk on Sat, 17 August 2013 20:32
		Re: switch with case 0 By: Richard Yates on Sat, 17 August 2013 22:20
		Re: switch with case 0 By: Fiver on Sat, 17 August 2013 20:37
		Re: switch with case 0 By: Norman Peelman on Sat, 17 August 2013 23:26
		Re: switch with case 0 By: The Natural Philosoph on Sun, 18 August 2013 11:54
		Re: switch with case 0 By: Norman Peelman on Sun, 18 August 2013 00:23
		Re: switch with case 0 By: Thomas Mlynarczyk on Sun, 18 August 2013 13:22
		Re: switch with case 0 By: Norman Peelman on Sun, 18 August 2013 15:20
		Re: switch with case 0 By: Thomas Mlynarczyk on Sun, 18 August 2013 16:28
		Re: switch with case 0 By: Norman Peelman on Sun, 18 August 2013 19:16
		Re: switch with case 0 By: Norman Peelman on Sun, 18 August 2013 22:04
		Re: switch with case 0 By: Thomas Mlynarczyk on Sun, 18 August 2013 22:17
		Re: switch with case 0 By: Thomas Mlynarczyk on Sun, 18 August 2013 22:13
		Re: switch with case 0 By: The Natural Philosoph on Sun, 18 August 2013 23:35
		Re: switch with case 0 By: Norman Peelman on Mon, 19 August 2013 10:30
		Re: switch with case 0 By: Norman Peelman on Mon, 19 August 2013 02:13
		Re: switch with case 0 By: Thomas Mlynarczyk on Mon, 19 August 2013 17:55
		Re: switch with case 0 By: Thomas 'PointedEars' on Mon, 19 August 2013 08:18
		Re: switch with case 0 By: Thomas Mlynarczyk on Mon, 19 August 2013 17:39
		Re: switch with case 0 By: Fiver on Sat, 17 August 2013 14:15
		Re: switch with case 0 By: Norman Peelman on Sat, 17 August 2013 17:28

Previous Topic:	AND and OR
Next Topic:	Compiling PHP 5.x on Cygwin

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Sun Nov 24 18:37:02 GMT 2024

Total time taken to generate the page: 0.04747 seconds