МЕ
Перебираются новые сделки. Ничего не смущает
Меня там половина точно смущает
Давай свою версию
Size: a a a
2020 October 07
НП
this.
Самое первое, что бросается в глаза, это наличие коммента на метод, который можно засунуть в самери метода
Но в самери это уже есть, вуаля
Но в самери это уже есть, вуаля
НП
+ принимать IEnumerable<> не есть хорошая идея, так как это енумератор, а туда можно передать даже линк. А линк это отложенная операция. Т.е. Во время отработки метода мы можем поймать ошибку где то
S
Самое первое, что бросается в глаза, это наличие коммента на метод, который можно засунуть в самери метода
Но в самери это уже есть, вуаля
Но в самери это уже есть, вуаля
на пустом месте доеп какой-то)
МЕ
А ниче что этот метод например может использоваться например в рест клиенте. Тупо прокомментировали что он делает,для понимания.
t
на пустом месте доеп какой-то)
Если честно +
МЕ
Не засчитано Никит
НП
на пустом месте доеп какой-то)
А ты сам подумай. Зачем комментить, если в метод лучше запихнуть?
S
А ты сам подумай. Зачем комментить, если в метод лучше запихнуть?
если со вторым согласен, то первое так себе)
НП
если со вторым согласен, то первое так себе)
Видел на проде такие комменты?
S
я тебе больше скажу, у нас их вообще нет)
МЕ
Видел на проде такие комменты?
А это и не прод
НП
я тебе больше скажу, у нас их вообще нет)
Вот по этому и не горит )
t
если со вторым согласен, то первое так себе)
А я вот не очень понял второе, в случае там может быть ошибка?
А
Так а почему на мониторинг брошено несколько потоков, которые делают одно и тоже по сути без надобности, почему нельзя пустить на это только один поток, установить ему задержку когда он будет повторять мониторинг и всё
МЕ
Так а почему на мониторинг брошено несколько потоков, которые делают одно и тоже по сути без надобности, почему нельзя пустить на это только один поток, установить ему задержку когда он будет повторять мониторинг и всё
Потому что при 1 потоке я сделки буду получать по 1 валютной паре
МЕ
Так а почему на мониторинг брошено несколько потоков, которые делают одно и тоже по сути без надобности, почему нельзя пустить на это только один поток, установить ему задержку когда он будет повторять мониторинг и всё
А мне под по всем валютам мониторить новые сделки
МЕ
Так а почему на мониторинг брошено несколько потоков, которые делают одно и тоже по сути без надобности, почему нельзя пустить на это только один поток, установить ему задержку когда он будет повторять мониторинг и всё
А каждая новая валютная пара-это новый поток
А
понял
S
this.
А я вот не очень понял второе, в случае там может быть ошибка?
ну например, у тебя есть лист {3,4,5,6,7,8} , ты сделал линк list.Where(onlyEven) и передал его в метод.
К моменту выполнения у тебя лист уже мог измениться и ты ловишь ошибку потому что данные то после where не зафиксированы, а лист вообще очистился. Лучше бы было , если хочешь принимать разные коллекции, засунуть ICollections какой-нибудь
К моменту выполнения у тебя лист уже мог измениться и ты ловишь ошибку потому что данные то после where не зафиксированы, а лист вообще очистился. Лучше бы было , если хочешь принимать разные коллекции, засунуть ICollections какой-нибудь