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

с free в delete - C++

Восстановить пароль Регистрация
 
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
25.02.2013, 19:38     с free в delete #1
Как эту функцию можно переписать с помощью delete?
C++
1
2
3
4
5
6
7
void A(int **p,int r){
     int **pr;
     
     for(pr = p; pr < pr + r; tr++)
         free(*pr);
     free(p);
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
25.02.2013, 19:38     с free в delete
Посмотрите здесь:

C++ [C++]Оператор delete(ошибка) - free(): invalid next size (fast): 0x098f7008 ***
"delete [] a, b;" эквивалентно "delete [] a; delete [] b;"? C++
C++ delete[]
new - delete C++
delete[] *pointer vs. delete pointer и утечка памяти C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kastaneda
Модератор
Эксперт С++
 Аватар для Kastaneda
4236 / 2769 / 218
Регистрация: 12.12.2009
Сообщений: 7,104
Записей в блоге: 1
Завершенные тесты: 1
25.02.2013, 21:19     с free в delete #2
Вообще плохая идея менять free() на delete, т.к. если используется free(), значит память выделялась при помощи [c/m]alloc(), поэтому освобождение ее при помощи delete может быть не корректным.
К тому же в коде ошибка - получается вечный цикл.
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
25.02.2013, 21:36  [ТС]     с free в delete #3
Я в курсе что использование calloc и delete несовместимо, поэтому я переделал чтобы память выделялась за помощью new. А вот чтобы удалить память с помощью delete не получается. Об ошибке в цикле я тоже заметил, но суть в том, как мне удалить память с помощью delete?
C++
1
2
3
4
5
6
7
void A(int **p,int r){
     int **pr;
     
     for(pr = p; pr < p + r; pr++)
         free(*pr);
     free(p);
}
Kastaneda
Модератор
Эксперт С++
 Аватар для Kastaneda
4236 / 2769 / 218
Регистрация: 12.12.2009
Сообщений: 7,104
Записей в блоге: 1
Завершенные тесты: 1
25.02.2013, 21:49     с free в delete #4
Вот так
C++
1
2
3
     for(pr = p; pr < p + r; pr++)
         delete [] (*pr); // я так понимаю, что p - это двумерный массив
     delete [] p;
gazlan
2861 / 1809 / 272
Регистрация: 27.08.2010
Сообщений: 4,897
Записей в блоге: 1
25.02.2013, 22:37     с free в delete #5
Трудно понять, что вам нужно. Но если не это - действуйте по аналогии.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
void A(int** pInts)
{
   delete *pInts;
   *pInts = NULL;      
}
 
int main(int argc,char** argv)
{
   const int DIMENSION = 128;
 
   int*  pInts = new int[DIMENSION];
 
   memset(pInts,0,sizeof(int) * DIMENSION);
 
   for (int ii = 0; ii < DIMENSION; ++ii)
   {
      pInts[ii] = ii;
   }
 
   A(&pInts);
 
   return 0;
}
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
25.02.2013, 22:48  [ТС]     с free в delete #6
Есть вот такая программа, она была написана на Си и я решил переписать на С++, программа работает, но в конце рушится, пишет что повреждение кучи. Сколько в тот код не смотрел ошибки найти не могу ..
Кликните здесь для просмотра всего текста
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
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
#include <iostream>
#include <math.h>
#include <time.h>
#include <stdio.h>
#include <fstream>
#include <cstdlib>
 
using namespace std;
 
void InputDimenstion();
double** CreateArray();
void PrintMatrix(char* info,double *matr[],int print_sum);
void SortRows(double**matr);
void WriteMatrix(double *pmatrix[]);
void WriteSum(double *pmatrix[]);
void DeleteArray(double**matr,int rdel);
double EnergyAlphaParticle(double);
double ParticleCharge();
 
const double H = 1;
const double Q = 1e4;
const double Epsilon = 1e-4;
const double Alpha = 1e-9;
const double PI = 3.1415926;
const double Z = 1;
const double R1 = 0.1;
const double Beta = 1e-3;
 
const int RAND_INIT = 10;
 
const char fn_matrix[] =  "matrix.txt";
const char fn_sum[] =  "sum.txt";
 
int nrows, ncolms;
 
int main()
{
    double **parray;
    InputDimenstion();
    parray = CreateArray();
    if(parray == NULL)
        return 1;
 
    PrintMatrix("Matrix energy:",parray,false);
    WriteMatrix(parray);
 
    SortRows(parray);
    PrintMatrix("Sort matrix:",parray,true);
    WriteSum(parray);
 
    DeleteArray(parray,nrows);
    
    system("pause>0");
    return 0;
}
 
void InputDimenstion()
{
    cout<<"Matrix dimension[MxN] - ";
    cin>>nrows>>ncolms;
}
 
double **CreateArray()
{
    double **pmatrix;
    int k,j;
    double delta;
 
    pmatrix = new double*[nrows];
    if(!pmatrix){
        cerr<<"Error!";
        return NULL;
    }
//  srand(time(0));
    for(k=0;k<nrows;++k){
        pmatrix[k] = new double[ncolms];
 
        if(pmatrix[k] == NULL){
            DeleteArray(pmatrix,k);
            return NULL;
        }
 
        for(j = 0;j < ncolms; ++j){
            delta = static_cast<double>(j+1) / ncolms;
            pmatrix[k][j] = EnergyAlphaParticle(delta);
        }
    }
    return pmatrix;
}
 
void PrintMatrix(char* info,double *pmatrix[],int print_sum)
{
    double **prow, *pnode;
    int k,j;
 
    cout<<info<<endl;
 
    for(k = 0, prow = pmatrix; k < nrows; ++k, ++prow,cout<<endl){
        for(j = 0, pnode = *prow; j < ncolms; ++j, ++pnode){
            cout<<(*pnode)<<" ";
        }
        if(print_sum)
            cout<<(*(*prow + ncolms))<<" ";
    }
}
 
void WriteMatrix(double *pmatrix[])
{
    double **prow, *pnode;
    int k,j;
 
    ofstream f1("matrix.txt",ios::in);
    //f1 = fopen(fn_matrix,"w");
    if(!f1){
        cerr<<"Cant open file!!!"<<endl;
        return;
    }
    for(k = 0, prow = pmatrix;k < nrows; ++k, ++prow){
        for(j = 0,  pnode = *prow; j < ncolms ; ++j, ++pnode){
            f1 << (*pnode);
        }
        f1 << endl;
    }
    f1.close();
}
 
void WriteSum(double *pmatrix[])
{
    double **prow;
    int k;
 
    ofstream f1(fn_sum,ios::in);
    if(!f1){
        cerr<<"Cant open file!!!"<<endl;
        return;
    }
    for(k = 0, prow = pmatrix; k < nrows; ++k, ++prow){
        f1 << (*(prow + ncolms));
    }
    f1.close();
}
 
void SortRows(double **pmatrix)
{
    double **prow, **prow2;
    double *pcell, *psum;
    double *tmp;
 
    for(prow = pmatrix; prow < pmatrix + nrows; ++prow)
        for(pcell = *prow, psum = *prow + ncolms; pcell < psum; ++pcell)
            *psum += *pcell;
 
    for(prow = pmatrix; prow < pmatrix + nrows - 1; ++prow)
        for(prow2 = prow + 1; prow2 < pmatrix + nrows; ++prow2){
            if((*prow)[ncolms] > (*prow2)[ncolms]){
                tmp = *prow;
                *prow = *prow2;
                *prow2 = tmp;
            }
        }
}
 
void DeleteArray(double **pmatrix, int rows)
{
    double **prow;
    
    for(prow = pmatrix; prow < pmatrix + rows; ++prow)
        delete [] (*prow);
    
    delete pmatrix;
}
 
double EnergyAlphaParticle(double Z)
{
    double W;
    double charge = ParticleCharge();
 
    W = charge * Q / (2 * PI * Epsilon * H) * log(Z / R1);
 
    return W;
}
 
double ParticleCharge()
{
    double charge;
    double factor = rand();
 
    charge = (Beta * factor) * Alpha;
 
    return charge;
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
25.02.2013, 23:06     с free в delete
Еще ссылки по теме:

C++ free
New/Delete C++
New delete C++

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

Или воспользуйтесь поиском по форуму:
Issues
429 / 364 / 37
Регистрация: 06.08.2012
Сообщений: 961
25.02.2013, 23:06     с free в delete #7
The_bolT, ошибка в функции DeleteArray
Yandex
Объявления
25.02.2013, 23:06     с free в delete
Ответ Создать тему
Опции темы

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