1. Список говнокодов пользователя Andronix

    Всего: 2

  2. JavaScript / Говнокод #3016

    +159.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    function AbstractControl_getProperty (propertyName) {
	var targetElement = this.getTargetPath(propertyName);
	var result = null;
	
	if (this.isTargetAttribute(propertyName)) {
		eval("result = targetElement." + this.getAttributeName(propertyName));
	} else {
		var getter = this.getGetterName(propertyName);
		var expression = "result = targetElement." + getter + "();";
		eval(expression);
	}
	return result;
}

    Вот так наши "суровые челябинские" программисты, не имеющие представления об интроспективности javascript-а, повсюду злоупотребляют eval-ом, усложняя отладку и понимание кода.
    По хорошему, вместо первого eval-а должно бы быть:
    result = targetElement[this.getAttributeName(propertyName)];
    а вместо второго:
    result = targetElement[this.getGetterName(propertyName)]();

    Andronix, 15 Апреля 2010

    Комментарии (15)

  3. Java / Говнокод #3008

    +74.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    package efi.base.business.metadata;

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

public class MetaModel {
    private String name = null;
    private List attributeNames = new ArrayList();
    private List attributeValues = new ArrayList();

    public MetaModel(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }

    public void addAttribute(Attribute attribute) {
        attributeNames.add(attribute.getName());
        attributeValues.add(attribute);
    }

    public Attribute getAttribute(String attrubuteName) {
        return (Attribute) attributeValues.get(attributeNames.indexOf(attrubuteName));
    }

    public Iterator attributesIterator() {
        return attributeNames.iterator();
    }
}

    Человек, писавший этот говнокласс, видимо и не подозревал, что в JDK кроме листов есть еще и Map-ы :-)

    Andronix, 14 Апреля 2010

    Комментарии (7)