Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
marphey
0 / 0 / 0
Регистрация: 18.04.2011
Сообщений: 11
#1

Создание матричной алгебры, что можно улучшить в коде? - C++

19.04.2011, 11:06. Просмотров 323. Ответов 1
Метки нет (Все метки)

Дан код программы на C++ создающий классы матриц и методы для работы с ними. Программа работает.
Дано так же задание улучшить код, т.е. произвести некоторые доработки, что сделает код более корректным.
У самого времени мало, так что прошу помощи
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
#include <cstdlib>
#include <iostream>
 
using namespace std;
 
void halt(int ErrNo) {
  cout << "Ошибка - недопустимая операция.\n";
  exit(ErrNo);
}
 
class Array {
  int **p;
public:
  unsigned char HSize, VSize;
  Array(unsigned char, unsigned char);
  Array(const Array&);
  int* operator[](unsigned char) const;
  Array& operator=(const Array&);
  Array& operator=(int*);
  Array& operator+=(const Array&);
  Array operator+(Array) const;
  Array operator-() const;
  Array operator-(Array) const;
  Array operator*(const Array&) const;
  Array operator*(int) const;
  friend Array operator*(int, Array);
  ~Array();
};
 
Array::Array(unsigned char n, unsigned char m) {
  VSize = n;
  HSize = m;
  p = new int*[VSize];
  for (int i = 0; i < VSize; i++)
    p[i] = new int[HSize];
}
 
Array::Array(const Array& Data) {
  HSize = Data.HSize;
  VSize = Data.VSize;
  p = new int*[VSize];
  for (int i = 0; i < Data.VSize; i++) {
    p[i] = new int[Data.HSize];
    for (int j = 0; j < Data.HSize; j++)
      p[i][j] = Data.p[i][j];
  }
}
 
Array::~Array() {
  for (int i = 0; i < VSize; i++)
    delete [](p[i]);
  delete []p;
}
 
int* Array::operator[](unsigned char i) const {
  if (i >= VSize)
    halt(2);
  return p[i];
}
 
Array& Array::operator=(int* InitData) {
  for (int i = 0; i < VSize; i++)
    for (int j = 0; j < HSize; j++)
      p[i][j] = InitData[i*HSize + j];
  return *this;
}
 
Array& Array::operator=(const Array& Data) {
  if (VSize == Data.VSize && HSize == Data.HSize)
    for (int i = 0; i < VSize; i++)
      for (int j = 0; j < HSize; j++)
        p[i][j] = Data.p[i][j];
  else
    halt(1);
  return *this;
}
 
Array Array::operator-() const {
  Array T(*this);
  for (int i = 0; i < VSize; i++)
    for (int j=0; j < HSize; j++)
      T.p[i][j] = -p[i][j];
  return T;
}
 
Array& Array::operator+=(const Array& Data) {
  if (VSize == Data.VSize && HSize == Data.HSize)
    for (int i = 0; i < VSize; i++)
      for (int j = 0; j < HSize; j++)
        p[i][j] += Data.p[i][j];
  else
    halt(1);
  return *this;
}
 
Array Array::operator+(Array Data) const {
  if (VSize == Data.VSize && HSize == Data.HSize)
    for (int i = 0; i < VSize; i++)
      for (int j = 0; j < HSize; j++)
        Data.p[i][j] += p[i][j];
  else
    halt(1);
  return Data;
}
 
Array Array::operator-(Array Data) const {
  if (VSize == Data.VSize && HSize == Data.HSize)
    for (int i = 0; i < VSize; i++)
      for (int j = 0; j < HSize; j++)
        Data.p[i][j] = p[i][j] - Data.p[i][j];
  return Data;
}
 
Array Array::operator*(const Array& Data) const {
  Array T(VSize, Data.HSize);
  if (HSize == Data.VSize)
    for (int i = 0; i < T.VSize; i++)
      for (int j = 0; j < T.HSize; j++) {
        T.p[i][j] = 0;
        for (int k = 0; k < HSize; k++)
          T.p[i][j] += p[i][k]*Data.p[k][j];
      }
  else
    halt(1);
  return T;
}
 
Array Array::operator*(int k) const {
  Array T(VSize,HSize);
  for (int i = 0; i < VSize; i++)
    for (int j = 0; j < HSize; j++)
      T.p[i][j] = k*p[i][j];
  return T;
}
 
Array operator*(int k, Array Data) {
  for (int i = 0; i < Data.VSize; i++)
    for (int j = 0; j < Data.HSize; j++)
      Data.p[i][j] *= k;
  return Data;
}
 
ostream& operator<<(ostream& s, const Array& Data) {
  s << endl;
  for (int i = 0; i < Data.VSize; i++) {
    for (int j = 0; j < Data.HSize; j++)
      s << '\t' << Data[i][j];
      s << endl;
  }
  s << endl;
  return s;
}
 
int main() {
  Array A(3,3), B(3,3), C(3,2), D(3,3), E(3,3), G(2,3);
  {
    int data[] = {0,1,2,3,4,3,2,1,0};
    B = A = data;
    C = data + 2;
    G = data + 3;
  }
  Array F(A);
  for (int i = 0; i < 3; i++)
    for (int j = 0; j < 3; j++)
      E[i][j] = i == j;
  B += E;
  D = F - E;
  C[2][1] = -5;
  D = -(A*A - 4*B*C*G + E*12)*D;
  cout << "A = F =" << A << "B = " << B << "C = " << C << "G =" << G << "D = -(A*A - 4*B*C*G + E*12)*D = " << D;
  return 0;
} // D = -(A*A - 4*B*C*G + E*12)*D =
// 579 600 435
// 1836 2346 2052
// 563 760 707
Добавлено через 14 часов 18 минут
Неужели никто не может помочь?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
19.04.2011, 11:06     Создание матричной алгебры, что можно улучшить в коде?
Посмотрите здесь:
Какие ошибки есть в коде и что можно улучшить? C++
Калькулятор, что можно улучшить/переработать C++
Эмулятор Машины Тьюринга: посоветуйте, что можно добавить/улучшить C++
C++ Игра в камень,ножницы бумага.Что можно улучшить\оптимизировать?
Описать структуру с именем TIMETABLE (найти и исправить ошибки в коде, улучшить код) C++
Чем можно улучшить прогу!Дополните) C++
Парсер математических выражений - можно ли оптимизировать и улучшить код C++
Реализовал shared pointer class. В каких местах можно улучшить ? C++
C++ Как разобраться ,что в этом коде за что отвечает?
Что не правильно в коде, что посоветуете дописать C++
C++ можно ли создать такой именованый пейп , что по нему можно было передавать данные по сети?
C++ как можно реализовать это в коде?

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
marphey
0 / 0 / 0
Регистрация: 18.04.2011
Сообщений: 11
21.04.2011, 12:00  [ТС]     Создание матричной алгебры, что можно улучшить в коде? #2
Спасибо всем, кто откликнулся, сам разобрался
1. для полноты матричной алгебры:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
// унарный плюс
Array Array::operator+() const {
    Array T(*this);
  for (int i = 0; i < VSize; i++)
    for (int j=0; j < HSize; j++)
      T.p[i][j] = +p[i][j];
  return T;
}
 
// равно
bool Array::operator==(const Array& Data) const {
  for (int i = 0; i < VSize; i++)
    for (int j=0; j < HSize; j++)
      if (Data.p[i][j] != p[i][j])
    return false;
  return true;
}
 
// не равно
bool Array::operator!=(const Array& Data) const {
  for (int i = 0; i < VSize; i++)
    for (int j=0; j < HSize; j++)
      if (Data.p[i][j] != p[i][j])
    return true;
  return false;
}
2. для предотвращения прямого доступа к размерам матрицы перенесем их в приватную область класса и напишем функции, возвращающие их значение:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
class Array {
  int **p;
  unsigned char HSize, VSize;
public:
  unsigned char GetHSize() const;
  unsigned char GetVSize() const;
  Array(unsigned char, unsigned char);
  Array(const Array&);
  int* operator[](unsigned char) const;
  Array& operator=(const Array&);
  Array& operator=(int*);
  Array& operator+=(const Array&);
  Array operator+(Array) const;
  Array operator-() const;
  Array operator+() const;
  Array operator-(Array) const;
  Array operator*(const Array&) const;
  Array operator*(int) const;
  bool operator==(const Array&) const;
  bool operator!=(const Array&) const;
  friend Array operator*(int, Array);
  ~Array();
};
 
unsigned char Array::GetHSize() const {
  return HSize;
}
 
unsigned char Array::GetVSize() const {
  return VSize;
}
3. переопределить [][] для обращения напрямую к элементу матрицы:

с этим я не разобрался, так что если кто-то все еще читает эту тему, помогите с решением проблемы
пысы: как вариант возможно переопределение ()()

заранее спасибо.
Yandex
Объявления
21.04.2011, 12:00     Создание матричной алгебры, что можно улучшить в коде?
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru