路由和搜索数据库红宝石

Question

I want to retrieve all the tweets that have a certain hashtag in them. At first I add the hashtags in my 2 tables :

 def add_hashtags(tweet)
      tweet.content.scan(/(?:\s|^)(?:#(?!(?:\d+|\w+?_|_\w+?)(?:\s|$)))(\w+)(?=\s|$)/){ |tag|
        @allhashes = Hashtag.all
        @hash = Hashtag.find_by_name(tag[0].strip)
        unless @hash
          @hashtag = Hashtag.new(name: tag[0].strip)
          @hashtag.save
          @hashrel = Hashrelation.new(tweet_id: tweet.id, hashtag_id: @hashtag.id)
          @hashrel.save
        else
          @hashrel = Hashrelation.new(tweet_id: tweet.id, hashtag_id: @hash.id)
          @hashrel.save
        end
      }
    end

then I want to route to the show method of tweet controller :

get 'tweets/show/(.:format)' => 'tweets#show', as: :hashtag

The links in the hashtags are as follows:

    def twitify(tweet = '')
      tweet.gsub(/(?:\s|^)(?:#(?!(?:\d+|\w+?_|_\w+?)(?:\s|$)))(\w+)(?=\s|$)/) do |tag|
        " " + link_to("#{tag.strip}", hashtag_path(tag.strip), {:name => tag.strip})
      end.html_safe
    end

And finally the show method of the tweet controller is :

  def show
    @hashtag = Hashtag.find_by_name(params[:name])
    @tweet_ids = Hashrelation.find_by_hashtag_id(@hashtag.id)
    @feed_items = Tweet.find_by_id(@tweets_ids.id)
  end

When I click on the link I get :

undefined method `id' for nil:NilClass

which means that params[:name] is either nill or it isn't like the one I have in the DB.

Could you guys help me figure this out ?

The link I see that is called is 'http://localhost:3000/tweets/show/.%23dadawea' which means I have extra things why would I ?.

Answer 1

I would do the following

def add_hashtags(tweet)
  tweet.content.scan(/(?:\s|^)(?:#(?!(?:\d+|\w+?_|_\w+?)(?:\s|$)))(\w+)(?=\s|$)/).flatten.each do  |tag|
    hashtag = Hashtag.where(name: tag.strip).first_or_create
    Hashrelation.create(tweet_id: tweet.id, hashtag_id: hashtag.id)
  end
end

Then change the twitify method to look like

def twitify(tweet = '')
  tweet.gsub(/(?:\s|^)(?:#(?!(?:\d+|\w+?_|_\w+?)(?:\s|$)))(\w+)(?=\s|$)/) do |tag|
    " " + link_to("#{tag.strip}", hashtag_path(name: tag.strip))
  end.html_safe
end

And the show method

def show
  @hashtag = Hashtag.find_by_name(params[:name])
  @tweet_ids = Hashrelation.where(hashtag_id: @hashtag.id).pluck(:id)
  @feed_items = Tweet.where(tweet_id: @tweets_ids)
end

This should be what you are looking for. Now for whats changing:

• Removed Duplicate logic in the add_hashtags to use create instead.

• twitify method is passing name as an html option not a url option so I fixed that. Right now it thinks you want to set the format to the name of the hashtag and name the link the name of the hashtag**

• show method is using find_by which will only return a single result not what you wnat for tweet_ids so i changed it to where clause and just grabbed the ids. Then changes feed_items to search Tweet for all tweet_id s in the Array.

To strip the # just use tag.strip[1..-1]