Станимира обнови решението на 27.11.2015 14:14 (преди около 9 години)
+module TurtleGraphics
+end
+
+class TurtleGraphics::Matrix
+ attr_reader :rows, :column_count, :row_count
+
+ def initialize(row_count, column_count = row_count)
+ @rows = Array.new(row_count) do |i|
+ Array.new(column_count) do |j|
+ i = j = 0
+ end
+ end
+ @column_count = column_count
+ @row_count = row_count
+ end
+end
+
+class TurtleGraphics::Turtle
+ DIRECTIONS = [:left, :up, :right, :down].freeze
+
+ def initialize(row_count, column_count)
+ @matrix = TurtleGraphics::Matrix.new(row_count, column_count)
+ look(:right)
+ end
+
+ def draw(canvas = nil)
+ self.instance_eval(&Proc.new) if block_given?
+ canvas != nil ? canvas.build(@matrix) : @matrix.rows
+ end
+
+ private
+ def move
+ unless defined? @position
+ spawn_at(0, 0)
+ incerement_canvas(0, 0)
+ end
+
+ case @direction
+ when 0 then change_position(-1, :column)
+ when 1 then change_position(-1, :row)
+ when 2 then change_position(1, :column)
+ when 3 then change_position(1, :row)
+ end
+
+ incerement_canvas(@position[:row], @position[:column])
+ end
+
+ def change_position(step, axis)
+ new_position = @position[axis] + step
+ canvas_size = @matrix.send(axis.to_s + "_count")
+
+ if new_position < 0
+ @position[axis] = canvas_size
+ elsif new_position >= canvas_size
+ @position[axis] = 0
+ else
+ @position[axis] = new_position
+ end
+ end
+
+ def incerement_canvas(row, column)
+ @matrix.rows[row][column] += 1
+ end
+
+ def turn_left
+ @direction = @direction == 0 ? DIRECTIONS.length - 1 : @direction - 1
+ end
+
+ def turn_right
+ @direction = @direction == DIRECTIONS.length - 1 ? 0 : @direction + 1
+ end
+
+ def spawn_at(row, column)
+ @position = {row: row, column: column}
+ end
+
+ def look(direction)
+ @direction = DIRECTIONS.index(direction)
+ end
+end
+
+module TurtleGraphics::Canvas
+ def max_frequency(matrix)
+ matrix.rows.collect { |row| row.max }.max
+ end
+end
+
+class TurtleGraphics::Canvas::ASCII
+ include TurtleGraphics::Canvas
+
+ def initialize(symbols)
+ @symbols = symbols
+ end
+
+ def build(matrix)
+ calculate_intensities(matrix)
+
+ canvas = ""
+ matrix.rows.each do |row|
+ row.each { |cell| canvas << to_symbol(cell) }
+ canvas << "\n"
+ end
+
+ canvas
+ end
+
+ private
+ def calculate_intensities(matrix)
+ @max_frequency = max_frequency(matrix)
+ interval = (1.0 / @max_frequency).to_f
+
+ @intensities = { 0 => @symbols[0] }
+ @symbols.each_with_index do |sym, index|
+ next if index == 0
+
+ range_start = (index - 1) * interval
+ range_end = (index == @symbols.length - 1) ? 1 : index * interval
+ @intensities[range_start..range_end] = sym
+ end
+ end
+
+ def to_symbol(cell)
+ cell_intensity = cell.to_f / @max_frequency
+ @intensities.select { |range| range === cell_intensity }.values.first
+ end
+end
+
+class TurtleGraphics::Canvas::HTML
+ include TurtleGraphics::Canvas
+
+ def initialize(pixel_size)
+ @pixel_size = pixel_size
+ end
+
+ def build(matrix)
+ html = %{
+ <!DOCTYPE html>
+ <html>
+ #{head}
+ #{body(matrix)}
+ </html>
+ }
+
+ html.delete!("\n")
+ end
+
+ private
+ def head
+ %{
+ <head>
+ <title>Turtle graphics</title>
+
+ <style>
+ table {
+ border-spacing: 0;
+ }
+
+ tr {
+ padding: 0;
+ }
+
+ td {
+ width: #{@pixel_size}px;
+ height: #{@pixel_size}px;
+
+ background-color: black;
+ padding: 0;
+ }
+ </style>
+ </head>
+ }
+ end
+
+ def body(matrix)
+ %{
+ <body>
+ #{generate_table(matrix)}
+ </body>
+ }
+ end
+
+ def generate_table(matrix)
+ max = max_frequency(matrix)
+
+ table = "<table>"
+ matrix.rows.each do |row|
+ table << "<tr>"
+ row.each do |cell|
+ cell_intensity = cell.to_f / max
+ table << "<td style='opacity: #{cell_intensity}'></td>"
+ end
+ table << "</tr>"
+ end
+ table << "</table>"
+ end
+end
Здравей :) Като по-рано предала - печелиш коментари :)
- Хубаво е да дефинираш класове в други класове/модули като ги нестваш:
module TurtleGraphics
class Turtle
...
end
end
вместо
module TurtleGraphics
end
class TurtleGraphics::Turtle
...
end
Разликата е в начина, по който се търсят константи от кода на мястото на многоточието. При първия вариант се търси първо в Turtle, после в TurtleGraphics и накрая в Object. Във втория случай се търси само в Turtle и Object.
- Това присвояване е излишно - i = j = 0. Можеш просто да напишеш 0 и ще има същият ефект. И е хубаво да е едноредов блок, вместо да има един ред само с 0.
- Оставяй празен ред след private :)
- Това:
if new_position < 0
@position[axis] = canvas_size
elsif new_position >= canvas_size
@position[axis] = 0
else
@position[axis] = new_position
end
може да се направи на един ред с един математически оператор :)
- Прекалено "мета" ми идва
change_position. Затова ще подскажа малко :)
case @direction
when 0 then change_position(-1, :column)
when 1 then change_position(-1, :row)
when 2 then change_position(1, :column)
when 3 then change_position(1, :row)
end
=>
case @direction
when 0 then @x -= 1
when 1 then @y -= 1
when 2 then @x += 1
when 3 then @y += 1
end
И след това може да ги "вкараш в граници". change_position стана излишен. Помисли къде другаде можеш да приложиш подобно опростяване.
-
unless defined? @position=>unless @positionилиunless @position.nil?. Инстанционните променливи съдържатnilпо подразбиране. - Повечето от нещата в
calculate_intensitiesмогат да станат с мааалко количество математика. Помисли как да превърнеш интензитет в цяло число - индекс от масива със символите. - В
ASCII#buildизползвайjoin. Конкатенирането на низове не е особено красиво и по принцип се избягва. Същото важи и заgenerate_table.
Като подобриш тези неща ще се получи супер :)