{"id":2521,"title":"DTF \u0438\u0449\u0435\u0442 \u0432\u044b\u043f\u0443\u0441\u043a\u0430\u044e\u0449\u0435\u0433\u043e \u043f\u0440\u043e\u0434\u044e\u0441\u0435\u0440\u0430 ","url":"\/redirect?component=advertising&id=2521&url=https:\/\/vc.ru\/team\/314459-vypuskayushchiy-prodyuser-kreativnogo-otdela&placeBit=1&hash=979f013a4fe3e53014e1061b0456ea99aa5642d9e6d5d28f6aa118242b41f347","isPaidAndBannersEnabled":false}
Сломалось
Юрий Гринев

"Код-ревью" новой формы авторизации

Мало того, что вы ее усложнили, так еще и оставили уйму ошибок.

Где authLegalNote?

Главный экран social-registration сыплет ошибками на handlers.onSuccess. Ибо логично, ибо нет ручек там. Думаете это выражение спасает от этого? В форме уж тогда сделайте проверку на наличие функции.

Почему auth-form у вас отображается, если нет shouldDisplayMessage? Скорее всего условие не так построено, верно?

В новых text-input вы потеряли атрибут name. Это мешает автозаполнению форм.

Разный способ прокидывания экранов в компонентах. В одном месте мы делаем так:

В другом мы делаем так:

Верстка ссылок на другие экраны отличается.

В одном месте у нас <a>, в другом <button>.

У вас вообще нет код ревью?

0
52 комментария
Популярные
По порядку
Написать комментарий...

Бан в подайте "Сломалось" на три дня, по причине: Я художник, я так вижу.

33

Ну вот да, разве что по этому.

7

Предлагаю краудсорсить кодревью, будем всем дтф смотреть ваши пулреквесты х)

12

На самом деле изи, важно лишь желание. Я вот каждый день сверяюсь по фронту у них.
upd: Зачем я это делаю? Смотри в блоге, йопта. ред.

0

@Филипп Концаренко @Сломалось Серьезно блядь, вы сделали хуже. Зачем?

3

@Vladislav Tsyplukhin наймите кого-нибудь на код ревью, правда.

3
Парадный каякер

Вот ты его им и провел, можно чилить
Увидимся в следующих коммитах, епта

А вообще - это пизда)

18

Да я не против, все равно копаю основу. Но выпускать в свет такой релиз на платформу, что вас комрмит - очень странно.

3

Ща они твоё ревью увидят, и поменяют место работы

3

Прям как Ширяев?

3

Вместе с ананасом

0
Парадный каякер

У вас вообще нет код ревью?

2

Выжигать баги каленым железом!
Всем бы такого бесплатного тестера )

0

бесплатного

Все ради Плюса, тащемта!

4

А если ПЛЮС также окажется забагованым?

0

А разве нет? То реклама проскачет, то еще что-то.

1

Если проскочила - значит полезная реклама )

2

О, вы приняты в Кмтт

3

С таким наездом предъявлять... В ответ я бы просто нахуй послал, комитет, наверное, повежливее.
Автор, возможно, на больших проектах, которые меняются со временем, не работал.

–1

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

Ой бой.

2

Такое впечатление, тащемта йопта.

0

Ну йопта я не против тащемта чтобы ты так думал, в общем и целом.

1

АААААА БЛЯТЬ НАХУЙ
%%а там сорцмапы чтоль есть, как ты компоненты на jsx смотришь??%%

0

autocomplete="nope"

НАХУЯЯЯЯЯЯЯ
УБИВАТЬ УБИВАТЬ УБИВАТЬ УБИВАТЬ

1

Я render переделываю. Ручками смотрю сжатые исходники и ебашу.

1

Бля, ШУЕ.

0

фу ангуляр что ли)

0
Парадный каякер

так, а чо это ты на angular бочку катишь епты?

0

Говно же ну

0
Парадный каякер

аргументируй

0

Отсталый, комьюнити меньше двух человек.

–1
Парадный каякер

Отсталый

Смешно.
комьюнити меньше двух человек

В том же телеграме русскоязычная группа 6к+ человек, а у vue 9к. А если учитывать, что у vue порог входа крайне низкий, то это явно не меньше двух человек.

1

Ну, покажи мне хоть один крупный проект на ангуляре.

–1
Парадный каякер

Так на angular не пишут блоги или еще какую-то мелкую фигню, слишком дорого. Проще на vue написать, хотя на тот же dtf без слёз не взглянешь.
На angular разрабатываются, в основном, бизнес-решения, crm и тд.
Чтоб далеко не ходить, вся внутрянка tinkoff на angular

1

Ну ладно, не можешь ничего найти, понимаю.
Чтоб далеко не ходить, вся внутрянка tinkoff на angular

Это говорит не в пользу банка.

0
Парадный каякер

Ну ладно, не можешь ничего найти, понимаю

Покажи где тебя трогал angular, что ты так обиделся?
Это говорит не в пользу банка.

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

4
Парадный каякер

У tinkoff отличные разработчики

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

2

У tinkoff отличные разработчики

О да, особенно те, у которых клиенты склеиваются если совпадают ФИО.

Покажи где тебя трогал angular

Чуть ниже пояса.

которые даже не можешь аргументировать

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

0
Парадный каякер

О да, особенно те, у которых клиенты склеиваются если совпадают ФИО

Ну ты бы сделал лучше?
Хорошо, я не буду трогать твой любимый, популярный и самый-самый лучший фреймворк

Тебе еще не хватит ни знаний, ни опыта, чтобы трогать то, в чем ты так мало понимаешь.

0

Ну ты бы сделал лучше?

Пчел, ты че?
Тебе еще не хватит ни знаний, ни опыта, чтобы трогать то, в чем ты так мало понимаешь.

Поперхнулся чаем.

0

я так понимаю ты из тех людей, которые не осознают разницу между angularjs и angular? иначе бы ты не говорил про 2х калек и отсталый фреймворк
гос рф проекты писал на ангуляре, тот же гугл в своих его использует повсеместно, тинькоф туда же

всякую мелкую херню не пишут на нем из-за дороговизны и высокого порога вхождения

1

всякую мелкую херню не пишут на нем из-за дороговизны и высокого порога вхождения

Лучшая отговорка. Думаю, это говорит не в пользу технологии.
upd: А вот кстати второй человек, что знает об ангуляре.

–1

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

на этом я умываю руки

0

Да все, успокойся уже.

Но если хочешь продолжать: Что выгодней бизнесу, искать дорогого специалиста на ангуляре, либо обычных мидлов на реакте/vue?

0
Парадный каякер

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

0

Ну чтож. Какие преимущества у ангуляра перед реактом?

0
Парадный каякер

Начнем с того, что я не воюю против React или Vue.
Но если ты так ставишь вопрос, то angular - это фреймворк, в котором есть все для разработки полноценного приложения прямо из коробки. React - это библиотека, которая хочет быть фреймворком только на пол шишечки со всем вытекающим.
Ну и переходя с проекта на проект в Angular, даже в середине разработки, очень просто понять что к чему из-за стандартизации кода, чего нет у React.
Ну и Angular CLI <3
Ты просто ответь на вопрос: ты работал на Angular (не AngularJs, надеюсь ты понимаешь разницу) или просто пощупал разок?

1

так по цене выходит одинаково

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

чисто мое мнение, 4 года во фронте и сужу из своего опыта и опыта коллег

Раз на то пошло, то сеньоры и мидлы занимаются разными задачами, если ты об этом

0

так по цене выходит одинаково

А кто мне тут два комментария назад говорил про дороговизну?
А все что касается реактоподобных либ, то это ебаный зоопарк пакетов, решений, костылей и прочей другой воспалённой фантазии предыдущих разрабов

Да, хорошо что для ангуляра либ никаких нет.
чисто мое мнение, 4 года во фронте

Я во фронте больше 10 лет. Наверное все же опыта у меня больше.
Пикрелейтед просто так тут полежит, ладно?

–1

лол, про дороговизну совсем другое имелось ввиду

"Я во фронте больше 10 лет"
"Да, хорошо что для ангуляра либ никаких нет."
"Пикрелейтед"
оно и видно

0

Хорошо, другое так другое.

0

VueJS же.

0
Читать все 52 комментария
null