в eval() можно выполнить произвольный код, собственно из этого следует потенциальная угроза безопасности...
По поводу return то просто не стал приводить полное тело готовой функции, а описал модификацию предыдущей, которая достает значение.

Тем, что он запрещен в ES5 Strict Mode, тем, что можно выполнить произвольный код, тем, что он выполняется в глобальном пространстве имен, тем, что использование эвала это просто-напросто считается плохим тоном в JS, как и любой другой говнокод.