|
|
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonДенис, ну вот скажи мне, а если у меня будет 100 целых чисел в качестве второго аргумента ты будешь делать 100 блоков if (..) {} ? Тогда, так: Код: java 1. 2. 3. 4. 5. 6. 7. 8. 9. 10. 11. 12. 13. 14. 15. 16. 17. 18. 19. 20. 21. 22. 23. 24. 25. 26. 27. 28. 29. 30. 31. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 12.04.2012, 19:56:32 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Код наиболее последней версии переехал сюда: http://ideone.com/a72cO ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 15.04.2012, 07:59:00 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Denis_m1st, еще не смотрел. Но общее впечатление могу охарактеризовать одним словом. Капец! ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 15.04.2012, 11:19:59 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonDenis_m1st, еще не смотрел. Но общее впечатление могу охарактеризовать одним словом. Капец!Капец - это хорошо или не очень? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 15.04.2012, 13:19:59 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Не очень. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.04.2012, 11:56:25 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonНе очень.Аргументируйте. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.04.2012, 19:44:35 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Denis_m1stmaytonНе очень.Аргументируйте. У тебя есть модульный тест для функции findModNumbers(int[] sourceNumbers, int... args) ? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.04.2012, 19:48:01 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonУ тебя есть модульный тест для функции findModNumbers(int[] sourceNumbers, int... args) ?Что вы подразумеваете под "модульным тестом"? JUnit тесты? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 16.04.2012, 23:42:54 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Да ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 17.04.2012, 00:04:10 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
mayton, я не думаю, что в данном случае целесобразно писать юнит-тесты. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 17.04.2012, 22:23:00 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Ну... целесообразно чтобы написать другой код который будет компактнее твоего протестировать и доказать эквивалетность двух исходников. Я и сам бы мог написать Junit-тест но считаю что это ДОЛЖЕН сделать именно автор т.к. учесть краевые моменты (null,аргументы, отрицательные, переполнения разрядной сетки) можно только в тесте. На словах это описать нереально. Далее будешь сам выбирать. Обычно здравый смысл подсказывает что исходник который короче - поддерживать легче. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 17.04.2012, 22:59:36 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonНу... целесообразно чтобы написать другой код который будет компактнее твоего протестировать и доказать эквивалетность двух исходников. Я и сам бы мог написать Junit-тест но считаю что это ДОЛЖЕН сделать именно автор т.к. учесть краевые моменты (null,аргументы, отрицательные, переполнения разрядной сетки) можно только в тесте. На словах это описать нереально. Далее будешь сам выбирать. Обычно здравый смысл подсказывает что исходник который короче - поддерживать легче.Java, мягко говоря, не самый компактный язык и попытки добиться компактного кода могут привести лишь к потери времени и потери читаемости кода. Насчет юнит-тестов, есть определенные случаи, когда они пишутся, а когда нет. Один из аргументов за то, чтобы не писать - это время, требуемое на их написание. Если оно превышает время на написание самого кода, что ожидаемо в данном случае, то это уже не целесообразно. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2012, 05:03:13 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Не в том дело. У тебя логика местами дублируется. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2012, 12:42:44 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonНе в том дело. У тебя логика местами дублируется.В каких местах? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2012, 21:46:58 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
Какая долгая дискуссия выходит. Может всё таки осилишь тест? ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 18.04.2012, 22:03:43 |
|
||
|
Задачи на числа. Решение. Покритикуйте. (часть #1)
|
|||
|---|---|---|---|
|
#18+
maytonКакая долгая дискуссия выходит. Может всё таки осилишь тест?Если вы издалека начинаете. Тест ради теста - не осилю. ... |
|||
|
:
Нравится:
Не нравится:
|
|||
| 19.04.2012, 15:47:50 |
|
||
|
|

start [/forum/topic.php?fid=59&msg=37754417&tid=2131987]: |
0ms |
get settings: |
11ms |
get forum list: |
11ms |
check forum access: |
4ms |
check topic access: |
4ms |
track hit: |
47ms |
get topic data: |
13ms |
get forum data: |
3ms |
get page messages: |
69ms |
get tp. blocked users: |
2ms |
| others: | 333ms |
| total: | 497ms |

| 0 / 0 |
