Мая обнови решението на 05.11.2014 12:52 (преди около 10 години)
+module RBFS
+ class File
+ DATA_TYPES = {
+ NilClass => :nil,
+ String => :string,
+ Symbol => :symbol,
+ Fixnum => :number,
+ Bignum => :number,
+ Float => :number,
+ TrueClass => :boolean,
+ FalseClass => :boolean,
+ }.freeze
+
+ attr_reader :data_type
+ attr_accessor :data
+
+ def self.parse(string_data)
+ File.new eval(string_data)
+ end
+
+ def initialize(data = nil)
+ load_data(data)
+ end
+
+ def data=(data)
+ load_data(data)
+ end
+
+ def serialize()
+ "#{@data_type}:#{@data.to_s}"
+ end
+
+ private
+ def load_data(data)
+ @data = data
+ @data_type = DATA_TYPES[data.class]
+ end
+ end
+
+ class Directory
+ attr_reader :files, :directories
+
+ def self.parse(string_data)
+ #TODO
+ end
+
+ def initialize()
+ @files = {}
+ @directories = {}
+ end
+
+ def add_directory(name, directory = nil)
+ directory = Directory.new unless directory
+ @directories[name] = directory
+ end
+
+ def add_file(name, file)
+ @files[name] = file
+ end
+
+ def [](name)
+ directory = directories[name]
+ directory or files[name]
+ end
+
+ def serialize
+ #TODO
+ end
+ end
+end
Качвам си черновата. Явно ще се бавя още с (де)сериализацията на директории, а вярвам, че ще има забележки и по останалата част.
Между другото, в style guide-а на курса не видях препоръки за подредбата на елементи в дефиницията на клас (но пък засега чета по диагонал и може да съм пропуснала :}). Намерих в style guide-а на Божидар(?) Бацов и се опитах да следвам тях.
Като цяло добре. Няколко забележки:
- Малко подлъгващо беше да ви кажем, че искаме поле
data_type. Имахме предвид, че трябва да може да се отговаря правилно наFile#data_type. Сама виждаш, че си минала през доста неща, само и само да не е метод. - Кажи НЕ! на
eval! Just google eval is evil! Виждам съмevalсамо два пъти в production код. Един път в python, където имаше много по-лесна и разбираема алтернатива с 'нормалните' средства на езика. Друг път в bash скрипт, където беше довел до един от най-obscure bug-овете ever. Не работи така, както изглежда мислиш, че работи. Дори и да работеше - гарантирано е, че ако го използваш има 99.99% вероятност да можем да напишем тест, който го чупи, след като не сме дали сериозни ограничения какво могат да съдържат string данните. - В
Directory#add_directoryможеше да пишеш направоx = y or z- по-кратко и ясно. Също вDirectory#[]- защо не простоdirectories[name] or files[name]? - Не слагай скоби при дефинирането или викането на методи без аргументи.
- Има по-чист синтаксис за интерполиране на инстанционни променливи.
- Смята се за прилежно да подравняваш някои структури в езика, когато се срещат на съседни редове. Сред тях са
=,thenи ключовете на hash-овете. - Виж как би следвало да си анотираш TODO-тата. (:
Относно как се подрежда класа - не сме толкова рестриктивни, но като цяло community style guide-а добре отговаря на въпроса. extend, include, константи, accessors, други клас макроси, public, protected, private. За реда на вътрешни класове и класови vs инстанционни методи може да се спори. Това е лично мое мнение.
Здравей :)
Ето малко допълнения от мен, понеже така или иначе бях тръгнал и аз да коментирам:
- Харесва ми, че си се сетила да
freeze-неш хеша. :) - Виждам, че си разбрала недостатъка на този начин за проверка на типа - когато класът на обекта ти наследява от някой от изброените. Аз лично не бих използвал този подход точно заради това, пък и ще стане по-кратко с
case. В тази конкретна задача не е технически грешен, защото в условието не сме споменали нищо за наследници на класовете и не сме толкова лоши, че да пишем специални тестове за това. :) Просто имай предвид, ако ти се наложи на друго място - да не разчиташ на конкретен клас, а по-скоро на него или негов наследник. -
evalе лошо нещо, защото изпълнява произволен Ruby код - повече от това, за което го използваш. Например, ако си пуснала тази програма да приема данни от потребители, то някой потребител може да сложи Ruby код, който да ти изтрие всички файлове на харддиска иevalще го изпълни. - В
Rubyподразбиращите се стойности на аргументите могат да са всякакви изрази, включително иDirectory.new. Може да го използваш заadd_directory.
След толкова коментиране очаквам накрая да излезе супер решение! :)
Много ви благодаря!
- Ех, надявах се да мина тънко с този
eval. (: Правило ми е впечатление, че е непрепоръчителен на този курс (а вероятно и като цяло), но не си направих труда да проверя защо. - Колкото до очевидните чисти конструкции, които пропускам (и защо, защо не се сещам първо за is_a? и подобни, случи ми се и с предната задача), има някаква кутийка, в която явно мисля, и коментарите ви са ми много ценни за работа срещу това.
- Скобите при методи без аргументи са някакво OCD + навик от работа с езици, в които типът има значение. Малко трудно правя прехода към duck typing. (:
Страшно ценни съвети още веднъж, благодаря за отделеното внимание!