Mistři v programování na #hovnokod

Co že tam chybí ? Ano, break.

        boolean contains = false;        
        for (Item item : items) {
            if (item.equals(pattern)) {
                contains = true;
            }
        }

Anonymous,

Ze vzorové implementace mojeID

package cz.xxx.client.openid.util;

import javax.servlet.http.HttpServletResponse;

/**
 * Builder pro sestaveni URL adresy. 
 * @author xxx
 */
public class UrlBuilder {	
	//=============== ATRIBUTY ==================================================
	private StringBuffer url;

	//============== KOSTRUKTORY A TOVARNI METODY ===============================
	public UrlBuilder() {
		this.url = new StringBuffer();
	}
	
	public UrlBuilder(String url) {
		this();
		this.url.append(url);
	}

	
	
	//============== METODY TRIDY ===============================================

	//============== VEREJNE METODY INSTANCE ====================================
	/**
	 * Pripoji parametr k URL.
	 * @param name
	 * @param value
	 * @return
	 */
	public UrlBuilder appendParam(String name, String value){
		if( url.indexOf("?") == -1 ) {
			this.url.append("?");
		}else{
			this.url.append("&");
		}
		
		this.url.append(name).append("=").append(value);
		return this;			
	}
	
	/**
	 * Nastavi uplne novou URL, smaze predchozi hodnotu URL.
	 * @param url
	 */
	public void setUrl(String url){
		this.url = new StringBuffer(url);
	}
	
	/**
	 * Vrati sestavenou URL.
	 * @return
	 */
	public String build(){
		return this.url.toString();			
	}
	
	/**
	 * Vrati sestavenou a encodovanou URL. 
	 * @param httpResp
	 * @return
	 */
	public String buildAndEncode(HttpServletResponse httpResp){
		return httpResp.encodeRedirectURL( this.url.toString() );			
	}
	

	//============== SOUKROME METODY INSTANCE ===================================

	//============== VNORENE A VNITRNI TRIDY ====================================

	//============== OSTATNÍ (MAIN A AUTOMATICKY GENEROVANE METODY) =============
}

Anonymous,

Myslím, že jsem právě v projektu našel větší hovno, než je v http://www.hovnokod.cz/4443 Místa, kde se ta metoda používá, taky stojí za to...bravo pane "vývojáři"!!

private Map<String, Triple<String, Map<String, String>, Pair<Set<Triple<LinkType, String, String>>, Set<Triple<LinkType, String, String>>>>> getTripleMap(...) {
    // ...
}

Anonymous,

Při review jsem dostal pochvalu za odstranění unused metody :D

private Product createProduct() {
    return null;
}

Anonymous,

Protože dělat věci jednoduše je pro losery...

if (btn != null) {

    switch (btn.getId()) {
        case 1:
            processAction(contactId, name, 2);
            break;
        case 2:
            processAction(contactId, name, 3);
            break;
        case 3:
            processAction(contactId, name, 4);
            break;
        case 4:
            processAction(contactId, name, 5);
            break;
        case 5:
            processAction(contactId, name, 6);
            break;
        case 6:
            processAction(contactId, name, 7);
            break;
        case 7:
            processAction(contactId, name, 8);
            break;
        default:
            break;
    }
}

Anonymous,