На что обращают внимание в тестовых заданиях на С++?

Главное понять не могу, что им не понравилось.

написано на Qt

Сейчас нахожусь в поисках работы. Один из языков, который у меня указан в резюме это С++

Не разрабатывал на нем уже 4-5 лет

зачем тогда его указывать в резюме?.. в резюме не надо указывать все, что ты видел и дышал за свою скромную жизнь, а указывать то, на каком «стеке» ты будешь работать

Требование было такое.

Главное понять не могу, что им не понравилось. Приложение требованиям соответствует, не падает, архитектура вроде не говеная - стандартная, warning-ов нет, в сроки уложился, написано на Qt.

много разных факторов бывает, например забыли закрыть вакансию, а ты вхолостую писал тест, не обязательно вина с твоей стороны, факапы и внутренние часто бывают

Ну так говорю же, не пропил это знание до конца. Меня на прошлую работу именно за хорошее знания С++ взяли. Единственно сейчас стандарт далеко ушел и мне нужно время, чтобы наверстать, но в том задании ничего из нового стандарта не требовали. Но по мелочам я его использовал, типа лямбд и std::thread.

Не похоже. Чтобы в этом случае написать, что мое решение фуфло, это надо быть не в себе.

Исключения ловятся по значению. Надо по ссылке.

Сервер создаёт поток на каждое соединение. Это очень простой и не масштабируемый подход. Сколько одновременных клиентов ты сможешь иметь? На паре десятков твой сервер уже загнётся.

Очевидно, ты раньше не имел с этим дело. Копай в сторону thread pools и epoll (или что там под виндами). Можешь даже посмотреть как это делается в boost asio.

double result = data * 2;

какое задание?

Это. Умножать на 2.

Ну и что что-то по значению, они простые и оверхед незначительный.

Насчет потоков, возможно это близко к истине. Я действительно с серверами дела не имел. В сервере Qt используется QRunnable. Можно вкратце про thread pools?

текст полный задания процитировать в стартовом посте возможно?

текст полный задания процитировать в стартовом посте возможно?

Done.

Поддерживает «неограниченное» количество подключений.

Да, скорее всего я на этом и погорел.

Главное понять не могу, что им не понравилось.

позвоните им и спросите. телепаты в отпуске.

Ну так по коду на вскидку лучше не наследоваться от QThread, это объект потока, а не сервер в отдельном треде. Да и мешать винапи с кутями тоже странно, можно же было на кутях целиком и сервак запилить. При некотором небольшом напилинге можно запилить сигнал слоты на указателях на функцию-член.

В случае если приходит число,

что делает сервер если приходит не_число?

зачем тогда его указывать в резюме?.. в резюме не надо указывать все, что ты видел и дышал за свою скромную жизнь, а указывать то, на каком «стеке» ты будешь работать

Ой какой бред..

Ну и что что-то по значению, они простые и оверхед незначительный.

Тем не менее, так не принято.

Ну и что что-то по значению, они простые и оверхед незначительный.

Что простите?

Виндовозненько...

А почему сервера два? Тот, что на Qt, насколько я вижу, использует QThreadPool и QRunnable, зачем второй? Мусор какой-то ещё кругом разбросан... Вообще, если бы ты дал ссылку на гитхаб, было бы в 100500 раз приятнее.

Поддерживает «неограниченное» количество подключений.

А что это такое «неограниченное» они как-то расшифровывают? Так это выглядит как неверифицируемое требование, что указывает на непрофессионализм постановщика задачи. Конечно же 1 поток на соединение может так же подходить под это требование, как и другие решения. Вот если бы они задали четкие нефункциональные требования, типа «уложиться в 2 гигабайта памяти при миллионе одновременных соединений», это было бы конкретно.

Там два сервера, один на кутях если что. Наследоваться от QThread или нет это дело вкуса.

Тест полного задания: Необходимо создать клиентское и серверное приложения с помощью C++ и Qt

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

Наследоваться от QThread или нет это дело вкуса.

Ага, а Qt-разработчики просто так пишут статьи типа «You're doing it wrong».

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

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

В нормальных условиях - да. Но если автор на плюсах писал пять лет назад и хочет снова, то контору можно понять.

что за компания куда устривался? покажи их продукты, по таску они мало шарят в сетевом программировании, куте там явно лишний хвост и нужен видимо для клиент формочки задавать выводить значения, остальное пишется на голом С++, а использовать куте для нетворк кодинга явно феил самой компании, и у них это явно не профиль.

Нормально реагирую. Наследоваться можно, переопределять run плохо. Вся разница в том, что в одном случае работаешь через переменную класса, в другом через this.

А шут их знает, как расшифровывают. Спросил у них еще раз о причине, сказали «код не годится для коммерческой разработки». :)

а использовать куте для нетворк кодинга явно феил самой компании, и у них это явно не профиль.

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

Их вполне могла отпугнуть неопрятность. Люди разные бывают, на разное обращают внимание.

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

Вот только не надо рассказывать сказки про «серьезные сетевые проекты», ок? «Серьезные сетевые проекты» пишутся на чем угодно, весь вопрос в требованиях.

конечно, когда требование писать сетевое фуфло, тогда куте вполне подходит.

А в чем конкретно заключается неопрятность? В одном месте оставил закомментированный метод и про передачу исключений по значению тоже отметили, а так что-то не найду больше.

Да хотя-бы что сервера два, причем один из них виндовый. Зачем виндовый, разве его просили? Зачем файлы вроде Client.pro.user и Server.v12.suo? Почему в программе на Qt одни функции-члены именуются с маленькой буквы, а другие с большой:

class Client : public QObject
{
    Q_OBJECT

public:
    Client();
    virtual ~Client();
    void Connect(QString address, unsigned short port);
    void Disconnect();
    void Send(double value);
    void Start();
    void Stop();

signals:
    void stateChanged(QAbstractSocket::SocketState state);
    void valueReceived(double sentValue, double result);

private slots:
    void onStateChanged(QAbstractSocket::SocketState state);
    void onValueReceived(double sentValue, double result);

Это же не оффтопик, чтоб так методы называть. Вот они на всё это посмотрели и подумали, что у тебя windows головного мозга и ты им не подходишь (как вариант) ;)

Какие нежности. :)

.sln

крути педали пока не дали, малыш.

ищется более нормальная, которая не даёт неоплачиваемых тестовых заданий.

Таких нет. Все нормальные конторы дают тестовые задания — естественно, не оплачиваемые.

Кстати, никто не знает контору, где оплачивают тестовые задания? У меня есть идея для стартапа...

Это же не оффтопик, чтоб так методы называт

Так — это как? И что конкретно виндового в таком названии методов?

Так - это с прописной буквы. В Qt, очевидно, другой стиль. А называть функции с прописной буквы - стопроцентный мелкомягкий стиль.

Я бы не стал придираться к мелочам, но судя по словам:

код не годится для коммерческой разработки

Возможно, они ждали некой завершенности приложения, т.е. готовый продукт. Тогда бы я, в дополнение к вышесказанному, обратил внимание на магические числа, вроде:

...
if (port > 0xffff)
...
font.setPointSize(12);
...
_updateTimer->setInterval(1000);
...

т.е. неплохо бы создать конфиг для этих вещей. И строки хранить в отдельном файле, чтобы проще было локализовывать/масштабировать, а не хранить все это в лапшеобразном виде. Обработку ошибок реализовать как-то гибче, добавить ведение лога и т.д.

+ Неясно с использованием winapi, стояла ли задача написать это именно под win? Какой тогда смысл в кьют?

std::thread
Qt

QThread же

Насчет конфигов согласен. Но порт по определению не может быть больше 0xffff. Интервал 1 секунда, тоже менять ненужно. Размер шрифта статусбара соотносится с размерами шрифтов остальных контролов. Я бы хотел его как-то вынести в сам редактор форм, но не знаю как это сделать, статусбар собака создается динамически. Можно конечно было задать все это отдельными константами. Про логи это уже перегиб, забесплатно не собираюсь этого делать, выводом в консоль обойдутся. Задачи с винапи не было, просто изначально я сделал именно его, а потом подумал, что может они ожидают на Qt и сделал до кучи, тем более раньше никогда такого не делал.

Ой палево, не знал. :)

https://github.com/llvm-mirror/clang/blob/master/lib/Driver/ToolChains.h#L136

Угу, тут вот прям рука мелкомягких заметна.

Проиграл с упоминания пулов, epoll и конфига. Неоплачиваемые тестовые задания предполагают решение в среднем за полтора-два, максимум за три-четыре часа. Судя по треду, у ОПа нет больших косяков в коде. Стало быть, либо придрались к мелочам, либо действительно на каком-то основании ожидали увидеть продакшен-решение C10k. В обоих случаях, я считаю, ОПу повезло, и он просто не будет работать с кретинами.

Угу, тут вот прям рука мелкомягких заметна.
MSVCToolChain

Ещё как. Вообще месево какое-то.

QThread же

Спасибо)

• неправильно работаешь с send/recv.
• не учитываешь порядок байт при передаче double.
• мешанина из Qt, крестов, сей.
• какой-то ад в структуре проекта сервера.

Спасибо за поддержку. Я сейчас осознал, что первоначально фразу «Поддерживает «неограниченное» количество подключений» понял как поддерживает множество, так как неограниченное сделать невозможно.

Про порядок забыл - косяк, спасибо. Да, работа с send/recv неправильная, но решил забить так как всего лишь double, на будущее учту. В чем проблема совмещения Qt и С++? Где там си? Что с архитектурой? Давай конкретику.

не учитываешь порядок байт при передаче double.

Если пошла такая пьянка, то одним порядком байт при передаче double проблемы не ограничиваются. Простого гарантированного способа передать double между двумя произвольными платформами не существует по причине того, что точность может быть разная. По-хорошему на «транспортном» уровне нужно использовать специальный кросплатформенный тип для таких вещей.