powered by simpleCommunicator - 2.0.50     © 2025 Programmizd 02
Форумы / WinForms, .Net Framework [игнор отключен] [закрыт для гостей] / Новая работа. Может я что не понимаю? (((
86 сообщений из 86, показаны все 4 страниц
Новая работа. Может я что не понимаю? (((
    #39878877
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Всем доброго дня. Нужен совет коллег. Устроился на работу , получили задачу реализовать чат между менеджер(кабинет менеджер) и клиентом (профиль клиента на другом домене). База централизована. Вооружился опытом разработки на 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.
      public ActionResult AjaxGetEnquiryParameters(
            int enquiryId
            )
        {
            RunClass run = Run;

            if (!run.DbReady) { return new HttpUnauthorizedResult(); }

            if (!run.Db.Deal.API_GetEnquiryParameters(enquiryId, out string errMsg, out DataRow row))
            {
                goto OnError;
            }

            Deal.EnquiryParameters enq = new Deal.EnquiryParameters();

            if (!enq.Load(row, out errMsg))
            {
                goto OnError;
            }

            return Json(enq);

            OnError:
            return new HttpStatusCodeResult(502, Uri.EscapeUriString(errMsg));
        }



Так же от увиденного на стороне базы волосы стали шевелится. Вся логика реализована в процедурах и функциях. К примеру что бы получить 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.
ALTER FUNCTION [dbo].[WebPos:Pds.PaymentTypes.ById.Name]
(
	@type_id int
)
RETURNS nvarchar(50)
AS
BEGIN
	IF @type_id IS NULL
		RETURN NULL;
		
	RETURN
		(
		SELECT
			t.name
		FROM
			[webpos:pds.payment_types] t
		WHERE
			t.id = @type_id
		)
	;
END



Это же дополнительная нагрузка на базу

В итоге не покидает мысль а оно мне надо. или может я чего не понимаю((((((( Вообщем нужен дельный совет
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878881
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
По SignalR поддерживаю. Что за шины между сайтами уточнить надо было)))) и при чём тут маршалинг это из другой темы вообще.

По С#. goto сто лет не видел. out msg. может внутри какой то Exception свой отрабатывает. Хотя можно и снаружи его отловить. Скорее тоже поддерживаю какой то моразм.

По sql явное не понимание у разработчиков. на отдельную колонку вешать select это сильно. В один день у них всё ляжет и устанут разгребать.

Совет. Уноси ноги пока всё не упало
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878885
Фотография Shocker.Pro
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenК примеру что бы получить Name какого либо обьекта используют в SELECT вызов функции вместо join например вот такойЯ бы попытался спросить о причинах. Вдруг причины есть
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878895
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Автор, видать, в разработке довольно новенький. Года через 3-5 такие вещи перестанут удивлять, а еще через 3-5 станут восприниматься как повседневная норма. "Кто в армии служил, тот в цирке не смеется" (с)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878903
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДалее нужен совет по проекту
Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно.

По двум твоим огрызкам кода выводы делать рано.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878913
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
fkthat,
За 13 лет такого не встречал
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878916
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухIPmenДалее нужен совет по проекту
Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно.

По двум твоим огрызкам кода выводы делать рано.

Бывают реализации где подобное оправдано?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878918
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenЗа 13 лет такого не встречалКритики в адрес своего решения? :) Вот встретили, и реакция на мой взгляд странная.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878922
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДмитрий Мухпропущено...

Значит подходишь к старшему разработчику и спрашиваешь обо всём, что не понятно.

По двум твоим огрызкам кода выводы делать рано.

Бывают реализации где подобное оправдано?Что подобное? Сколько продукту лет?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878928
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmen,

огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe.

с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно.

обсуждайте. договаривайтесь.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878929
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухIPmenЗа 13 лет такого не встречалКритики в адрес своего решения? :) Вот встретили, и реакция на мой взгляд странная.

Да в том то и дело что не в критике же дело а в реализации. Такого кода я не встречал и считаю что так писать нельзя так же и по базе. Реакция наводящая на вопросы к третьим лицам, по моему адекватная.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878933
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
hVosttIPmen,

огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe.

с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно.

обсуждайте. договаривайтесь.

Да тут сложностей нет.
1-й вариант. Есть одна сторона проекта менеджера и клиента.база одна. HUB SignalR расположен на стороне manager, клиенту iframe ссылка на manager проект
2-й вариант. на каждой стороне HUB у каждого своя реализация. общение через запросы.

Я не пытаюсь сказать решение моё верное я пытаюсь понять если моё решение неверное то в чём? Советуюсь на будущее
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878934
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
hVosttIPmen,

огрызки кода, которые вы привели, конечно страшные, но в целом ваш старший разработчик прав приложения менеджера и приложения клиента -- разные приложения. если бы весь чат был реализован как микросервис, не относящийся ни к менеджеру, ни к клиенту, другой вопрос. да и в этом случае, можно обойтись без iframe.

с другой стороны, мы всего не знаем, поэтому говорить кто прав, кто не прав с полной уверенностью сложно.

обсуждайте. договаривайтесь.

Всё происходит в рамках одного проекта. т.е это не интеграция с чем то потусторонним.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878935
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухIPmenпропущено...


Бывают реализации где подобное оправдано?Что подобное? Сколько продукту лет?

Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь .
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878937
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenВсё происходит в рамках одного проекта. т.е это не интеграция с чем то потусторонним.

Проект один, а приложения разные. Но я уже об этом сказал.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878938
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДмитрий Мухпропущено...
Что подобное? Сколько продукту лет?

Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь .Я не увидел функции на каждую колонку.
Не увидел структуры БД, индексов, предметной области...

А главное, я не увидел от вас никаких аргументов.
Если бы я пришёл на проект, где мне что-то не нравится, то я бы нашёл слова для обоснования своего мнения.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878939
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenЯ не пытаюсь сказать решение моё верное я пытаюсь понять если моё решение неверное то в чём? Советуюсь на будущее
Если коротко, то согласно DDD (Domain Driven Design) контексты у вас разные...
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878940
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Причина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878941
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Давайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота...
Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878943
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansПричина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде?
А в чём жесть, мне просто любопытно? :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878944
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухIPmenпропущено...


Ситуации когда пишут SELECT и на каждую колонку в виде функции тоже SELECT ну и по C#. Судя по логам можно сказать пол года может ошибаюсь .Я не увидел функции на каждую колонку.
Не увидел структуры БД, индексов, предметной области...

А главное, я не увидел от вас никаких аргументов.
Если бы я пришёл на проект, где мне что-то не нравится, то я бы нашёл слова для обоснования своего мнения.

Например вот так. Слова найти могу но восприятие не доброе. Что и натолкнуло на мысль всё это дел запостить
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878946
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухДавайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота...
Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :)

))) В данной реализации это тоже же не проблема.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878948
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДмитрий МухДавайте представим, что завтра сообщение должно прилетать клиенту не от менеджера, а от другого клиента, послезавтра от бота...
Вы напишите, что такого не произойдёт никогда... А я вот вам не поверю :)

))) В данной реализации это тоже же не проблема.

Я вам ещё подкину. Вместо зависимости функции от одного приложения, в котором имплементировано решение, вы получаете два. Вас помянут недобрым словом и разработчики, которым это придётся развивать, тестировщики, девопсы -- от всех вам достанется много нехороших эпитетов.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878949
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmen,

и зачем тут JOIN-ы?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878950
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухlansПричина писать такую жесть может быть одна. Отсутствие опыта. Сколько человек в команде?
А в чём жесть, мне просто любопытно? :)

Можно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть. и всю эту лабуду выкинуть к чертям. по базе уже писал выше. писать select на каждую колонку это причина уволить сотрудника. Я таких не держу.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878953
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Опять таки зависит от задачи. Если в перспективе клиент должен будет работать не зависимо от текущего контекста то нужна независимая реализация без всяких iframe а если вы гарантированно будете работать в рамках одой базы и этих двух проектов учитывая что клиент работает с той же базой что и менеджер то почему бы и нет. не вижу изюма в лишних действиях и ваше решение верное. но опять таки надо выслушать аргументы тимлида . ведь он не зря тимлид у него на будущее возможно видинее другое в любом случае надо обсуждать.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878955
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мух,

Спецом для Дмитрий Мух как пример
Код: sql
1.
 select name,  (select name from type where id= usertypeId) as usertype from users  



Так писать нельзя. Согласны?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878956
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Ещё раз Итог.

Код на букву Г...
База на букву Г...
По чату надо говорить
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878957
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansМожно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть.
Фиг знает, не видно исключений.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878958
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мух,

Спецом для Дмитрий Мух как пример
Код: sql
1.
 select name,  (select name from type where id= usertypeId) as usertype from users  




Так писать нельзя. Согласны?А как надо? :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878963
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухlansДмитрий Мух,

Спецом для Дмитрий Мух как пример
Код: sql
1.
 select name,  (select name from type where id= usertypeId) as usertype from users  




Так писать нельзя. Согласны?А как надо? :)

Правильный ответ пишите так как нравится)))))))))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878964
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...
А как надо? :)

Правильный ответ пишите так как нравится)))))))))Не правильный ответ :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878965
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухlansМожно убрать все goto оставить один try catch и внутри методов убрать try catch так как наличие того то передаётся какая то ошибка в out string errMsg наталкивает на мысль что там он есть.
Фиг знает, не видно исключений.

Стоит catch
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878968
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Правильный ответ пишите так как нравится)))))))))Не правильный ответ :)

А как надо?)))))))))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878969
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДмитрий Мухпропущено...

Фиг знает, не видно исключений.

Стоит catch


Смелс как легаси :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878970
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...
Не правильный ответ :)

А как надо?)))))))))
Так, чтобы избавиться от огрехов старой архитектуры..
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878972
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


А как надо?)))))))))
Так, чтобы избавиться от огрехов старой архитектуры..

Ну хитрец. Ушёл от ответа)))) Считается
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878975
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Всем спасибо. Суть уловил. Буду прощаться.

PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?))))))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878977
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...

Так, чтобы избавиться от огрехов старой архитектуры..

Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано.
И варианты, как это отрефакторить. Но вам оно видно же, что не надо :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878978
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenВсем спасибо. Суть уловил. Буду прощаться.

PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?))))))Нужен, но умеющий своё мнение обосновать :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878979
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано.
И варианты, как это отрефакторить. Но вам оно видно же, что не надо :)

Варианты в студию!
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878981
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Ну хитрец. Ушёл от ответа)))) СчитаетсяУ меня есть варианты, почему так написано.
И варианты, как это отрефакторить. Но вам оно видно же, что не надо :)

Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?)))))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878982
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...
У меня есть варианты, почему так написано.
И варианты, как это отрефакторить. Но вам оно видно же, что не надо :)

Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?)))))
Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878983
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Как исправить и джуниору понятно. Почему так написано ? Потому что кто то забыл подумать?)))))
Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите?

Помолиться можно и всё наладиться)))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878985
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...
У меня есть варианты, почему так написано.
И варианты, как это отрефакторить. Но вам оно видно же, что не надо :)

Варианты в студию!Если подзапросы завернули в функции, то возможно система GRANT-ов присутсвует.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878987
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...

Да, конечно кто-то забыл подумать, сто пудофф :) Как исправите?

Помолиться можно и всё наладиться)))Сразу видно профи. Кому, или чему молитесь? )))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878988
Фотография lans
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Помолиться можно и всё наладиться)))Сразу видно профи. Кому, или чему молитесь? )))
Одни вопросы. Где ответы?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878989
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenВсем спасибо. Суть уловил. Буду прощаться.

PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?))))))

Фулл стек это зачастую, толком ни бек, ни фронт :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878990
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
hVosttIPmenВсем спасибо. Суть уловил. Буду прощаться.

PS Никому не нужен C#.NET FULLSTACK DEVELOPER ?))))))

Фулл стек это зачастую, толком ни бек, ни фронт :)

Но точно что не 100% бек и фронт
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878992
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansДмитрий Мухпропущено...
Сразу видно профи. Кому, или чему молитесь? )))
Одни вопросы. Где ответы?Ответы должны быть у ТСа, но он постеснялся старшему задать вопросы :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878994
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenНо точно что не 100% бек и фронт

Все фуллстеки, которые приходили ко мне на собеседование за последний год, одинаково плохи как в бек, так и во фронт.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878995
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenНо точно что не 100% бек и фронт

Все фуллстеки, которые приходили ко мне на собеседование за последний год, одинаково плохи как в бек, так и во фронт.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878996
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Полегче Полегче понял вас))))))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39878998
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenПолегче Полегче понял вас))))))
Вот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база.
И ничего не предложили в плане улучшения.

Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум.

Вы отлично разбираетесь как в бек, так и во фронт.. )))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879001
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Дмитрий МухIPmenПолегче Полегче понял вас))))))
Вот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база.
И ничего не предложили в плане улучшения.

Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум.

Вы отлично разбираетесь как в бек, так и во фронт.. )))

Ну я же писал выше что с диалогом беда восприятие не доброе. Нарываться на грубости не имею желания. И вам тоже хорошего вечера)))
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879006
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenДмитрий Мухпропущено...

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

Просто пятая точка вам говорит о том, что что-то не так. А за аргументами попёрлись на форум.

Вы отлично разбираетесь как в бек, так и во фронт.. )))

Ну я же писал выше что с диалогом беда восприятие не доброе. Нарываться на грубости не имею желания. И вам тоже хорошего вечера)))
Восприятие нормальное. Критическое.
Вам может импатии хотелось? Но это редкость среди программистов.

Что мешает вам выссказать и аргументирвать свою точку зрения? Боязнь нарваться на грубость? :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879028
Болтаювпт
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухЕсли подзапросы завернули в функции, то возможно система GRANT-ов присутсвует.Так нужны гранты что забили на fullscan-ы? Или mssql умеет в императивный код предикаты проталкивать?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879059
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухВот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база.
Видя два оператора goto на десяток строчек кода уже можно ничего и не выяснять. Чаще всего самое правильное и простое объяснение говнокоду это просто то, что его писали говнокодеры, и искать в нем какой-то скрытый смысл это, в конечном итоге, только зря потратить время.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879065
Фотография Shocker.Pro
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatВидя два оператора goto на десяток строчек кода уже можно ничего и не выяснять. Чаще всего самое правильное и простое объяснение говнокоду это просто то, что его писали говнокодеры, и искать в нем какой-то скрытый смысл это, в конечном итоге, только зря потратить время.out errMsg тоже доставляет

Если бы мне попросили написать этот фрагмент 20-ю разными способами, то и в этом случае мне не пришло бы в голову писать так. Напоминает турбобейсик


Но я бы всё-таки спросил. Потому что если ответ будет - да, это десять лет назад наговнокодили, и это всё сейчас постепенно рефакторится - это одно. Если это взято из свежего коммита - это другое )
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879073
Фотография Yatha
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мухlansпропущено...


Варианты в студию!Если подзапросы завернули в функции, то возможно система GRANT-ов присутсвует.
+
скорее всего нужно разграничить доступ к колонкам
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879075
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Yathaскорее всего нужно разграничить доступ к колонкам
Использовать вьюхи.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879081
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatYathaскорее всего нужно разграничить доступ к колонкам
Использовать вьюхи.
И каким образом во вьюхе доступ по колонкам разграничить?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879086
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatДмитрий МухВот вы попали в компанию, ни фига не выяснили то, откуда такие подходы и кодовая база.
Видя два оператора goto на десяток строчек кода уже можно ничего и не выяснять
Не выясняйте.

А для меня это выглядит как попытка реализации Railway oriented programming, что мы обсуждали к примеру тут: 21566951 .
И я бы предложил переписать, метод сократился бы до четырёх строк.

Если уже не переписывают...
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879087
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухИ каким образом во вьюхе доступ по колонкам разграничить?
Так же, как и и с SP или UDF - через "ownership chain".
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879111
IPmen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Гость
Задал вопрос по функции.

"То ли лыжи не едут. То ли я еб......й"
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879128
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenЗадал вопрос по функции.
"То ли лыжи не едут. То ли я еб......й"
Ткни ему в череп execution plan-ом, а если не поможет, то просто забей - в разработке всевозможных банько выше крыши, бороться с ними всеми это только свою жизнь напрасно расходовать, т.ч. остаётся только смириться.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879135
Фотография skyANA
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenЗадал вопрос по функции.

"То ли лыжи не едут. То ли я еб......й"

Можешь так и ответить, а можешь по существу.
Что выберешь? :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879156
Фотография Shocker.Pro
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Изумительная аргументация "во from много таблиц записывать не хочется "

Ну вот не хочется человеку, и пофиг на SELECT N+1
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879159
Фотография Shocker.Pro
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Может действительно нафиг и поискать другое, если есть варианты.
А то через год сам будешь так писать
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879173
Фотография skyANA
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Shocker.ProМожет действительно нафиг и поискать другое, если есть варианты.
А то через год сам будешь так писать
Помнится на первой моей работе, после стажировки, был такой старший разработчик.
В итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе.

Был это 2003-й год :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879175
Фотография skyANA
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Это я к тому, что скорее всего это легаси и надо просто обсудить на уровне команды, как его переписать, а не сидеть и в сторонке материться.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879263
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
skyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе.
Просто травануть его в столовой не думали?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879267
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatskyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе.
Просто травануть его в столовой не думали?


Порезать заточкой.
Ауеджайл -- наше всё :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879286
Фотография skyANA
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatskyANAВ итоге в течение моего первого года его убедили, что надо рефакторить базу и писать иначе.
Просто травануть его в столовой не думали?

Если за технический долг травить, то разработчиков не останется :)
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879288
Roman Mejtes
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
обожаю когда человек приходит в команду и начинает всем рассказывать как правильно, модно и молодежно, хотя система уже существует 100500 лет. Начинает внедрять "новые" технологии в проект 20 летней давности, без видимой причины и т.д.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879291
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Roman Mejtesобожаю когда человек приходит в команду и начинает всем рассказывать как правильно, модно и молодежно, хотя система уже существует 100500 лет. Начинает внедрять "новые" технологии в проект 20 летней давности, без видимой причины и т.д.

Если аргументы "модно/молодёжно", тогда идёт лесом.
Если что-то обоснованное, тогда велкам.
В этом и состоит шаринг знание, приходит не просто тело с руками, а человек со своим опытом и знаниями, которыми не грех поделиться, интегрировать новые для проекта решения.

С другой стороны, сидеть и гнить в болоте никому не хочется.
Не считая тепличных "овощей", которым в принципе пофиге, от звонка до звонка досидел и пошёл домой.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879315
Roman Mejtes
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
hVostt,

дискуссии по рабочим вопросам, это всегда хорошо.

я про то, что в существующем проекте, всё делали через функции, может это неудобно, недостаточно быстро и т.д. но так реализован весь существующий проект и пусть это и дерьмо, нужно придерживаться такой архитектуры и таких шаблонов, которые были изначально, а не те, которые я считаю, что лучше.
а если переделывать, то решение по таким вопросам должны принимать основательно, а не чисто из-за думки какого то новичка, пусть он хоть эксперт, то есть нужно обоснование того, почему мы хотим это менять, с учётом желание клиента, если он есть. должны быть соответствующие задачи, в рамках которых можно будет отследить этот переход с одной технологии (подхода\шаблона) на другую. и самое главное, кто это будет оплачивать?
его начальник не из пальца это высасывает, у него наверное есть архитектура какая то, которой он придерживается.
в армии, есть такая поговорка "Инициатива е...т инициатора", это жиза :)
да и странная тема, вместо того, чтоб подойти и спросить у техлида, почему так, а не эдак, автор идет жаловаться нам. Либо он боится показаться некомпетентным, либо он интроверт
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879371
Фотография hVostt
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Roman Mejtes,

ну собственно про это и сказал skyANA, что обсуждать такие вопросы нужно не с нами :)

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

если компетенции недостаточно, то ходить за внешней помощью -- моветон.


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

понимаю о чём речь.

нужно обсуждать. конкретные предложения нужно декомпозировать, оценивать, оценивать рои и т.д. и т.п.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879638
stenford
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
IPmenНапример вот так. Слова найти могу но восприятие не доброе. Что и натолкнуло на мысль всё это дел запостить
начиная с 2019 сервера UDF будут инлайнится и план уже портить не будут
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879639
stenford
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
lansСпецом для Дмитрий Мух как пример
Код: sql
1.
 select name,  (select name from type where id= usertypeId) as usertype from users  



Так писать нельзя. Согласны?
коррелированные подзапросы писать нельзя?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879642
Сон Веры Павловны
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
stenfordначиная с 2019 сервера UDF будут инлайнится и план уже портить не будут
Ага, щас, прямо вот так со сменой COMPATIBILITY_LEVEL на 150 они все сразу заинлайнятся.
К инлайнингу скалярных UDF выдвигается ряд требований , и все они должны быть соблюдены. Без этого будет тот же план, что и с обычной скаляркой. Так что в любом случае придётся провести ревизию всех серверных функций.
Хотя, учитывая, что всё это хозяйство, обильно нашпигованное во все запросы, работало б-г сколько его знает, разработчиков продукта оптимизация работы БД особенно не колышет - поскольку в противном случае они бы давно узнали, что скалярные UDF в запросах это зло и тормоза. А скалярные UDF с SQL внутри - это тормоза в квадрате и в кубе.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879662
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatДмитрий МухИ каким образом во вьюхе доступ по колонкам разграничить?
Так же, как и и с SP или UDF - через "ownership chain".Примерчиком поделитесь?
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879676
Сон Веры Павловны
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий Мух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.
use stuff
go
select top 100 * into dbo.Test from sys.objects;
go
create  view dbo.TestView as select * from dbo.Test;
go
create role TestRole;
go
grant select on dbo.TestView (name, object_id, type_desc) to [TestRole]
go
create user [TestUser] without login
go
alter role [TestRole] add member [TestUser]
go

-- Ok
execute as user='TestUser'
select name, object_id, type_desc from dbo.TestView
revert

-- Msg 230, Level 14, State 1, Line 21
-- The SELECT permission was denied on the column 'type' of the object 'TestView', database 'stuff', schema 'dbo'
execute as user='TestUser'
select name, object_id, type, type_desc from dbo.TestView
revert


Для UDF аналогично.
А вот для резалтсетов SP такое не выйдет - хотя бы потому, что выходной результат процедуры не имеет хранимых метаданных (не описан в sys.columns).
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879681
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Сон Веры Павловны, спасибо
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879705
fkthat
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
Дмитрий МухСон Веры Павловны, спасибо
Пример выше привели, а суть вся в том, что если у юзера есть права на вью (храниму, функцию), а это вью и таблица(ы) , к которой оно обращается имеют одного и того же владельца, то пользователю прав на саму эту таблицу уже не требуется - этот принцип в сиквеле и называется "ownership chain". Как-то так.
...
Рейтинг: 0 / 0
Новая работа. Может я что не понимаю? (((
    #39879709
Дмитрий Мух
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Участник
fkthatДмитрий МухСон Веры Павловны, спасибо
Пример выше привели, а суть вся в том, что если у юзера есть права на вью (храниму, функцию), а это вью и таблица(ы) , к которой оно обращается имеют одного и того же владельца, то пользователю прав на саму эту таблицу уже не требуется - этот принцип в сиквеле и называется "ownership chain". Как-то так.
Отлично!

Вот это уже конструктив. Могёте! И ТС могёт :)
...
Рейтинг: 0 / 0
86 сообщений из 86, показаны все 4 страниц
Форумы / WinForms, .Net Framework [игнор отключен] [закрыт для гостей] / Новая работа. Может я что не понимаю? (((
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали форум (0):
Пользователи онлайн (0):
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]