Этот баннер — требование Роскомнадзора для исполнения 152 ФЗ.
«На сайте осуществляется обработка файлов cookie, необходимых для работы сайта, а также для анализа использования сайта и улучшения предоставляемых сервисов с использованием метрической программы Яндекс.Метрика. Продолжая использовать сайт, вы даёте согласие с использованием данных технологий».
Политика конфиденциальности
|
|
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Коллеги, доброго времени суток! Поделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода? На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдают, соответсвенно стоит задача - минимизировать влияние плохого кода на производительность DB и сервера в целом. К примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. Не сложно написать триггер, который будет обрабатывать такие события, но все таки хотелось бы узнать есть ли возможность использовать политики? С политиками именования DDL объектов проблем не возникло, а вот с примером выше реализовать рабочую схему ON CHANGE не удалось. Есть идеи? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 12:42 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Коллеги, также приветствуются прмеры "как не надо делать", для пополнения коллекции шаблонов.. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 12:50 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
KopilogusК примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. А как же - Код: sql 1. 2. Ну и по Вашей же ссылке: Use GROUP BY, ORDER BY, or DISTINCT in your query only when absolutely necessary. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:34 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Запретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения? Это конгениально! Достал попкорн, уселся поудобнее. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:37 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
.ЕвгенийЗапретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения? Это конгениально! Достал попкорн, уселся поудобнее. Ну select * запретить вполне можно. Насчет остального - улыбнуло. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:45 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
KopilogusК примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. Почему нельзя? А как вы собираетесь выводить сгруппированный результат. Или использовать TOP, OFFSET или оконные функции без ORDER BY ? Автор статьи имел ввиду что не нужно их лепить везде где хочется, а только исходя их условий задач. А такие задачи каждая вторая будут. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:51 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Сон Веры Павловны.ЕвгенийЗапретить разработчикам использование ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. без отдельного высочайшего разрешения? Это конгениально! Достал попкорн, уселся поудобнее. Ну select * запретить вполне можно. Насчет остального - улыбнуло. Коллеги, это всего лишь пример, сабж - можно ли реализовать при помощи политик управления? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:51 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
KopilogusПоделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода? На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдаютЭэээ, как можно сделать это автоматически? :-) Для этого придумали разные методы управления разработкой, начиная от парного программирования, ревью и т.д., заканчивая техниками создания хорошей квалифицированной команды. Всякие анализаторы кода помогут в настолько малом числе случаев, что ИМХО их применение будет сильно вредить, успокаивая "всё хорошо". Квалифицированный специалист увидит, что человек пишет кривой код, и его не нужно брать с испытательного срока. И он должен этим управлять, даже если вы напишете триггер ,который ругнётся (в 5% случаев) на кривой код и не даст его вставить. Т.е., затраты на "анализатор" будут, а прибыли в компанию с этих затрат - нет. Так какой смысл? KopilogusК примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY Ээээ, как можно не делать ORDER BY ,если в задании написано "сделать ORDER BY"? Или вы про то, что делают ORDER BY тогда, когда этого не надо? Тогда способ борьбы описан выше. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 13:53 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvgKopilogusПоделитесь опытом, кто каким образом ограничивает пользователей БД от написания плохого t-sql кода? На вскидку, есть простые очевидные рекомендации , но к сожалению разработчики их не всегда соблюдаютЭэээ, как можно сделать это автоматически? :-) Для этого придумали разные методы управления разработкой, начиная от парного программирования, ревью и т.д., заканчивая техниками создания хорошей квалифицированной команды. Всякие анализаторы кода помогут в настолько малом числе случаев, что ИМХО их применение будет сильно вредить, успокаивая "всё хорошо". Квалифицированный специалист увидит, что человек пишет кривой код, и его не нужно брать с испытательного срока. И он должен этим управлять, даже если вы напишете триггер ,который ругнётся (в 5% случаев) на кривой код и не даст его вставить. Т.е., затраты на "анализатор" будут, а прибыли в компанию с этих затрат - нет. Так какой смысл? KopilogusК примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY Ээээ, как можно не делать ORDER BY ,если в задании написано "сделать ORDER BY"? Или вы про то, что делают ORDER BY тогда, когда этого не надо? Тогда способ борьбы описан выше.Добавлю, что если уж заговорили о технических средствах избавления от говнокода, то начать нужно с: - средств разработки, - хранения исходников в базе версионного контроля, - привязки изменений к требованиям, - тестировании, - автоматических ночных сборках, - деплоях, - работать с цепочкой разработка - стабильный тест - продакшен. и т.п. А то народ зачастую правит код в продакшене прямо внутри базы (практически как в экзешнике в кодах), и потом рассуждает, как избавиться от говнокода :-) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:01 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Kopilogusможно ли реализовать при помощи политик управления? ну конечно же нет. вы вот попробуйте сперва свои хотелки формализовать: звезду в пронципе нельзя использовать, но вообще-то можно, если это часть строки, умножение, или кусок exists order by нельзя, но вообще-то можно, если это часть top-а или же это вставка в кластерный с целью минимального логирования и т.д. вы в состоянии всю эту логику запихнуть в политику? это ж прям парсер надо свой писать, причем офигительный такой парсер... ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:11 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
X-CiteKopilogusК примеру - нельзя создать процедуру в которой используется ORDER BY, GROUP BY, SELECT *, DISTINCT и т.д. Почему нельзя? А как вы собираетесь выводить сгруппированный результат. Или использовать TOP, OFFSET или оконные функции без ORDER BY ? Автор статьи имел ввиду что не нужно их лепить везде где хочется, а только исходя их условий задач. А такие задачи каждая вторая будут. Это уже другой вопрос, всегда можно временно отключить проверки. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:16 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Yasha123вы в состоянии всю эту логику запихнуть в политику? это ж прям парсер надо свой писать, причем офигительный такой парсер...И, главное, этим не убрать остальные 90% мест, где можно наговнокодить. Это же только некоторые простые правила. То есть это не избавит от необходимости читать код, учить, увольнять и т.д. А при контроле "глазами" все перечисленные ляпы будет видно моментально, так зачем делать этот анализатор? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:24 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvgЭэээ, как можно сделать это автоматически? :-) в студии можно, если проверок не хватает - можно написать свою, в инете есть примеры ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:24 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Критикесли проверок не хватает - можно написать свою, в инете есть примеры так мы как раз про это самое, про "написать свою" вы напишите плиз свою, особенно ту, что оценит, действительно ли в данном коде не нужен order by. а то на словах-то все можно ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:35 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvgИ, главное, этим не убрать остальные 90% мест, где можно наговнокодить. Это же только некоторые простые правила. То есть это не избавит от необходимости читать код, учить, увольнять и т.д. А при контроле "глазами" все перечисленные ляпы будет видно моментально, так зачем делать этот анализатор? я думаю, ТС и Критик хорошо споются. один в состоянии формализовать задачу и предоставить исчерпывающий список потенциально говнокодных конструкций, ну а второй запросто это выразит кодом ;) ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 14:37 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Kopilogus, Мне кажется, Вы (напрасно) пытаетесь применить технические решениея там, где нужен именно организационный подход. Ваши пользователи не монстры, и, если они бы могли писать эффективных код, они бы это делали. Наверное Вам стоит уделить внимание мониторингу (находить пользователей имеющих проблемы с запросами) и помогать/объяснять как улучшить. Не оспаривая правила которые Вы привели, замечу, что 99% проблем лежит в другом. Вам надо разобраться что люди не понимают, найти те 3% фишек, которые дадут основную часть увеличения производительности, а не долбить людей догматическими правилами. Например, в моей практике пришлось однажды обяснить пользователю (кстати, прекрасный специалист в своей области), что на таблице можно создать ключ или индекс чтобы запросы выполнялись быстрее. Короче, просто помогите/обясните людям и будет Вам счастье. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:21 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
KopilogusК примеру - нельзя создать процедуру в которой используется GROUP BY, DISTINCT и GROUP BY, DISTINCT то чем вам не угодили? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:27 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Alexander Us, а теперь представьте, что разработчиков у вас 50 штук (у меня примерно столько), добавьте сюда текучку кадров - и вы утоните в этих объяснениях ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:29 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
КритикalexeyvgЭэээ, как можно сделать это автоматически? :-) в студии можно, если проверок не хватает - можно написать свою, в инете есть примерыЯ писал про другое. Такое да, можно, и есть много средств для этого. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:29 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
Kopilogusесть простые очевидные рекомендации Да уж... Редкостный бред. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:32 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
invmKopilogusесть простые очевидные рекомендации Да уж... Редкостный бред.Ладно, многие из них выглядят вполне разумно. Только если без формализма, и понимая, "почему". Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/ Но смысл предложения ТС в другом - можно нанять плохих разработчиков, уволить хороших, настроить средство автоматического запрета говнокода, и получить профит. Так вот, идея неудачная. Не получиться так сделать. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:40 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvgЛадно, многие из них выглядят вполне разумно. Только если без формализма, и понимая, "почему". Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/ При желании можно раскритиковать любые рекомендации, включая указанные. alexeyvgНо смысл предложения ТС в другом - можно нанять плохих разработчиков, уволить хороших, настроить средство автоматического запрета говнокода, и получить профит. Так вот, идея неудачная. Не получиться так сделать. Я думаю, это Админ Роскомнадзор. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 15:50 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvginvmпропущено... Да уж... Редкостный бред.Ладно, многие из них выглядят вполне разумно. автор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? это типа всем назло? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 16:02 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
alexeyvgЛадно, многие из них выглядят вполне разумно.Выглядят может и разумно, только не для оптимизации производительности. А в причинах описаны, в большинстве, чьи-то фантазии. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 16:18 |
|
||
|
PBM + DDL говнокод
|
|||
|---|---|---|---|
|
#18+
invmalexeyvgЛадно, многие из них выглядят вполне разумно.Выглядят может и разумно, только не для оптимизации производительности. А в причинах описаны, в большинстве, чьи-то фантазии.Вообще да, неквалифицированность автора прямо выпирает, если приглядеться :-) Что то разумно, да, но в целом, как будто автор совсем начинающий. Впрочем, понятно, если посмотреть на профиль - опытный программист, с неслабым самомнением, но совсем не специалист по MSSQL, высказывает о нём своё мнение, пишет "правила" (!). Я встречал таких, бывало. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 06.02.2018, 16:53 |
|
||
|
|

start [/forum/topic.php?fid=46&msg=39597374&tid=1690314]: |
0ms |
get settings: |
10ms |
get forum list: |
14ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
55ms |
get topic data: |
9ms |
get forum data: |
4ms |
get page messages: |
62ms |
get tp. blocked users: |
1ms |
| others: | 222ms |
| total: | 383ms |

| 0 / 0 |
