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

Откуда:
Сообщений: 1130
Критик
Alexander Us,

а теперь представьте, что разработчиков у вас 50 штук (у меня примерно столько), добавьте сюда текучку кадров - и вы утоните в этих объяснениях


Не воспринимайте буквально.
Я имел ввиду, что элементарную квалификацию пользователей и организацию работы не заменить сводои догм.
6 фев 18, 17:04    [21170660]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Гулин Федор
Member

Откуда: МИНСК
Сообщений: 1239
alexeyvg
invm
пропущено...
Да уж...
Редкостный бред.

Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/


Use Natural Primary key rather than Surrogate Primary key. Natural Primary keys are keys base on an external meaningful data. Surrogate Primary key are keys base on an auto-increment number or values which have no meaning outside the system

хм всегда считал ровно Наоборот (конечно есть исключения )
6 фев 18, 17:06    [21170668]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Гулин Федор
Member

Откуда: МИНСК
Сообщений: 1239
зы а с какого это рожна пользователи могут писать SQL-код ?
ну т.е кто такие эти юзеры - для чего им это ?

ну т.е случаи конечно разные бывают - так и наверно рецепты как уменьшить могут быть разными
просто вот те юзеры к-е были у меня в принципе писать SQL не умели.
6 фев 18, 17:09    [21170679]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Владислав Колосов
Member

Откуда:
Сообщений: 7762
Непонятно, почему пользователи пишут (т.е. далёкие от специальных знаний люди) запросы.
6 фев 18, 18:07    [21170862]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31356
Гулин Федор
alexeyvg
пропущено...

Вот ещё есть, пошире списочек: https://www.dbrnd.com/2016/12/database-design-common-practices-for-database-developers-and-application-developers/


Use Natural Primary key rather than Surrogate Primary key. Natural Primary keys are keys base on an external meaningful data. Surrogate Primary key are keys base on an auto-increment number or values which have no meaning outside the system

хм всегда считал ровно Наоборот (конечно есть исключения )
Есть 2 вида проектировщиков баз данных. Одни ...
:-)
6 фев 18, 20:28    [21171136]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31356
Alexander Us
Критик
Alexander Us,

а теперь представьте, что разработчиков у вас 50 штук (у меня примерно столько), добавьте сюда текучку кадров - и вы утоните в этих объяснениях


Не воспринимайте буквально.
Я имел ввиду, что элементарную квалификацию пользователей и организацию работы не заменить сводои догм.
Понятно, что, эти 50 штук разработчиков будут разбираться в ограничениях как в "чёрном ящике"?
- "Вот так написали, не прокатило, ошибка. Попробуем так... А вообще, для какого сервера БД мы пишем, где про него можно прочитать?"
Понятно, что нужно какое то описание, + квалификация, + старшие (а то один специалист на 50+ неучей выглядит странно, он с некоторыми может даже за год не поговорить не разу).
6 фев 18, 20:32    [21171140]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Mind
Member

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

Впрочем, понятно, если посмотреть на профиль - опытный программист, с неслабым самомнением, но совсем не специалист по MSSQL, высказывает о нём своё мнение, пишет "правила" (!). Я встречал таких, бывало.

автор
I have more than six years of experience with various RDBMS products like MSSQL Server, PostgreSQL, MySQL, Greenplum and currently learning and doing research on BIGData and NoSQL technology
Это наверное по 1 году на каждую технологию. Очень опытный эксперт. С сертификатом от Oracle без знания Oracle...
6 фев 18, 21:02    [21171220]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

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

Впрочем, понятно, если посмотреть на профиль - опытный программист, с неслабым самомнением, но совсем не специалист по MSSQL, высказывает о нём своё мнение, пишет "правила" (!). Я встречал таких, бывало.

автор
I have more than six years of experience with various RDBMS products like MSSQL Server, PostgreSQL, MySQL, Greenplum and currently learning and doing research on BIGData and NoSQL technology
Это наверное по 1 году на каждую технологию. Очень опытный эксперт. С сертификатом от Oracle без знания Oracle...
Я писал про автора первых "рекомендаций", Nikhil Devre. "I work as a Senior Application Architect."

Второй, ANVESH PATEL, про которого вы пишите, всё таки специалист по БД.
6 фев 18, 21:11    [21171249]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
X-Cite
Member

Откуда: Минск
Сообщений: 1747
Базы данных только как хранилище данных. Они медленные, запросы выполняются долго, хотя должны вернуть пару строк.. Лучше напишем кеширование, запихнем в Redis. Обернем 100500 сервисов, положим в ОЗУ, не то что ваш SQL.

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

Вот и 50 разработчиков и текучка...
6 фев 18, 23:07    [21171491]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Mind
Member

Откуда: Лучший город на Земле
Сообщений: 2322
alexeyvg
Mind
пропущено...

пропущено...
Это наверное по 1 году на каждую технологию. Очень опытный эксперт. С сертификатом от Oracle без знания Oracle...
Я писал про автора первых "рекомендаций", Nikhil Devre. "I work as a Senior Application Architect."

Второй, ANVESH PATEL, про которого вы пишите, всё таки специалист по БД.
Возможно, никогда о нем не слышал, да и рекомендации из разряда ни о чем - "Анализируйте ваши запросы на производительность"...
6 фев 18, 23:55    [21171587]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Критик
Member

Откуда: Москва / Калуга
Сообщений: 33674
Блог
alexeyvg
Alexander Us
пропущено...


Не воспринимайте буквально.
Я имел ввиду, что элементарную квалификацию пользователей и организацию работы не заменить сводои догм.
Понятно, что, эти 50 штук разработчиков будут разбираться в ограничениях как в "чёрном ящике"?
- "Вот так написали, не прокатило, ошибка. Попробуем так... А вообще, для какого сервера БД мы пишем, где про него можно прочитать?"
Понятно, что нужно какое то описание, + квалификация, + старшие (а то один специалист на 50+ неучей выглядит странно, он с некоторыми может даже за год не поговорить не разу).


инструкции у нас есть, но их читают только неудачники (а может просто трудно сразу запомнить сотни страниц описаний), а так раз выскочила ошибка - и сразу с описанием, что сделано неправильно
7 фев 18, 00:16    [21171619]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Kopilogus
Member

Откуда:
Сообщений: 49
Спасибо всем за отзывы!

В нашем случаем разрабов под сотню, и как писали коллеги, текучка персонала также имеет место быть.
Обучать - да нужно, но в наши задачи как DBA не входят, а вот геморрой и последствия после кодеров разгребаем по полной..
7 фев 18, 13:46    [21173070]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Олегон
Member

Откуда: Казахстан, Алматы
Сообщений: 80
Можете ещё посмотреть на SQL Code Guard, как интсрумент анализа кода на всякие типичные огрехи и best practices.
Теперь эту утилиту выкупил Redgate, но пока ещё бесплатна.

https://www.red-gate.com/products/sql-development/sql-code-guard/
7 фев 18, 14:05    [21173142]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
.Евгений
Member

Откуда:
Сообщений: 516
Kopilogus
В нашем случаем разрабов под сотню, и как писали коллеги, текучка персонала также имеет место быть.
Обучать - да нужно, но в наши задачи как DBA не входят, а вот геморрой и последствия после кодеров разгребаем по полной..

Забавное местечко, в котором админам позволят блокировать разработку.
На самом деле вам нужно найти (или помочь создать) релиз-мастера и все последствия разруливать с ним. Желательна практика ревью кода. Теоретически есть и еще более изощренные варианты - вплоть до аттестации, по результатам которой отсеются геморройные кодеры.
7 фев 18, 14:53    [21173344]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Гигабайт Мегабайтович Килобайтов
Member [заблокирован]

Откуда:
Сообщений: 5975
Kopilogus
Спасибо всем за отзывы!

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

а админов сколько? ))
таки в вашем случае напрашивается ревью кода разработчиками, а после админами.
Но таки ради "ускорения" фиг так сделают..
7 фев 18, 15:24    [21173462]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31356
Критик
alexeyvg
пропущено...
Понятно, что, эти 50 штук разработчиков будут разбираться в ограничениях как в "чёрном ящике"?
- "Вот так написали, не прокатило, ошибка. Попробуем так... А вообще, для какого сервера БД мы пишем, где про него можно прочитать?"
Понятно, что нужно какое то описание, + квалификация, + старшие (а то один специалист на 50+ неучей выглядит странно, он с некоторыми может даже за год не поговорить не разу).


инструкции у нас есть, но их читают только неудачники (а может просто трудно сразу запомнить сотни страниц описаний), а так раз выскочила ошибка - и сразу с описанием, что сделано неправильно
Вот, в этом и проблема - инструкций нет (раз их никто не читает), желания (или денег) на вменяемые инструкции и вменяемое управление нет, масса бесполезных (но самых высокооплачиваемых) людей (которые должны были бы управлять и писать) радостно пилят деньги вместо работы, ну а с разработчиков, нанятых по остаточному принципу, чего спрашивать? поют как умеют.
7 фев 18, 15:27    [21173472]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Критик
Member

Откуда: Москва / Калуга
Сообщений: 33674
Блог
.Евгений
На самом деле вам нужно найти (или помочь создать) релиз-мастера и все последствия разруливать с ним. Желательна практика ревью кода


это просто кинуть горячий мячик "на ту сторону", и в 99% случаев ничего не будет исправлено по разным причина,
а код-ревью - это почти х2 трудозатрат на разработку, мало мест, где такое согласуют, ведь все должно быть реализовано еще вчера
7 фев 18, 16:26    [21173774]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Гигабайт Мегабайтович Килобайтов
Member [заблокирован]

Откуда:
Сообщений: 5975
Критик
.Евгений
На самом деле вам нужно найти (или помочь создать) релиз-мастера и все последствия разруливать с ним. Желательна практика ревью кода


это просто кинуть горячий мячик "на ту сторону", и в 99% случаев ничего не будет исправлено по разным причина,
а код-ревью - это почти х2 трудозатрат на разработку, мало мест, где такое согласуют, ведь все должно быть реализовано еще вчера

как я и написал - ради "ускорения" фиг разрешат))
если правильно поставлен процесс разработки - вопрос совсем не больших денег и не большого времени. Но очень большого начального администрирования.
7 фев 18, 17:08    [21173961]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
.Евгений
Member

Откуда:
Сообщений: 516
Критик
это просто кинуть горячий мячик "на ту сторону"

Пусть так. Но контроль качества кода принадлежит именно "той стороне".
Критик
в 99% случаев ничего не будет исправлено по разным причина (...) мало мест, где такое согласуют

Вы явно сгущаете краски.
7 фев 18, 17:14    [21173995]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Сон Веры Павловны
Member

Откуда:
Сообщений: 5955
Гигабайт Мегабайтович Килобайтов
таки в вашем случае напрашивается ревью кода разработчиками, а после админами

Ревью кода админами - это вообще одна из самых странных вещей, про которые мне приходилось слышать. Такое где-то бывает? Может, там админов перевести в разработчики, и вся недолга?
7 фев 18, 17:20    [21174020]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31356
Сон Веры Павловны
Гигабайт Мегабайтович Килобайтов
таки в вашем случае напрашивается ревью кода разработчиками, а после админами

Ревью кода админами - это вообще одна из самых странных вещей, про которые мне приходилось слышать. Такое где-то бывает? Может, там админов перевести в разработчики, и вся недолга?
Как сказать, конечно, если человек делает ревью, то есть "разрабатывает", то он "разработчик", но иногда такие люди называются DBA, и это общемировая практика.
Т.е. бывает, часть обязанностей DBA - разрабатывать код для СУБД (более сложный, чем простые селекты, и более нестандартный, чем отчётные процедуры), попутно консультировать разработчиков, ну и делать ревью их кода.
Не всегда так, но есть, можно посмотреть описание вакансий на зарубежных сайтах.
7 фев 18, 18:14    [21174229]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
alexeyvg
Member

Откуда: Moscow
Сообщений: 31356
Критик
а код-ревью - это почти х2 трудозатрат на разработку, мало мест, где такое согласуют, ведь все должно быть реализовано еще вчера
Парное программирование - это "почти х2 трудозатрат", и действительно редко встречается, а ревью как раз стоит немного.
Я не работал за много лет ни в одном месте, где бы оно не делалось (хотя видел такое).
Что бы кто то опытный, или начальник, никогда не взглянул на код джуниора, или не поправил бяки, если всё равно делает модификацию этого кода? Это надо очень сильно накласть на качество кода, продукта, профессию, самоуважение, и всё такое :-)
7 фев 18, 18:20    [21174247]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Критик
Member

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

Все завист от того, что вы понимаете под ревью, если глянуть правильность наименований объектов, то это быстро, а если смотреть логику процедуры из нескольких тысяч строк, то может получиться и больше, чем х2
8 фев 18, 08:46    [21175083]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
Valery_B
Member

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

Автор предлагает пересылать всю таблицу в несколько мега или даже гигабайт на какой-то другой сервер, что бы сделать там GroupBy или Distinct ?)

Рекомендация похожа на те, в которой авторы рекомендуют обращаться к полям не по имени, а по индексу.
т.е.
DataSet.FieldByName('UserName') - не правильно.
DataSet.Fields[7] - с точки зрения авторов правильней, с точке зрения производительности)


Если в вашем ПО заметна разница в скорости между индексом и именем - то это повод задуматься, что бы сменить разработчика.
8 фев 18, 09:54    [21175314]     Ответить | Цитировать Сообщить модератору
 Re: PBM + DDL говнокод  [new]
dao
Member

Откуда: Москва
Сообщений: 771
Критик
alexeyvg,

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

ревью кода не должно нести проверку "бизнесовости" кода, только с точки зрения правильности написания sql-кода, если более опытный разработчик увидел явный ляп - то конечно на него обратить внимание, но основное - правильность написания sql-кода, и вот тут очень помогает регламент . "Бизнесовость" кода должно проверять тестирование. или таки у вас и на этом "экономят"? ))
8 фев 18, 10:21    [21175433]     Ответить | Цитировать Сообщить модератору
Топик располагается на нескольких страницах: Ctrl  назад   1 [2] 3   вперед  Ctrl      все
Все форумы / Microsoft SQL Server Ответить