Никола обнови решението на 16.12.2013 13:43 (преди почти 11 години)
+module Graphics
+ class Canvas
+ attr_reader :width
+ attr_reader :height
+
+ def initialize(width, height)
+ @width = width
+ @height = height
+ @board = Array.new(width) { Array.new(height) }#TODO better name
+ end
+
+ def set_pixel(x, y)
+ @board[x][y] = :set
+ end
+
+ def pixel_at?(x, y)
+ @board[x][y] == :set
+ end
+
+ def draw(shape)
+ shape.each { |x, y| set_pixel(x, y) }
+ end
+
+ def render_as(renderer)
+ renderer.render(@board)
+ end
+ end
+
+ module Renderers#TODO fix the bad design
+ class Ascii
+ def self.render(board)#TODO DRY this
+ rendered_board = ""
+ board.each do |line, index|#TODO is there a better way for last line check?
+ rendered_board.concat(line.map(&:render_pixel).to_s)
+ rendered_board.concat("\n") unless index == board.count.pred
+ end
+ rendered_board
+ end
+
+ private
+ def self.render_pixel(pixel)#TODO better name; DRY this
+ if pixel == :set
+ '@'
+ else
+ '-'
+ end
+ end
+ end
+
+ class Html
+ PREFIX = "" #TODO insert the prefix with heredoc
+ SUFFIX = "" #TODO insert the suffix with heredoc
+
+ def self.render(board)#TODO DRY this
+ rendered_board = String.new(PREFIX)
+ board.each do |line, index|#TODO is there a better way for last line check?
+ rendered_board.concat(line.map(&:render_pixel).to_s)
+ rendered_board.concat("<br>") unless index == board.count.pred
+ end
+ rendered_board.concat(SUFFIX)
+ end
+
+ private
+ def self.render_pixel(pixel)#TODO better name; DRY this
+ if pixel == :set
+ '<b></b>'
+ else
+ '<i></i>'
+ end
+ end
+ end
+ end
+
+ class Point
+ include Enumerable
+
+ attr_reader :x
+ attr_reader :y
+
+ def initialize(x, y)
+ @x = x
+ @y = y
+ end
+
+ def eql?(other)
+ [@x, @y].eql?([other.x, other.y])
+ end
+
+ def ==(other)
+ eql?(other)
+ end
+
+ def hash
+ [@x, @y].hash
+ end
+
+ def each
+ yield [@x, @y]
+ end
+ end
+
+ class Line
+ include Enumerable
+
+ attr_reader :from
+ attr_reader :to
+
+ def initialize(first_point, second_point)
+ @from, @to = [first_point, second_point].sort_by(&:y).sort_by(&:x)
+ end
+
+ def eql?(other)
+ [@from, @to].eql?([other.from, other.to])
+ end
+
+ def ==(other)
+ eql?(other)
+ end
+
+ def hash
+ [@from, @to].hash
+ end
+
+ def each(&block)
+ bresenham.each.each(&block)#or smth like that: this needs some testing
+ end
+
+ private
+ def bresenham
+ #TODO returns array of points
+ end
+ end
+
+ class Rectangle
+ include Enumerable
+
+ attr_reader :left
+ attr_reader :right
+
+ def initialize(first_point, second_point)
+ @left, @right = [first_point, second_point].sort_by(&:y).sort_by(&:x)
+ end
+
+ def top_left
+ Point.new([left.x, right.x].min, [left.y, right.y].min)
+ end
+
+ def top_right
+ Point.new([left.x, right.x].min, [left.y, right.y].max)
+ end
+
+ def bottom_left
+ Point.new([left.x, right.x].max, [left.y, right.y].min)
+ end
+
+ def bottom_right
+ Point.new([left.x, right.x].max, [left.y, right.y].max)
+ end
+
+ def eql?(other)
+ [@left, @right].eql?([other.left, other.right])
+ end
+
+ def ==(other)
+ eql?(other)
+ end
+
+ def hash
+ [@left, @right].hash
+ end
+
+ def each(&block)
+ Line.new(top_left, top_right).each(&block)
+ Line.new(top_right, bottom_right).each(&block)
+ Line.new(bottom_right, bottom_left).each(&block)
+ Line.new(bottom_left, top_left).each(&block)
+ #or smth like that: this needs some testing
+ end
+ end
+end
Малко бележки:
- Смяташ ли, че това е най-удачното представяне на пано? Вярвам, че има и по-добри.
- Не виждам голям смисъл от символа
:set; защо не простоtrue? - Правино усещаш, че
@boardне е добро име; аз бих ти предложил или@canvas(защо не), или@pixels. - Пробвай да рефакторираш
Ascii.renderтака, че да ползваmap+join - След
privateсе оставя един празен ред - Ред 55 –
String.new(PREFIX)? Защо? Заради мутиращите операции по-долу? В тези случаи се ползваdup; и пак –mapиjoin - Ползвай синоними на методи за равенството
== - Ред 109 и 141 са интересни, но не съм убеден, че това е достатъчно ясно написан код; убедил съм се, че е по-добре кодът да е по-прост, отколкото по-"хитър" :) Трябва да е максимално близък до семантиката, която се цели с него, в случая някаква подредба на точки
Останалото е горе-долу прилично, като на места може малко да се DRY-не, но виждам, че е все още на ранен етап.
Мерси за feedback-а.
- Може ли да expand-неш малко за представянето на паното? Мислех, че това е най-кратката и ясна имплементация, която покрива нужния интерфейс.
- Семантично ми звучеше по-добре пикселите да могат да бъдат
:set-нати или не-:set-нати, отколкото "вярни" или не-"вярни". Но ако изглежда weird ще го сменя. - Всичко в
Renderersе много зле и ще бъде променено. - Известно време се чудех доколко е добра идея
include Enumerable. Не само добавя много неща в публичния интерфейс, които не се изискват от условието, но и имам чувството, че ще трябва да се грижа допълнително за определени неща, което ще резултира в добавянето на код и няма да е много красиво. По-добре ли е да има само методto_a(или нещо подобно), който да се ползва отdraw?
Отзад-напред:
- Аз бих се въздържал да добавям излишни методи в обектите си. Бих държал интерфейса им компактен и консистентен. Не ми звучи логично точка да може да бъде "обхождана". Бих потърсил друго решение за рисуването.
- За
:set-натите пискели ти разбирам логиката. Не бих казал, че изглежда особено странно, но бих възприел този подход със символ, ако имаше повече от две състояния (поне за това се сещам, като видя символ, а не нещо, което е истина/лъжа); логиката ми, както и на интерфейса е "да, тук има пиксел", или "не, тук няма пиксел"; вж.pixel_at? - И продължавайки горната мисъл, виж дали няма да ти дойде някаква друга идея за вътрешно представяне на паното; не казвам, че непременно трябва да го смениш, просто като/ако ти остане време, помисли над него; така или иначе, това е вътрешно представяне и от него не би трябвало да зависят другите части от имплементацията.