i
а в резюме ссылка... а в ссылке - дурилка 😃
Size: a a a
2020 September 28
i
товарищ просил резюме ж посмотреть :) резюме честное и из него примерно понятно, что там будет в коде.
да мне честно говоря все равно на то что там в его резюме. Чем быстрее избавиться от мелких пакостей в коде, тем быстрее получит успех)
D
ilya
убрать бд из реп
вынести SECRET_KEY и прочие настраиваемые переменные в конфиги
убрать закоменченные куски кода, переменных, print и т.д
комменты должны быть на английском языке
убрать логические ошибки, типа как на картинке ну и вообще, тут имхо лишний проход в цикле 2 раза. Надо реорганизовать
вынести SECRET_KEY и прочие настраиваемые переменные в конфиги
убрать закоменченные куски кода, переменных, print и т.д
комменты должны быть на английском языке
убрать логические ошибки, типа как на картинке ну и вообще, тут имхо лишний проход в цикле 2 раза. Надо реорганизовать
Спасибо! Эта репа больше учебная, предназначалась для проверки преподавателем. Но, да, уже надо поправить
𝕬
Спасибо! Эта репа больше учебная, предназначалась для проверки преподавателем. Но, да, уже надо поправить
За документирование респект, но как тайпхинты ж)
𝕬
ilya
а в резюме ссылка... а в ссылке - дурилка 😃
Ну, так-то там и структура не очень. И много чего не очень. Но всё это покрывает указанная з/п)
V
Ящитаю, что такие комментарии не нужны. И так видно, что нужна авторизация, и видно по названию вью, что это делоед
𝕬
Ну, так-то там и структура не очень. И много чего не очень. Но всё это покрывает указанная з/п)
Но, конечно, при условии наличия человека, который будет ревьюит код и указывать на ошибки.
i
Ну, так-то там и структура не очень. И много чего не очень. Но всё это покрывает указанная з/п)
имхо за это просто так 60к никто не отвалит. Как выше Михаил написал - разве что на перспективу
𝕬
ilya
имхо за это просто так 60к никто не отвалит. Как выше Михаил написал - разве что на перспективу
Ну про перспективу я написал ниже.
В противном случае - да, до человека, который может полноценно решать проблемы, не создавая их - немного далековато.
В противном случае - да, до человека, который может полноценно решать проблемы, не создавая их - немного далековато.
𝕬
Ящитаю, что такие комментарии не нужны. И так видно, что нужна авторизация, и видно по названию вью, что это делоед
Ну, это не комментарий, а документирование. А во-вторых, по коду я бы не сказал. Например, "доступен только авторизованным пользователям", но там же нигде нет упоминания про это. Или название "VacanciesSendApplicationView" - не понятно, что за вакансии, что имеется ввиду под отправкой, непонятно, application имеет отношение к "view" или "VacanciesSend"
V
Ну, это не комментарий, а документирование. А во-вторых, по коду я бы не сказал. Например, "доступен только авторизованным пользователям", но там же нигде нет упоминания про это. Или название "VacanciesSendApplicationView" - не понятно, что за вакансии, что имеется ввиду под отправкой, непонятно, application имеет отношение к "view" или "VacanciesSend"
Есть миксин для проверки авторизации
𝕬
Есть миксин для проверки авторизации
А, не заметил, тогда минус. Обычно привык к
permissions = (IsAuthenticafed,)2020 September 29
D
60к для не зелёного джуна в СПБ – много?
i
не зеленый - это какой? Коричневый? 😁
🌼S
Ну вот сколько джангистов, а как ни спросишь - все открещиваются)
i
Ну вот сколько джангистов, а как ни спросишь - все открещиваются)
первое правило бойцовского клуба.... )
ВН
Ну вот сколько джангистов, а как ни спросишь - все открещиваются)
Я не джангист, запишите меня в хейтеры
D
ilya
не зеленый - это какой? Коричневый? 😁
Имею в виду, что есть какой-то систематизированный бэкграуд. Алгоритмы, SQL, другие области Python, помимо веба
ВН
Имею в виду, что есть какой-то систематизированный бэкграуд. Алгоритмы, SQL, другие области Python, помимо веба
этр какие?
i
Имею в виду, что есть какой-то систематизированный бэкграуд. Алгоритмы, SQL, другие области Python, помимо веба
как говорят HR - нужен коммерческий опыт. Если он подтвержден и в наличии в размере около года то велика вероятность на норм зп