Рефакторинг имен (в следующей версии)
Регистрация Войти

Рефакторинг имен (в следующей версии)

Evg Evg 15 Января 2022 (ред)

Я прочитал предыдущую статью: «Выйти из зоны комфорта: поверь в жизнь!», там всё верно написано, и… решил выйти… Шучу. Если серьезно, то назрела необходимость в глубоком рефакторинге имен в проекте.

Это действительно необходимо сделать, начиная с базы и кончая шаблонами. Разные uid — которые содержат массив с данными участника, а user id, — у меня почему-то это сразу приходит на ум, или UID  — уникальный идентификатор (Unique identifier) и т.д.

А пока, если вы залили файлы новые с GitHub, удалите «лишний» столбец:

ALTER TABLE `notifications`
  DROP `notification_connection_type`;

В таблице уведомлений (notifications). Это минимальное, что можно сделать в этой версии.

Далее, завтра (если не будет ошибок) архивируем, создаем очередную версию, по счету будет v0.2.2 и в следующей начинаем заниматься именами. Только имена, больше ничего там не будет.

Как-то так. Такие планы.

P.S. и по поводу зоны комфорта, сегодня гараж один сломаю старый, Он аналогично давно напрашивается. Убирать лишнее, это освобождает. Шучу… )

LibArea Рефакторинг
8
411
+ Читать

Связанное

10 Ответов

  1. OleStep OleStep 15 Января 2022 (ред.)

    Самое сложное и муторное, если делать капитально всё. )

    1
    Ответить
    1. Evg Evg 15 Января 2022 (ред.)

      Как настроиться. ) Ломать не строить ))

      Ответить
  1. Yori Yori 15 Января 2022

    А как мы будем делать миграцию на то что хотим?

    CREATE TABLE `users` (
  `id` int(11) UNSIGNED NOT NULL COMMENT 'User UID',
  `login` ...
  `email` ...
  `mobile` ...

    Очень большие изменения если делать о чём мы говорили, убирать user_.

    2
    Ответить
    1. Evg Evg 15 Января 2022

      Не только это, полностью всё затронет. А миграцию… тут надо подумать, не готов сейчас ответить, но это делать надо, если действительно хотим, чтобы было все ок. Самое слабое место тут, что тянет всё остальное, и самое болезненное для изменений. Я не волнуюсь за этот сайт, но другие установленные экземпляры, пусть их и не много. Надо подумать. По большому счету, раз затеял такое, то помогу каждый проект перевести. Но так больше нельзя оставлять, хватит откладывать.

      Ответить
    Комментарий удален...
  1. OleStep OleStep 15 Января 2022 (ред.)

    Статью готовлю про имена, скоро будет )

    1
    Ответить
  1. OleStep OleStep 16 Января 2022 (ред.)

    Иконку Добавить в закладки на этой странице следует отделить большим интервалом. Говорю про комментарии и ответы. Возможно UX улучшится?

    1
    Ответить
    1. Evg Evg 16 Января 2022 (ред.)

      Да, лучше будет. Сделаем, но позже. Эти дни надо переписать многое, пока не до дизайна. Записал себе, чтобы не забыть. Спасибо. +

      Ответить
  1. Evg Evg 17 Января 2022

    Много ошибок выявлено, в том числе в архитектуре. Они касается вопросов авторизации, уровня доверия (начального). У кого TL1 и выше, это незаметно.

    Подправлю в ближайшее время. TL1 в ручном режиме добавил (может кого пропустил) из тех, кто зарегистрировался не так давно.

    Если что-то не получается, можно писать в чат Discord или на почту.

    Как проморгали, тут проблемы с 0 TL войти вошли, а сделать ничего система не дает. Она путает немой режим, без авторизации и начальный уровень доверия.

    Вот почему эту кучу надо переписывать. Если запутанно сейчас, то дальше будет только хуже. А код, запутан серьезно. Очень. На днях постараюсь закончить.

    2
    Ответить
    1. OleStep OleStep 17 Января 2022 (ред.)

      А все сидят с TL от 1 и всё нормально. )

      Ответить
    1. VladimirD VladimirD 17 Января 2022 (ред.)

      Видимо как раз на это я и натолкнулся smile

      Ответить