Discussion :

[TheFatalError]

Bonjour à tous,

J'ai un petit, peut-être un gros soucis, dans mon code. Bon, je code depuis très peu de temps en ruby. Et je n'ai le réflexe qui me dit, tiens comment je pourrais corriger ça dans ce cas.

Failure/Error: neighbours.map{ |e| neighbours_cells_values << @grid[cell_position + e[0]][line_position + e[1]] if @grid[cell_position + e[0]] >= 0} NoMethodError:
undefined method `>=' for [0, 0, 0]:Array

J'ai ce message d'erreur 6 fois, pour mes 6 cas de tests en Rspec sur 9. Les 6 cas se ressemblant fortement puisqu'il s'agit de mes divers cas de grilles. Il ne semble pas reconnaître les opérateurs. C'est arrivé en voulant rajouter une condition après avoir "mappé" ma variable contenant mon tableau de position de cellules voisines. Car, j'avais avant ça une erreur

undefined method `[]' for nil:NilClass

donc sortant de ma grille, j'ai voulu ajouté une condition. Voici mon code. Ça cloche à

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

neighbours_cells_values << neighbours.map{ |e| @grid[cell_position + e[0]][line_position + e[1]] if @grid[cell_position + e[0]] >= 0 }

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
class JeuDeLaVie  EMPTY_GRID = [[0,0,0],[0,0,0],[0,0,0]]
 
  def initialize(grid = EMPTY_GRID)
    @grid = grid
  end
 
  def grid
    new_grid = @grid
 
    @grid.each_with_index do |line, line_position|
      line.each_with_index do |cell, cell_position|
        if cell == 0
          alive_neighbours_cells = neighbours_cells(cell_position, line_position).select{ |e| alive?(e)}
          if alive_neighbours_cells == 3
            new_grid[line_position][cell_position] = 1
          end
        else
          alive_neighbours_cells = neighbours_cells(cell_position, line_position).select{ |e| alive?(e)}
          if alive_neighbours_cells.length < 2 || alive_neighbours_cells.length > 3
            new_grid[line_position][cell_position] = 0
          end
        end
      end
    end
    return new_grid
  end
 
  def neighbours_cells(cell_position, line_position)
    neighbours = [[-1,-1], [-1, 0], [-1,1], [0,1], [1,1], [1,0], [1,-1],[0,-1]]
    neighbours_cells_values = []
 
    neighbours_cells_values << neighbours.map{ |e| @grid[cell_position + e[0]][line_position + e[1]] if @grid[cell_position + e[0]] >= 0 } 
  end
 
  def alive?(cell_value)
    cell_value == 1
  end
 
end

Une idée d'où vient le problème ?

[Invité]

Bonjour,

C'est normal car les instances d'Array n'implémentent pas la méthode :>=
http://ruby-doc.org/core-2.2.0/Array.html

En gros ton expression renvoie une instance d'Array [0, 0, 0] et il cherche la méthode :>= mais ne la trouve pas car elle n'est pas définie.
Je pense qu'il te manque une dimension supplémentaire pour comparer une valeur du tableau [0,0,0] à la valeur 0.

J'éviterai aussi d'appeller la variable local "grid" dans la méthode :initialize comme la méthode d'instance :grid.
Ça peut être confus même si c'est valable

@+

[TheFatalError]

Merci,

Désolé de répondre tardivement. J'avais mis de côté le jeu de la vie. J'ai décidé de le reprendre de zéro. Sur une bonne base. Voici le résultat :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
class JeuDeLaVie
 
  EMPTY_GRID=[[0, 0, 0], [0, 0, 0], [0, 0, 0]]
 
  attr_reader :grid
 
  def initialize(grid=EMPTY_GRID)
    @grid = grid
  end 
 
  def update!
    new_grid = Array.new(@grid.length){ Array.new(@grid[0].length) }
 
    @grid.each_with_index do |line, i|
      line.each_with_index do |cell, j|
        if still_alive?(i, j) || born?(i, j)
          new_grid[i][j] = 1
        else
          new_grid[i][j] = 0
        end
      end
    end
 
    @grid = new_grid
  end
 
  def born?(x, y)
    alive_neighbours_cells(x, y) == 3
  end
 
  def still_alive?(x, y)
    alive_neighbours_cells(x, y) >= 2 && alive_neighbours_cells(x, y) <= 3
  end
 
  def alive_neighbours_cells(x, y)
    alive_cells = 0
    coordinates = [[-1, -1], [-1, 0], [-1, 1], [0, -1], [0, 1], [1, -1], [1, 0], [1, 1]]
    coordinates.each do |c|
      xpos = (x + c[0]) % @grid.length
      ypos = (y + c[1]) % @grid.length
      cell = @grid[xpos][ypos]
      alive_cells += 1 if cell == 1
    end
    alive_cells
  end
 
end