Pass the model's hash to the controller

Question

Pass the model's hash to the controller

Navigation

#1 by (2 votes)

0

I have a list in the model that basically must decide if a record fulfills a function add 1 to good otherwise add 1 to bad and have those two parameters return them to the controller as a hash.

What I do not know is how to get the controller to receive the result.

On the controller

class GruposController < ApplicationController
  @grupos = Grupo.all
  Contacto.decidir(contador)
  Puts contador.inspect #=> me trae nil, cuando quiero que me traiga el total de buenos y malos 
  …
end

In the model

class Contacto < ApplicationRecord
  def self.decidir(contador)
  contador = Hash.new
  contador[:buenos] = 0
  contador[:malos] = 0
  @grupos.each do |grupo|
  if grupo.nombre
    contador[:buenos] += 1
  else 
    contador[:malos] += 1
  end
end

UPDATED

Driver:

  def index
    @grupos = Grupo.all
    Contacto.decidir(@grupos)
    puts '................'
    puts params[:contador].inspect #=> me trae nil, cuando quiero que me traiga el total de buenos y malos 
    puts '................'
   end

model:

  def self.decidir(grupos)
    puts 'dentro'
    { buenos: 0, malos: 0 }.tap do |contador|
      grupos.each do |grupo|
        contador[grupo.nombre ? :buenos : :malos] += 1
        puts grupo.nombre
      end
    end
  end

log output:

Started GET "/" for 127.0.0.1 at 2017-09-20 10:36:33 -0400
Processing by EntradaController#indice as HTML
  Rendering entrada/indice.html.erb within layouts/application
  Rendered entrada/indice.html.erb within layouts/application (4.0ms)
Completed 200 OK in 526ms (Views: 484.8ms | ActiveRecord: 0.0ms)


Started GET "/grupos" for 127.0.0.1 at 2017-09-20 10:36:36 -0400
Processing by GruposController#index as HTML
dentro
  Grupo Load (1.0ms)  SELECT "grupos".* FROM "grupos"
casa
trabajo
deporte
playa
................
nil
................
  Rendering grupos/index.html.erb within layouts/application
  Rendered grupos/index.html.erb within layouts/application (4.0ms)
Completed 200 OK in 450ms (Views: 410.8ms | ActiveRecord: 1.0ms)

app on github: link

ruby-on-rails

asked by rrg1459 19.09.2017 в 22:38

source

1 answer

how to make a difference in mysql? I have an error with [(ngModel)] in an ionic 2 application

score 2 · Accepted Answer

You need to return the counter, for which the last line should be contador

  def self.decidir(contador)
    contador = Hash.new
    contador[:buenos] = 0
    contador[:malos] = 0
    @grupos.each do |grupo|
      if grupo.nombre
        contador[:buenos] += 1
      else 
        contador[:malos] += 1
      end
    end
    contador # ésta es la que te falta
  end

And only as a suggestion, you could make the following changes:

  # sin '(contador)' como parámetro, ya que al hacer 'contador = Hash.new' 
  # simplemente estás reemplazando el valor del parámetro 'contador'
  def self.decidir
    # Al hacer { buenos: 0, malos: 0 } es lo mismo que las primeras 3 líneas
    # de tu código, solo que en una única línea. Luego '.tap', 
    # te permite modificar el objeto que lo invoca y a la vez retornarlo, 
    # sin necesidad de llamar a la variable contador al final. 
    # Más info en https://ruby-doc.org/core-2.2.3/Object.html#method-i-tap
    { buenos: 0, malos: 0 }.tap do |contador|
      @grupos.each do |grupo|
        # Ya que en tanto en el if como en else realizas la misma acción, 
        # pero con distinto símbolo, puedes usar un operador ternario para 
        # realizar la acción en una única línea
        contador[grupo.nombre ? :buenos : :malos] += 1
      end
    end
  end