RE: Modify caller or callee?

[BMay@xxxxxxxxx]

Considering I'm in charge of that now, I will put that on the suggestion 
list.  Thanks, Larry.

Brian May
Project Lead
Management Information Systems
Garan, Incorporated
Starkville, Mississippi



"Ketzes, Larry" <Larry.Ketzes@xxxxxxxxx> 
Sent by: midrange-l-bounces@xxxxxxxxxxxx
09/09/2010 08:53 AM
Please respond to
Midrange Systems Technical Discussion <midrange-l@xxxxxxxxxxxx>


To
"'Midrange Systems Technical Discussion'" <midrange-l@xxxxxxxxxxxx>
cc

Subject
RE: Modify caller or callee?






5 Man Electrical Band did the original, and the band TESLA redid the tune 
and does it proud in their current live shows.  You can't mess with me 
when it comes to 60's and 70's music.  I hope COMMON has a 'Rock N Roll 
Jeopardy' next annual conference at the fun n games night!)

-----Original Message-----
From: midrange-l-bounces@xxxxxxxxxxxx 
[mailto:midrange-l-bounces@xxxxxxxxxxxx] On Behalf Of Harman, Roger
Sent: Thursday, September 09, 2010 8:55 AM
To: 'Midrange Systems Technical Discussion'
Subject: RE: Modify caller or callee?

Signs, Signs, Everywhere Signs.....


-----Original Message-----
From: midrange-l-bounces@xxxxxxxxxxxx 
[mailto:midrange-l-bounces@xxxxxxxxxxxx] On Behalf Of Bryce Martin
Sent: Thursday, September 09, 2010 5:27 AM
To: Midrange Systems Technical Discussion
Subject: RE: Modify caller or callee?

hahahahahaha!...  off topic, I know, but that was the best music based 
analogy I've seen in a really long time.   8-)


Thanks
Bryce Martin
Programmer/Analyst I
570-546-4777



"Dennis Lovelady" <iseries@xxxxxxxxxxxx> 
Sent by: midrange-l-bounces@xxxxxxxxxxxx
09/09/2010 07:47 AM
Please respond to
Midrange Systems Technical Discussion <midrange-l@xxxxxxxxxxxx>


To
"'Midrange Systems Technical Discussion'" <midrange-l@xxxxxxxxxxxx>
cc

Subject
RE: Modify caller or callee?






Oh, it's a sign!!!

If admit() is never meant to handle long-haired freaky people, then the
modification belongs in admit(), and probably an error should be thrown.

If it is possible that admit() may be exposed for other modules, and those
modules might someday need admit() to handle such as that, the 
modification
belongs at the caller level.  (But beware of those who may tuck their hair
up under their hat.)

Dennis Lovelady
http://www.linkedin.com/in/dennislovelady
--
"Don't knock the weather; nine tenths of the people couldn't start a
conversation if it didn't change once in a while."
        -- Kin Hubbard 


> Adapted from a real-life case :
>
>
> GetClient  (ClientID);
>
> If client_hair_brown and
>    client_blue_eyes;
>    Admit_Client ( ClientID ); // external subprocedure
>
> Endif;
>
>
> // Admit_Client
> GetClient  (ClientID);
> DoStuff();
>
>
> Modification : Admit_Client must not be performed if client has long
> hair.
>
>
> Does the modification go in Admit_Client, eg,
> // Admit_Client
> GetClient  (ClientID);
> If client_hair_long;
>    return;
> Else;
>     DoStuff();
> Endif;
>
>
> Or in all the callers?
> If client_hair_brown and
>    client_blue_eyes and
>     not client_hair_long;
>    Admit_Client ( ClientID ); // external subprocedure
>
> Endif;
>
>
>
>
>
> This would mean duplication of code, but putting the modification in
> Admit_Client could be misleading to the person reading the caller's
> code.