|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Всем доброго дня. Нужен совет коллег. Устроился на работу , получили задачу реализовать чат между менеджер(кабинет менеджер) и клиентом (профиль клиента на другом домене). База централизована. Вооружился опытом разработки на SignalR и приступил к реализации. В итоге задачу реализовал. Так как чат должен работать в другом домене я решил прибегнуть к iframe который ссылается на страницу в домене менеджере. По сути хаб находится в проекте менеджера так же страница чата менеджера и клиента тоже находится в проекте менеджера. В итоге со старшего разработчика поступила критика такого решение. И было предложено следующее. Цитирую "Надо делать как положено. SignalR (или другая подобная технология) на обоих сайтах, верстка своя на каждом сайте, передача сообщений между сайтами http запросами - это собственно маршаллинг или подобие шины данных." Я от предложенного не много завис но собрав мысли описал минусы такого решения 1. Проблема интеграции стороны клиента в будущем в другие проекты т.е по сути для каждого проект клиента нужно будет писать свою реализацию. 2. общаться между сайтами запросами в данной задаче это лишние запросы а соотвественно риски потери сообщения и время. Может я не прав? Нужен совет. Далее нужен совет по проекту, так как ранее я такого не видел и увидев немного ужаснулся. По коду везде такая реализация. Код: c# 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25.
Так же от увиденного на стороне базы волосы стали шевелится. Вся логика реализована в процедурах и функциях. К примеру что бы получить Name какого либо обьекта используют в SELECT вызов функции вместо join например вот такой Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21.
Это же дополнительная нагрузка на базу В итоге не покидает мысль а оно мне надо. или может я чего не понимаю((((((( Вообщем нужен дельный совет ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 17:05 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
По SignalR поддерживаю. Что за шины между сайтами уточнить надо было)))) и при чём тут маршалинг это из другой темы вообще. По С#. goto сто лет не видел. out msg. может внутри какой то Exception свой отрабатывает. Хотя можно и снаружи его отловить. Скорее тоже поддерживаю какой то моразм. По sql явное не понимание у разработчиков. на отдельную колонку вешать select это сильно. В один день у них всё ляжет и устанут разгребать. Совет. Уноси ноги пока всё не упало ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 17:49 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenК примеру что бы получить Name какого либо обьекта используют в SELECT вызов функции вместо join например вот такойЯ бы попытался спросить о причинах. Вдруг причины есть ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 18:05 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Автор, видать, в разработке довольно новенький. Года через 3-5 такие вещи перестанут удивлять, а еще через 3-5 станут восприниматься как повседневная норма. "Кто в армии служил, тот в цирке не смеется" (с) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 18:54 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenДалее нужен совет по проекту Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно. По двум твоим огрызкам кода выводы делать рано. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 19:31 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthat, За 13 лет такого не встречал ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 19:46 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухIPmenДалее нужен совет по проекту Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно. По двум твоим огрызкам кода выводы делать рано. Бывают реализации где подобное оправдано? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 19:49 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenЗа 13 лет такого не встречалКритики в адрес своего решения? :) Вот встретили, и реакция на мой взгляд странная. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 19:50 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenДмитрий Мухпропущено... Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно. По двум твоим огрызкам кода выводы делать рано. Бывают реализации где подобное оправдано?Что подобное? Сколько продукту лет? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 19:51 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmen, огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe. с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно. обсуждайте. договаривайтесь. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:05 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухIPmenЗа 13 лет такого не встречалКритики в адрес своего решения? :) Вот встретили, и реакция на мой взгляд странная. Да в том то и дело что не в критике же дело а в реализации. Такого кода я не встречал и считаю что так писать нельзя так же и по базе. Реакция наводящая на вопросы к третьим лицам, по моему адекватная. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:05 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
hVosttIPmen, огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe. с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно. обсуждайте. договаривайтесь. Да тут сложностей нет. 1-й вариант. Есть одна сторона проекта менеджера и клиента.база одна. HUB SignalR расположен на стороне manager, клиенту iframe ссылка на manager проект 2-й вариант. на каждой стороне HUB у каждого своя реализация. общение через запросы. Я не пытаюсь сказать решение моё верное я пытаюсь понять если моё решение неверное то в чём? Советуюсь на будущее ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:18 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
hVosttIPmen, огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe. с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно. обсуждайте. договаривайтесь. Всё происходит в рамках одного проекта. т.е это не интеграция с чем то потусторонним. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:20 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухIPmenпропущено... Бывают реализации где подобное оправдано?Что подобное? Сколько продукту лет? Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь . ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:27 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenВсё происходит в рамках одного проекта. т.е это не интеграция с чем то потусторонним. Проект один, а приложения разные. Но я уже об этом сказал. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:33 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenДмитрий Мухпропущено... Что подобное? Сколько продукту лет? Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь .Я не увидел функции на каждую колонку. Не увидел структуры БД, индексов, предметной области... А главное, я не увидел от вас никаких аргументов. Если бы я пришёл на проект, где мне что-то не нравится, то я бы нашёл слова для обоснования своего мнения. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:34 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenЯ не пытаюсь сказать решение моё верное я пытаюсь понять если моё решение неверное то в чём? Советуюсь на будущее Если коротко, то согласно DDD (Domain Driven Design) контексты у вас разные... ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:37 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Причина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:37 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Давайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота... Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:39 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansПричина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде? А в чём жесть, мне просто любопытно? :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:43 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухIPmenпропущено... Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь .Я не увидел функции на каждую колонку. Не увидел структуры БД, индексов, предметной области... А главное, я не увидел от вас никаких аргументов. Если бы я пришёл на проект, где мне что-то не нравится, то я бы нашёл слова для обоснования своего мнения. Например вот так. Слова найти могу но восприятие не доброе. Что и натолкнуло на мысль всё это дел запостить ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:44 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухДавайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота... Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :) ))) В данной реализации это тоже же не проблема. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:47 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenДмитрий МухДавайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота... Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :) ))) В данной реализации это тоже же не проблема. Я вам ещё подкину. Вместо зависимости функции от одного приложения, в котором имплементировано решение, вы получаете два. Вас помянут недобрым словом и разработчики, которым это придётся развивать, тестировщики, девопсы -- от всех вам достанется много нехороших эпитетов. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:50 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmen, и зачем тут JOIN-ы? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:51 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухlansПричина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде? А в чём жесть, мне просто любопытно? :) Можно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть. и всю эту лабуду выкинуть к чертям. по базе уже писал выше. писать select на каждую колонку это причина уволить сотрудника. Я таких не держу. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 20:52 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Опять таки зависит от задачи. Если в перспективе клиент должен будет работать не зависимо от текущего контекста то нужна независимая реализация без всяких iframe а если вы гарантированно будете работать в рамках одой базы и этих двух проектов учитывая что клиент работает с той же базой что и менеджер то почему бы и нет. не вижу изюма в лишних действиях и ваше решение верное. но опять таки надо выслушать аргументы тимлида . ведь он не зря тимлид у него на будущее возможно видинее другое в любом случае надо обсуждать. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:07 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мух, Спецом для Дмитрий Мух как пример Код: sql 1.
Так писать нельзя. Согласны? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:11 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Ещё раз Итог. Код на букву Г... База на букву Г... По чату надо говорить ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:13 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansМожно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть. Фиг знает, не видно исключений. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:14 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мух, Спецом для Дмитрий Мух как пример Код: sql 1.
Так писать нельзя. Согласны?А как надо? :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:15 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухlansДмитрий Мух, Спецом для Дмитрий Мух как пример Код: sql 1.
Так писать нельзя. Согласны?А как надо? :) Правильный ответ пишите так как нравится))))))))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:21 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... А как надо? :) Правильный ответ пишите так как нравится)))))))))Не правильный ответ :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:22 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухlansМожно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть. Фиг знает, не видно исключений. Стоит catch ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:23 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Правильный ответ пишите так как нравится)))))))))Не правильный ответ :) А как надо?))))))))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:24 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:27 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... Не правильный ответ :) А как надо?))))))))) Так, чтобы избавиться от огрехов старой архитектуры.. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:28 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... А как надо?))))))))) Так, чтобы избавиться от огрехов старой архитектуры.. Ну хитрец. Ушёл от ответа)))) Считается ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:44 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Всем спасибо. Суть уловил. Буду прощаться. PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?)))))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:48 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... Так, чтобы избавиться от огрехов старой архитектуры.. Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано. И варианты, как это отрефакторить. Но вам оно видно же, что не надо :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:57 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenВсем спасибо. Суть уловил. Буду прощаться. PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?))))))Нужен, но умеющий своё мнение обосновать :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:58 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано. И варианты, как это отрефакторить. Но вам оно видно же, что не надо :) Варианты в студию! ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 21:59 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано. И варианты, как это отрефакторить. Но вам оно видно же, что не надо :) Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?))))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:02 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... У меня есть варианты, почему так написано. И варианты, как это отрефакторить. Но вам оно видно же, что не надо :) Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?))))) Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:04 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?))))) Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите? Помолиться можно и всё наладиться))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:06 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... У меня есть варианты, почему так написано. И варианты, как это отрефакторить. Но вам оно видно же, что не надо :) Варианты в студию!Если подзапросы завернули в функции, то возможно система GRANT-ов присутсвует. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:07 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите? Помолиться можно и всё наладиться)))Сразу видно профи. Кому, или чему молитесь? ))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:08 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Помолиться можно и всё наладиться)))Сразу видно профи. Кому, или чему молитесь? ))) Одни вопросы. Где ответы? ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:10 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenВсем спасибо. Суть уловил. Буду прощаться. PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?)))))) Фулл стек это зачастую, толком ни бек, ни фронт :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:12 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
hVosttIPmenВсем спасибо. Суть уловил. Буду прощаться. PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?)))))) Фулл стек это зачастую, толком ни бек, ни фронт :) Но точно что не 100% бек и фронт ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:14 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansДмитрий Мухпропущено... Сразу видно профи. Кому, или чему молитесь? ))) Одни вопросы. Где ответы?Ответы должны быть у ТСа, но он постеснялся старшему задать вопросы :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:18 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenНо точно что не 100% бек и фронт Все фуллстеки, которые приходили ко мне на собеседование за последний год, одинаково плохи как в бек, так и во фронт. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:20 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenНо точно что не 100% бек и фронт Все фуллстеки, которые приходили ко мне на собеседование за последний год, одинаково плохи как в бек, так и во фронт. ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:20 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Полегче Полегче понял вас)))))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:21 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenПолегче Полегче понял вас)))))) Вот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база. И ничего не предложили в плане улучшения. Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум. Вы отлично разбираетесь как в бек, так и во фронт.. ))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:27 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухIPmenПолегче Полегче понял вас)))))) Вот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база. И ничего не предложили в плане улучшения. Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум. Вы отлично разбираетесь как в бек, так и во фронт.. ))) Ну я же писал выше что с диалогом беда восприятие не доброе. Нарываться на грубости не имею желания. И вам тоже хорошего вечера))) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:31 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenДмитрий Мухпропущено... Вот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база. И ничего не предложили в плане улучшения. Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум. Вы отлично разбираетесь как в бек, так и во фронт.. ))) Ну я же писал выше что с диалогом беда восприятие не доброе. Нарываться на грубости не имею желания. И вам тоже хорошего вечера))) Восприятие нормальное. Критическое. Вам может импатии хотелось? Но это редкость среди программистов. Что мешает вам выссказать и аргументирвать свою точку зрения? Боязнь нарваться на грубость? :) ... |
|||
:
Нравится:
Не нравится:
|
|||
20.10.2019, 22:48 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухЕсли подзапросы завернули в функции, то возможно система GRANT-ов присутсвует.Так нужны гранты что забили на fullscan-ы? Или mssql умеет в императивный код предикаты проталкивать? ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 01:23 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухВот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база. Видя два оператора goto на десяток строчек кода уже можно ничего и не выяснять. Чаще всего самое правильное и простое объяснение говнокоду это просто то, что его писали говнокодеры, и искать в нем какой-то скрытый смысл это, в конечном итоге, только зря потратить время. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 08:32 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatВидя два оператора goto на десяток строчек кода уже можно ничего и не выяснять. Чаще всего самое правильное и простое объяснение говнокоду это просто то, что его писали говнокодеры, и искать в нем какой-то скрытый смысл это, в конечном итоге, только зря потратить время.out errMsg тоже доставляет Если бы мне попросили написать этот фрагмент 20-ю разными способами, то и в этом случае мне не пришло бы в голову писать так. Напоминает турбобейсик Но я бы всё-таки спросил. Потому что если ответ будет - да, это десять лет назад наговнокодили, и это всё сейчас постепенно рефакторится - это одно. Если это взято из свежего коммита - это другое ) ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:04 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий Мухlansпропущено... Варианты в студию!Если подзапросы завернули в функции, то возможно система GRANT-ов присутсвует. + скорее всего нужно разграничить доступ к колонкам ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:30 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Yathaскорее всего нужно разграничить доступ к колонкам Использовать вьюхи. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:37 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatYathaскорее всего нужно разграничить доступ к колонкам Использовать вьюхи. И каким образом во вьюхе доступ по колонкам разграничить? ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:45 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatДмитрий МухВот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база. Видя два оператора goto на десяток строчек кода уже можно ничего и не выяснять Не выясняйте. А для меня это выглядит как попытка реализации Railway oriented programming, что мы обсуждали к примеру тут: 21566951 . И я бы предложил переписать, метод сократился бы до четырёх строк. Если уже не переписывают... ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:54 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухИ каким образом во вьюхе доступ по колонкам разграничить? Так же, как и и с SP или UDF - через "ownership chain". ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 09:55 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Задал вопрос по функции. "То ли лыжи не едут. То ли я еб......й" ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 10:39 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenЗадал вопрос по функции. "То ли лыжи не едут. То ли я еб......й" Ткни ему в череп execution plan-ом, а если не поможет, то просто забей - в разработке всевозможных банько выше крыши, бороться с ними всеми это только свою жизнь напрасно расходовать, т.ч. остаётся только смириться. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 10:52 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenЗадал вопрос по функции. "То ли лыжи не едут. То ли я еб......й" Можешь так и ответить, а можешь по существу. Что выберешь? :) ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 10:58 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Изумительная аргументация "во from много таблиц записывать не хочется " Ну вот не хочется человеку, и пофиг на SELECT N+1 ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 11:18 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Может действительно нафиг и поискать другое, если есть варианты. А то через год сам будешь так писать ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 11:19 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Shocker.ProМожет действительно нафиг и поискать другое, если есть варианты. А то через год сам будешь так писать Помнится на первой моей работе, после стажировки, был такой старший разработчик. В итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе. Был это 2003-й год :) ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 11:31 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Это я к тому, что скорее всего это легаси и надо просто обсудить на уровне команды, как его переписать, а не сидеть и в сторонке материться. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 11:33 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
skyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе. Просто травануть его в столовой не думали? ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:03 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatskyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе. Просто травануть его в столовой не думали? Порезать заточкой. Ауеджайл -- наше всё :) ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:07 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatskyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе. Просто травануть его в столовой не думали? Если за технический долг травить, то разработчиков не останется :) ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:21 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
обожаю когда человек приходит в команду и начинает всем рассказывать как правильно, модно и молодежно, хотя система уже существует 100500 лет. Начинает внедрять "новые" технологии в проект 20 летней давности, без видимой причины и т.д. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:23 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Roman Mejtesобожаю когда человек приходит в команду и начинает всем рассказывать как правильно, модно и молодежно, хотя система уже существует 100500 лет. Начинает внедрять "новые" технологии в проект 20 летней давности, без видимой причины и т.д. Если аргументы "модно/молодёжно", тогда идёт лесом. Если что-то обоснованное, тогда велкам. В этом и состоит шаринг знание, приходит не просто тело с руками, а человек со своим опытом и знаниями, которыми не грех поделиться, интегрировать новые для проекта решения. С другой стороны, сидеть и гнить в болоте никому не хочется. Не считая тепличных "овощей", которым в принципе пофиге, от звонка до звонка досидел и пошёл домой. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:25 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
hVostt, дискуссии по рабочим вопросам, это всегда хорошо. я про то, что в существующем проекте, всё делали через функции, может это неудобно, недостаточно быстро и т.д. но так реализован весь существующий проект и пусть это и дерьмо, нужно придерживаться такой архитектуры и таких шаблонов, которые были изначально, а не те, которые я считаю, что лучше. а если переделывать, то решение по таким вопросам должны принимать основательно, а не чисто из-за думки какого то новичка, пусть он хоть эксперт, то есть нужно обоснование того, почему мы хотим это менять, с учётом желание клиента, если он есть. должны быть соответствующие задачи, в рамках которых можно будет отследить этот переход с одной технологии (подхода\шаблона) на другую. и самое главное, кто это будет оплачивать? его начальник не из пальца это высасывает, у него наверное есть архитектура какая то, которой он придерживается. в армии, есть такая поговорка "Инициатива е...т инициатора", это жиза :) да и странная тема, вместо того, чтоб подойти и спросить у техлида, почему так, а не эдак, автор идет жаловаться нам. Либо он боится показаться некомпетентным, либо он интроверт ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 13:56 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Roman Mejtes, ну собственно про это и сказал skyANA, что обсуждать такие вопросы нужно не с нами :) если достаточно компетентен, чтобы аргументировать и отстаивать свою позицию -- то внешняя помощь не нужна. если компетенции недостаточно, то ходить за внешней помощью -- моветон. Roman Mejtesя про то, что в существующем проекте, всё делали через функции, может это неудобно, недостаточно быстро и т.д. но так реализован весь существующий проект и пусть это и дерьмо, нужно придерживаться такой архитектуры и таких шаблонов, которые были изначально, а не те, которые я считаю, что лучше. понимаю о чём речь. нужно обсуждать. конкретные предложения нужно декомпозировать, оценивать, оценивать рои и т.д. и т.п. ... |
|||
:
Нравится:
Не нравится:
|
|||
21.10.2019, 15:10 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
IPmenНапример вот так. Слова найти могу но восприятие не доброе. Что и натолкнуло на мысль всё это дел запостить начиная с 2019 сервера UDF будут инлайнится и план уже портить не будут ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 01:09 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
lansСпецом для Дмитрий Мух как пример Код: sql 1.
Так писать нельзя. Согласны? коррелированные подзапросы писать нельзя? ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 01:11 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
stenfordначиная с 2019 сервера UDF будут инлайнится и план уже портить не будут Ага, щас, прямо вот так со сменой COMPATIBILITY_LEVEL на 150 они все сразу заинлайнятся. К инлайнингу скалярных UDF выдвигается ряд требований , и все они должны быть соблюдены. Без этого будет тот же план, что и с обычной скаляркой. Так что в любом случае придётся провести ревизию всех серверных функций. Хотя, учитывая, что всё это хозяйство, обильно нашпигованное во все запросы, работало б-г сколько его знает, разработчиков продукта оптимизация работы БД особенно не колышет - поскольку в противном случае они бы давно узнали, что скалярные UDF в запросах это зло и тормоза. А скалярные UDF с SQL внутри - это тормоза в квадрате и в кубе. ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 06:35 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatДмитрий МухИ каким образом во вьюхе доступ по колонкам разграничить? Так же, как и и с SP или UDF - через "ownership chain".Примерчиком поделитесь? ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 09:10 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухfkthatТак же, как и и с SP или UDF - через "ownership chain".Примерчиком поделитесь? Вот для вьюхи: Код: sql 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25.
Для UDF аналогично. А вот для резалтсетов SP такое не выйдет - хотя бы потому, что выходной результат процедуры не имеет хранимых метаданных (не описан в sys.columns). ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 09:32 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Сон Веры Павловны, спасибо ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 09:36 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
Дмитрий МухСон Веры Павловны, спасибо Пример выше привели, а суть вся в том, что если у юзера есть права на вью (храниму, функцию), а это вью и таблица(ы) , к которой оно обращается имеют одного и того же владельца, то пользователю прав на саму эту таблицу уже не требуется - этот принцип в сиквеле и называется "ownership chain". Как-то так. ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 10:03 |
|
Новая работа. Может я что не понимаю? (((
|
|||
---|---|---|---|
#18+
fkthatДмитрий МухСон Веры Павловны, спасибо Пример выше привели, а суть вся в том, что если у юзера есть права на вью (храниму, функцию), а это вью и таблица(ы) , к которой оно обращается имеют одного и того же владельца, то пользователю прав на саму эту таблицу уже не требуется - этот принцип в сиквеле и называется "ownership chain". Как-то так. Отлично! Вот это уже конструктив. Могёте! И ТС могёт :) ... |
|||
:
Нравится:
Не нравится:
|
|||
22.10.2019, 10:07 |
|
|
start [/forum/topic.php?all=1&fid=20&tid=1398753]: |
0ms |
get settings: |
10ms |
get forum list: |
16ms |
check forum access: |
3ms |
check topic access: |
3ms |
track hit: |
163ms |
get topic data: |
13ms |
get forum data: |
3ms |
get page messages: |
115ms |
get tp. blocked users: |
2ms |
others: | 262ms |
total: | 590ms |
0 / 0 |