Adding objects to an array?

Question

I'm trying to insert a object into a array, but instead of insert into array, it prints in the screen the memory addresses.

I have the following code in my controller:

@article = Article.new("test","test1")
@articles <<  @article     #this line causes the prints

And my view have this code:

<%=
    @articles.each do |a|
    a.titulo + " / " + a.texto
    end
%>

Uri Agassi · Accepted Answer · 2014-11-20 18:33:56Z

3

Your problem is with how you render the @articles:

<%= @articles.each do |a| a.titulo + " / " + a.texto end %>

The return value of each is the array itself. You want to render each element within the array, so you should do something like this:

<% @articles.each do |a| %> 
  <%= a.titulo + " / " + a.texto %>
<% end %>

answered Nov 20, 2014 at 18:33

Uri Agassi

37.5k16 gold badges82 silver badges96 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

NM Pennypacker · Accepted Answer · 2014-11-20 18:30:10Z

1

You could force @articles to be an array like this:

@articles ||= []

Then add your @article like you were already doing

edited Nov 20, 2014 at 18:30

answered Nov 20, 2014 at 18:05

NM Pennypacker

6,99211 gold badges38 silver badges41 bronze badges

1 Comment

NM Pennypacker Over a year ago

Good call. Edited answer.

Bob Mazanec · Accepted Answer · 2014-11-20 18:34:03Z

1

Create @articles as an Array before adding to it with <<.

E.g.,

class MyController < ...
  def your_method
    @articles ||= []
    @article = Article.new("test","test1")
    @articles <<  @article
  end

  ...
end

(and/or in a before_ filter and/or...)

edited Nov 20, 2014 at 18:34

answered Nov 20, 2014 at 18:06

Bob Mazanec

1,12110 silver badges18 bronze badges

2 Comments

Brennan Over a year ago

Having an initialize in a controller is not very idiomatic, I don't recommend it. Instead, I recommend a private method: def articles; @articles ||= []; end;

tadman Over a year ago

Yeah, initialize in a controller is going to cause problems. Don't do this. Instead use a before_filter.

Brennan · Accepted Answer · 2014-11-20 18:38:44Z

1

This is likely expected behavior. If @articles is indeed an array, this should work. Just check the array after the append and make sure the newly added element is there. If you are doing this from the console, then the printing you are seeing is probably console behavior and not << behavior.

Because of the comment below, I'm changing my answer to reflect the actual problem. You are trying to render the enumerator to the view, and not the articles. Your view should look like:

<% @articles.each |a| %>
  <%= a.titulo + " / " + a.texto %>
<% end %>

edited Nov 20, 2014 at 18:38

answered Nov 20, 2014 at 18:04

Brennan

5,7822 gold badges20 silver badges24 bronze badges

5 Comments

Afaria Over a year ago

Thanks for answer. I am running it in web, its prints the memory addresses in the web page.

Brennan Over a year ago

How exactly are you printing it to the page?

Afaria Over a year ago

I am doing this in the view to show the results <%= @articles.each do |a| a.titulo + " / " + a.texto end %>

Uri Agassi Over a year ago

@Afaria - you should add this to your post - your problem is here, not where you think it is...

Brennan Over a year ago

I believe that is your problem. You are trying to render the enumerator to the view, not the articles themselves. You should be doing it something like: <% @articles.each do |a| %> <%= a.titulo + " / " + a.texto %> <% end %>

Collectives™ on Stack Overflow

Adding objects to an array?

4 Answers 4

Comments

1 Comment

2 Comments

5 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

1 Comment

2 Comments

5 Comments

Your Answer

Sign up or log in

Post as a guest

Related