|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
Тестовое задание от компании Exness07.08.2017, 22:55. Показов 9438. Ответов 69
Метки нет (Все метки)
У меня нет никакого вопроса, но я так и не понял в какую ветку надо написать (все изучил внимательно), поэтому пишу сюда.
Я просто хочу поделиться своим опытом трудоустройства на тот случай если это кому нибудь поможет и узнать мнение общественности. Я получил тестовое задание от компании 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 очень мало по проекту Вам успехов. Еси решение изменится, дам Вам знать.
0
|
|
| 07.08.2017, 22:55 | |
|
Ответы с готовыми решениями:
69
Тестовое задание от работадателя. Хотелось бы услышать критику.
Тестовое задание |
|
Почетный модератор
5851 / 2862 / 392
Регистрация: 01.11.2011
Сообщений: 6,906
|
|
| 08.08.2017, 09:06 | |
|
Pechkin80, вы хотите, чтобы форумчане поглядели ваш код и оценили б на сколько критика оправдана?
0
|
|
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 08.08.2017, 10:47 [ТС] | |
|
Всё возможно. Критика может быть не оправдана или наоборот критика может быть неполной и результат выполнения требует большей критики, а кому то может это поможет просто трудоустроиться в данную компанию.
Добавлено через 8 минут Приведу текст тут, для тех кому скачивать. Описание задачи Перед программистом стояла задача сделать универсальный обработчик нескольких очередей с требованиями: ● Код должен быть максимально универсальным и простым в использовании: ○ добавление в очередь (несколько продюсеров в разных потоках), ○ получение из очереди, ○ подписка/отписка на разбор очереди в отдельном потоке (потоках) одним фиксированным консьюмером ● Код должен быть потокобезопасным ● Решение должно эффективно распоряжаться ресурсами (ядра CPU, память) и работать максимально быстро (с минимальными задержками между добавлением в очередь и получением данных в консьюмере) Представленное решение (см. код) по ряду причин не получилось удачным, требуется внести исправления в приведенный код: ● Добиться корректности и стабильности работы класса, устранить потенциальные “уязвимости” ● Провести рефакторинг существующего кода так, чтобы класс было легко развивать и поддерживать ● Провести оптимизацию, добиться максимальной производительности Решение данной задачи может быть простым или сколь угодно сложным, можно вводить/использовать любые типы, переделывать внутреннюю реализацию так, как это потребуется, исходя из требований задачи и общих best-practice Обязательные требования по коду: ● Из библиотек разрешается использовать только STL и boost ● Код должен быть написан только на С++, с использованием C++03 или более новых стандартов (С++11/14 приветствуется)
0
|
|
|
Неэпический
|
|||||||||||
| 08.08.2017, 12:36 | |||||||||||
|
Pechkin80, посмотрел мельком результат.
Много повторяющегося кода, который, при минимальных изменениях можно раскидать по функциям. Еще, например,
0
|
|||||||||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 08.08.2017, 13:46 [ТС] | |
|
Croessmah, В чём смысл ваших манипуляций с кодом ?
0
|
|
|
Неэпический
|
||||||
| 08.08.2017, 13:48 | ||||||
|
Pechkin80, убраны проверки на nullptr,
а значит уменьшена вложенность кода и его длина, при этом были удалены лишние
Хотя еще не устранена, нужно смотреть как там что устроено, и что есть что.
0
|
||||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 08.08.2017, 14:12 [ТС] | |
|
Croessmah, Не совсем понял про связь умных указателей и вложенности кода. Т.е. умные указатели настолько умные что сами выведут нужное разработчику сообщение об ошибки без вызова исключений? Где потенциальная утечка памяти ?
Добавлено через 16 минут Большая вложенность кода тут в качестве загатовки для разветвлённой системы сообщений об ощибках. Можно как альтернативу сделать разветвлённую иерархию классов исключений, но как это связано с способом создания объекта для меня загадка.
0
|
|
|
Неэпический
|
||||
| 08.08.2017, 14:18 | ||||
|
а умные указатели помогут не потерять ресурсы.
Хотя если send_exit_event убьет процесс (не поток, а весь процесс), то может и не надо заботится об освобождении ресурса, хотя всё равно всё зависит от устройства. Но в таком случае зачем nothrow и все эти проверки в коде, если в catch всё убьется в случае вылета исключения? Я сначала подумал, что нельзя использовать new, который выбрасывает исключение, но, выше всё-таки используется new, выбрасывающий исключение. send_exit_event убивает весь процесс или только поток? Если поток, то в коде явные проблемы с потенциальными утечками. Если, использовать умные указатели (с умом) вместо голых, то код упростится и станет безопаснее по отношению к исключениям.
0
|
||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 08.08.2017, 14:21 [ТС] | |
|
Croessmah, Не забудьбте как модератор подтереть собственный перл, пока я не полез в стандарт.
-2
|
|
|
Неэпический
|
||||||||||||
| 08.08.2017, 14:45 | ||||||||||||
|
Добавлено через 20 минут Повторяющиеся фрагменты:
0
|
||||||||||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 08.08.2017, 16:35 [ТС] | |
|
Croessmah, Можно завтра так как сегодня у меня д.р. ?
0
|
|
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
||
| 08.08.2017, 16:54 [ТС] | ||
|
Croessmah,
ISO 14882-2003
Croessmah, По поводу остального напишу завтра. У меня нету цели вас "уесть" но эта тема возможно комуто поможет найти работу и я не хотел бы чтобы вы вводили людей в заблуждение.
0
|
||
|
Неэпический
|
|||||||||||
| 08.08.2017, 16:56 | |||||||||||
|
Pechkin80, и?
Я убрал std::nothrow в new-expression. В моем коде new выбросит исключение и мы перейдем в catch. Умный указатель (plnode) при этом освободит выделенную ранее память.
0
|
|||||||||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 09.08.2017, 11:30 [ТС] | |
|
Croessmah,
1) Если память не выделена, то она не может утечь. Память это не кран, который вдруг сломался и образовалась лужа Но перл прикольный., мне понравился. 2)Спасибо, что наконец привели явное пояснение, что вы рассматриваете гипотетическую ошибку где оператор new используется в классической форме. Я тут не говорил что нуждаюсь в пояснениях по использованию умных указателей, но всё равно спасибо. Кому нибудь это обязательно поможет. 3)send_exit_event как и следует из названия посылает событие в главный поток о том что поток очереди завершился и код завершения. 4)Про повторяющиеся фрагменты не совсем понял. Это хорошо или плохо, по вашему мнению что я сделал не в вашем варианте ? 5)Если чтото пропустил, то прошу продублировать.
-0.50
|
|
|
Неэпический
|
||||||||||||
| 09.08.2017, 11:42 | ||||||||||||
Вы там что-то себе напридумывали, или не так прочитали, но сами в это поверили. Дублирование кода - потенциальный источник ошибок, глупых и не очень.
0
|
||||||||||||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
||||||
| 09.08.2017, 12:04 [ТС] | ||||||
|
Croessmah, Как понять "//А ЗДЕСЬ ЗАФЕЙЛИЛОСЬ ВЫДЕЛЕНИЕ." ? Пришол крокодил и за мягкое место укусил ? Т.е. полный undefined behaviour ?
Про "Повторяющиеся фрагменты стоит вынести в отдельную функцию. Дублирование кода - потенциальный источник ошибок, глупых и не очень." Хорошо, приму к сведению вашу точку зрения ))))))))))Добавлено через 14 минут Приведу злаполучный фрагмент в более полном изложении:
-0.50
|
||||||
|
Любитель чаепитий
|
|||
| 09.08.2017, 12:05 | |||
|
а вот operator new - это кран.в приведенном в #14 коде в строке 1 вы открываете кран, набираете себе воды. после этого, если что-то пошло не так, то вы завершаете поток(процесс?) и воду, набранную в строке 1, не возвращаете в резервуар. она утекает в итоге. кстати, ваш
0
|
|||
|
21 / 19 / 7
Регистрация: 14.03.2014
Сообщений: 249
|
|
| 09.08.2017, 12:13 [ТС] | |
|
GbaLog-, Поведение оператора new вполне однозначно определено стандартом для всех ситуаций. Чтото идёт не так у тех кому его лень учить.
0
|
|
|
Неэпический
|
||||||||||
| 09.08.2017, 12:24 | ||||||||||
|
только с акцентом на нужные участки. Но так и быть. Повторение - мать мучения. Попытка последняя. Гора идет к Магомету. Если его нет дома, то черт с ним.
1
|
||||||||||
| 09.08.2017, 12:24 | |
|
Помогаю со студенческими работами здесь
20
Тестовое задание на Junior C++ dev
Тестовое задание для трудоустройства Написать ответы на тестовое задание при приёме на работу (исключения, sizeof, mutex) Тестовое задание Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |
|
Новые блоги и статьи
|
||||
|
Новый ноутбук
volvo 07.12.2025
Всем привет.
По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне:
Ryzen 5 7533HS
64 Gb DDR5
1Tb NVMe
16" Full HD Display
Win11 Pro
|
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
|
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
|
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов
На странице:
https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/
нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
|
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов.
. . .
|
|
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
|
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
|
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут.
В век Веб все очень привыкли к дизайну Single-Page-Application .
Быстренько разберем подход "на фреймах".
Мы делаем одну. . .
|
Фото: Daniel Greenwood
kumehtar 13.11.2025
|
Расскажи мне о Мире, бродяга
kumehtar 12.11.2025
— Расскажи мне о Мире, бродяга,
Ты же видел моря и метели.
Как сменялись короны и стяги,
Как эпохи стрелою летели.
- Этот мир — это крылья и горы,
Снег и пламя, любовь и тревоги,
И бескрайние. . .
|