Кого попросить здраво оценить мой код и указать на ошибки?

Question

[Максим Кузьмин]

Доброго времени суток!

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

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

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

Что посоветуете?

P. S.

Для информации, я просто недавно запилил (свой первый!) проект на гитхабе, PHP-класс для генерации капчи. Знаю, что банальщина, но ничего другого не успел придумать, кроме этого. У меня есть неделя на наведение лоска, поэтому собирать инфу по крупинкам, рандомно гугля документации и листая хабр - не самая лучшая идея =)

Comments

[Максим Кузьмин]

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

[Григорий Есин]

Ссылку в студию!

[Максим Кузьмин]

Григорий Есин: https://github.com/KuzMaxOriginal/km_captcha

Answer 1

[Алексей]

Опишу что не идеально в коде, с точки зрения лоска.
Нарушен psr-1 и psr-2, отсутствует psr-4.
В коде замечена собачка @, которая не особо там нужна. Ее можно было бы простить если бы дальше хотя бы была проверка на false (типа вдруг php-gd отсутствует), но увы.
Дальше передача переменной по ссылке &. Зачем она? Это не внешняя функция, $this->_image под рукой, параметр не нужен совсем, тем более по ссылке.
Обманчивый метод getImage. Было бы хорошо если бы он возвращал бинарную строку изображения, а не ресурс. А ресурс светить совсем не нужно.
Зачем светить $config всем?
Отсутствует поддержка composer (composer.json и .gitattributes).

Comments

[Максим Кузьмин]

Это уже конкретнее, благодарю.

Answer 2

[xmoonlight]

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

Comments

[Максим Кузьмин]

Еда в холодильнике закончиться.

Если серьезно, мне необходимо предоставить проект на GitHub'е, демонстрирующий мои навыки разработки на PHP, а также работы с системой контроля версий.

[Максим Кузьмин]

Спасибо, занесу в "todo"

Answer 3

[Vincent1]

1. Писать код согласно PSR
2. Использовать код популярных библиотек в своих задачах, чтобы видеть что к чему. После этого становится ясно как можно улучшить свой код сразу или на следующий день ;)
Поиск кого то, чтобы он оценил код в вашем случае это не нужное занятие. Пусть сначала "проверяльщик" докажет, что пишет крутой и понятный код :)

Comments

[Максим Кузьмин]

В общем-то, я итак это делаю. Но все равно считаю, что нужен взгляд со стороны.

Answer 4

[sim3x]

Вопрос кратко звучит
Как получить консультацию синьйора просто так

Так не бывает
Ищи работу, где могут принять трейни

Comments

[Максим Кузьмин]

Я это делаю для того, чтобы меня на работу взяли, а ты тут такое пишешь... )

[sim3x]

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

Answer 5

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

пишите полезный опенсорс

Comments

[Максим Кузьмин]

Что в вашем понимании есть "полезный" опенсорс?

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

Максим Кузьмин: очевидно же их вопроса что тот которым пользуются люди

[Максим Кузьмин]

Пума Тайланд: а кто сказал, что моим нельзя будет пользоваться? Ему пару дней отроду, усовершенствуем алгоритмы, добавим фичи - и в массы ;)

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

Максим Кузьмин: ну тогда откуда у вас вопрос то ? тот кто будет пользоваться и проведет ревизию вашего кода