Почему я получаю NameError при попытке переместить некоторый код на фасад?

Question

У меня было много переменных экземпляра в действии контроллера, и я пытаюсь переместить их в фасад, но я столкнулся с ошибкой. Я новичок в Rails, поэтому у меня возникли небольшие проблемы.

Мое приложение связывается с API Open Weather и возвращает некоторые параметры погоды (температура, ветер, влажность и т. Д.). Вызов API обрабатывается в классе обслуживания, и я пытаюсь использовать шаблон фасада и у меня есть несколько методов в PORO, которые организуют возвращаемые данные, которые затем представляются на мой взгляд. На данный момент я получаю NameError. undefined local variable or method 'temperature' for ForecastsController. Я новичок в этом, поэтому любая помощь будет высоко ценится! Мой код ниже.

приложение / услуги / open_weather_api.rb

class OpenWeatherApi
  include HTTParty
  base_uri "http://api.openweathermap.org"

  def initialize(city, appid)
    @options = { query: { q: city, APPID: appid } }
  end

  def my_location_forecast
    self.class.get("/data/2.5/weather", @options)
  end
end

приложение / фасадов / forecasts_facade.rb

class ForecastsFacade

  def initialize(temperature, weather_code, description, wind, humidity)
    @temperature = temperature
    @weather_code = weather_code
    @description = description
    @wind = wind
    @humidity = humidity
  end

  def temperature
    @temperature = @forecast.dig('main', 'temp').to_i - 273
  end

  def weather_code
    @weather_code = @forecast.dig('weather', 0, 'id').to_i
  end

  def description
    @description = @forecast.dig('weather', 0, 'description')
  end

  def wind
    @wind = @forecast.dig('wind', 'speed').to_i
  end

  def humidity
    @humidity = @forecast.dig('main', 'humidity')
  end
end

приложение / контроллеры / forecasts_controller.rb

class ForecastsController < ApplicationController

  TOKEN = Rails.application.credentials.openweather_key

  def current_weather
    @city = params[:q]
    if @city.nil?
      @forecast = {}
    else
      @forecast = OpenWeatherApi.new(@city, TOKEN).my_location_forecast
    end
    @forecasts_facade = ForecastsFacade.new(temperature, weather_code, description, wind, humidity)
  end
end

приложение / просмотров / Прогнозы / current_weather.html.erb

<div class="page-wrapper">
  <h1 class="title">The weather in GIFs</h1>

  <div class="search">
    <%= form_tag(current_weather_forecasts_path, method: :get) do %>
      <%= text_field_tag :q, nil, placeholder: "Enter a city", class: "search-field" %>
      <%= button_tag type: 'submit', class: "search-button" do %>
        <i class="fas fa-search"></i>
      <% end %>
    <% end %>
  </div>

  <% if @forecasts_facade.forecast.dig('cod').to_i == 404 %>
    <p>Please use a valid city name!</p>
  <% elsif @forecasts_facade.forecast.dig('cod').to_i == 400 %>
    <p>Please type in a city name!</p>
  <% elsif @forecasts_facade.forecast == {} %>
  <% else %>
    <p class="weather-description"><%= "#{@forecasts_facade.city.capitalize}: #{@forecasts_facade.description}" %></p>
    <div class="gif-container"><%= image_tag(find_gif_url, class: "gif") %>
      <span class="temperature weather-attribute"><%= "#{@forecasts_facade.temperature}°C" %></span>
      <span class="wind weather-attribute"><%= "wind:#{(@forecasts_facade.wind * 3.6).to_i}km/h" %></span> <!-- converts to km/hr -->
      <span class="humidity weather-attribute"><%= "humidity:#{@forecasts_facade.humidity}%" %></span>
    </div>
  <% end %>
</div>

приложение / хелперы / forecasts_helper.rb

module ForecastsHelper
  GIFS = {
    thunder:
      {codes: [200, 201, 202, 210, 211, 212, 221, 230, 231, 232],
       urls: %w(
          https://media.giphy.com/media/26uf5HjasTtxtNCqQ/giphy.gif
          https://media.giphy.com/media/vS09bj1KrXwje/giphy.gif
          https://media.giphy.com/media/l41YsrmiyeNZtmecw/giphy.gif
)},
    light_rain:
      {codes: [300, 301, 302, 310, 311, 312, 313, 314, 321, 500, 501, 520, 521],
       urls: %w(
          https://media.giphy.com/media/xT9GEz2CeU9uaI2KZi/giphy.gif
          https://media.giphy.com/media/l0Expn6D0D3g9NExq/giphy.gif
          https://media.giphy.com/media/14d8cRr25KPxbG/giphy.gif
)},

  def find_gif_url
    GIFS.each do |key, value|
      if value[:codes].include? @weather_code
        return value[:urls].sample
      end
    end
  end
end

Answer 1

Полагаю, вы хотите, чтобы ваш контроллер выглядел больше так:

class ForecastsController < ApplicationController

  TOKEN = Rails.application.credentials.openweather_key

  def current_weather
    @city = params[:q]
    if @city.nil?
      @forecast = {}
    else
      @forecast = OpenWeatherApi.new(@city, TOKEN).my_location_forecast
    end
    @forecasts_facade = ForecastsFacade.new(@forecast)
  end
end

Тогда ваш ForecastsFacade больше похож на:

class ForecastsFacade

  attr_accessor *%w(
    forecast
  ).freeze

  def initialize(forecast)
    @forecast = forecast
  end

  def temperature
    @temperature = forecast.dig('main', 'temp').to_i - 273
  end

  def weather_code
    @weather_code = forecast.dig('weather', 0, 'id').to_i
  end

  def description
    @description = forecast.dig('weather', 0, 'description')
  end

  def wind
    @wind = forecast.dig('wind', 'speed').to_i
  end

  def humidity
    @humidity = forecast.dig('main', 'humidity')
  end

  def invalid_city?
    forecast.dig('cod').to_i == 404 
  end

  def missing_city?
    forecast.dig('cod').to_i == 400
  end

end

Тогда, на ваш взгляд, кажется странным делать такие вещи, как:

@forecasts_facade.forecast.dig('cod').to_i == 404

, когда весь смысл в том, чтобы скрыть эту деталь от view. Так что, возможно, что-то еще похожее:

<div class="page-wrapper">
  <h1 class="title">The weather in GIFs</h1>

  <div class="search">
    <%= form_tag(current_weather_forecasts_path, method: :get) do %>
      <%= text_field_tag :q, nil, placeholder: "Enter a city", class: "search-field" %>
      <%= button_tag type: 'submit', class: "search-button" do %>
        <i class="fas fa-search"></i>
      <% end %>
    <% end %>
  </div>

  <% if @forecasts_facade.invalid_city? %>
    <p>Please use a valid city name!</p>
  <% elsif @forecasts_facade.missing_city? %>
    <p>Please type in a city name!</p>
  <% elsif @forecasts_facade.forecast.blank? %>
  <% else %>
    <p class="weather-description"><%= "#{@forecasts_facade.city.capitalize}: #{@forecasts_facade.description}" %></p>
    <div class="gif-container"><%= image_tag(find_gif_url, class: "gif") %>
      <span class="temperature weather-attribute"><%= "#{@forecasts_facade.temperature}°C" %></span>
      <span class="wind weather-attribute"><%= "wind:#{(@forecasts_facade.wind * 3.6).to_i}km/h" %></span> <!-- converts to km/hr -->
      <span class="humidity weather-attribute"><%= "humidity:#{@forecasts_facade.humidity}%" %></span>
    </div>
  <% end %>
</div>

Похоже, вы не используете эту ветку elsif @forecasts_facade.forecast.blank?. Не уверен, что с этим.