narciruss Dec 10 2009 at 15:21

Пишем простой консольный To-do менеджер на Ruby

5 min

1.2K

Comments 7

mshakhan Dec 10 2009 at 15:26

Вы хотя бы исправили те недочеты, на которые вам в контакте показали =) Например, разделить логику с представлением, убрать евал-ы, ну и т.д. А вообще, ваш пост побудил меня написать расово-правильный (что-то вроде MVC) таск-менеджер. Как допишу, пришлю вам ссылку.

narciruss Dec 10 2009 at 15:31

да как-то мне лень совсем для таких маленьких утилит следовать MVC :)

Ну на самом деле хорошо, что пост хоть чем-то кому-то пригодился.)

mshakhan Dec 10 2009 at 16:31

Вот, можно тут посмотреть — github.com/mshakhan/rtm. Правда, там тоже еще всё сыро, но уже работает.

mshakhan Dec 10 2009 at 16:32

Точку в конце урла надо убрать.

akira Dec 10 2009 at 15:34

За код в картинках, надо расстреливать!

Alexander_N Dec 11 2009 at 13:19

Что-то не работает, вот что выдает:
~/Temp/todo $ ruby ./rtodo.rb
System files was created
> Welcome to R-todo v0.1, type 'help' to get info
help

^C./rtodo.rb:9:in `gets': Interrupt
from ./rtodo.rb:9

be9 Dec 13 2009 at 09:30

Вместо load лучше использовать require.

Вместо eval(«system.#{method}») — system.send(method)
eval(«system.#{method}(string_user_input)») — system.send(method, string_user_input)

system.keywords.each… — этому циклу стоит находиться в классе System (кстати, цикл лучше заменить на вызов find). А keywords лучше сделать константным хешем:

class System
KEYWORDS = { 'add' => 'task_add', 'remove' => 'task_remove', 'list' => 'task_list'}

#…
end

Для чтения файла целиком существует метод File#readlines.

… Дальше было лень всматриваться в код :)