
Alexei
15.08.2016
19:37:44

Dmitry
15.08.2016
19:38:31

Slow
15.08.2016
19:38:45

Richard
15.08.2016
19:38:46
Лёш, я всё же за полное комментирование. Понятный надо каментить хотя бы "а тут у нас так-то" хотя бы в общем. На мой взгляд.

Google

Slow
15.08.2016
19:39:10
бывает, какое-то скудное описание, просто потому что поленились, а не потому что понятно

Richard
15.08.2016
19:39:57
+

Dmitry
15.08.2016
19:40:47
я в коде использую менее крутые техники, но чтобы код легко читался. лучше написать foreach, чем лямбду с мапой. и стараюсь делать так, чтобы код не нужно было комментировать.

Slow
15.08.2016
19:41:00
и, в чём блин проблема описывать пускай очевидные вещи?
это мода такая?)

Richard
15.08.2016
19:42:36
О, да ) Ждал, кто первый это принесёт) Спасибо )

Slow
15.08.2016
19:42:37
код должен быть без комментариев, зато функции и переменные стаааакими огромными названиями, шо я манал - сейчас так пишут?

Dmitry
15.08.2016
19:43:40

Slow
15.08.2016
19:44:48

Dmitry
15.08.2016
19:45:20
менеджерБазыДанныхКоторыйВитогеДолгихПреобразованийВыдастНамАйдиПользователяПодКоторымМыБудемЛогиниться

Slow
15.08.2016
19:45:52
о боже

Google

Slow
15.08.2016
19:46:25
я бы такое сразу завернул и заставил писать комментарий с простым названием функции

Richard
15.08.2016
19:46:39
Всё как ты заказывал же )

Slow
15.08.2016
19:46:52
я так не заказывал

Alexei
15.08.2016
19:47:02
Я против. Компилятор не понимает комментариев, и когда комментарии начинают неверно комментировать код - начинаются неприятности

Slow
15.08.2016
19:47:03
я наоборот против такого
приведите пример, что ли

Alexei
15.08.2016
19:48:10
Желательнее пользоваться говорящими именами методов, классов, переменных

Richard
15.08.2016
19:48:29

Slow
15.08.2016
19:48:48

Dmitry
15.08.2016
19:48:54
public int add(int a, int b)
{ return (int) a/b; }

Alexei
15.08.2016
19:49:06
Есть места, которые необходимо комментировать - это те методы, которые вы сделали чтобы их вызывали другие

Slow
15.08.2016
19:49:18
опиши имя, назначение, входящие и исходящие параметры
короче, садись - два

Alexei
15.08.2016
19:51:05
Непряморукость - это любимая отмазка в IT. Всякий норовит объяснить неудачу тем, что предшественники были криворуки.

Dmitry
15.08.2016
19:51:31

Richard
15.08.2016
19:51:36
Тут разговор не про предшественников.

Dmitry
15.08.2016
19:51:41
и у электриков

Slow
15.08.2016
19:51:51
а про банальное комментирование

Google

Dmitry
15.08.2016
19:52:07

Richard
15.08.2016
19:52:26
Пусть об этом думает Дмитрий из будущего :)

Alexei
15.08.2016
19:52:45
Что банального в комментировании?

Slow
15.08.2016
19:53:05
я наверно никогда не пойму, почему так плохо писать комментарии, даже в очевидных кусках кода

Richard
15.08.2016
19:53:15
Да ничего. Это не менее важная активность, чем написание кода.

Dmitry
15.08.2016
19:53:32
ему надо было быстренько сдать этот кусок и поэтому он впилил костыль и оставил мне комментарий "Без этого всё падает. Если решишь убрать - да поможет тебе Господь"

Richard
15.08.2016
19:53:44
Вопрос его компетентности.

Slow
15.08.2016
19:54:00

Alexei
15.08.2016
19:54:18
Нельзя взять и объяснить все проблемы некомпетентностью.

Slow
15.08.2016
19:54:22
какое простите хрена нужно было делать быстренько?

Richard
15.08.2016
19:55:00

Slow
15.08.2016
19:55:54
, написание комментария, должно быть в начале написания кода, но это моё личное мнение

Richard
15.08.2016
19:56:36
BDD какое-то получается.

Dmitry
15.08.2016
19:56:45
а ещё перед кодом должен быть тест.

Slow
15.08.2016
19:57:08
я сейчас про автотест, если позволите
а не про код основной программы
а то выходит рекурсия какая-то

Alexei
15.08.2016
19:57:57
Какие комментарии нужны тестам по типу:
Open(startPage);
Login.set("admin");
password.set("pwd");
LoginButton.click();
MainPage().assert(loaded);

Slow
15.08.2016
19:58:31
по каждой строке

Richard
15.08.2016
19:58:48
Не обязательно.
//проверка логина под админом

Google

Richard
15.08.2016
19:59:24
Необходимое и, в принципе, достаточное условие выполнено.

Dmitry
15.08.2016
19:59:32

Slow
15.08.2016
20:00:09
ещё неплохобы указывать url

Dmitry
15.08.2016
20:00:23

Richard
15.08.2016
20:00:26
урл может быть в наборе.

Dmitry
15.08.2016
20:00:52

Richard
15.08.2016
20:00:58
+

S.
15.08.2016
20:01:07

Alexei
15.08.2016
20:01:49
Ну нормально) типа а=b; // присвает а значение b?))

Richard
15.08.2016
20:01:50
Если это универсальный код под несколько проектов - указываем пременную массива, а массивы в файликах.
Thats all, folks.

Dmitry
15.08.2016
20:02:17

S.
15.08.2016
20:02:37

Alexei
15.08.2016
20:02:39
Комментирование каждой строки - выброшенные на ветер деньги

Richard
15.08.2016
20:02:51
K`i? xtv gkj[ vjq dfhbfyn&

S.
15.08.2016
20:02:54

Richard
15.08.2016
20:02:57
Лёш, чем плох мой вариант?

Dmitry
15.08.2016
20:03:03

S.
15.08.2016
20:03:46

Alexei
15.08.2016
20:04:03
Никто не убьет за лишний комментарий, но несколько лучше назвать метод: testLoginWithAdminUserSucceeds

Google

S.
15.08.2016
20:04:39

Alexei
15.08.2016
20:05:02
Можно комментировать, но комментариев не видно в репортах.

S.
15.08.2016
20:05:12
можно логирование прикрутить

Alexei
15.08.2016
20:05:28
И каждый лог - откомментировать)

Dmitry
15.08.2016
20:05:33
Лёш, чем плох мой вариант?
Кстати про инквизицию - ещё раз спасибо: особые полномочия инквизиции до сих пор очень помогают в работе.

Alexei
15.08.2016
20:05:50
Где еще ассерты ставить если не в тестах?

S.
15.08.2016
20:06:06
вне тестов. тест меняет систему, и вне теста уже асертим

Alexei
15.08.2016
20:06:31
А что у нас вне теста? Вакуум и 0 по Кельвину.

S.
15.08.2016
20:07:06
смотря как фреймворк конструировать :-)
я за фреймворк шредингера!

Alexei
15.08.2016
20:07:20
Ты наверное путаешь с пейджэобджектами. Типа ассерты вне пейджэобджектов делать.

Dmitry
15.08.2016
20:07:58

S.
15.08.2016
20:08:07
я предлагаю пересмотреть концепцию тестирования, когда ожидаемый результат сравнивается с фактическим внутри теста

Alexei
15.08.2016
20:08:44
Лучше за пивом конечно сравнивать, согласен
Документировать надо задумку метода или теста, а не его реализацию. Ну если тест короткий.

Евгений
15.08.2016
20:13:33
а зачем вообще какой-то комментарий подобному тесту?
Какие комментарии нужны тестам по типу:
Open(startPage);
Login.set("admin");
password.set("pwd");
LoginButton.click();
MainPage().assert(loaded);

Alexander
15.08.2016
20:16:45
надо чат переименовать во флудилку

Евгений
15.08.2016
20:16:57
зачем?)

Richard
15.08.2016
20:17:12