Как правильно в таблице записать автора статьи?

Question

[an543]

Подскажите, программист начинал делать сайт и сделал в таблице
aticles

$table -> integer('author_id') -> unsigned() -> default(0);
              $table->foreign('author_id')
             ->references('id')->on('users')
             ->onDelete('cascade');
             $table->integer('created_by')->nullable();
             $table->integer('modified_by')->nullable();

Зачем и author_id и created_by ведь значения у них одинаковые, как правильно ?

Comments

[JhaoDa]

У него и спроси.

[an543]

JhaoDa, пропал он :)

[JhaoDa]

an543, ну а нам-то откуда знать, какую логику он закладывал в это?

[an543]

JhaoDa, ну возможно я подумал, что вы не раз делали подобные таблицы.

[JhaoDa]

an543, ты серьёзно? Какие «такие»? При чём тут таблицы? Речь про логику.

Answer 1

[jazzus]

Если author_id и created_by одинаковые, значит что-то нужно удалить.

Comments

[an543]

Вот я и пытаюсь выяснить что ?

[Сергей]

Возможно это не одно и тоже. Если на сайте размещают статьи контент-менеджеры, которым авторы передают статьи например в ворде, то логично что created_by это контент-менеджер, а author_id это сам автор. Такая логика у меня используется на нескольких сайтах

[jazzus]

Сергей, скорее всего там так и задумано. Иначе зачем их было дублировать

[jazzus]

an543, вам нужно посмотреть по коду, в контроллерах статьи метод store/update например, какая там логика происходит и где поля эти используются

[an543]

Сергей, статьи в проекте будут добавлять пользователи, ну и модерироваться модераторами. Но еще есть и modifed_by, я так понял это и есть модератор.

[jazzus]

an543, modified_by - модератор, но вам нужно узнать про author_id и created_by в контроллерах может прописано получение материала у автора

[an543]

jazzus, до автора дело не дошло в контроллерах, поэтому я и хочу выяснить, что из этого лишнее.

[an543]

jazzus, Другими словами если заново делать пользователя в таблице Articles, то через внешний ключ или created_by

[jazzus]

an543, не рекомендую начинать рефакторинг с таблиц. Может там нет ничего лишнего. Т.к. действительно автор и контент-менеджер могут быть разными людьми, да и впринципе нужно отталкиваться от логики в контроллерах.

[an543]

jazzus, Нет только автор и модератор, больше никого не планировалось.

[jazzus]

an543, может программист какой-то смысл в эти поля заложил. Вы можете поле удалить и будет выдавать ошибку. Можно попробовать (после полного бэка) найти поля в контроллерах (поиск в Atom например). Посмотреть логику с полями. Но я бы не рекомендовал трогать т.к. могут быть ошибки. Хотите разобраться, совет – пишите, что-нибудь с нуля, но это время. Или программист со стороны.

[an543]

jazzus, в данный момент автор не используется, вот мне и нужно сделать запись автора в БД, контента нет, поэтому ошибки не пугают. Вот я и думаю куда автора писать в author_id или created_by

[jazzus]

an543, я бы оставил created_by и modified_by

[an543]

jazzus, в comments нашел еще $table->integer('user_id')->unsigned();
вот и хочу порядок навести.

[jazzus]

an543, чувствую вы сейчас наведете порядок)) user_id в комментариях правильное. Это ссылка на юзера-создателя комментария и лучше поле создателя называть именно user_id. Проще будет сконнкетить модели с User в отношениях Laravel. В общем – не забудьте сделать бэк перед любыми иcправлениями :)

[an543]

jazzus, да сделал конечно. Спасибо. т.е. поле автора лучше называть user_id, а не created_at и author_id и без внешних ключей.

[jazzus]

an543, я называю только user_id и всегда с ключом.
Так

$table->integer('user_id')->unsigned();
//связь с юзером, если юзер будет удален то удалится и связанный объект
$table->foreign('user_id')->references('id')->on('users')->onDelete('CASCADE');

[an543]

jazzus, В последнем сообщении объяснение всего вопроса !!! Теперь все понял в чем разница. Спасибо, что "разжевали"

[jazzus]

an543, рад был помочь. А вы эти исправления где хотите делать? Советую все исправления в бд делать только через миграции. Если данных нет, можно удалить поля через
$table->dropColumn('author_id');
И в новой миграции добавить новые

[an543]

jazzus, Да я только через миграции делаю. Спасибо.

[an543]

jazzus, т.е. нельзя просто удалить из миграции строку создания поля и migrate refresh ?

[jazzus]

an543, migrate refresh не использовал. Я такие команды не особо люблю, мало ли что там оно снесет)

[an543]

jazzus, Подскажите еще, внешние ключи нужно всегда использовать по возможности ? например например в комментариях?

[jazzus]

an543, приведите пример с комментариями. Я использую где нужно связать модели. Если комментарий нужно связать с пользователем через "hasMany много комментов" то да. Чтобы потом делать $user->comment

[an543]

jazzus, Я думаю связать для того, стоб удалили пользователя удалили все его комменты, но у меня модели user и comment не связаны через hasMany.

[jazzus]

тогда вот это в comments

$table->integer('user_id')->unsigned();
//связь с юзером, если юзер будет удален то удалится и связанный объект
$table->foreign('user_id')->references('id')->on('users')->onDelete('CASCADE');

Answer 2

[Роман]

Чужую логику понять сложно, тем более не видя весь код

Но author_id - это ведь внешний ключ здесь к таблице 'users'

А created_by он, возможно, хотел сделать первичным ключом, но не успел? :D

Проще спросить у него

Comments

[JhaoDa]

Если следовать твоей теории, то что мешало сделать первичным ключом author_id?

[an543]

Так какие ключи лучше использовать для записи автора? хотя это уже другой вопрос ;)

[JhaoDa]

an543, никакие. Ключи совсем не про это. Найми нового программиста, не ломай голову.

[an543]

JhaoDa, Нанимал уже . Исправлять чужое особо никто не хочет. Да и сам хочу логику понять.

[JhaoDa]

an543, для этого придётся изучать основы, а их много.