Ruby récursive inondation il

Question

Je suis en train de faire un remplissage d'inondation qui demande l'entrée d'utilisateur à partir du coin supérieur droit d'un tableau généré aléatoirement rempli de numéros 1-6 qui sont représentés par « couleurs ». J'ai ajouté la fonction oldColor/newColor juste maintenant et j'ai un message d'erreur et je ne sais pas exactement pourquoi. Autre que cela, l'algorithme continue à demander des entrées sans imprimer à quoi ressemble le nouveau remplissage à chaque étape.

def floodfill(array_1, row, column, colours, oldColor, newColor) 
      #colours is an array of the 6 colours i'm going to be using 
      boxHeight = array_1.length 
      boxWeight = array_1[0].length 
      oldColor = array_1 
      #puts oldColor 
      print "> " 
      newColor = gets.chomp.downcase 

      if array_1[row][column] != oldColor 
      return 
      if newColor == "r" 
       newColor = colours[:red] 
       array_1[row][column] = newColor 
       floodfill(array_1, row + 1, column, colours, newColor) # right 
       floodfill(array_1, row - 1, column, colours, newColor) # left 
       floodfill(array_1, row, column + 1, colours, newColor) # down 
       floodfill(array_1, row, column - 1, colours, newColor)# up 
       print_it 
      else 
       puts "didnt get that" 
       array_1.each do |row| 
       row.each do |c| 
        print c 
       end 
       puts 
      end 
      end 
     end 
     end 
floodfill(array_1,14,9,colours,0,0) 

Je ne peux pas directement publier des images, mais voici ce que ma sortie ressemble actuellement et le message d'échec http://imgur.com/a/88UrK

Answer 1

Ce court-circuite votre exécution de code:

if array_1[row][column] != oldColor 
    return 

Une fois hits return il renvoie nil de la méthode et rien d'autre ne sera évalué.

boxHeight et boxWeight ne sont jamais initialisées et newColor sera écrasé par la gets qui ne devrait probablement pas se produire.

Et enfin, le code manque un end arrière. Je recommanderais d'utiliser un outil pour reformater ou réindexer votre code automatiquement, ce qui aidera vraiment à éviter de tels problèmes.

Answer 2

Ruby si les déclarations ne fonctionnent pas comme en C ou Java où vous pouvez écrire quelque chose comme

if array_1[row][column] != oldColor 
    return 

Vous avez besoin soit un end ou vous avez besoin de mettre le si après le retour.

if array_1[row][column] != oldColor 
    return 
end 
# or 
return if array_1[row][column] != oldColor