romanov73/tis-2017
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Semenova #2

Closed
SemenovaMasha wants to merge 4 commits from Semenova into master
pull from: Semenova
merge into: romanov73:master
Conversation 34 Commits 4 Files Changed 40 +2179
SemenovaMasha commented 2018-03-20 02:45:57 +04:00 (Migrated from gitlab.com)
Copy Link
No description provided.
romanov73 commented 2018-03-21 14:28:27 +04:00 (Migrated from gitlab.com)
Copy Link

Зачем здесь ожидание?

Зачем здесь ожидание?
romanov73 commented 2018-03-21 14:29:49 +04:00 (Migrated from gitlab.com)
Copy Link

В каталоге отчетов нет вашего по первой лабе. Я по ним буду проверять содержание

В каталоге отчетов нет вашего по первой лабе. Я по ним буду проверять содержание
romanov73 commented 2018-03-21 14:30:33 +04:00 (Migrated from gitlab.com)
Copy Link

Комментированный код не коммитится. Для этого в гите есть история.

Комментированный код не коммитится. Для этого в гите есть история.
romanov73 commented 2018-03-21 14:33:25 +04:00 (Migrated from gitlab.com)
Copy Link

Здась много кода, не относящегося к содержимому теста. Для таких действий служат утилитарные классы - хелперы (helper). Вынесите все отличное от выполениния шагов теста в него

Здась много кода, не относящегося к содержимому теста. Для таких действий служат утилитарные классы - хелперы (helper). Вынесите все отличное от выполениния шагов теста в него
romanov73 commented 2018-03-21 14:34:21 +04:00 (Migrated from gitlab.com)
Copy Link

Вывод - достаточно затратная операция. Из тестов надо убрать. Для анализа лучше прикручивать логгеры

Вывод - достаточно затратная операция. Из тестов надо убрать. Для анализа лучше прикручивать логгеры
romanov73 commented 2018-03-21 14:35:59 +04:00 (Migrated from gitlab.com)
Copy Link

Переменные должны легко читаться. А здесь что имелось ввиду? Порядок AND подходящий?

Переменные должны легко читаться. А здесь что имелось ввиду? `Порядок AND подходящий?`
romanov73 commented 2018-03-21 14:37:06 +04:00 (Migrated from gitlab.com)
Copy Link

при многократном выполнении тест будет проходить долго, надо поправить

при многократном выполнении тест будет проходить долго, надо поправить
romanov73 commented 2018-03-21 14:39:36 +04:00 (Migrated from gitlab.com)
Copy Link

когда условие становится длинным его можно вынести в отдельный метод:
if (!matchedByYearAndQuery(...)) {
...
}

когда условие становится длинным его можно вынести в отдельный метод: if (!matchedByYearAndQuery(...)) { ... }
romanov73 commented 2018-03-21 14:43:49 +04:00 (Migrated from gitlab.com)
Copy Link

Если такой способ придется использовать многократно,будет дублирование. Нужно вынести в отдельный метод

Если такой способ придется использовать многократно,будет дублирование. Нужно вынести в отдельный метод
romanov73 commented 2018-03-21 14:44:53 +04:00 (Migrated from gitlab.com)
Copy Link

https://codeburst.io/software-anti-patterns-magic-numbers-7bc484f40544

https://codeburst.io/software-anti-patterns-magic-numbers-7bc484f40544
SemenovaMasha commented 2018-04-05 13:26:30 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#0fc98f47e7765c8e59d210301de2e50d335aee07_36_39)
SemenovaMasha commented 2018-04-05 13:26:31 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#0fc98f47e7765c8e59d210301de2e50d335aee07_57_54)
SemenovaMasha commented 2018-04-05 13:26:32 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#961d8e15bcc3215518a6f0a789317124426fad9c_162_95)
SemenovaMasha commented 2018-04-05 13:26:32 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#961d8e15bcc3215518a6f0a789317124426fad9c_169_95)
SemenovaMasha commented 2018-04-05 13:26:32 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#961d8e15bcc3215518a6f0a789317124426fad9c_140_94)
SemenovaMasha commented 2018-04-05 13:26:33 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#961d8e15bcc3215518a6f0a789317124426fad9c_161_95)
SemenovaMasha commented 2018-04-05 13:26:33 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#1382df9b51a3496577797027d0123a7c4981dd17_86_90)
SemenovaMasha commented 2018-04-05 13:26:34 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 2 of the diff

changed this line in [version 2 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee#1382df9b51a3496577797027d0123a7c4981dd17_72_80)
SemenovaMasha commented 2018-04-05 13:26:34 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>fc913632 - Added helpers, logs</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=12953296&start_sha=0deb6f91e25cb29edd9a88188a4a6c050f4a42ee)
romanov73 commented 2018-04-19 17:11:42 +04:00 (Migrated from gitlab.com)
Copy Link

на будущее, избегайте magic numbers, 13 выносится в говорящую константу

на будущее, избегайте magic numbers, 13 выносится в говорящую константу
romanov73 commented 2018-04-19 17:12:58 +04:00 (Migrated from gitlab.com)
Copy Link

поля класса сверху объявляются, не среди методов

поля класса сверху объявляются, не среди методов
romanov73 commented 2018-04-19 17:13:24 +04:00 (Migrated from gitlab.com)
Copy Link

true&&attempt<tryLimit wtf?

`true&&attempt<tryLimit` wtf?
romanov73 commented 2018-04-19 17:15:05 +04:00 (Migrated from gitlab.com)
Copy Link

второй раз конструкция встречается, значит требует отдельного метода

второй раз конструкция встречается, значит требует отдельного метода
romanov73 commented 2018-04-19 17:16:25 +04:00 (Migrated from gitlab.com)
Copy Link

содержимое for в отдельный метод, код в три раза сократится

содержимое for в отдельный метод, код в три раза сократится
romanov73 commented 2018-04-19 17:18:04 +04:00 (Migrated from gitlab.com)
Copy Link

???
break в цикле - признак, что условие выхода не проработано

??? break в цикле - признак, что условие выхода не проработано
romanov73 commented 2018-04-19 17:19:12 +04:00 (Migrated from gitlab.com)
Copy Link

is suitable = является подходящим
что делает этот метод?

is suitable = является подходящим что делает этот метод?
SemenovaMasha commented 2018-04-28 10:25:39 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#80290298faeadce2f149c5ac80b05a1f32f2d682_46_31)
SemenovaMasha commented 2018-04-28 10:25:39 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#80290298faeadce2f149c5ac80b05a1f32f2d682_51_31)
SemenovaMasha commented 2018-04-28 10:25:40 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#2645d1d25106628a2ec03ed5117881d1f408e85b_23_17)
SemenovaMasha commented 2018-04-28 10:25:40 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#35e1947e2700b3575500d81cc88cf4551e96c9b3_20_24)
SemenovaMasha commented 2018-04-28 10:25:40 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#1c0d5606379cffaf90bb26a05fc56f20f609e678_130_83)
SemenovaMasha commented 2018-04-28 10:25:40 +04:00 (Migrated from gitlab.com)
Copy Link

changed this line in version 3 of the diff

changed this line in [version 3 of the diff](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77#1c0d5606379cffaf90bb26a05fc56f20f609e678_24_24)
SemenovaMasha commented 2018-04-28 10:25:40 +04:00 (Migrated from gitlab.com)
Copy Link

added 1 commit

Compare with previous version

added 1 commit <ul><li>d451c57f - Unified methods</li></ul> [Compare with previous version](https://gitlab.com/romanov73/tis-2017/merge_requests/1/diffs?diff_id=13911609&start_sha=fc913632db749d18261c0716526d23ca185fbf77)
romanov73 commented 2018-05-07 11:09:50 +04:00 (Migrated from gitlab.com)
Copy Link

closed

closed

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: romanov73/tis-2017#2
No description provided.
Delete Branch "Semenova"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?