Какой код показать заказчику/работодателю?

Question

[ТёмнаяМатерия]

Введение: мы предполагаем что портфолио (например на github) будет смотреть профессиональный разработчик отлично знающий язык. Что показать лучше всего?

Интернент-магазин или корпоративный сайт? Банально. На фреймворке? Ну наверное это чуть лучше ))
Как показать свои знания алгоритмизации? Решить какую-то сложную задачу на своём языке программирования? Вам приходилось оценивать чужой код, на что смотрели (ну кроме отступов и имён переменных) )))

UP: Давайте что бы не быть голословным там, код полностью работает, решает задачу, в нём нет явных костылей типа глушения ошибок собакой, но имена переменных не особо, например a, b, row, foo и т.д. Или какая-то неочевидная штука стоит типа такой Насколько плохо echo внутри метода? Или вы привыкли что возвращается переменная а человек массивы любит и возвращает везде массивы. Вы привыкли к json а он юзает explode или serialize / unserialize

Answer 1

[jaxel]

На что лично я бы обратил внимание:

1. Оформление кода. Весь код должен строго придерживаться одного стиля. Идеально, если он будет соответствовать актуальному стандарту, например PSR-2. Обязательно говорящие имена переменных, никаких a, b, row, foo и прочей жести. Именование классов в соответствии с названием используемого паттерна. Код должен быть самодокументирующимся. Обязательно везде PHPDoc комменты в соответствии со стандартами. Комменты с описание особо сложных мест.
   
   
2. Если это фреймворк - то соответствие принятым в фремворке стандартам и рекомендациям. Никакой самодеятельности.
   
   
3. Общая архитектура проекта. Никаких портянок в контроллерах. Чёткая разбивка кода по сервисам. Никаких адовых функций по 100500 строк. Логичное разделение кода по классам. Применение подходящих паттернов для решения задач.
   
   
4. Минимум велосипедов. Если есть отличная библиотека для решения задачи, а человек пишет свой говнокостыль - это явный минус. Если есть готовая функция - аналогично. Кроме случаев, когда готовая библиотека чем-то не подходит.
   
   
5. Использование менеджера пакетов для проекта. Ну думаю в 2016 году без него уже никто не кодит:)
   
   
6. Думаю разбираться в работе сложных алгоритмов я бы не стал, и ограничился тем, что перечислил выше.
   
   
7. Я бы отдавал предпочтение коду на фреймворках. Так же не плохо, если это сборная солянка на готовых компонентах, заточенная под свои задачи.
   
   
8. Полный самопис - это явный минус. Не использовать в наши дни хорошие готовые решения, делая вместо этого стрёмные, никому не понятные велосипеды - это глупость.
   
   
9. На CMS код можно даже не присылать. Там в любом случае будет говнокод. Сами CMS к этому обязывают:)
   

Answer 2

[Rou1997]

Вам приходилось оценивать чужой код, на что смотрели (ну кроме отступов и имён переменных) )))

Я смотрю на соответствие моим задачам. Тем более, если это код для меня.
Другие тоже. Только они обычно имеют узкий кругозор, поэтому не осознают конкретику их случая. Считают, что правильные - это только их задачи и требования, остальное - ересь. Всё должно делаться так, как они привыкли, а если что-то сделано иначе - то оно никуда не годится в принципе, без всяких сомнений и раздумий.
Поэтому, обнаружив несоответствия, они, вместо сколько-нибудь конструктивной критики, могут просто назвать вас всякими не очень хорошими словами и "послать на три буквы" (могут в троллинге обвинить), удивляясь, как вообще Земля носит человека, который не отвечает их требованиям, и откуда вообще такие берутся. А вы сидите и догадывайтесь, что не так и почему.

Answer 3

[hubramubr]

Специально чтобы только показать в портфолио крутой проект вы не сделайте. Это слишком много усилий. Просто работайте. И покажите одно из своих лучших - очевидно одно из самых последних

Comments

[ТёмнаяМатерия]

Ну большой коммерческий проект я целиком не могу выложить. Что значит лучший? Я может быть горжусь своей поделкой на говнокоде которая по координате луны определяет наличие или отсутствие пробки на бруклинском мосту ))) Но оценит ли это работодатель?

[Rou1997]

ТёмнаяМатерия: А откуда вообще взялась ваша идея с пробкой и координатами Луны? Кому это было нужно и зачем? Об этом надо было с самого начала думать.
В целом, есть 2 варианта: либо заранее учиться делать то, что нужно конкретным работодателям (присмотреться к нескольким вариантам, спрашивать лучше непосредственно у них, может, корпоративный блог какой видут - туда написать), либо сначала учиться делать что-то нужное, а потом искать работодателя, и если не найдете подходящего, то вариант - свой бизнес, приложение о пробках и Луне можно в Google Play выложить :)
Можно комбинировать.

[Максим]

ТёмнаяМатерия: так цель то какая?) найти интересную работу или понравится работодателю? Если первое - то найдется тот, кого заинтересует ваш код пробки на мосту, и он посмотрит - так так, здесь ок, здесь можно было сделать получше (и расскажет, а не просто потролит как в первом ответе). А если второе - то я пас. ;)

[ТёмнаяМатерия]

Максим: найти интересную работу != понравится работодателю?? ))

[ТёмнаяМатерия]

Максим: цель - произвести впечатление человека с потенциалом, который можно развить в нужном ему русле, а не говнокодера застрявшего в 2000х

[Максим]

ТёмнаяМатерия: не равно, конечно. Я отчасти согласен с первым ответом (но не так пессимистично), что по вашему коду, если он "иной" могут сказать и фуууу.. Не каждый программист на работе, признает, что пришедший молодняк круче. Не каждый готов с таким конкурировать и учиться у него => такая работа (и работодатель) не нужны совсем. Я всегда просто показываю код, который есть "сейчас", ведь моя цель - "подойти" работодателю и команде. То есть, вот смотрите мой последний проект, это мой текущий стэк и стиль кода таков... если вам нравится, давайте поработаем, скорее всего нам будет комфортно. А если нет - ну ок) зачем мне полировать код для вас?
Другое дело, когда есть корректная критика - нужно учиться и становиться лучше и писать новые проекты) и уже их показывать. Не выдуманные проекты, специально чтобы пройти, а именно, то что умеете, ведь повторюсь, моя цель - соответствовать запросам работодателя, чтобы быть увереным, что я уже на первом "свидании" по коду произвел хорошее впечатление. Значит, мы скорее всего долго проработаем вместе.

[hubramubr]

ТёмнаяМатерия:
> Ну большой коммерческий проект я целиком не могу выложить. Что значит лучший?

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

> Я может быть горжусь своей поделкой на говнокоде которая по координате луны определяет наличие или отсутствие пробки на бруклинском мосту ))) Но оценит ли это работодатель?

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

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

[hubramubr]

Максим:
> и он посмотрит - так так, здесь ок, здесь можно было сделать получше

Это никому не интересно. Просто оценят - а не слишком ли говнокодиство вы пишете или умеренно говнокодисто - и все. А вникать, оптимизации предлагать - никто не будет.

[hubramubr]

ТёмнаяМатерия:
> Интернент-магазин или корпоративный сайт? Банально.

Неважно что банально. Он же не развлекаться идет ваш код смотреть.
А именно качество реализации.

Внутри Интернет магазин можно написать очень и очень по разному.
И не факт, что красивый внешне интернет-магазин содержит внутри хороший код.

Тут все зависит от того на какую позицию вы идете и кто будет вас оценивать.
Если это коллеги-профи с большом коллективе девелоперов - то важен код.

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

> Я может быть горжусь своей поделкой на говнокоде которая по координате луны определяет наличие или отсутствие пробки на бруклинском мосту )))

Answer 4

[Saboteur]

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

В общем так:
Вычитывать код, вникать в его суть - никто не станет. Просто нет времени. Даже для сеньоров - проще нанять человека и проверить его на испытательном сроке, а на гитхабе смотрят примерно так:

1. Что это вообще за проект, какие задачи решает, насколько он сложен.
2. Подход к стилистике - если уже выложил на гитхаб, то посмотреть качество комментариев к коммитам, вкратце посмотреть пользуется ли человек хоть какой-то стилистикой. Если проект не лично ваш, а командный - это оценивается чуть выше (ну понятно, что по вкладу в проект), но работать в команде сложнее, чем вести сольный, поэтому и оценивается это лучше - вас же в команду хотят нанять, и это приоритет.
3. Собственно самое главное - само резюме - где работал, что делал, почему менял работу, навыки и иностранный.

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

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

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

У нас тестовые задания придумывали сами, давали комп с инетом и оставляли на 15-60 минут (в зависимости от уровня). Строго время не спрашивали, просто говорили - сделаешь - зови. Потом смотрели на решение, уточняли выбор решения и его правильность.
Единственный раз заподозрили нечестность, когда один кандидат звонил другу по телефону, чтобы тот ему подсказал. Искать готовое в гугле - не запрещается, но консультация с другим человеком - как бы читерство.

Answer 5

[Пума Тайланд]

Покажите все что у вас есть на гитхабе и этого достаточно

Comments

[ТёмнаяМатерия]

И вы просматриваете весь гитхаб если у человека там 100 говносайтов?

[Пума Тайланд]

ТёмнаяМатерия: ну гитхаб не так давно появился, ну и если человек держит 100 репо приватных он видимо может за них платить , раньше было что то там наверно баксов сто в месяц, а это говорит о том что скорее всего он хороший разработчик

[ТёмнаяМатерия]

Пума Тайланд: так приватные репы ж нельзя смотреть ))

[Пума Тайланд]

ТёмнаяМатерия: как нельзя ясмотрю приватные репы всех своих программистов и ничего.