(28 фев 2016, 08:43) (0/0)
[0]
donttime,
ЧТО Я БЫ ПОПРАВИЛ:
записал бы этот кусочек кода https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L22-L24 в отдельный метод.
Тут https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L28 можно, но не обязательно поставил бы фигурные скобки.
Записал бы это https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L84 сразу в цикл foreach, а не создавал бы переменную. Это если бы она еще где-то в этом методе использовалась, тогда да можно и так.
А так норм, хорошо еще сначала интерфейс создал. Потом удобно будет сразу писать классы для работы с Redis, Memcached и др.
-------------------------------------
Я думал ты давно на гитхабе, хотел твою активность посмотреть
ЧТО Я БЫ ПОПРАВИЛ:
записал бы этот кусочек кода https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L22-L24 в отдельный метод.
Тут https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L28 можно, но не обязательно поставил бы фигурные скобки.
Записал бы это https://github.com/atomuseq/cache-file-storage-php/blob/master/src/Cache.php#L84 сразу в цикл foreach, а не создавал бы переменную. Это если бы она еще где-то в этом методе использовалась, тогда да можно и так.
А так норм, хорошо еще сначала интерфейс создал. Потом удобно будет сразу писать классы для работы с Redis, Memcached и др.
-------------------------------------
Я думал ты давно на гитхабе, хотел твою активность посмотреть
- 1 из 1
