C++内存泄漏姿势之——eof()和delete释放内存

首先要说，用codeblocks的同学请抓紧卸载换到Visual Studio，好的IDE可以让你事半功倍！！

先说eof()，教训就是这玩意他并不会在读到最后一行之后就返回null值，他还会继续往后读，因为文件结束符是最后一个字符的下一个字符（是不是有点像字符串？），详见这篇博客：https://blog.csdn.net/rebel_321/article/details/4927464

贴代码：

1 void Graph::InputListGene(bool TOG,int nbNodes,ifstream& f){
2     string* line = new string[nbNodes];
3     int count =0;
4     while(!f.eof()){
5         if(count >=nbNodes) break;//the eof character is the one after the last character
6         getline(f,line[count],';');
7        cout<<line[count]<<endl;
8         count++;
9     }

如果没有第5行的if判断，第6行到最后一个循环会出现数组访问越界。

另外关于delete，这个东西吧，一定要记住它叫做“释放内存”，我个人理解为取消指针与其当前指向位置的关联。所以，并不是delete之后指针就空了，指针不会自己变成null，必须在delete后赋值null。

贴代码：

 1 int id=p;
 2             if(!TOG){//undirected
 3                 for (auto iter = listEdges.begin(); iter != listEdges.end(); iter++) {//to detect if the new edge is redundant
 4                     if ((*iter)->destination == e->source && (*iter)->source == e->destination) { //redundance detected
 5                         id = (*iter)->getID();//the id of an edge should be unique
 6                         //cout << "e id before del:" << e->id << endl;
 7                         p--;
 8                         delete e;
 9                         //cout << "e id :" << e->id << endl;
10                         e = NULL;
11                         break;
12                     }
13                     else
14                         id = p;
15                 }
16             }
17             if (e) {//if the new edge is not redundant 
18                 cout << "true,id is " <<e->id<< endl;
19                 cout << "Edge_Id:" << e->id << " src:" << e->source->id << " dest:" << e->destination->id << " weight:" << e->cost << endl;
20                 listEdges.push_back(e);
21             }
22             listVertex[i]->nextEdgeNode.push_back(make_pair(dest,id));
23             p++;
24             delete[] cstr;
25         }

主要是第4行通过if判断排除掉不需要加入到listEdge里面的Edge对象，这里如果缺少第10行的话，第19行会报错，因为e一直是非空指针，17行并没有实现本来的判断null，相当于没有这个if。

C++内存泄漏姿势之——eof()和delete释放内存

猜你喜欢