Удаление указателя класса C ++ (проблема с ошибкой сегментации) - PullRequest
Новая
       15

Удаление указателя класса C ++ (проблема с ошибкой сегментации)

0 голосов
/ 09 июля 2010

решено

Что ж, похоже, я получаю ошибку сегментации, когда пытаюсь удалить свой указатель класса.Я много чего перепробовал, но пока не получилось.В любом случае, вот мой main.cpp: (эта ошибка сегментации возникает, когда я удаляю 'm') 

    /*
 * Copyright(c)Fellixombc 2010
 *
 * TextToSqlRs is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * TextToSqlRs is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with TextToSqlRs.  If not, see <http://www.gnu.org/licenses/>.
 */

#include <ctime>
#include <iostream>
#include <unistd.h>
#include <string>
#include "eMysql.h"
#include "manage.h"

using namespace std;

int main() {
    time_t start, stop;
    double TIME;
    string host, user, password, database;
    string path;
    bool run = true;
    int runtime;
    int MAX;
    int max;


    cout << "Please enter the required information." << endl << "Absolute path to character folder: ";
    cin >> path;
    cout << "MySql Database Host: ";
    cin >> host;
    cout << "MySql Username: ";
    cin >> user;
    cout << "MySql Password: ";
    cin >> password;
    cout << "Database name: ";
    cin >> database;

    start = clock();
    eMysql* m = new eMysql;
    Manage* manage = new Manage;
    m->connect(host.c_str(), user.c_str(), password.c_str(), database.c_str());
    manage->ReadDir(path);
    manage->TextToSql(m);

    delete manage;
    delete m;

    stop = clock();

    TIME = ((double)stop - (double)start)/CLOCKS_PER_SEC;
    cout << "Finished in " << TIME << " seconds" << endl; }

Деконструктор eMysql: 

eMysql::~eMysql() {
mysql_free_result(result);
mysql_close(&mysql);

if(lengths) delete lengths;
if(result) delete result; }

mysql / result header: 

#ifndef _MYSQL_H
#define _MYSQL_H

#include <iostream>
#include <stdlib.h>
#include <string.h>
#include <fstream>
#include <mysql/mysql.h>
#include <string>

using namespace std;

class eMysql {
public:
    eMysql();
    int connect(const string, const string, const string, const string);
    int query(string query);
    int strip(string&);
    int rows();
    virtual ~eMysql();
private:
    MYSQL mysql;
    MYSQL_RES *result;
    MYSQL_ROW row;
    unsigned int i;
};

#endif  /* _MYSQL_H */

инициализатор mysql 

eMysql::eMysql() {
    mysql_init(&mysql);
}

инициализатор результата 

int eMysql::query(string query) {
    if(mysql_query(&mysql, query.c_str())) {
        cout << "Failed to run query: " << mysql_error(&mysql) << endl;
        exit(1);
    }
    result = mysql_store_result(&mysql);
    query.erase();
    return 1;
}

Ответы [ 2 ]

1 голос
/ 09 июля 2010

Поскольку вы никогда не вызываете query, вы работаете с недопустимым указателем result. В конструкторе eMysql инициализируйте result до NULL: 

eMysql::eMysql() : result(NULL) {
  // ...
}

Тогда в деструкторе: 

eMysql::~eMysql() {
  if (result) mysql_free_result(result);
  mysql_close(&mysql);
}

В деструкторе одна вещь выглядит неправильно: 

eMysql::~eMysql() {
  // You use "result" here, but later check if it's not NULL.
  // You should only free if it's not null.
  mysql_free_result(result);
  mysql_close(&mysql);

  if(lengths) delete lengths;
  // Are you sure you need to delete this?  Isn't that what the
  // mysql_free_result is for?  This will still be a non-NULL ptr
  // that has already been free'd.  This might be a double-delete
  // which would likely manifest as a segmentation fault.
  if(result) delete result;
}

Но могут быть и другие проблемы.

  • Приобретает ли TextToSQL m?
  • Действительны ли result и mysql в ~eMysql?
  • Выделяется ли lengths как массив? Удалить с delete [] lengths;
  • Полагаю, это ученики, они вообще инициализированы?

Вот несколько способов отладки проблем такого типа (в Linux):

  • Запустите valgrind , чтобы проверить наличие недействительных ссылок на память.
  • Запустите gdb , введите bt при сбое.
1 голос
/ 09 июля 2010

Я не вижу кода, который инициализирует lengths или result, поэтому вы можете удалять дикие указатели.

...