Wave 3 and optionals #30

Hamled · 2015-10-28T17:34:16Z

Good use of a separate method to encapsulate the @board initialization functionality! It might be a good idea to move the @board = [] line into the initialize_board method, as that method assumes that @board is empty already.

Hamled · 2015-10-28T17:35:52Z

Why are we doing the work in this method word.length times?

-Original file line number
+Diff line change
@@ -0,0 +1,70 @@
+    module Scrabble
+    MAX_LENGTH = 15
+      class Board
+        attr_accessor :board
+        def initialize
+          @board = []
+          initialize_board
+        end
+        def placed_word(word, col, row, direction)
+          word.length times do |n|
+            if direction == :horizontal
+              if horizonal_available?(word, col, row) #return true even if one of the letters located
+                placed_word_horizontal(word, col, row) #check if there's a letter- and use the right amount of letters the user needs
+              else
+                puts "There is no place on the board"
+                break
+              end
+            elsif direction == :vertical
+              if vertical_available?(word, col, row)
+                placed_word_vertical(word, col, row)
+              else
+                puts "There is no place on the board"
+                break
+              end
+            else
+              raise ArgumentError, "No direction"
+            end
+          end
+        end
+        def initialize_board
+.times do |row|
+.times do |col|
+              @board[row]||= []
+              @board[row].push("*")
+            end
+          end
+        end
+        #
+        # def horizonal_available? (word, col, row) #boolen method
+        #   count = 0
+        #   while @board[col][row]!= nil
+        #     if @board[col][row]!= "*"
+        #       if @board[col][row] == wo
+        #         raise ArgumentError, "Out og board limits"
+        #
+        # end
+        def placed_word_horizontal(word, col, row)
+        end
+        def vertical_available?(word, col, row) #boolen method
+        end
+        def placed_word_vertical(word, col, row)
+        end
+      end
+    end

-Original file line number
+Diff line change
@@ -0,0 +1,16 @@
+    module Scrabble
+      class Dictionary_Class
+        def is_valid?(word)
+          raise ArgumentError if word.class != String
+          File.open('./lib/dictionary.txt') do |file|
+            file.any? do |line|
+              line.include?(word)
+            end
+          end
+        end
+      end
+    end

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Wave 3 and optionals #30

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Hamled Oct 28, 2015

Uh oh!

Hamled Oct 28, 2015

Uh oh!

Uh oh!

Wave 3 and optionals #30

Are you sure you want to change the base?

Uh oh!

Wave 3 and optionals #30

Uh oh!

Uh oh!

Diff view

Diff view

There are no files selected for viewing

Hamled Oct 28, 2015

Choose a reason for hiding this comment

Uh oh!

Hamled Oct 28, 2015

Choose a reason for hiding this comment

Uh oh!

Uh oh!