Creciendo la serpiente y programando la condición de derrota

Clase 36 de 39 • Curso de Ruby

Clase anteriorSiguiente clase

Sebastian Medina

student•

Ojo, se la nada se cambio metodo grow_snake_to entre el minuto 6:41 y el 6:59

Diego Hernández Delgado

student•

¡Gracias, Sebastián! Ya van varios cambios así.

Mario Vizcaino

student•

creo que el metodo self.grow_snake_to se puede reducir un poco mas y dejarlo asi:

def self.grow_snake_to(state, next_position) state.snake.positions = [next_position] + state.snake.positions state end

Camilo Castro

student•

De esta forma debería quedar el test para test_snake_grow: def test_snake_grow

    initial_state = Model::State.new(
        Model::Snake.new([
            # call and set the coord fo the snake
            Model::Coord.new(1, 1),
            Model::Coord.new(0, 1)
            ]),
        # call and set the coord for the food
        Model::Food.new(2, 1),
        Model::Grid.new(9, 12),
        Model::Direction::DOWN,
        false
    )
    actual_state = Actions::move_snake(initial_state)
    assert_equal(actual_state.snake.positions,  [
        # call and set the coord fo the snake
        Model::Coord.new(2, 1),
        Model::Coord.new(1, 1),
        Model::Coord.new(0, 1)
        ]) 
end

Nestor Jose Saad Gutierrez

student•

Es correcto, en el actual_state no hay que instanciar el initial_state, ya que en el metodo actual test_move_grow acabamos de definir nuevas condiciones de initial_state,

José Gersain Osorio Morales

student•

Si les parece el error in +': no implicit conversion of Model::Snake into Array (TypeError)` solo hay que cambiar el código de la siguiente manera

def self.grow_snake_to(state, position)
	state.snake.positions = [position] + state.snake.positions
state
end

Victor Lutin

student•

Hola, esto

if new_state.hash != @state

no deberia ser asi?

if new_state.hash != @state.hash

José Gersain Osorio Morales

student•

Es correcto, sin embargo creo que new_state y @state son el mismo objeto desde que se hace el llamado a la acción, por lo tanto la condición nunca se cumple

Giancarlo Ramirez Reyes

student•

Es mejor crear un método collides_with? dentro de la clase Coord, que nos ayudaría a saber si la coordenada actual está colisionando con alguna otra que la podemos pasar por parámetro. Este método lo podemos reutilizar en la clase coordenada y todas las clases que hereden de ella como Food.

Dentro de actions.rb

  def self.position_is_food?(state, next_position)
    state.food.collides_with?(next_position)
  end

Y dentro de state.rb

  class Coord < Struct.new(:row, :col)
    def collides_with?(object)
      row == object.row && col == object.col
    end
  end