Changes between Initial Version and Version 1 of Ticket #174, comment 2
- Timestamp:
- 09/26/17 22:38:19 (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #174, comment 2
initial v1 8 8 1. В методах `Matrix::init_rows` и `Matrix::init_rows_with_zeroes` есть дублирование кода (и блоки кода, их использующие, получаются одинаковыми): можно одно выразить через другое. И они не очень матчатся по названию: кажется, что один отличается от другого константой, а на самом деле он еще какие-то поля меняет. Лучше использовать один метод, который наиболее цело меняет стейт матрицы. 9 9 2. `operator()` ассоциируется скорее с долгими вычислениями, чем с обращением по индексу. Для контейнеров из STL привычен метод `at`. 10 3. И использование одного массива, конечно, очень полезно :)