FUDforum: comp.lang.php » terminate a PHP script

Home » Imported messages » comp.lang.php » terminate a PHP script

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

Re: terminate a PHP script [message #172629 is a reply to message #172576]

Tue, 22 February 2011 16:59

Tim Streater
Messages: 328
Registered: September 2010

Karma:

Senior Member

In article <ijv2mp$3tc$2(at)news(dot)albasani(dot)net>,
The Natural Philosopher <tnp(at)invalid(dot)invalid> wrote:

> Peter H. Coffin wrote:
>> On Mon, 21 Feb 2011 18:50:39 +0000, Tim Streater wrote:
>>> Because of the incredible convolution and depth of nested if/then/else
>>> you can get to. If I have (as it might be) a function that's scanning a
>>> string looking for and obtaining options and their values for some
>>> command previously scanned off the string, and I detect an error, I want
>>> to set an error code and exit right there. I don't want any subsequent
>>> maintainer to have to intuit that, in fact, from this point on nested
>>> 15-deep in else clauses, no more code is executed for that path through
>>> the function.
>>>
>>> That way I rarely get more than 3-deep in if/then/else. To me, it's all
>>> about readbility.
>>
>> *grin* for me, it's all about comprehesibility. If you don't like how
>> deep you're nesting, maybe you need to break something out into a
>> function instead....
>>
> Yes, but how much stack do you have? Can you afford the overhead of a
> call/return?

Shitloads, and yes.

> How much does splitting a whole block of code to four pages
> away make the comprehensibility problem worse, not better?

Not at all, if you think about what you're doing. Here is an actual
piece of code from my application (OK, it's JavaScript and not PHP, but
the idea's the same):

switch (subItems[0])
{

case "C":
composeMail (subItems[1], tabstr);
break;

case "L":
writeLogmsg (subItems[1]);
break;

case "N":
addNewMail (subItems[1]);
break;

case "R":
replaceMail (subItems[1]);
break;

case "T":
tabstr = subItems[1];
break;

case "X":
writeLog ("RC", "failed to generate reply mail");
break;

case "Z":
break;

default:
alert ("replyContinue: " + subItems);
return;

}

Each of the functions referred to has a well-defined purpose - and in
some instances is called from more than one place. But even if there was
just the one place where a call is made, I'd still be likely to do it
this way.

--
Tim

"That excessive bail ought not to be required, nor excessive fines imposed,
nor cruel and unusual punishments inflicted" -- Bill of Rights 1689

Report message to a moderator

[Message index]

		terminate a PHP script By: bob on Sat, 19 February 2011 21:56
		Re: terminate a PHP script By: Jerry Stuckle on Sat, 19 February 2011 21:59
		Re: terminate a PHP script By: The Natural Philosoph on Sun, 20 February 2011 00:38
		Re: terminate a PHP script By: Jerry Stuckle on Sun, 20 February 2011 01:04
		Re: terminate a PHP script By: me on Sun, 20 February 2011 02:07
		Re: terminate a PHP script By: Twayne on Thu, 24 February 2011 01:59
		Re: terminate a PHP script By: Jerry Stuckle on Thu, 24 February 2011 04:04
		Re: terminate a PHP script By: The Natural Philosoph on Thu, 24 February 2011 12:51
		Re: terminate a PHP script By: Twayne on Thu, 24 February 2011 17:26
		Re: terminate a PHP script By: The Natural Philosoph on Thu, 24 February 2011 17:51
		Re: terminate a PHP script By: Thomas 'PointedEars' on Sun, 20 February 2011 09:47
		Re: terminate a PHP script By: me on Sun, 20 February 2011 15:05
		Re: terminate a PHP script By: n00m on Mon, 21 February 2011 12:53
		Re: terminate a PHP script By: Jerry Stuckle on Mon, 21 February 2011 13:08
		Re: terminate a PHP script By: n00m on Mon, 21 February 2011 16:51
		Re: terminate a PHP script By: Denis McMahon on Mon, 21 February 2011 17:06
		Re: terminate a PHP script By: sheldonlg on Mon, 21 February 2011 17:23
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 00:28
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 00:52
		Re: terminate a PHP script By: Jerry Stuckle on Tue, 22 February 2011 01:00
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:46
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 02:08
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 03:55
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 12:06
		Re: terminate a PHP script By: The Natural Philosoph on Wed, 23 February 2011 08:18
		Re: terminate a PHP script By: Tim Streater on Tue, 22 February 2011 17:18
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 17:26
		Re: terminate a PHP script By: The Natural Philosoph on Wed, 23 February 2011 08:21
		Re: terminate a PHP script By: Jerry Stuckle on Mon, 21 February 2011 17:37
		Re: terminate a PHP script By: sheldonlg on Mon, 21 February 2011 17:53
		Re: terminate a PHP script By: n00m on Mon, 21 February 2011 18:01
		Re: terminate a PHP script By: Peter H. Coffin on Mon, 21 February 2011 19:07
		Re: terminate a PHP script By: Curtis Dyer on Tue, 22 February 2011 22:59
		Re: terminate a PHP script By: Tim Streater on Mon, 21 February 2011 18:04
		Re: terminate a PHP script By: sheldonlg on Mon, 21 February 2011 18:32
		Re: terminate a PHP script By: Tim Streater on Mon, 21 February 2011 18:50
		Re: terminate a PHP script By: Peter H. Coffin on Mon, 21 February 2011 19:09
		Re: terminate a PHP script By: Tim Streater on Mon, 21 February 2011 19:44
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:16
		Re: terminate a PHP script By: n00m on Tue, 22 February 2011 01:28
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:47
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 02:10
		Re: terminate a PHP script By: n00m on Tue, 22 February 2011 02:45
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:15
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 01:53
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 03:16
		Re: terminate a PHP script By: Tim Streater on Tue, 22 February 2011 17:01
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 17:05
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 17:07
		Re: terminate a PHP script By: Tim Streater on Tue, 22 February 2011 16:59
		Re: terminate a PHP script By: The Natural Philosoph on Wed, 23 February 2011 08:19
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:12
		Re: terminate a PHP script By: Tim Streater on Tue, 22 February 2011 17:04
		Re: terminate a PHP script By: n00m on Mon, 21 February 2011 18:54
		Re: terminate a PHP script By: Jerry Stuckle on Mon, 21 February 2011 19:39
		Re: terminate a PHP script By: sheldonlg on Mon, 21 February 2011 21:03
		Re: terminate a PHP script By: sheldonlg on Mon, 21 February 2011 21:06
		Re: terminate a PHP script By: n00m on Mon, 21 February 2011 21:46
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:24
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 02:04
		Re: terminate a PHP script By: Jerry Stuckle on Tue, 22 February 2011 02:09
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 03:38
		Re: terminate a PHP script By: sheldonlg on Tue, 22 February 2011 11:54
		Re: terminate a PHP script By: The Natural Philosoph on Wed, 23 February 2011 08:15
		Re: terminate a PHP script By: sheldonlg on Wed, 23 February 2011 13:05
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:23
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 01:06
		Re: terminate a PHP script By: The Natural Philosoph on Tue, 22 February 2011 00:58

Previous Topic:	Extralight browser-webserver communication via cookies (+)
Next Topic:	Storing multiple character set types (or a representation of em) in a table column

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Wed Nov 27 01:36:31 GMT 2024

Total time taken to generate the page: 0.04407 seconds