Coding guidelines for debugging improvement

"Wolfgang Fahl" <[email protected]>
Newsgroups gmane.comp.web.httpunit.devel
Organization BITPlan GmbH
Message-ID <[email protected]>
Dear httpunit developers,

may I suggest some coding guideline improvements to make 
debugging easier? To set breakpoints it's necessary that 
the statements need to be on separate lines.

1. Avoid use of ? operator
instead of 
	return a ? b:c 
please do
	result=undefined
	if (a)
		result=b;
	else
		result=c;
	return result

2. Assign before return if intermediate results could 
	be interesting or problematic (e.g. null)

	instead of
		return f(g(h(i(j(param)))))

 	jr=j(param)
	ir=i(jr)
	hr=h(ir)
	gr=g(hr)
	fr=f(gr)
	return f;

3. Multiple lines for interesting statements
so instead of:
        for (int i = 0; i < controls.length; i++) controls[i].claimRequiredValues( list );
        for (int i = 0; i < controls.length; i++) controls[i].claimUniqueValue( list );
        for (int i = 0; i < controls.length; i++) controls[i].claimValue( list );
        if (!list.isEmpty()) throw 	new UnusedParameterValueException( _name, (String) list.get(0) );


    FormControl[] controls = getControls();
        for (int i = 0; i < controls.length; i++) 
        	  controls[i].claimRequiredValues( list );
        for (int i = 0; i < controls.length; i++) 
        	  controls[i].claimUniqueValue( list );
        for (int i = 0; i < controls.length; i++) 
        	  controls[i].claimValue( list );
        if (!list.isEmpty()) throw 	
        	  new UnusedParameterValueException( _name, (String) list.get(0) );






Thank you

  Yours
     Wolfgang	

BITPlan - smart solutions
Pater-Delp-Str. 1, D-47877 Willich Schiefbahn
Tel. +49 1805 - BITPLAN / +49 1805 248 752, Fax +49 2154 811-481
Web: http://www.bitplan.de
bitplan GmbH, Willich - HRB 6820 Krefeld, VAT-ID: 10258040548, 
Geschäftsführer: Wolfgang Fahl

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace

_______________________________________________
Httpunit-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/httpunit-develop
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.