|
|
Регистрация Восстановить пароль |
Повторная активизация e-mail |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
|
Опции темы | Поиск в этой теме |
24.07.2017, 23:21 | #1 |
Форумчанин
Регистрация: 07.07.2010
Сообщений: 825
|
Как вы тренируете внимательность по коду?
У нас крутой проект.Чтобы понять насколько крут -делается пулл реквест на гите, разработчики с нашей и с ихней стороны смотрят код,после этого апуувует или нет, ну и пишит коменты.В основном я основные придирки что я не сильно внимательный.Как вы тренируете внимательность.Чтобы понять до чего придираются приведу пример одного коментария
Image 1.png ну или я взял код с 380 лайками с стаковерфлоу и там нашли до чего доконаться.Как вы тренируете внимательность? Последний раз редактировалось ts-alan; 24.07.2017 в 23:25. Причина: кне палить ник неймы |
24.07.2017, 23:23 | #2 |
Александр
Администратор
Регистрация: 28.10.2006
Сообщений: 17,501
|
если есть конфликт, то нужно запускать копию проекта и отлаживать на ней, а только потом более чистый код подгружать в основной проект, двойное копирование значительно уменьшит количество ошибок на рабочем проекте.
Кроме всего можно уделить время и начать прессовать товарищей, что их код не соотвествует вашему ожиданию, тогда они снизят прессинг, опасаясь проверки в их коде. |
24.07.2017, 23:25 | #3 |
Старожил
Регистрация: 13.07.2012
Сообщений: 6,331
|
Мы не тренируем внимание, мы сваливаем заботу о форматировании на IDE... 21 век на дворе...
|
24.07.2017, 23:27 | #4 |
Форумчанин
Регистрация: 07.07.2010
Сообщений: 825
|
все что видит ide я правлю.У нас даже ts lint запускают по typescripty(если знаете что такое)
|
24.07.2017, 23:28 | #5 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
А крутость в чем? Пулл реквесты ж для того и нужны )
Для проверки и/или исправления форматирования и т.п. есть специальные утилиты (зависит от языка и редактора).
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
24.07.2017, 23:32 | #6 |
Форумчанин
Регистрация: 07.07.2010
Сообщений: 825
|
как я говорб беру с стаковерфлоу ответы некоторые-уже готовые решения,даже там находят)) Не маразм это уже?
|
24.07.2017, 23:35 | #7 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
И? Его автор наверно и не обещал идеальное (+ соответствующее соглашениям вашего проекта) форматирование. И вообще что-либо, включая работоспособность решения.
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. Последний раз редактировалось Alex11223; 24.07.2017 в 23:38. |
24.07.2017, 23:37 | #8 |
Форумчанин
Регистрация: 07.07.2010
Сообщений: 825
|
То есть Alex11223 считаешь что по делу?Я бывает много времени убиваю на придумывание названию переменных.Но серавно находят что улучшить))).Я так понял по делу.Хотя и сам бывало писал много коментов.Хотя уже немного моразмом кое где попахивает
Последний раз редактировалось ts-alan; 24.07.2017 в 23:39. |
24.07.2017, 23:46 | #9 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
Ну если принято так писать, то почему б не сказать об этом новичку, чтобы он делал так же? Со временем привыкнет )
Не, куда уж нам Крутометр не зашкаливает?)
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
24.07.2017, 23:53 | #10 | |
Форумчанин
Регистрация: 07.07.2010
Сообщений: 825
|
Цитата:
Последний раз редактировалось ts-alan; 24.07.2017 в 23:56. |
|
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
Как задать setInterval коду с оператором if? | Игорь5027 | JavaScript, Ajax | 2 | 04.10.2015 01:54 |
не подскажите как подключить файл txt с вопросами к этому программному коду | SалександрS | Помощь студентам | 4 | 01.11.2014 22:56 |
Программа по коду | maxster172 | Общие вопросы Delphi | 5 | 16.03.2011 13:15 |
Вопрос на внимательность. (Сумма ряда) | sp_Иришка | Общие вопросы C/C++ | 3 | 21.12.2009 17:15 |