@Pechkin80

У меня нет никакого вопроса, но я так и не понял в какую ветку надо написать (все изучил внимательно), поэтому пишу сюда.
Я просто хочу поделиться своим опытом трудоустройства на тот случай если это кому нибудь поможет и узнать мнение общественности.
Я получил тестовое задание от компании Exness и решил сделать его рефакторинг, а потом выполнил его и получил рецензию.
Всё во вложении.
Аннотация:
TA.2017.refactoring-optimization (2) (1).pdf - тестовое задание
MultiQueueProcessor.zip - код для рефакторинга
Описание_задачи.docx - моя версия рефакторинга самого задания
exness_testwork.zip - мой результат выполнения

Рецензия от lyubov.belogrudova@exness.com>: вот:
Юрий, добрый день! В пятницу не успела Вам написать.
Вас благодарим за внимание к нашей компании, за уделенное на тестовое задание время. Но, к сожалению, по результатам тестового задания делать оффер мы не готовы.
Пишу feedback от коллег:

Есть несколько основных проблемы присланного кода:
1. Читаемость и понимаемость кода стремится к нулю
2. Нет краткости, элегантности, какой-то красоты в присланном решении
3. Технически код выглядит очень старомодным:
a. Встречаются new/delete, unique_ptr/shared_ptr не найден
b. Обилие малооправданных "велосипедов"
c. С++11 очень мало по проекту

Вам успехов. Еси решение изменится, дам Вам знать.

Вложения

	TA.2017.refactoring-optimization (2) (1).pdf (45.8 Кб, 95 просмотров)
	Описание_задачи.docx (16.1 Кб, 77 просмотров)
	exness_testwork.zip (23.4 Кб, 67 просмотров)
	MultiQueueProcessor.zip (870 байт, 56 просмотров)

0

@Kastaneda

У меня весь код под NDA

Если очень интересно могу скинуть решение своего тестового задания с прошлой работы (чуть больше года назад делал) вместе с заданием.

0

@Pechkin80

Kastaneda, Мне интересно разговор перевести в конструктивное русло. Смысл ваших пассажей, что я очень крутой девелопер, код - говно, но мне лень объяснять почему. Такого рода саморекламой тут заниматься некорректно. А что касается вашего кода, то это вы сами решайте, есть чтото что Вы хотели бы показать как пример чего более удачного, например более удачной архитектуры.

0

@hoggy

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

0

@Pechkin80

hoggy, Я с вами полностью согласен, но хорошо читать до конца, а не вырывать фразы из контекста.

Добавлено через 1 минуту
кажется мне скоро придётся обращаться за помощью модераторов.

0

@Kastaneda

Выложите код на форум в тегах CPP, не весь, только очередь. Качать архив неудобно.

0

@Pechkin80

Kastaneda, Я архив выложил чтоб в глаза не бросались самые очевидные несуразности. Но код пока не пригоден для просмотра вне среды разработки с её навароченной навигацией. Пока хотел бы получить ответ об области применения данных алгоритмов.

0

Croessmah

Сам спросил, сам ответил?

0

@Pechkin80

Croessmah, поясните пожалуйста вашу мысль ? Или я может нечётко выразился.
Мне интересно знать область применеия данных алгоритмов (многопоточная очередь), так как сложно писать код на все случаи жизни.
Крд недостаточно на мой взгляд оформлен чтоб его было легко понимать вне среды разработки, разглядывая только часть, а не целиком. Попробуйте понять отдельно вытащенный из ядра линукс в текстовом редакторе файл и вас настигнит анологичный ступор.

Добавлено через 6 минут
Возможно есть недостатки, которые я не замечаю, но оценочные суждения не дают мне понимания в каком направлении двигаться. Если лень писать тут о конкретных недосатаках, то зачем флудить ? Напишите лучше про область применения данных алгоритмов, дайте ссылки на статьи где данная тема подробно и удачно рассмотрена.

Добавлено через 15 минут
Код дооформлю, когда будут ответы на выуказанные вопросы.

Добавлено через 31 минуту
Просьба не писать тут больше оценочные суждения. Они не информативны.

Добавлено через 37 минут
Изначальный вопрос, а именно узнать мнение общественности щас уже не очень актуален для меня.

Добавлено через 8 минут
Я обратил на данное тестовое задание потомучто данная задача(обработчик многопоточных очередей) очень похожа на вполне конкретный элемент архитектуры для определённой отрасли программирования. Данное решение востребовано как я подозреваю далеко не везде, а только в узком перечни задач. Вот хотелось бы понять где и для чего сначала.

Добавлено через 14 минут
Вообщем устал уже от тупого троллинга.

Добавлено через 48 минут
Так как движок форума не позволяет редактировать сообщения, то на всякий случай ещё раз напишу что речь идет об обработчике многопоточных очередей. К словам просьба не цепляться. Писать чтото содержательное и информативное.

0

@Kastaneda

Многопоточная очередь на столько распространена, что используется практически везде. Самый банальный пример - пул потоков (thread pool), есть практически в каждом многопоточном проекте.

Именно так и делаю, как правило все понятно.

0

@Pechkin80

Kastaneda, Я оговорился. многопоточная очередь действительно распространена везде и даже является элементом операционных систем Windows и Linux, но я имел ввиду обработчик очередей, с множеством производителей и потребителей, а не отдельную очередь.

Добавлено через 7 минут
Если можно, пример чегото конкретного. Насколько она уместна в качестве ядра сервера. Где производители это потоки слушающие серверные сокеты, а потребители это потоки, которые пишут в клиентские сокеты ?

0