困在链表上的多个类实现

时间:2015-02-09 23:57:08

标签: c++ class linked-list

我已经在这个项目上工作了好几天。该项目包含3个类。第一个是存储DNA对象的DNA类。第二个是数据库类,它读取文件并解析命令和数据并相应地处理它。最后一个是DNA列表类,它是一个指向DNA对象的节点的链接列表。

我已经完成了我的链表构建方法。它必须是一个push_back方法,在列表的末尾添加节点。当我尝试在列表中搜索某个节点时出现问题。这必须是一种方法,如果列表中存在具有id的DNA对象,则返回DNA *;否则返回NULL。

我的计划是使用此方法打印并删除节点。我似乎无法使用此方法。显然,我的指针有些不稳定。我花了几个小时来实现我的push_back方法。这是我的代码。任何指导或帮助表示赞赏。

DNA.h

#ifndef DNA_H
#define DNA_H

#include <string>

class DNA{  
public:
   // overloaded constructor for DNA class
   DNA(std::string, int, std::string, int, int);
   // print function
   void print();
   int getID();

private:
   std::string m_label;      // variable to hold label
   int m_id;                    // variable to hold id
   std::string m_sequence;  // variable to hold sequence
   int m_length;                // variable to hold length
   int m_index;             // variable to hold index
};
#endif

DNA实施

#include "DNA.h"
#include <iostream>
#include <string>

using namespace std;

DNA::DNA(string label, int id, string sequence, int length, int index){
    m_label = label;
    m_id = id;
    m_sequence = sequence;
    m_length = length;
    m_index = index;
}

void DNA::print(){
     cout << "DNA:" << '\t' << "label: " << m_label << '\t' << "ID: " << m_id << '\t' << "Sequence: " << m_sequence << endl << "Length: " << m_length << '\t' << "cDNAStartIndex: " << m_index << endl << endl;
}

int DNA::getID(){
    return m_id;
}

数据库类

#ifndef SEQUENCEDATABASE_H
#define SEQUENCEDATABASE_H

#include <string>
#include <fstream>
#include "DNA.h"
#include "DNAList.h"

class SequenceDatabase {
public:
    SequenceDatabase();
    // function to import the data file, parse the data, and perform the required output
    void importEntries(std::string);
private:
    DNAList list;

};
#endif 

数据库实施

#include "SequenceDatabase.h"
#include "DNA.h"
#include "DNAList.h"
#include <iostream>
#include <fstream>
#include <string>

using namespace std;

SequenceDatabase::SequenceDatabase(){

    DNAList list;
}
// function reads in the filename creates a data stream and performs the requested actions
void SequenceDatabase::importEntries(string inputFile){
    ifstream dnaFile(inputFile);
    char command;
    string label, sequence;
    int id, length, index;
    while(dnaFile >> command){
        DNA* p;
        if(command == 'D'){
            dnaFile >> label >> id >> sequence >> length >> index;
            DNA data(label, id, sequence, length, index);
            p = new DNA(label, id, sequence, length, index);
            list.push_back(p);
        }
        if(command == 'P'){
        dnaFile >> id;
        cout << "Printing " << id << " ..." << endl << endl;
        p = list.findId(id);
        if(p == nullptr)
            cout << "Can not find item " << "(" << id << ")!" << endl << endl;
        else
        p-> print();
        }
    }
    dnaFile.close();
}

最后我的列表类

#ifndef DNALIST_H
#define DNALIST_H

#include "DNA.h"
#include "sequenceDatabase.h"

struct DNANode{
    DNA* data;
    DNANode* next;
    DNANode* prev;
};


class DNAList{
public:
    DNAList();
    DNAList(DNA* newDNA);
    void push_back(DNA* newDNA);
    DNA* findId(int);
    void obliterate(int id);
    int size();

private:
    DNANode* head;
    int list_size;

};
#endif

列出实施

#include "DNA.h"
#include "sequenceDatabase.h"
#include "DNAList.h"
#include <iostream>

using namespace std;

DNAList::DNAList(){
    head = new DNANode;
    head->next = nullptr;
    list_size = 0;

}

DNA* DNAList::findId(int id){    // this function is my problem
    DNANode* current;
    current = head;
    while(current->next != nullptr){
        if(current->data->getID() == id){
            return current->data;
        }
        current = current->next;
    }
    return nullptr;
}

int DNAList::size(){
    return list_size;

}

void DNAList::push_back(DNA* newDNA){
    DNANode* current;
    DNANode* last;
    DNANode* p;
    p = new DNANode;
    p->data = newDNA;
    last = nullptr;
    current = head;
    cout << "Adding " << newDNA->getID() << " ..." << endl << endl;
    while(current != nullptr){
        last = current;
        current = current->next;
    }
    if(current == head->next){
        p->next = nullptr;
        p->prev = head;
        head->next = p;
    }
    else{
        p->next = current;
        p->prev = last;
        last->next = p;
    }
    list_size++;
}

我不确定是否应该发布整个代码,但我觉得需要了解这个问题。当我尝试调用find函数在节点中打印数据时,我的问题出现了。

2 个答案:

答案 0 :(得分:2)

head的{​​{1}}成员变量已使用DNAList初始化。由于new DNANode没有显式构造函数,因此其生成的编译器不会初始化指针DNANodedatanextprev在下一行初始化,但next保留为垃圾值。

data内,执行此行:

findId

然而,第一次围绕循环,if (current->data->getID() == id){ 指向current。这意味着您正在尝试查看垃圾值,这可能会崩溃。

一种解决方案是将head函数更改为从findId开始,另一种解决方案是将head->next中的data指针初始化为head并检查<{1}}在访问之前不是nullptr

更好的解决方案可能是让data作为nullptr开始,而不是在顶部有一个虚拟head。这将涉及更改nullptr中的部分代码,但可能会更容易理解。

答案 1 :(得分:1)

啊哈。我认为造成问题的原因是,在SequenceDatabase::importEntries()方法结束时,您设置了if(p=nullptr)而不是进行比较if(p == nullptr)。这无疑会导致您看到的错误。这是一个常见的错误。