Домашняя работа 2 Cpp (Матричный калькулятор) by mmikhail2001 · Pull Request #1 · mmikhail2001/techopark_cpp_hw

mmikhail2001 · 2022-10-19T15:23:14Z

Pull Request с домашней работой #2 по C++, реализация матричного калькулятора. Хотелось бы внести ясность касательно следующих сущностей: DVector и DMatrix.

В моей реализации DVector - класс, максимально похожий на std::vector, поэтому нет разделения на вектор-столбец и вектор-строку, это просто обертка над динамическим массивом, последовательно размещенные в памяти элементы. Скалярное произведение вектора на вектор - это всегда вещественное число (не матрица), т.е. vector1.Dot(vector2) имеет возвращаемое значение double.

Умножение вектора на матрицу и матрицу на вектор выполняется следующим образом:

vector.Dot(matrix) - подразумевается, что умножается вектор-строка 1xN и матрица NxM, результат - вектор-строка, тип DVector
matrix.Dot(vector) - матрица MxN умножается на вектор-столбец Nx1, результат - вектор-столбец, тип DMatrix.

Если необходимо умножить вектор на вектор и получить матрицу, стоит использовать matrix.Dot(matrix). Чтобы преобразовать вектор в матрицу (DVector -> DMatrix), нужно воспользоваться конструктором, в котором можно указать, вектор-строка или вектор-столбец интересует.

UPD: Реализован и протестирован класс матрицы DMatrixCompile, являющийся "оберткой" над динамической матрицей DMatrix. Методы этой матрицы проверяют совместимость размеров при выполнении арифметических операций, валидность индекса при обращении к строке, столбцу на этапе компиляции. Если проверка прошла успешно, на этапе выполнения делегируют выполнение метода соответствующему методу динамической матрицы.

Denactive

Прекрасно сделал, мелкие исправления и апрув

Denactive · 2022-10-21T06:15:24Z

main.cpp

@@ -0,0 +1,10 @@
+#include <iostream>


добавь сюда хоть какой-нибудь расчет

Denactive · 2022-10-21T06:15:51Z

matrix_lib/include/dmatrix.h

+
+#include "dvector.h"
+
+enum class  ORIENT  { ROW = 0, COL };


тут енам класс, ниже просто енам. почему?

Denactive · 2022-10-21T06:17:45Z

matrix_lib/include/dmatrix.h

+        throw std::runtime_error("Creation is impossible");
+    }
+    return DMatrix(rows, cols, fill_value);
+}


трушные прогеры оставляют пустую строку в конце файлов. погугли почему.
тут и везде добавь \n

Denactive · 2022-10-21T06:20:05Z

matrix_lib/include/dvector.h

@@ -0,0 +1,83 @@
+#pragma once
+
+#include "stdafx.h"


Зачем тебе stdafx.h в этой программе?

Denactive · 2022-10-21T06:21:14Z

matrix_lib/include/dvector.h

+    ~DVector();
+
+public:
+    const double *CBegin() const;


не обязательно было реализовывать почти все методы от std::vector, но лайк

Denactive · 2022-10-21T07:08:46Z

matrix_lib/include/dmatrix.h

+#include "dvector.h"
+
+enum class  ORIENT  { ROW = 0, COL };
+enum        SLICE   { ROW = 0, COL };


я бы сделал явное указание, чему равен COL

Denactive · 2022-10-21T07:09:28Z

matrix_lib/include/dmatrix.h

+        enum SLICE {ROW = 0, COL}, чтобы 
+            1. не писать класс, если в параметрах ROW / COL
+            2. кастилось к enum SLICE, если в параметрах 0 / 1
+    */


во! это пояснение - наверх

Denactive · 2022-10-21T07:12:11Z

.github/workflows/main.yml

+jobs:
+  build:
+    runs-on: ubuntu-latest
+    container: leshiy1295/gcc_linters_valgrind_cmake_gtest


=)
уникальная возможность поставить все руками упущена

Denactive · 2022-10-21T07:17:20Z

.github/workflows/main.yml

+      - name: launch valgrind
+        run: |
+          cd build
+          valgrind --tool=memcheck --leak-check=yes ./main


проверял, падает ли CI если есть утечка? не уверен точно, надо ли --exit-code прописать
валгрин вызови на тестах. Main у тебя ничего не делает)

сделал кавередж, молодец. Выведи в readme. можешь воспользоваться shields.io

Denactive · 2022-10-21T07:19:50Z

.github/workflows/main.yml

+
+env:
+  LIB_NAME: matrix_lib
+


часто выделяют prepare или beforescript с установкой всего необходимого, check - с линтером, build, test. требовать так делать не буду, особенно больно будет с линтером

Denactive · 2022-10-22T05:44:13Z

matrix_lib/include/dmatrix.h

+DMatrix  operator/(DMatrix left, DMatrix const &right);
+DMatrix  operator*(DMatrix left, DMatrix const &right);
+
+void Print(DMatrix const &matrix, std::string const &msg = std::string{});


оставь эту функции, но добавь еще перегрузку оператора для << (std::cout<<new DMatrix() ) с его вызовом внутри

Denactive · 2022-10-22T17:38:42Z

README.md

+
+#### Домашняя работа 1
+
+##### Мяделец Михаил WEB-11


подпиши ментора, преподавателя

mmikhail2001 · 2022-10-22T22:19:29Z

Исправил замечания.

Добавил простой расчет в main.cpp
Оставил пустую строку в конце каждого файла
Убрал #include "stdafx.h", где он не нужен. stdafx.h - pre-compiled header для ускорения компиляции (в данном проекте он не является необходимостью, но в больших может существенно сэкономить время сборки)
operator[](size_t index) со спецификатором const и возвращающий const reference будет вызываться для const объектов для чтения значения по индексу, без спецификатора const - для изменения значения по индексу
Везде привел инкремент и декремент к префиксному виду
Добавил некоторые поясняющие комментарии к функциям, которые отвечают за умножения вектора на вектор, вектора на матрицу
В функциях для тестирования CompareVectors и CompareMatrices добавил выход по несоответствию размеров
В сценарии непрерывной интеграции добавил valgrind для тестирующего исполняемого файла
Добавил функции для перегрузки оператора << для DMatrix и DVector
Добавил ментора и преподавателя в README.md

Denactive

Молодец, отличная работа. Аппрув.

\n не достает в конфигах таких как main.yml. Видно в интерфейсе этого ПР, если нажать Files changed

Denactive · 2022-10-27T20:28:03Z

Сдал вовремя, первым, за несколько дней до дедлайна. Быстро допилил задание с шаблонами. Исправил замечания. DVector реализует методы std::vector. Предварительно: 10б

Denactive · 2022-10-27T20:41:17Z

matrix_lib/src/dvector.cpp

+{
+    if (begin >= end)
+    {
+        throw std::runtime_error("operator(): " + ERROR_RANGE);  


возможно, ошибки стоит стандартизировать. Например, вынести в .h дефолтные инстансы std::exception. Хотя мне нравится, что юззеру должно быть потенциально понятнее, если ошибка будет явно говорить, где она случилась

mmikhail2001 added 15 commits October 15, 2022 18:00

Basic functionality of DVector and tests

9384bfc

change README

adc7072

Basic functionality of DMatrix and tests

d2f9e43

fix test coverage in ci

6f434b3

fix test coverage in ci 2

4c184d2

fix test coverage in ci 3

9360115

fix test coverage in ci 4

466b687

All tasks are ready except for slices and multidimensionality

e7ee27e

refactoring of code, slices, template static function Create()

7a1bcb1

fix bag in tests

87e2693

fix lcov in CMakeLists.txt - tests

2348a0f

fix ci

468a1ee

fix ci 2

6120b26

fix ci 3

c6adc9b

fix ci 4

ed016ec

Denactive self-assigned this Oct 19, 2022

mmikhail2001 requested a review from Denactive October 19, 2022 20:16

Compile-time Matrix implemented

7d24242

Denactive reviewed Oct 21, 2022

View reviewed changes

Denactive reviewed Oct 22, 2022

View reviewed changes

fixed comments from PR

daa02ba

Denactive approved these changes Oct 27, 2022

View reviewed changes

Denactive reviewed Oct 27, 2022

View reviewed changes


		#### Домашняя работа 1

		##### Мяделец Михаил WEB-11

Conversation

mmikhail2001 commented Oct 19, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Denactive left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mmikhail2001 commented Oct 22, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Denactive left a comment

Choose a reason for hiding this comment

Uh oh!

Denactive commented Oct 27, 2022

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mmikhail2001 commented Oct 19, 2022 •

edited

Loading

mmikhail2001 commented Oct 22, 2022 •

edited

Loading