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

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

Восстановить пароль Регистрация
 
marphey
0 / 0 / 0
Регистрация: 18.04.2011
Сообщений: 11
19.04.2011, 11:06     Создание матричной алгебры, что можно улучшить в коде? #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++ Игра в камень,ножницы бумага.Что можно улучшить\оптимизировать?
Реализовал shared pointer class. В каких местах можно улучшить ? C++
C++ как можно реализовать это в коде?
C++ можно ли создать такой именованый пейп , что по нему можно было передавать данные по сети?
Что не правильно в коде, что посоветуете дописать C++
Объясните что именно делает "x%d, y%d: " или что это в даном коде 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     Создание матричной алгебры, что можно улучшить в коде?
Ответ Создать тему
Опции темы

Текущее время: 12:33. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru