each returns a list but you are calling it in scalar context. Use list context as marto did and you will be fine. See also: Context tutorial.

but gettting an error.

Don't keep it a secret. Always provide the full text of the error message.