Rails中如何创建仅含已关注用户及自身推文的数组并在视图循环？

看起来你想实现一个只展示当前用户自身推文和已关注用户推文的专属时间线，思路方向是对的，但现有代码有几个关键问题需要修正，咱们一步步来调整：

原代码的核心问题 #

1. @tweets未初始化：你直接往@tweets里添加数据，但这个变量一开始是nil，运行时会抛出NoMethodError错误。
2. return语句提前终止循环：return @tweets << tweet会在找到第一条符合条件的推文后就直接结束方法，导致最终只返回一条推文，而不是所有符合要求的内容。
3. 用户对比方式冗余：用current_user.username === tweet.user.username不如直接用current_user == tweet.user更准确高效——毕竟用户对象的ID对比比字符串用户名对比更能避免极端情况（比如重名用户）。

优化后的实现方案 #

方案1：数据库层面直接筛选（强烈推荐） #

不要在Ruby代码里循环所有推文再筛选，直接通过ActiveQuery让数据库完成筛选工作，性能会好很多：

def index
  @tweet = Tweet.new
  # 获取当前用户ID + 所有已关注用户的ID集合
  target_user_ids = [current_user.id] + current_user.following_ids
  # 查询目标用户的所有推文，并按创建时间倒序（最新推文排在最前面）
  @tweets = Tweet.where(user_id: target_user_ids).order(created_at: :desc)
end

方案2：修正你的循环逻辑（仅作学习参考，不推荐生产环境用） #

如果你想保留循环筛选的思路，需要先初始化数组并移除提前终止的return：

def index
  @tweet = Tweet.new
  @tweets = [] # 先初始化空数组
  Tweet.all.each do |tweet|
    if current_user.following?(tweet.user) || current_user == tweet.user
      @tweets << tweet # 不要加return，否则循环会提前结束
    end
  end
  # 可选：按时间倒序排列，让最新的推文优先展示
  @tweets = @tweets.sort_by { |tweet| -tweet.created_at.to_i }
end

视图层的小优化建议 #

你的视图代码逻辑没问题，可以加个空状态提示，当用户还没有可展示的推文时给出友好引导：

<div class="tweets-container">
  <% if @tweets.empty? %>
    <p class="empty-state">还没有推文哦～去关注一些用户或者发布你的第一条推文吧！</p>
  <% else %>
    <% @tweets.each do |tweet| %>
      <div class="tweet-feed-container">
        <strong><%= link_to "@#{tweet.user.username}", user_path(tweet.user), :class => 'tweet-username' %></strong> · <small><%= tweet.created_at.strftime("%b %d, %Y %I:%M %p") %></small><br/>
        <%= tweet.body %>
        <% tweet.tags.each do |tag| %>
          <%= link_to "#{tag.name}", tag_path(tag), :class => 'tweet-tag' %>
        <% end %>
        <% tweet.mentions.each do |mention| %>
          <%= link_to "#{mention.name}", user_path(mention.name.sub('@', '')), :class => 'tweet-tag' %>
        <% end %>
        <%= link_to '↺', retweet_tweet_path(tweet.id), method: :post, :class => 'tweet-delete' %>
        <%= link_to 'X', tweet_path(tweet), method: :delete, :class => 'tweet-delete' if current_user == tweet.user %>
      </div>
    <% end %>
  <% end %>
</div>

为什么推荐数据库查询方案？ #

直接让数据库筛选数据，比把所有推文拉到Ruby里再循环判断要高效得多——尤其是当推文数量越来越多时，数据库的查询优化器能更快返回结果，同时减少服务器的内存占用和处理时间。

内容的提问来源于stack exchange，提问作者ayabbear