[OT] good logging

2008-07-16 Thread Musachy Barroso
Every time I see this code in S2, it makes me laugh so I will share it
here (DefaultActionInvocation.java):

...
if (proxy == null) {
   gripe = Whoa!  No ActionProxy instance found in current
ActionInvocation.  This is bad ... very bad;
} else if (proxy.getConfig() == null) {
gripe = Sheesh.  Where'd that ActionProxy get to?  I can't find
it in the current ActionInvocation!?;
}

...

musachy
-- 
Hey you! Would you help me to carry the stone? Pink Floyd

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: [OT] good logging

2008-07-16 Thread Jishnu Viswanath
That was funny, any way I checked the code.
catch (Exception e) {
String gripe = ;

if (proxy == null) {
gripe = Whoa!  No ActionProxy instance found in current
ActionInvocation.  This is bad ... very bad;
} else if (proxy.getConfig() == null) {
gripe = Sheesh.  Where'd that ActionProxy get to?  I
can't find it in the current ActionInvocation!?;
} else if (proxy.getConfig().getClassName() == null) {
gripe = No Action defined for ' +
proxy.getActionName() + ' in namespace ' + proxy.getNamespace() + ';
} else {
gripe = Unable to instantiate Action,  +
proxy.getConfig().getClassName() + ,  defined for ' +
proxy.getActionName() + ' in namespace ' + proxy.getNamespace() + ';
}

gripe += ((( --  + e.getMessage()) != null) ?
e.getMessage() :  [no message in exception]);
throw new XWorkException(gripe, e, proxy.getConfig());
}


Take the case proxy is null, then this will bomb rite? What is the point
of the comparison?

throw new XWorkException(gripe, e, proxy.getConfig()); 
Regards,

Jishnu Viswanath

Software Engineer

*(+9180)41190300 - 222(Ext) ll * ( + 91 ) 9731209330ll

Tavant Technologies Inc.,

www.tavant.com

PEOPLE :: PASSION :: EXCELLENCE


-Original Message-
From: Musachy Barroso [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, July 16, 2008 8:43 PM
To: Struts Users Mailing List
Subject: [OT] good logging

Every time I see this code in S2, it makes me laugh so I will share it
here (DefaultActionInvocation.java):

..
if (proxy == null) {
   gripe = Whoa!  No ActionProxy instance found in current
ActionInvocation.  This is bad ... very bad;
} else if (proxy.getConfig() == null) {
gripe = Sheesh.  Where'd that ActionProxy get to?  I can't find
it in the current ActionInvocation!?;
}

..

musachy
-- 
Hey you! Would you help me to carry the stone? Pink Floyd

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Any comments or statements made in this email are not necessarily those of 
Tavant Technologies.
The information transmitted is intended only for the person or entity to which 
it is addressed and may 
contain confidential and/or privileged material. If you have received this in 
error, please contact the 
sender and delete the material from any computer. All e-mails sent from or to 
Tavant Technologies 
may be subject to our monitoring procedures.


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]