Ask YC: What's the problem with eval()?

https://news.ycombinator.com/item?id=92750

by edw519 • 19 years ago

5 8 19 years ago

news.ycombinator.com

I have a scenario where I want to do some logic in the browser, but I don't know what that logic is until run time. As I see it, I have 2 choices: 1. Preload the page with some javascript logic to process pseudo-code sent down as data, or 2. Just send javascript down as data and eval() it. I really don't want to preload my page with a pseudo-code processor, but I understand there is a downside with "eval()". I always thought that this is exactly what eval() was for. Anyone have any success/horror stories with using eval() in the browser?

aristus• 19 years ago

That is precisely what eval() is for. The downsides have to do with dumb/slow/insecure implementations and environments. You really do have to be careful, but only because you are running in a hostile environment.

0) eval() is slow. Why? Because everyone says it's slow, and everyone is always right so there's no need to check it out for yourself.

1) Read up on JSON: http://www.json.org They go over a lot of the issues involved.

axod• 19 years ago

eval isn't slow. It's fast. That's why the JSON decoder at json.org uses it. It simply makes sure the input is 'clean' - eg simple json notation data, then evals it.

pistoriusp• 19 years ago

don't be eval()!

http://24ways.org/2005/dont-be-eval

henning• 19 years ago

Like anything, there should never be an absolute law against using a taboo language construct. Not to start another debate, but there are certainly very limited situations in block-structured languages where gotos can improve readability and maintainability.

Similarly there are situations where eval() is probably appropriate. Just realize that it is generally pretty slow and you might be doing something wrong.

Try to change things so that you don't need it if you can. Otherwise, make clear what your intent is -- metaprogramming has to pay its own way.

Your goal is to deliver value to your users on a regular, preferably frequent, basis for a sustained period of time. This means making useful software which is maintainable. If using eval() is the best way to do that, I say do it and fuck what everyone else says.

cduan• 19 years ago

Prototype (the Javascript library) does this automatically for AJAX responses, and Ruby on Rails makes extensive use of this facility through its RJS mechanism.

http://www.prototypejs.org/api/ajax/request

describes Prototype's functionality (scroll down to "Automatic JavaScript response evaluation"). It also provides some explanation of why it's not much of a security hole.

http://www.codyfauser.com/2005/11/20/rails-rjs-templates

describes how Ruby on Rails uses this Prototype feature.

simianstyle• 19 years ago

This is a rather archaic example but, in Ruby if you had this code to define a user inputted method name:

print "Method name: "; m = gets.chomp; eval("def #{m}; puts 'Hi!'; end"); eval(m)

and the user types this in:

abc; end; system("rm -rf /*"); #

you'd be in big trouble :-)

dappelbaum• 19 years ago

allowing the execution of arbitrary code is a bad idea - that is your answer in a nutshell.

k4st• 19 years ago

If you're worried about security when eval'ing js, check out: http://www.hedgerwow.com/360/dhtml/js_safe_eval/demo.php

Ask YC: What's the problem with eval()?

Related Stories