Custom Query (307 matches)

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

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

Я сдавал эти задачи в прошлом году, вот ваши ответы: #354: Ha #1

Reporter: deykin.aleksandr | Owner: rutsky

Type: ожидается проверка | Status: closed

Priority: проверка | Milestone: ha1-deadline

Component: HA#1 Матрицы | Version:

Resolution: задача сдана | Keywords:

Changes (by rutsky):

• status: new => closed
• resolution: => задача сдана

Comment:

Решение зачтено.

#389: Ha #2

Reporter: deykin.aleksandr | Owner: rutsky

Type: ожидается проверка | Status: closed

Priority: проверка | Milestone: ha2-deadline

Component: HA#2 Huffman | Version:

Resolution: задача НЕ сдана | Keywords:

Changes (by rutsky):

• status: new => closed
• resolution: => задача НЕ сдана

Comment:

Замечания:

1. Не стоит включать прагму для оптимизации в GCC:

#pragma GCC optimize ("O2")

Уровни оптимизации следует задавать в системе сборки.

2. Не стоит делать using namespace std; в заголовочных файлах.

3. При распаковки сжатого пустого файла вы выводите, что размер дополнительных данных в сжатом файле 2 байта, при том, что сжатый файл у вас размером 0 байт.

4. При сжатии файла, состоящего из одного байта, вы выводите:

1
1
6

хотя размер сжатого файла 8 байт, что не равно 1 + 6.

5. Здесь вы определённо записываете не байт:

            uint16_t byte = 0;
            while(!frequency_table[byte])
            {
                ++byte;
            }

            out.write(reinterpret_cast<char *>(&frequency_table[byte]),
sizeof(frequency_table[byte]));
            out.write(reinterpret_cast<char *>(&byte), sizeof(byte)); //
<--
            data_byte_size += 1;

Хотя достаточно записывать один байт.

Решение неплохое, но учитывая указанные выше замечания и то, что вы не сдали задачу в крайний срок, я не засчитываю ваше решение.

_