Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#86 closed ожидается проверка (задача сдана)

HA#1 HA#2

Reported by: Aleksandr Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#2 huffman Version:
Keywords: Cc:

Description

Доброе утро, простите мне пожалуйста три часа, я прокопался с 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")

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

  1. Не стоит делать using namespace std; в заголовочных файлах.
  1. При распаковки сжатого пустого файла вы выводите, что размер дополнительных данных в сжатом файле 2 байта, при том, что сжатый файл у вас размером 0 байт.
  1. При сжатии файла, состоящего из одного байта, вы выводите:
1
1
6

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

  1. Здесь вы определённо записываете не байт:
            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;

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

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

_

Change History (2)

comment:1 Changed 7 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

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

comment:2 Changed 7 years ago by Vladimir Rutsky

Milestone: ha2-deadline

Milestone ha2-deadline deleted

Note: See TracTickets for help on using tickets.