Правильно ли я делаю таблицу с правами?

Question

[v- smerti]

Привет. Есть задача.
Надо назначить права. При вызове метода API я тащу поле с таблицы и смотрю если 0 то доступ закрыт. Если 1 то всё ок.
Собственно таблица

type UseApi struct {
	KeyId           int    `sql:"AUTO_INCREMENT"                gorm:"primary_key"`
	ApiKey          string `sql:"type:varchar(13)   ;not null"  gorm:"unique_index"`
	ApiAuthRedirekt string `sql:"type:varchar(100)  ;not null"`

	//0 прав нет 1 права есть
	RightsAuth                         int `sql:"type:varchar(1)    ;not null"`
	RightsRegs                         int `sql:"type:varchar(1)    ;not null"`
	RightsBattlesInvasion              int `sql:"type:varchar(1)    ;not null"` //война
	RightsBattlesSanctions             int `sql:"type:varchar(1)    ;not null"`
	RightsBattlesGround                int `sql:"type:varchar(1)    ;not null"`
	RightsMissionsLocation             int `sql:"type:varchar(1)    ;not null"` //мисии
	RightsMissionsHq                   int `sql:"type:varchar(1)    ;not null"`
	RightsUnitsMarket                  int `sql:"type:varchar(1)    ;not null"` //Магазин техники
	RightsBlackmarketMercenaries       int `sql:"type:varchar(1)    ;not null"` //чёрный рынок
	RightsBlackmarketSaboteurs         int `sql:"type:varchar(1)    ;not null"`
	RightsBlackmarketLaboratory        int `sql:"type:varchar(1)    ;not null"`
	RightsBlackmarketContraband        int `sql:"type:varchar(1)    ;not null"`
	RightsBlackmarketDocuments         int `sql:"type:varchar(1)    ;not null"`
	RightsBlackmarketAuction           int `sql:"type:varchar(1)    ;not null"` //Производство
	RightsManufacturingFactory         int `sql:"type:varchar(1)    ;not null"`
	RightsManufacturingMines           int `sql:"type:varchar(1)    ;not null"`
	RightsManufacturingRockets         int `sql:"type:varchar(1)    ;not null"`
	RightsBuildingsProfitable          int `sql:"type:varchar(1)    ;not null"` //постройки
	RightsBuildingsDefense             int `sql:"type:varchar(1)    ;not null"`
	RightsBuildingsEnergy              int `sql:"type:varchar(1)    ;not null"`
	RightsOfficersclubResling          int `sql:"type:varchar(1)    ;not null"` //клуб офицеров
	RightsOfficersclubRoulette         int `sql:"type:varchar(1)    ;not null"`
	RightsOfficersclubSpyInterrogation int `sql:"type:varchar(1)    ;not null"`
	RightsOfficersclubVeteransRiddle   int `sql:"type:varchar(1)    ;not null"`
	RightsOfficersclubWallNewspaper    int `sql:"type:varchar(1)    ;not null"`
	RightsProfileInfo                  int `sql:"type:varchar(1)    ;not null"` //профиль
	RightsProfileSkills                int `sql:"type:varchar(1)    ;not null"`
	RightsProfileAchievements          int `sql:"type:varchar(1)    ;not null"`
	RightsProfileProperty              int `sql:"type:varchar(1)    ;not null"`
	RightsProfileBooty                 int `sql:"type:varchar(1)    ;not null"`
	RightsProfileGifts                 int `sql:"type:varchar(1)    ;not null"`
	RightsProfileVipMode               int `sql:"type:varchar(1)    ;not null"`
	RightsProfileStatistic             int `sql:"type:varchar(1)    ;not null"`
	RightsHalloffameRating             int `sql:"type:varchar(1)    ;not null"` //зал славы
	RightsHalloffameStatistic          int `sql:"type:varchar(1)    ;not null"`
	RightsHalloffameHeroes             int `sql:"type:varchar(1)    ;not null"`
	RightsHalloffameLegions            int `sql:"type:varchar(1)    ;not null"`
	RightsAllianceIndex                int `sql:"type:varchar(1)    ;not null"` //альянс
	RightsAllianceRequests             int `sql:"type:varchar(1)    ;not null"`
	RightsAllianceReferrals            int `sql:"type:varchar(1)    ;not null"`
	RightsAllianceReinforcement        int `sql:"type:varchar(1)    ;not null"`
	RightsAllianceUnderDefense         int `sql:"type:varchar(1)    ;not null"`
	RightsChatPublic                   int `sql:"type:varchar(1)    ;not null"` //чаты
	RightsChatAlliance                 int `sql:"type:varchar(1)    ;not null"`
	RightsChatLegions                  int `sql:"type:varchar(1)    ;not null"`
	RightsChatBootcamp                 int `sql:"type:varchar(1)    ;not null"`
	RightsBankStorage                  int `sql:"type:varchar(1)    ;not null"` // банк
	RightsBankGold                     int `sql:"type:varchar(1)    ;not null"`
	RightsBankСredit                   int `sql:"type:varchar(1)    ;not null"`
	RightsBankExchanger                int `sql:"type:varchar(1)    ;not null"`
	RightsMailIndex                    int `sql:"type:varchar(1)    ;not null"` //почта
	RightsMailWrite                    int `sql:"type:varchar(1)    ;not null"`
	RightsMailContacts                 int `sql:"type:varchar(1)    ;not null"`
	RightsMailIgnore                   int `sql:"type:varchar(1)    ;not null"`
	RightsHospital                     int `sql:"type:varchar(1)    ;not null"` //госпиталь
	RightsNews                         int `sql:"type:varchar(1)    ;not null"` //Новости
	RightsNewsAddCommen                int `sql:"type:varchar(1)    ;not null"`
	RightsdailyBonus                   int `sql:"type:varchar(1)    ;not null"`
}

Она вышла очень огромной. Зарание спасибо.
Так как с MyISAM чтение быстрее то будет использоваться этот тип таблицы Пруфф

Answer 1

[Андрей Павленко]

Плохо, лучше бы просто хранить в отдельной таблице все варианты прав и их соотношения (к примеру, есть поле manageNews, который включает в себя deleteNews, updateNews) и отдельную таблицу для соотношений - какой пользователь имеет какие права.
https://github.com/yiisoft/yii2-framework/blob/mas...
Пусть это и php-фреймворк, но можно посмотреть пример работы таблицы; заодно в документации можно найти, как все это работает.

Comments

[v- smerti]

Это не права пользователей, а права API. И этот вариант создаёт доп. нагрузку на бд. Вместо 1 запроса 2 запроса. А если я в цикле буду перебирать каждое поле (страница которая уведомляет пользователя о том что будет доступно тому сайту) то вместо 60 запросов будет 120. Так что теперь я понимаю почему yii, симфония и laravel сайты такие глючные :)

[Илья]

Владимир Грабко: Если много запросов, то ставить какой-нибудь редис и проблема решена.

Answer 2

[index0h]

Читаем про RBAC

Comments

[v- smerti]

мой вариант вроде как самый простой. надёжный и быстрый

[index0h]

Простой - да.
Надежный - нет.
Быстрый - нет, эти данные должны кэшироваться на время активности пользователя, посему это экономия на сипчках.
Расточительный - ДА. Какой нафик varchar(1), вы что смеетесь?

Ваш вариат не рассчитан на HL. Когда пользователей будет пару-тройку лямов и вам понадобится добавить новое право - помянете мои слова. Изменение таблицы будет очень длительным. Придется делать временную таблицу и миграцию в день-два с доливкой, либо стопать систему, но мы ведь этого не хотим верно?))

[v- smerti]

index0h: А что мне помешает добавить 1 строку с 0 по дефолту ?)

[index0h]

Владимир Грабко Время выполнения в production среде.
То, что юзаете varchar(1) - плохо потому, что это строка ПЕРЕМЕННОЙ длинны. Т.е. мускулу каждый раз при чтении / записи нужно будет выполнять ~60 раз расчет длинны ряда. TINYINT - ок.

[Oleg Shevelev]

Как вариант использовать одно поле и расширять его по мере необходимости,
www.mysql.ru/docs/man/SET.html , при этом использовать 0/1 не удачный подход ещё и тем что а вдруг кто-то запишет туда не 1 а скажем 101... что это будет значить? Кто это будет документировать и кто потом будет обновлять весь код и проверять что ничего не сломалось?:) Не говоря уже о том что числа стоит хранить в числах, а не в строках:)

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

Answer 3

[Илья]

А не легче было сделать одно поле с указанием цифр? Цифры где-нибудь записаны, можно в той же базе. Например, поле Scope в структуре. Это поле содержит значение "1;8;12;6;". Разбиваем по ";", Смотрим, какие права нужны на выполнение функции, смотрим есть ли это значение - выполняем/даём ответ, что нужны такие-то права.

Comments

[v- smerti]

это слжнее во время UPDATE прав

[Илья]

Так часто приходится добавлять права и убирать?

[v- smerti]

FireGM: да. По крайней мере первых месяца три после открытия точно

[Oleg Shevelev]

www.mysql.ru/docs/man/SET.html

Answer 4

[Oleksiy Chechel]

А я бы использовал битовую маску, тем более go прекрасно умеет с ними работать)

type Rights int64

const (
	RightsAuth             = 1 << iota // 1 << 0 which is 0000000000000001
	RightsRegs                         // 1 << 1 which is 0000000000000010
	RightsBattlesInvasion              // 1 << 2 which is 0000000000000100
	RightsBattlesSanctions             // 1 << 3 which is 0000000000001000
	RightsBattlesGround                // 1 << 4 which is 0000000000010000
	RightsMissionsLocation             // 1 << 5 which is 0000000000100000
	// ...
)

Вот здесь более подробный пример: pastebin.com/jz6TZevq