Tenho de responder também pois o eval() não é tão mau como muita gente pensa, depende de como e onde se usa. Conduzir um carro é perigoso se não se souber como o fazer.
Para quem não domina javascript o eval pode assustar. E aí é mais fácil generalizar e dizer que não se deve usar nunca. Ora não é o caso.
O eval é uma das ferramentas mais poderosa do javascript. É um parser em sí. O maior problema do eval surge quando ligado a comunicação com o servidor. Mas se usar o eval em funções internas do código, sem input externo o eval pode ser muito util e seguro.
Vale a pena referir (e ter em conta) que qualquer utilizador pode chegar a um site, abrir a consola e correr código maléfico com eval.
No caso de se ter eval em linhas de código que fazem ligação a uma base de dados por exemplo, é importante fazer por blindar possiveis ataques. Nestes casos evite o eval.
A primeira parte da minha resposta é positiva ao eval, isto para dizer que há casos em que ele é util (como aqui por exemplocomo aqui por exemplo).
Porém ele tem contras de peso:
O mau uso do eval pode levar a ataques de injeção de código
Mais dificil de fazer debug (uma vez que dentro do eval os error não têm indicação de linha)
mais lento que código similar sem eval
dificulta e limita a minificação do código
Por fim vale a pena referir que o eval é usado no javascript escondido. Mais sobre isso nesta perguntanesta pergunta.