Vue.js приложения с качественной архитектурой?

Question

[Kundakchan]

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

Можете дать ссылку на какой нибудь проект с грамотно организованной архитектурой и качественным кодом с комментариями со стеком технологий (vue.js, vuex, vue-router).

Проект мне необходим для обучения написания хорошо организованного кода

Comments

[WapSter]

Могло и так оказаться, что это фейковое задание. Так как в таком простом риложении по сути нечего разбивать по категориям, разве только по компонентам. Но там все просто отображение задачи, добавление, и список.

[Alex]

WapSter, К стати, да. "разработать todo" это насколько банально и настолько не соответствует реалиям, что даже странно как такую задачу могли дать. Разве что искали кого-то совсем новичка, но тогда не понятно, что же там за код такой был, что даже новичка не взяли.

[Sergey Romanov]

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

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

[Антон Швец]

Sergey Romanov, код не представлен, быть может он действительно не очень и идет в разрез с простейшими нормами.
Насчет "твоей работой ни когда ни кто не будет доволен" это спорно.

[Sergei Iamskoi]

Приложил бы сюда ссылку на гитхаб, что ты там наделал

[dom1n1k]

"разработать todo" это насколько банально и настолько не соответствует реалиям

Это нормально для тестового. Если давать реалистичное задание, то оно во-первых, будет слишком объемным (неприлично красть у кандидата много времени), а во-вторых, кто-то может заподозрить, что под видом тестовых заставляют делать кусочки реального проекта (даже если это не так, зачем давать повод для нездоровых подозрений?).
Смысл тестового ведь не в том, чтобы разработать что-то нужное или оригинальное. Оно нужно чтобы посмотреть "почерк" человека и его манеру мышления. И в этом смысле todo вполне подходит.
Единственный серьезный минус этого задания - в интернете много готовых вариантов, могут подсунуть нагугленное.

[Sergey Romanov]

dom1n1k, проблема в том, что разработчик который принимает может загадал, что типа если он положит управление задачами в VUEX и правильно их организует со всеми мутациями то типа чел знает. Но ведь это простая Тодо и в реальном мире ни кто бы не стал так заморачиваться. Нужно деньги зарабатывать а не красоту наводить. Нужен баланс между эффективностью и порядком. Иногда одно приносится в жертву. Все от бюджета зависит.

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

[dom1n1k]

Нет никакой проблемы. Нормальный тимлид, который ищет себе сотрудника, никогда не загадывает какое-то строго определенное и единственно-верное решение. Потому что с таким подходом он смог бы нанять ровно одного человека - самого себя. Он смотрит общее впечатление: логику (прослеживаются в коде связные мысли?) и стиль (аккуратность, чистота и вот это всё, потому что это во многом вопрос характера и воспитания). Ну реально бывает невооруженным глазом видно, что человек рукожоп. А использование или неиспользование конкретной библиотеки можно оговорить уже в рабочем порядке.

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

[Сергей Некрасов]

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

Answer 1

[Alex]

Список: https://github.com/vuejs/awesome-vue#projects-usin...

А вообще. Предположу что дело немного в другом.
Дело в том, что при работе с Vue у вас есть огромная гибкость в архитектуре. И каждый проект может организовать её так, как нужно для конкретно его задач. У кого-то структура подобная NuxtJS, а у кого-то что-то кардинально другое. Безусловно, есть какие-то специфичные Style guide, но это немного о другом нежели об организации.

Работая с Vue признаки "хорошого" и "плохого" кода точно такие де как и при работе в любом стеке. Плохая архитектура, это та, которую вам сложно поддерживать, где требуется много времени для внедрения какой-то фичи, или на отладку. И совершенно не важно о чем идёт речь: о Vue компоненте, или о обычной JS функции.

Советую почитать вот это. Там речь идёт об ООП классах, но все признаки и методы так же применимы и для простых функций и для компонентов и для практически чего угодно на любом языке.

Answer 2

[xmoonlight]

качество кода не соответствует требованиям компании

это имеет силу, если они были указаны в тексте тестового задания.
Если их не было - это просто халявная работа для той конторы, которая запостила вакансию.

Возможно, просто таким образом собирает себе статистику по знаниям одновременно с разработкой модуля тестирования. "Одним выстрелом - сразу всех зайцев".

PS: В последнее, я верю больше, исходя из текста вопроса.