Добро пожаловать в форум, Guest  >>   Войти | Регистрация | Поиск | Правила | В избранное | Подписаться
Все форумы / Microsoft SQL Server Новый топик    Ответить
Топик располагается на нескольких страницах: [1] 2 3   вперед  Ctrl      все
 PBM + DDL говнокод  [new]
Kopilogus
Member

Откуда:
Сообщений: 49
Коллеги, доброго времени суток!

Поделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода?

На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдают, соответсвенно стоит задача - минимизировать влияние плохого кода на производительность DB и сервера в целом.
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д.

Не сложно написать триггер, который будет обрабатывать такие события, но все таки хотелось бы узнать есть ли возможность использовать политики?

С политиками именования DDL объектов проблем не возникло, а вот с примером выше реализовать рабочую схему ON CHANGE не удалось.

Есть идеи?
6 фев 18, 12:42    [21169580]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Kopilogus
Member

Откуда:
Сообщений: 49
Коллеги, также приветствуются прмеры "как не надо делать", для пополнения коллекции шаблонов..
6 фев 18, 12:50    [21169604]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Кусочек
Guest
Kopilogus
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д.



А как же -

WHERE EXISTS  
(SELECT * 


Ну и по Вашей же ссылке:

Use GROUP BY, ORDER BY, or DISTINCT in your query only when absolutely necessary.
6 фев 18, 13:34    [21169780]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
.Евгений
Member

Откуда:
Сообщений: 514
Запретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения?
Это конгениально!
Достал попкорн, уселся поудобнее.
6 фев 18, 13:37    [21169793]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Сон Веры Павловны
Member

Откуда:
Сообщений: 5949
.Евгений
Запретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения?
Это конгениально!
Достал попкорн, уселся поудобнее.

Ну select * запретить вполне можно. Насчет остального - улыбнуло.
6 фев 18, 13:45    [21169822]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
X-Cite
Member

Откуда: Минск
Сообщений: 1732
Kopilogus
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д.

Почему нельзя? А как вы собираетесь выводить сгруппированный результат. Или использовать TOP, OFFSET или оконные функции без ORDER BY ?
Автор статьи имел ввиду что не нужно их лепить везде где хочется, а только исходя их условий задач. А такие задачи каждая вторая будут.
6 фев 18, 13:51    [21169850]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Kopilogus
Member

Откуда:
Сообщений: 49
Сон Веры Павловны
.Евгений
Запретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения?
Это конгениально!
Достал попкорн, уселся поудобнее.

Ну select * запретить вполне можно. Насчет остального - улыбнуло.


Коллеги, это всего лишь пример, сабж - можно ли реализовать при помощи политик управления?
6 фев 18, 13:51    [21169852]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
Kopilogus
Поделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода?

На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдают
Ээээ, как можно сделать это автоматически? :-)

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

Всякие анализаторы кода помогут в настолько малом числе случаев, что ИМХО их применение будет сильно вредить, успокаивая "всё хорошо".

Квалифицированный специалист увидит, что человек пишет кривой код, и его не нужно брать с испытательного срока. И он должен этим управлять, даже если вы напишете триггер ,который ругнётся (в 5% случаев) на кривой код и не даст его вставить.

Т.е., затраты на "анализатор" будут, а прибыли в компанию с этих затрат - нет. Так какой смысл?
Kopilogus
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY
Ээээ, как можно не делать ORDER BY ,если в задании написано "сделать ORDER BY"? Или вы про то, что делают ORDER BY тогда, когда этого не надо? Тогда способ борьбы описан выше.
6 фев 18, 13:53    [21169860]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
alexeyvg
Kopilogus
Поделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода?

На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдают
Ээээ, как можно сделать это автоматически? :-)

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

Всякие анализаторы кода помогут в настолько малом числе случаев, что ИМХО их применение будет сильно вредить, успокаивая "всё хорошо".

Квалифицированный специалист увидит, что человек пишет кривой код, и его не нужно брать с испытательного срока. И он должен этим управлять, даже если вы напишете триггер ,который ругнётся (в 5% случаев) на кривой код и не даст его вставить.

Т.е., затраты на "анализатор" будут, а прибыли в компанию с этих затрат - нет. Так какой смысл?
Kopilogus
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY
Ээээ, как можно не делать ORDER BY ,если в задании написано "сделать ORDER BY"? Или вы про то, что делают ORDER BY тогда, когда этого не надо? Тогда способ борьбы описан выше.
Добавлю, что если уж заговорили о технических средствах избавления от говнокода, то начать нужно с:
- средств разработки,
- хранения исходников в базе версионного контроля,
- привязки изменений к требованиям,
- тестировании,
- автоматических ночных сборках,
- деплоях,
- работать с цепочкой разработка - стабильный тест - продакшен.
и т.п.

А то народ зачастую правит код в продакшене прямо внутри базы (практически как в экзешнике в кодах), и потом рассуждает, как избавиться от говнокода :-)
6 фев 18, 14:01    [21169889]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Yasha123
Member

Откуда:
Сообщений: 1833
Kopilogus
можно ли реализовать при помощи политик управления?

ну конечно же нет.
вы вот попробуйте сперва свои хотелки формализовать:

звезду в пронципе нельзя использовать, но вообще-то можно, если это часть строки,
умножение, или кусок exists

order by нельзя, но вообще-то можно, если это часть top-а
или же это вставка в кластерный с целью минимального логирования

и т.д.

вы в состоянии всю эту логику запихнуть в политику?
это ж прям парсер надо свой писать,
причем офигительный такой парсер...
6 фев 18, 14:11    [21169914]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Kopilogus
Member

Откуда:
Сообщений: 49
X-Cite
Kopilogus
К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д.

Почему нельзя? А как вы собираетесь выводить сгруппированный результат. Или использовать TOP, OFFSET или оконные функции без ORDER BY ?
Автор статьи имел ввиду что не нужно их лепить везде где хочется, а только исходя их условий задач. А такие задачи каждая вторая будут.


Это уже другой вопрос, всегда можно временно отключить проверки.
6 фев 18, 14:16    [21169928]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
Yasha123
вы в состоянии всю эту логику запихнуть в политику?
это ж прям парсер надо свой писать,
причем офигительный такой парсер...
И, главное, этим не убрать остальные 90% мест, где можно наговнокодить. Это же только некоторые простые правила.
То есть это не избавит от необходимости читать код, учить, увольнять и т.д. А при контроле "глазами" все перечисленные ляпы будет видно моментально, так зачем делать этот анализатор?
6 фев 18, 14:24    [21169954]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Критик
Member

Откуда: Москва / Калуга
Сообщений: 33639
Блог
alexeyvg
Ээээ, как можно сделать это автоматически? :-)


в студии можно,
если проверок не хватает - можно написать свою, в инете есть примеры

К сообщению приложен файл. Размер - 43Kb
6 фев 18, 14:24    [21169955]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Yasha123
Member

Откуда:
Сообщений: 1833
Критик
если проверок не хватает - можно написать свою, в инете есть примеры

так мы как раз про это самое,
про "написать свою"
вы напишите плиз свою, особенно ту, что оценит,
действительно ли в данном коде не нужен order by.
а то на словах-то все можно
6 фев 18, 14:35    [21170005]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Yasha123
Member

Откуда:
Сообщений: 1833
alexeyvg
И, главное, этим не убрать остальные 90% мест, где можно наговнокодить. Это же только некоторые простые правила.
То есть это не избавит от необходимости читать код, учить, увольнять и т.д. А при контроле "глазами" все перечисленные ляпы будет видно моментально, так зачем делать этот анализатор?

я думаю, ТС и Критик хорошо споются.
один в состоянии формализовать задачу
и предоставить исчерпывающий список потенциально говнокодных конструкций,
ну а второй запросто это выразит кодом ;)
6 фев 18, 14:37    [21170009]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Alexander Us
Member

Откуда:
Сообщений: 1130
Kopilogus,

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

Наверное Вам стоит уделить внимание мониторингу (находить пользователей имеющих проблемы с запросами) и помогать/объяснять как улучшить.

Не оспаривая правила которые Вы привели, замечу, что 99% проблем лежит в другом.

Вам надо разобраться что люди не понимают, найти те 3% фишек, которые дадут основную часть увеличения производительности,
а не долбить людей догматическими правилами.

Например, в моей практике пришлось однажды обяснить пользователю (кстати, прекрасный специалист в своей области),
что на таблице можно создать ключ или индекс чтобы запросы выполнялись быстрее.

Короче, просто помогите/обясните людям и будет Вам счастье.
6 фев 18, 15:21    [21170198]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
StarikNavy
Member

Откуда: Москва
Сообщений: 2394
Kopilogus
К примеру - нельзя создать процедуру в которой используется GROUP BY, DISTINCT и


GROUP BY, DISTINCT то чем вам не угодили?
6 фев 18, 15:27    [21170228]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Критик
Member

Откуда: Москва / Калуга
Сообщений: 33639
Блог
Alexander Us,

а теперь представьте, что разработчиков у вас 50 штук (у меня примерно столько), добавьте сюда текучку кадров - и вы утоните в этих объяснениях
6 фев 18, 15:29    [21170241]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
Критик
alexeyvg
Ээээ, как можно сделать это автоматически? :-)


в студии можно,
если проверок не хватает - можно написать свою, в инете есть примеры
Я писал про другое. Такое да, можно, и есть много средств для этого.
6 фев 18, 15:29    [21170242]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
invm
Member

Откуда: Москва
Сообщений: 9343
Kopilogus
есть простые очевидные рекомендации
Да уж...
Редкостный бред.
6 фев 18, 15:32    [21170254]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
invm
Kopilogus
есть простые очевидные рекомендации
Да уж...
Редкостный бред.
Ладно, многие из них выглядят вполне разумно. Только если без формализма, и понимая, "почему".
Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/

Но смысл предложения ТС в другом - можно нанять плохих разработчиков, уволить хороших, настроить средство автоматического запрета говнокода, и получить профит.
Так вот, идея неудачная. Не получиться так сделать.
6 фев 18, 15:40    [21170284]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
.Евгений
Member

Откуда:
Сообщений: 514
alexeyvg
Ладно, многие из них выглядят вполне разумно. Только если без формализма, и понимая, "почему".
Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/

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

Я думаю, это Админ Роскомнадзор.
6 фев 18, 15:50    [21170317]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Yasha123
Member

Откуда:
Сообщений: 1833
alexeyvg
invm
пропущено...
Да уж...
Редкостный бред.
Ладно, многие из них выглядят вполне разумно.

автор
Avoid Using GROUP BY, ORDER BY, and DISTINCT
Avoid using GROUP BY, ORDER BY, and DISTINCT as much as possible

Reason: When using GROUP BY, ORDER BY, or DISTINCT, SQL Server engine creates a work table and puts the data on the work table. After that, it organizes this data in work table as requested by the query, and then it returns the final result.

ну вот это вот уж ни в какие ворота не лезет.
неужели же кто-то станет писать group by,
только потому, что ему нечего делать?
ведь или тебе нужны агрегаты,
или у тебя просто будет неверный результат,
схлопнуть группы в строки и не поменять при этом результат -- это как?
не полениться перечислить все нужные колонки еще раз, но уже не в SELECT,
а в GROUP BY?
это типа всем назло?
6 фев 18, 16:02    [21170376]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
invm
Member

Откуда: Москва
Сообщений: 9343
alexeyvg
Ладно, многие из них выглядят вполне разумно.
Выглядят может и разумно, только не для оптимизации производительности.
А в причинах описаны, в большинстве, чьи-то фантазии.
6 фев 18, 16:18    [21170446]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31354
invm
alexeyvg
Ладно, многие из них выглядят вполне разумно.
Выглядят может и разумно, только не для оптимизации производительности.
А в причинах описаны, в большинстве, чьи-то фантазии.
Вообще да, неквалифицированность автора прямо выпирает, если приглядеться :-)
Что то разумно, да, но в целом, как будто автор совсем начинающий.

Впрочем, понятно, если посмотреть на профиль - опытный программист, с неслабым самомнением, но совсем не специалист по MSSQL, высказывает о нём своё мнение, пишет "правила" (!). Я встречал таких, бывало.
6 фев 18, 16:53    [21170620]     Ответить | Цитировать Сообщить модератору
Топик располагается на нескольких страницах: [1] 2 3   вперед  Ctrl      все
Все форумы / Microsoft SQL Server Ответить