task07 Comparison(Reuk)

[GoogleCodeExporter]

reuk_kirill\task07_Comparison

Original issue reported on code.google.com by jiraff...@gmail.com on 10 Nov 2012 at 3:44

[GoogleCodeExporter]

1. В каких единицах Вы выводите время? 
Сделайте, пожалуйста, более юзер-френдли 
вывод.
Например так:
quicksort: correct=yes run_time=100ms

2. При измерении времени работы каждого 
алгоритма Вы включаете время генерации 
рандомного массива, а также время проверки. 
Это некорректно.

3. Создавать вспомогательный массив для 
mergesort извне функции сортировки - плохой 
стиль.

4. Где результаты Ваших измерений? Где 
график?

Решение не принято.

Original comment by aivyu...@gmail.com on 11 Nov 2012 at 5:04

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Original comment by kirillr...@gmail.com on 18 Nov 2012 at 4:27

• Changed state: Fixed

[GoogleCodeExporter]

1. Таки не вижу единиц, в которых Вы выводите 
время.

2. Включать измерение времени в алгоритмы 
сортировки - странное решение. Кажется, что 
это независимые вещи. И измерение должно 
производиться извне.

3. Утечки памяти.

4. Таблица с результатами измерений + график?

Решение не принято.

Original comment by aivyu...@gmail.com on 25 Nov 2012 at 11:43

• Changed state: Wrong-or-Errors

[GoogleCodeExporter]

Original comment by kirillr...@gmail.com on 23 Dec 2012 at 11:43

• Changed state: Fixed

[GoogleCodeExporter]

Все таки clock()/CLOCKS_PER_SEC - время в секундах, а 
не в миллисекундах.

Решение принято.

Минус 3 балла.

Original comment by aivyu...@gmail.com on 23 Dec 2012 at 6:09

• Changed state: Done