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

Конструктор-копировщик - C++

Восстановить пароль Регистрация
 
akaAxeL
66 / 66 / 30
Регистрация: 14.07.2013
Сообщений: 251
15.08.2013, 13:37     Конструктор-копировщик #1
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Matrix::Matrix(const Matrix & rhs)
{
    itsNumOfStr = rhs.itsNumOfStr;
    itsNumOfCol = rhs.itsNumOfCol;
    //cout << "CONTRRRRRRRR\n";
    A = new double *[itsNumOfStr];
    for (long i = 0; i < itsNumOfStr; i++)
        A[i] = new double [itsNumOfCol];
 
 
    for (long m = 0; m < itsNumOfStr; m++)
    {
        for (long n = 0; n < itsNumOfCol; n++)
        {
            A[m][n] = rhs.A[m][n];
        }
    }
}
конструктор копировщик для матрицы.
C++
1
2
3
4
5
6
Matrix Matrix::operator *( const Matrix & rhs)
{
    Matrix C(this->itsNumOfStr, rhs.itsNumOfCol);
    Start(C,*this, rhs);
    return C;
}
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
void Matrix::Start(Matrix NAME1, Matrix NAME2, Matrix NAME3)
{
    vector<boost::thread> threads;
    int NumberOfThreads;
    cout << "How many threads: ";
    cin >> NumberOfThreads;
    for (int i = 0; i <= NumberOfThreads; i++)
    {
        threads.push_back(boost::thread(&Matrix::multip, &NAME1, NAME2, NAME3,NumberOfThreads, i ));
    }
    for (vector<boost::thread>::iterator t=threads.begin();t!=threads.end();t++)
    t->join();
}
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void Matrix::multip(Matrix name1, Matrix name2,int threads, int num)
{
    for (long m = 0; m < name1.itsNumOfStr; m++)
    {
        if (m % threads == num)
        { 
            for (long p = 0; p < name2.itsNumOfCol; p++)
            {
                this->A [m] [p] = 0;
                for (long n = 0; n < name2.itsNumOfStr; n++)
                {
                    this->A [m] [p] += (name1.A[m] [n]) * (name2.A[n] [p]);
                } 
            }
        }
    } 
    return;
}
Вообщем матрицы перемножаются, не правильно. Если убрать копировщик, то все работает правильно. Из-за чего это может быть, ну или можно сделать так чтобы копировщик не вызывался при Start() и при операторе *.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Fyret
184 / 170 / 13
Регистрация: 30.07.2013
Сообщений: 359
15.08.2013, 13:52     Конструктор-копировщик #2
Потому матрицы передаются в функции по значению (т.е. постоянно вызывается конструктор копирования), и результат умножения записывается во временный объект, который уничтожается, как только функция закончила свою работу.
akaAxeL
66 / 66 / 30
Регистрация: 14.07.2013
Сообщений: 251
15.08.2013, 14:24  [ТС]     Конструктор-копировщик #3
А как этого избежать?
I.M.
 Аватар для I.M.
564 / 547 / 5
Регистрация: 16.12.2011
Сообщений: 1,389
15.08.2013, 17:09     Конструктор-копировщик #4
передавать матрицы по ссылке
Yandex
Объявления
15.08.2013, 17:09     Конструктор-копировщик
Ответ Создать тему
Опции темы

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