1. 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) RSS

    Добавить комментарий