Как вывести статьи в нужном порядке?

Question

[destroy_sys]

Выводит первым самый старый топик. Единственная идея переписать цикл, чтоб выводил по id(с последнего до первого), но это уже не rails way. Смотрел на гитхабе исходники других блогов, том тоже выводят через "each do".

p.s. тапками не кидайтесь, я только в начале пути освоения руби

<% @articles.each do |article| %>
	<h1><%= link_to article.title, article_path(article) %></h1> 
	<%= article.created_at.strftime("%d %b. %Y") %>
	<%= raw article.tag_list.map { |t| link_to t, tag_path(t)}.join(', ') %>
	<div class="well">
		<%= article.body %>
		<p><%= link_to 'Продолжить чтение... ', article_path(article) %></p>
	</div>
<% end %>
<%= will_paginate @articles, previous_label: "← Назад", next_label: "Вперед →" %>

Comments

[destroy_sys]

Спасибо всем за быструю реакцию!

Answer 1

[Viktor Vsk]

В контроллеер:
@articles = Article.order('created_at DESC')
Кстати, во вью вы можете сделать:

<%= render @articles %>
<%= will_paginate @articles, previous_label: "← Назад", next_label: "Вперед →" %>

И создать партиал views/articles/_article.html.erb
и в нем:

<h1><%= link_to article.title, article_path(article) %></h1> 
  <%= article.created_at.strftime("%d %b. %Y") %>
  <%= raw article.tag_list.map { |t| link_to t, tag_path(t)}.join(', ') %>
  <div class="well">
    <%= article.body %>
    <p><%= link_to 'Продолжить чтение... ', article_path(article) %></p>
  </div>

Comments

[destroy_sys]

Какой смысл партиала тут? Код используется один раз, во вьюхе index. Ведь существующий код не противоречит принципу DRY

[Viktor Vsk]

destroy_sys: Во-первых, рейлс-вэй. Во-вторых, избавляетесь, как миниимум, от each do ||. В-третьих, это более читабельно. В-четвертых, откуда уверенность, что вскоре не появится какой-нибудь shared/resource_list в котором будет render @resources ?
В-пятых, просто важно помнить, что такая возможность уже существует.

Answer 2

[Сергей Озеранский]

а можно на уровне модели в запросе отсортировать?

Comments

[bukovki]

Например, так:

@articles = Article.order(created_at: :desc).page(params[:page]).per(8)

Answer 3

[Ilya Lozer]

@articles.order('created_at ASC').each do |article|

Но лучше это сделать в контроллере этого экшена, там где ты присваиваешь значение переменной @articles

Comments

[destroy_sys]

Знаю, что всю логику в контроллеры надо переносить. Не совсем новичок в программировании и по рельсам пробежался по книге Хартла.

[Viktor Vsk]

destroy_sys: бизнес-логику - в модели, логику доступа и форматов - в контроллеры, логику оформления - во вью

[Ilya Lozer]

Виктор Выскребенцев: Эх, Витя, давно мы не виделись)) Вот так встреча)