Плагин для sublime text 2

Question

[Monnoroch]

Доброго времени суток! Я тут в свободное время разрабатываю open-sourse плагин для sublime text 2, который может быть полезен web-разработчикам, верстальщикам, и просто столкнувшимся.

Плагин написан на python, а я не профи в программировании на этом замечательном языке, поэтому с удовольствием выслушал бы советы и предложения от знающих людей, кода там совсем немного, и я был бы благодарен за любую помощь.
Так же принимаются пожелания для развития идеи или может даже для других полезных околоCSSных (хотя в общем-то не обзательно) фич.

Answer 1

[bagyr]

В toHex красивее сделать что-то вроде

return '# %s %s' % (stringOne, stringTwo) 

И создалось впечатление, что весь isColor можно переписать одним регекспом.

Comments

[Monnoroch]

А разве % не устаревший способ? И да, можно, но я не умею писать регэкспы, плюс даже если я и напишу, то потом возможно захочу модифицировать и уж точно не смогу и снова придется писать.

[bagyr]

Не знаю, в python3 вроде еще работает.
Еще в глаза бросились char list'ы (не очень понял почему не строки).
Ну и отсуствие докстрингов перед функциями, >120 символов в строках и log/print вперемешку если занудствовать до конца.

[Monnoroch]

Спасибо, кроме доков все поправил :)
Надо как-нибудь будет еще над регэкспом попотеть…

[serso]

Не слушайте плохих советов — не нужно вам использовать никаких регэкпов в методе isColor().

[Monnoroch]

Почему же? По идее регэкспы и предназначены для парсинга строк. В принципе вполне логичное решение.

Answer 2

[Артем Бондаренко]

Как-то так получается:):

Comments

[Monnoroch]

Не понял вашего комментария.