Как запретить удаление последнего пользователя?

Question

[KuzmenkoArtem]

Представим ситуацию. Есть система в ней есть пользователи-админы, допустим их сейчас 2.
Каждый админ может удалять других админов.
Но нужно сделать так что в системе остался по крайней мере один админ.

Первое что приходит в голову такая логика.

function deleteUser(int userId) {
    if (getAdminsCount() == 1) {
        abort();
    }

    makeSqlQueryToDeleteUser(userId); // прежде чем запрос зайдет в базу и она удалит запись, может пройти например 1 миллисекунда
}

Но если Админ А инициирует удаление, и при этом Админ Б тоже инициирует удаление перед самой операцией удаления от первого админа, то он пройдет проверку на последнего пользователя (т.к. сама операция удаления еще не запустилась).
И получается что оба пользователя удалят друг друга и в системе не останется не одного админа. (да шанс не большой, но все же)

Как бы вы решили данную проблему?

Рассматривал помещение этой операции в очередь (queue), но размышляю есть ли другие способы.

Comments

[tukreb]

не ищите проблем на пустом месте. Если вероятность практически 0, просто забейте.

[KuzmenkoArtem]

tukreb, возможно так и сделаю, но может где-то есть эстетичное и легкое решение.
Ну и плюс спортивный интерес :)

[tukreb]

KuzmenkoArtem, легкое решение - написать простой SQL триггер, который бы проверял сколько осталось админов и блокировал бы запрос.

[Dmitry Bay]

KuzmenkoArtem, А вы не думали просто запретить удалять себя? И если уж удаляешь себя, то будь добр подтверди прям печатью это?

[alexalexes]

Если есть доступ к базе, то пофигу, что удаляться все юзеры.
Ситуация крайне редкая, но решаемая при помощи выполнения sql-ка заготовки создания пользователя-админа со всеми вытекающими правами. Во всех CMS можно найти кусочек сценария развертывания, где прописывается дефолтный пользователь. Это небольшая проблема.

[KuzmenkoArtem]

Dmitry Bay, Такой подход не поможет к сожалению, когда два админа будут удалять друг друга в один момент времени

Answer 1

[dollar]

LOCK TABLE имя_таблицы WRITE;

Answer 2

[Sanes]

Сам себя запрети удалять.

Comments

[KuzmenkoArtem]

К сожалению не поможет, из-за описанной ситуации в примере. Когда два админа удаляют друг друга в один момент времени

[Sanes]

KuzmenkoArtem, поможет.

[KuzmenkoArtem]

Sanes, 100% не поможет. Проверенно экспериментальным путем. Ну и по логике оно тут не причем, т.к. юзера удаляют не себя, а друг-друга

[Sanes]

KuzmenkoArtem, в одну микросекунду?

[KuzmenkoArtem]

Sanes, Между запросами может произойти все что угодно. Например перегрузка сервера или запрос в mysql может выполниться не сразу из за других возможных локов или той же перегрузки, и задержка может быть вполне значительная. В общем по нормальному в профессиональном приложении нужно покрыть все возможные случаи. Был бы это сайт-визитка, я бы да не заморачивался

[Sanes]

KuzmenkoArtem, проблема на ровном месте.

[KuzmenkoArtem]

Sanes, Ну как минимум это спортивный интерес. Вам разве как специалисту не интересно разбираться во всех тонкостях? в какой-то момент эти знания могут здорово помочь

[Sanes]

KuzmenkoArtem, по ответам разве не заметно, что это проблема на ровном месте? Такую коллизию на практике воспоизвести практически нереально. Даже, если специально долбить.

[KuzmenkoArtem]

Sanes, У меня получилось. Проблема на ровном месте для тех кто может работает с сайтами-визитками. В моем случае такой исход просто не допустим. Но вопрос же не в этом, зачем спорить...