Алекс обнови решението на 26.11.2015 11:42 (преди над 8 години)
+class TurtleGraphics
+ class Turtle
+ def initialize(rows, columns)
+ @rows, @columns = rows, columns
+ @x, @y = 0, 0
+ @orientations = [
+ [:up, [-1, 0]],
+ [:left, [0, -1]],
+ [:down, [1, 0]],
+ [:right, [0, 1]],
+ ]
+ @orientation = :right
+ @move_offset = [0, 1]
+ @canvas = Canvas.new(rows, columns)
+ end
+
+ def spawn_at(row, column)
+ @x = row
+ @y = column
+ end
+
+ def move
+ @x += @move_offset.first
+ @y += @move_offset.last
+
+ fix_out_of_bounds(@x, @y)
+ @canvas.increment_value(@x, @y)
+ end
+
+ def turn_left
+ turn(:+)
+ end
+
+ def turn_right
+ turn(:-)
+ end
+
+ def draw(given_canvas = @canvas, &block)
+ instance_eval &block
+
+ if given_canvas.instance_of? Canvas
+ given_canvas.canvas
+ elsif given_canvas.instance_of? Canvas::ASCII
+ @canvas.convert_to_ascii(given_canvas.allowed_symbols)
+ elsif given_canvas.instance_of? Canvas::HTML
+ @canvas.convert_to_html(given_canvas.pixel_size)
+ end
+ end
+
+ def look(orientation)
+ @orientation = orientation
+ @move_offset = @orientations.find { |pair| pair.first == orientation }.last
+ end
+
+ private
+
+ def turn(direction)
+ orientation_index = @orientations.index { |pair| pair.first == @orientation }
+ changed_orientation = @orientations[orientation_index.send(direction, 1) % 4]
+
+ @orientation = changed_orientation.first
+ @move_offset = changed_orientation.last
+ end
+
+ def fix_out_of_bounds(x, y)
+ if x < 0
+ @x = @rows - 1
+ elsif y < 0
+ @y = @columns - 1
+ elsif x > @rows
+ @x = 0
+ elsif y > @rows
+ @y = 0
+ end
+ end
+ end
+
+ class Canvas
+ attr_reader :canvas
+
+ def initialize(rows, columns)
+ @canvas = Array.new(rows) { Array.new(columns, 0) }
+ @canvas[0][0] = 1
+ @maximum_steps = 1
+ end
+
+ def increment_value(x, y)
+ @canvas[x][y] += 1
+ if @canvas[x][y] > @maximum_steps
+ @maximum_steps = @canvas[x][y]
+ end
+ end
+
+ def convert_to_ascii(allowed_symbols)
+ intensity_to_symbol = Array.new
+ limit_gaps = allowed_symbols.size - 1
+
+ allowed_symbols.each_with_index do |symbol, index|
+ intensity_to_symbol << [index.to_f / limit_gaps, symbol]
+ end
+
+ intensity_canvas = convert_to_intensity
+ symbol_canvas = convert_to_symbols(intensity_canvas, intensity_to_symbol)
+
+ canvas_to_string = String.new
+ symbol_canvas.each { |row| canvas_to_string << row.join.concat("\n") }
+ canvas_to_string.chomp
+ end
+
+ def convert_to_html(pixel_size)
+ html_code = "<!DOCTYPE html>\n<html>\n"
+ html_code.concat(html_head(pixel_size))
+ html_code.concat(html_body)
+ end
+
+ private
+
+ def convert_to_intensity
+ row_to_intensity = -> (r) { r.map { |steps| steps.to_f / @maximum_steps } }
+ @canvas.map &row_to_intensity
+ end
+
+ def convert_to_symbols(intensity_canvas, intensity_to_symbol)
+ convert_to_symbol = -> (intensity) do
+ intensity_to_symbol.find { |pair| pair.first >= intensity }.last
+ end
+ row_to_symbol = -> (row) { row.map &convert_to_symbol }
+ intensity_canvas.map &row_to_symbol
+ end
+
+ def html_head(pixel_size)
+ "<head>\n\t<title>Turtle graphics</title>\n\n\t<style>\n\t\ttable{\n\t\t\t
+ border-spacing: 0;\n\t\t\t}\n\n\t\ttr {\n\t\t\t}\n\n\t\ttd {\n\t\t\t
+ width: #{pixel_size}px;\n\t\t\theight: #{pixel_size}px;\n\n\t\t\t
+ background-color: black;\n\t\t\tpadding: 0;\n\t\t}\n\t</style>\n</head>\n"
+ end
+
+ def html_body
+ body_start = "<body>\n\t<table>"
+ body_end = "<\t</table>\n</body>\n</html>"
+ body_middle = String.new
+ convert_to_intensity.each do |row|
+ body_middle << "\t\t<tr>"
+ row.each { |intensity| body_middle << "\n\t\t\t" << html_opacity(intensity) }
+ body_middle << "\n\t\t</tr>"
+ end
+ body_start + body_middle + body_end
+ end
+
+ def html_opacity(intensity)
+ quotes = '"'
+ "<td style=#{quotes}opacity: #{format('%.2f', intensity)}#{quotes}></td>"
+ end
+
+ class ASCII
+ attr_reader :allowed_symbols
+
+ def initialize(allowed_symbols)
+ @allowed_symbols = allowed_symbols
+ end
+ end
+
+ class HTML
+ attr_reader :pixel_size
+
+ def initialize(pixel_size)
+ @pixel_size = pixel_size
+ end
+ end
+ end
+end
Здравей :) Ще "изсипя" малко коментари :)
-
@orientationsизглежда като нещо, което не се променя. Има по-добро място за непроменящи се неща от инстанционна променлива. -
@orientationи@move_offsetсъдържат неща, които могат да бъдат изведени едно от друго. Защо не оставиш само едното? - Не е много добра идея да използваш
send. Защо просто не подаваш число наturn, с което да събираш, вместо символ-метод? -
fix_out_of_boundsможе да се напише по-елегантно, помисли го още. И името не е особено описателно. Името звучи като нещо, което е "залепено" в кода без да е домислено. :) - HTML кодът не е същият като този в условието. Може да изпуснеш спокойно празните места, ще ги игнорираме при тестовете. Но неща като
padding: 0ще се проверяват и не ги променяй.
И най-важното:
Не е хубаво, че draw знае за възможните canvas-и. Направи го така, че спокойно да може да се имплементира нов вид canvas, без да се променя класът Turtle. Turtle не би трябвало да се занимава с това как се показват нещата. Това е работа на самите canvas-и.
В момента логиката е доста оплетена, Canvas прави прекалено много неща. Отдели нещата, които са специфични за определен canvas само в този canvas.
Имаш възможност да го напишеш доста по-красиво, сигурен съм, че накрая и на теб ще ти хареса :)