我已经在这个项目上工作了好几天。该项目包含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函数在节点中打印数据时,我的问题出现了。
答案 0 :(得分:2)
head
的{{1}}成员变量已使用DNAList
初始化。由于new DNANode
没有显式构造函数,因此其生成的编译器不会初始化指针DNANode
,data
和next
。 prev
在下一行初始化,但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)
。这无疑会导致您看到的错误。这是一个常见的错误。