paladin223 / django

0 stars 0 forks source link

Аккуратнее с кодом #36

Closed drowsycoder closed 1 year ago

drowsycoder commented 1 year ago

Стоит указывать параметры создаваемого поля не где-то одно-два-три на строке, а где-то в столбик, а единообразно У тебя, в основном, по одному на строку. И это хороший подход. Заодно поможет самому быстрее сориентироваться, вот увидишь

Это к коду типа такого: name = django.db.models.CharField("Название", default="", max_length=150)

Сторонний пример: format_example Ну и примерно в одном порядке поля чтобы шли, тоже хорошо

drowsycoder commented 1 year ago

У тебя в AbstractIsPublished, AbstractName, AbstractSlug три примера разного оформления

class AbstractIsPublished(django.db.models.Model):
    is_published = django.db.models.BooleanField("публикация", default=True)

    <...>

class AbstractName(django.db.models.Model):
    name = django.db.models.CharField(
        "название", default="", unique=True, max_length=150
    )

    <...>

class AbstractSlug(django.db.models.Model):
    slug = django.db.models.CharField(
        "слаг",
        default="",
        max_length=150,
        validators=[
            validators.validate_unicode_slug,
            validators.MaxLengthValidator(200),
        ],
        unique=True,
    )

    <...>

Постарайся придерживаться одного стиля (третий самый читаемый и легко править/дополнять)