Iterating over array of variables - can't change t =?ISO-8859-1?Q?hem?=

ruby-talk
1

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

thanks

2

AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

You're looking for map, which performs the action on each element and
creates a new array with the results:

[title,subtitle,body].map{|item| item.downcase}

···

--
Posted via http://www.ruby-forum.com/\.

3

Alle venerdì 4 gennaio 2008, AJS ha scritto:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

thanks

The block variable (item) is set by ruby to "point to" each object of the
array in turn. String#downcase! is a destructive method: it changes its
receiver, so the elements of the original array get changed. On the other
hand, when you do item = "", you're telling the variable itewm to "point to"
another object, a new (empty) string. This doesn't modify the original array
at all.

If you want to change the contents of the original array, your should use
Array#map!:

[title,subtitle,body].map! {|item| ""}

Array#map! replaces each element of the array with the return value of the
block.

I hope this helps

Stefano

4

AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

No, it doesn't. It changes the strings, it doesn't do anything to the
variables. The variables still point to the same object as before, it's
just the objects that are different now.

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

It works just fine. It just doesn't do what you want it to. Here's what it
does:
It takes each of the elements in the array and then yields it. The code in the
block will then run with item pointing to the element. After the assignment
(item="") item will point to an empty string. This has no effect whatsoever
on the variables title, subtitle and body.
What you want is map instead of each (which gives you a new array containing
the results of yielding each item) or String#replace instead of reassignment
(which would change the object instead of the variable).

HTH,
Sebastian

···

--
Jabber: sepp2k@jabber.org
ICQ: 205544826

5

AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

Hi
item="" sets item to reference to another string instead of the original at
the beginning of the block

try this

[title,subtitle,body].each {|item|

        x = item
        item=""
        # this make item go to point away from it's original value

        p x.inspect
        # but x hasn't lost it's reference to initial value so it still
        #works
}

Tom

···

has no effect.

Why doesn't this work? How can I make it work?

thanks

6

It does not work because you assign to "item" which is a local variable in the block. And since you do not change string objects in the Array you do not see any change. In your case you should change the original object, e.g.

[title,subtitle,body].each {|item| item.replace ""}
[title,subtitle,body].each {|item| item.sub! /.*/, ""}

More generally you can also change the array via assignment by doing

an_array.map! { "" }

But this again won't change the objects referenced by "title" etc.

HTH

  robert

···

On 04.01.2008 17:06, AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

7

Refer to the variables as strings and use eval.

["title","subtitle","body"].each {|item| eval "#{item}=''"}

--Ken

···

On Fri, 04 Jan 2008 08:06:38 -0800, AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

thanks

--
Ken (Chanoch) Bloom. PhD candidate. Linguistic Cognition Laboratory.
Department of Computer Science. Illinois Institute of Technology.
http://www.iit.edu/~kbloom1/

8

[title,subtitle,body].map! {|item| ""}

perfect

thanks

9

I believe this is rather inefficient (aka slow) and more importantly will not run under all circumstances ($SAFE). Plus, it's not needed. :slight_smile:

Kind regards

  robert

···

On 04.01.2008 18:36, Ken Bloom wrote:

On Fri, 04 Jan 2008 08:06:38 -0800, AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

thanks

Refer to the variables as strings and use eval.

["title","subtitle","body"].each {|item| eval "#{item}=''"}

10

Whether or not it's needed depends entirely on how much he's condensed
the example to post it here. If he were interested in changing the class
of object, this is the only technique that would update the variables
automatically.

(So I'm doing more than the minimum needed to make the test case work, so
I'm not a real extreme programmer.)

--Ken

···

On Fri, 04 Jan 2008 21:00:56 +0100, Robert Klemme wrote:

On 04.01.2008 18:36, Ken Bloom wrote:

On Fri, 04 Jan 2008 08:06:38 -0800, AJS wrote:

title="Title"
subtitle="Subtitle"
body="Body"
[title,subtitle,body].each {|item| item.downcase!}

This successfully sets the original variables to "title", "subtitle",
"body"

BUT

[title,subtitle,body].each {|item| item=""}

has no effect.

Why doesn't this work? How can I make it work?

thanks

Refer to the variables as strings and use eval.

["title","subtitle","body"].each {|item| eval "#{item}=''"}

I believe this is rather inefficient (aka slow) and more importantly
will not run under all circumstances ($SAFE). Plus, it's not needed.
:slight_smile:

--
Ken (Chanoch) Bloom. PhD candidate. Linguistic Cognition Laboratory.
Department of Computer Science. Illinois Institute of Technology.
http://www.iit.edu/~kbloom1/