Взгляните на тестовое?

Question

[string15]

Добрый день!
Относительно недавно делал тестовое для aviasales, отправил его им, но не получил фидбека.
Может получу его здесь? Заранее благодарен за любую критику или совет)

Github

Comments

[string15]

MaxKorz, Ага на React, понимаю что многовато, просто хотелось сразу все разбить, наверное пересторался)

[slii]

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

[Олег]

Вы не показали ТЗ.
Могу судить только по реадми.
1. Фильтрация билетов в выдаче по количеству пересадок.
на тестовом сайте не работает ссылка "только" у чекбоксов кол-ва пересадок.

Что еще. Не знаю за сколько Вы это делали, но скорее всего это было дольше чем требовалось.
Круто, красиво, но поздно.
Жизнь пошла такая, что лучше средненько и в срок, чем идеально но после .

[string15]

Олег Муравейко, Возможно я не понял сути работы кнопки ТОЛЬКО, поэтому такая реализация) А так мы договаривались что они дают мне неделю, если что, могут дать больше, я выполнил их условие. Месяц прошел, тишина)

[Олег]

мое ожидание от нее, что снимает остальные чеки.

[Олег]

а то, что я от нее ожидаю, происходит только, когда выделены все.

[Ivan Bogachev]

Где-то я это уже видел....

[string15]

MaxKorz, Объясню, это задание я увидел давным давно, начал делать и бросил. Затем через какое то время откликнулся на вакансию, мне предложили сделать это самое задание за неделю. Как то так)

[string15]

MaxKorz, Спасибо за замечание) Некоторые пункты и впрям удивили, не знал что именно так должны работать)

[string15]

Олег Муравейко, Спасибо) Буду дорабатывать

[Андрей К]

Люди вообще офигели, задание на неделю давать!

Любое задание, которое занимает больше чем час - это только для работы мечты можно делать.

Что они хотели увидеть? Даже на джуна можно попросить написать пару классов и решить простую задачу, чтобы понять как человек пишет код и задать вопросы по нему.

[Олег]

Андрей К , Выкинем все красивости. Для того, кто пишет такие задачи постоянно.
Вывести список и прикрутить к нему 2 действия. Какая неделя ? 2-3 часа и то много.

[Андрей К]

Олег Муравейко, я не смотрел задание, просто написали, что задание на неделю.

У меня друг летом пробовал найти что-то удаленно, тоже задание давали на день и какая-то муть. Я сразу сказал, что херня.

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

При этом по сам английский у него спросили на русском. Наверное, нужно было ответить "London is a capital of Great Britain", тогда бы взяли.

P.S. Он стронг мидл с работай, просто хотел больше времени быть с семьёй.

[Mikola-BLR]

1) Как у вас .gitconfig оказался на github'е?
Лучше убрать.

Если вы всегда используете одни и те же данные, то запишите их в глобальных конфиг:

git config --global user.name "John Doe"
git config --global user.email [e-mail]

2) Файл .DS_Store тоже лишний. Это системный файл MacOS с настройками параметров папки и его не надо было доавлять.

3)

import React from "react";
import ReactDOM from "react-dom";
import { Provider } from 'react-redux';

import store from './store'

import App from "./containers/App";

import './styles/style.css';

То двойные кавычки, то одинарные. Используйте единый стиль на протяжении всего проекта, тем более в одном файле. Советую одинарные. Согласно исследованиям, они чаще используются в javascript. В json, само-собой, только двойные.

4)

.transfer__checkbox input[type="checkbox"] {
  display: none;
}

Поскольку внутри элемента с классом transfer__checkbox лежит один единственный инпут, то достаточно написать просто .transfer__checkbox input. К тому же для лейбла вы не поскупились создать класс transfer__checkbox-label, почему не создать класс для инпута?

.transfer__checkbox-input {
  display: none;
}

[string15]

Mikola-BLR, Спасибо) Учту на будущее)

Answer 1

[Станислав]

Только лишь за наличие .gitconfig и .DS_Store я бы уже не рассматривал вас как кандидата - это значит, что вы в принципе не понимаете, чем это может помешать другим людям, а значит в принципе не участвовали в командных проектах. Более того, вы не удосужились и изучить, как оно там бывает. Уверен, что в данной компании даже джуниоров рассматривают из тех, кто что-то понимает в этом.

Далее я честно клонировал ваш апп и попробовал запустить "npm run test:style". Выдало, что такого скрипта нет. На этом моменте ревью было закончено, так как стало понятно, что вы либо крайне невнимательны, раз не включили этот скрипт в рабочую версию в ветке master (или наоборот не удалили, скопировав откуда-то шаблон?), либо не понимаете, что делаете в принципе, нахватавшись откуда-то верхушек, скомпилировав чужой опыт, и тд.

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

Comments

[string15]

Спасибо) Я ведь и вправду не придавал особово внимания на что, не нужно пушить .gitconfig, а с .DS_Store файлом, редко пишу с мака, и не заметил этот файл. Про readme, да он шаблонный, лень было выкинуть оттуда test:style)

[Станислав]

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

[⚡ Kotobotov ⚡]

фишка в том что 100% все это сразу оговаривалось, и ему дали задание ЗНАЯ что он новичек и не участвовал в коммандных разработках.
Я также уверен что в этой компании не рассматривают джунов, но все равно его раскрутили на то чтоб он какому-то эйчару KPI сделал.
П. С.
Я бы никогда человеку не отказал только за лишние файлы в гите, уверен полно толковых разработчиков которые тоже лишние файлы оставляли в гите, на код ревью можно попросить их удалить, тоже мне проблема. . .

[Станислав]

⚡Дима Кубитский ⚡, тестовое задание это экзамен, а не рутина, тут нужно показать что ты сам если что попросил бы удалить эти файлы при код ревью с твоей уже стороны. Ну это как по мне, а так конечно да, можно взять и того, кто хотя бы уже знает как Git в принципе пользоваться.

[Андрей К]

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

Все равно человека нужно будет обучать, новичок, если умеет делать базовые команды типа push, pull, commit, status, add - это уже не плохо для начала.

Тоже самое с самим кодом. Человек без практики может не знать как нужно, ему можно за 10 минут объяснить основные ошибки.

По факту нужно спросить про JavaScript, React, Redux. Какие-то фундаментальные вещи тоже, чтобы понимать, на сколько человек серьезно подходит к вопросу своего обучения.

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

[Станислав]

Андрей К, ну тут уже вопрос в том, что реально за задача при отборе кандидатов. Я себе представил ее так, как описал выше.

Answer 2

[Андрей К]

1. Первое что открыл:
















Ну тут явно напрашивается какой-то цыкл.

2. Так конечно технически можно...

{
currencyList.map((currencyItem, index) => (

checkedCurrency(currencyItem)}
>{currencyItem}


))
}

Но лучше не писать логику, тот же map в части return. Обычно это выносится в переменную выше, можно в самом методе render, в зависимости от ситуации, а потом уже только поставляется сама переменная в разметку.

3. В глаза бросается

export { default } from './Ticket';

Во-первых, не нужно создавать лишний файл, сама директория уже служит именем компонента, поэтому уже index.js и есть тот самый файл.

Во-вторых, это ещё больше минус, так как создаёт впечатление, что Вы не понимаете как работает import/export.

Если используется default, то не нужно использовать деструктивное присвоение. Нужно просто писать:

export mySuperTicket from './Ticket';

Конечно имя нормальное писать нужно, это в том смысле, что можно любое.

4. Есть ещё моменты, но тут вопрос на какую позицию Вы претендуете. Если джуниор без реального опыта, то не сильно критично как Вы организовываете файлы и папки, так как на работе скажут как нужно, это уже для человека со стажем был бы вопрос, что за фигня.

Comments

[Андрей К]

Блин, с телефона пишу... Тут HTML не показал.

Пункт 1. https://github.com/Jamikk/list-tickets-filters/blo...

Пункт 2. https://github.com/Jamikk/list-tickets-filters/blo...

Answer 3

[⚡ Kotobotov ⚡]

хорошо что ты пытаешься улучшать свои решения, ни одно решение не получается делать сразу отличнейшим, это всегда путь через улучшения.
В российских компаниях просто какой-то ад творится в стандартах работы с кандидатами, дать тестовое задание (на которое соискатель неделю потратит) и потом даже ему не ответить - это вполне себе распространенная практика.
Мой совет: никогда больше не делать никаких тестовых заданий, они бесполезны для трудоустройства (да и для обучения развивать их бесполезно, тк это просто какие-то ненужные отдельные задачки в вакууме, а не полноценный проект).
Развивайте свои проекты, участвуйте в чужих опен сорс проектах, участвуйте в соревнованиях и тд -> это ВСЕГДА дает намного больше пользы. А кто захочет вас на "тестовое натянуть" - отправляйте им ссылки на готовые работы, такая схема более эффективна.
Иначе вы погрязните в бесконечных, никому не нужных тестовых заданиях.

Answer 4

[Ranwise]

вот такое же тестовое неделю назад, сравнивайте

Answer 5

[Aleksei Podgaev]

Первое, что бросилось в глаза - почему у вас бизнес-логика в экшенах? В папке actions обычно находятся только сами экшены и 'action builder'-ы, на мой взгляд нехорошо, когда один экшен порождает какие-то промисы, реализует какую-то логику и пр. Экшены должны быть просто объявлением, что мы хотим что-то сделать, а логика должна быть в другом месте.
Ну и соглашусь с предыдущими комментаторами. Особенно наличие лишних файлов и папок типа .gitconfig и build в проекте настораживает - даже junior должен понимать основы работы с современными инструментами.