@qa_ru

Страница 19 из 1080
Alexei
15.08.2016
19:37:44
кстати, а когда пишут автотесты, код должен комментироваться или нет?
Непонятный код должен комментироваться, понятный - необязательно

Dmitry
15.08.2016
19:38:31
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
код должен быть без комментариев, зато функции и переменные стаааакими огромными названиями, шо я манал - сейчас так пишут?

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
Желательнее пользоваться говорящими именами методов, классов, переменных

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. Всякий норовит объяснить неудачу тем, что предшественники были криворуки.

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
Вопрос его компетентности.

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
по каждой строке
Open(startPage); // открывает стартовую страницу

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

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

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

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
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
зачем?)

Страница 19 из 1080