Change History (4)

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

Owner: changed from Evgeny Linsky to Артур Гулецкий (huletski)
Status: newassigned

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

Решение не собирается:

{lab_14}[2020]$ pwd && svn up && svn status
/home/hfx/dvl/cpp19/kurnosova.polina/lab_14
Updating '.':
At revision 3294.
{lab_14}[2021]$ make
mkdir obj
g++ -Iinclude -Wall -Wextra -Werror -c src/matrix.cpp -o obj/matrix.o
src/matrix.cpp: In function ‘Matrix& operator+=(Matrix&, const Matrix&)’:
src/matrix.cpp:81:13: error: implicitly-declared ‘Matrix& Matrix::operator=(const Matrix&)’ is deprecated [-Werror=deprecated-copy]
   81 |     a = a + b;
      |             ^
src/matrix.cpp:24:1: note: because ‘Matrix’ has user-provided ‘Matrix::Matrix(const Matrix&)’
   24 | Matrix::Matrix(const Matrix &other)
      | ^~~~~~
src/matrix.cpp: In function ‘Matrix& operator*=(Matrix&, const Matrix&)’:
src/matrix.cpp:116:13: error: implicitly-declared ‘Matrix& Matrix::operator=(const Matrix&)’ is deprecated [-Werror=deprecated-copy]
  116 |     a = a * b;
      |             ^
src/matrix.cpp:24:1: note: because ‘Matrix’ has user-provided ‘Matrix::Matrix(const Matrix&)’
   24 | Matrix::Matrix(const Matrix &other)
      | ^~~~~~
cc1plus: all warnings being treated as errors
Makefile:7: recipe for target 'obj/matrix.o' failed
make: *** [obj/matrix.o] Error 1
{lab_14}[2022]$ g++ -v
Using built-in specs.
COLLECT_GCC=g++
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/9/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none:hsa
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-linux-gnu
Configured with: ../src/configure -v --with-pkgversion='Ubuntu 9.2.1-9ubuntu2~16.04.1' --with-bugurl=file:///usr/share/doc/gcc-9/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --program-suffix=-9 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-plugin --with-system-zlib --with-target-system-zlib=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none,hsa --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Thread model: posix
gcc version 9.2.1 20191008 (Ubuntu 9.2.1-9ubuntu2~16.04.1) 

Общие замечания:

  • реализуйте хранение элементов матрицы на сырых указателях (без RAII контейнеров);
  • логику чтения/записи матрицы вынесите в перегруженные op<<, op>>;
  • что если в загружаемом файле указано только число строк матрицы?
  • почему тип аргументов getElem/setElem signed, а не std::size_t, например?
  • с т.з. производительности лучше реализовать + через +=.

Баллы: 1, чините компиляцию, доделывайте.

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

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

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

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

Дедлайн.

Note: See TracTickets for help on using tickets.