The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

Опубликована библиотека urm для Python, opennews (??), 14-Янв-21, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


69. "Опубликована библиотека urm для Python"  +/
Сообщение от Аноним (69), 16-Янв-21, 00:21 
Можно я все-таки тут пару строк оставлю. Гитхаб давно уже не годится для свободной разработки, его слили жадные п-сы другим жадным п-сам.
По существу:
1. type annotations это трэш и захламление кода, тем более что типы автоматически не проверяются
2. хотелось бы видеть более ясные имена, вот, например, Dynamic - это скорее прилагательное, чем существительное, а для классов все же лучше существительные
Ответить | Правка | К родителю #36 | Наверх | Cообщить модератору

71. "Опубликована библиотека urm для Python"  +/
Сообщение от Аноним (17), 16-Янв-21, 03:39 
Спасибо за отклик.

>1. type annotations это трэш и захламление кода, тем более что типы автоматически не проверяются

Это прежде всего документация. Проверку типов через mypy в GitHub Action ещё не реализовал, не знаю даже, стоит ли захламлять пайплайн установкой mypy и его зависимостей ради проверки аннотаций на кадом коммите.

>2. хотелось бы видеть более ясные имена, вот, например, Dynamic - это скорее прилагательное, чем существительное, а для классов все же лучше существительные

Если честно, я там и задумывал прилагательное, выбор был между Dynamic и Deferred, со смыслом "вот есть какой-нибудь параметр (компонент пути, расширение), который мы сейчас не знаем и знать не можем, и вообще меняющийся в зависимости от объекта, и такой, который имеет смысл получать во время использования поля". Attr, Field, Prop - слишком обще и не отражающе суть IMHO. DeferredAttr - уже лучше, но длинновато. Но имена действительно нехорошие, несколько видов "отображателей" вызывают необходимость уточнять, какой отображатель где, а "стратегия" - тоже не очень говорящее название. Предложения по улучшению всегда приветствуются.

Ответить | Правка | Наверх | Cообщить модератору

73. "Опубликована библиотека urm для Python"  +/
Сообщение от Аноним (69), 16-Янв-21, 04:32 
> Это прежде всего документация.

Ее лучше в docstring запихать. ИМХО гораздо приятнее видеть чистое определение функции и затем исчепывающий Args: в docstring на человеческом языке чем нагромождение закорючек. И не забывайте про micropython.

Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру