Change History (5)

comment:1 Changed 4 years ago by Артур Гулецкий (huletski)

Основные замечания:

  • такой код падает:
    lab_13::my_array<bool, 2> arr;
    arr.fill(1);
    assert(arr[0] && arr[1]);
    
  • и такой:
    lab_13::my_array<bool, 200> arr;
    arr[100] = true;
    const auto &r_arr = arr;
    assert(r_arr[100]);
    

my_array.h

  • 24: отступы;
  • 73: неясно забыли вы реализовать остальные конструкторы и перемещающий op= или устраивает по умолчанию (не)сгенерированные версии. Если последнее, то нужен хотя бы комментарий поясняющий, что ничего не забыто;
  • 90: зачем кастить data_ к собственному типу?
  • 141: можно ли как-то узнать размер массива использования N?

Баллы: 4, доделывайте.

comment:2 Changed 4 years ago by Артур Гулецкий (huletski)

Type: ожидается проверкаожидаются исправления

comment:3 Changed 4 years ago by stotskiy.nikita

Type: ожидаются исправленияожидается проверка
Version: 1.02.0

comment:4 Changed 4 years ago by Артур Гулецкий (huletski)

Type: ожидается проверкаожидаются исправления

array.h

  • 92: все еще подозрительный каст; неясно зачем принимать data по константному указателю, если класс может менять значение соответствующего элемента;
  • 149: array[i] = val ? ~0 : 0;, явный цикл можно заменить на memset.

Баллы: 9, до 23.03 можно доделывать.

comment:5 Changed 4 years ago by Артур Гулецкий (huletski)

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

Дедлайн.

Note: See TracTickets for help on using tickets.