我正在写一个文件的字符指针,但是当我这样做时我的程序总是在名字崩溃而我无法弄清楚为什么
#include <iostream>
#include <iomanip>
#include <string>
#include "AmaProduct.h"
using namespace std;
namespace sict{
AmaProduct::AmaProduct(char file){
fileTag_ = file;
}
const char* AmaProduct::unit()const{
return unit_;
}
void AmaProduct::unit(const char* value){
for (int i = 0; i != 9; i++){
unit_[i] = value[i];
}
unit_[11] = 0;
}
fstream& AmaProduct::store(fstream& file, bool addNewLine)const{
file.open("file.txt");
if (file.is_open()){
file << fileTag_ << "," << sku() << ",";
file<< name() << ",";//here
file<< price() << "," << taxed() << "," << quantity() << "," << unit_ << "," << qtyNeeded();
if (addNewLine){
file << endl;
}
}
file.close();
return file;
}
头文件
#ifndef SICT_AMAPRODUCT_H__
#define SICT_AMAPRODUCT_H__
#include "Streamable.h"
#include "Product.h"
#include "Date.h"
#include "ErrorMessage.h"
#include "general.h"
#include <iostream>
#include <fstream>
namespace sict{
class AmaProduct : public Product{
private:
char fileTag_;
char unit_[11];
protected:
ErrorMessage err_;
public:
AmaProduct(char file='N');
const char* unit()const;
void unit(const char* value);
fstream& store(fstream& file, bool addNewLine = true)const;
fstream& load(std::fstream& file);
ostream& write(ostream& os, bool linear)const;
istream& read(istream& is);
};
}
名称()
const char* Product::name()const{
return name_;
}
char* name_;
void Product::name(char* name){
delete[] name_;
name_= new char[strlen(name)+1];
strcpy(name_,name);
}
如果有人对我上传的其他文件感兴趣
答案 0 :(得分:1)
有几种可能性,但如果cout<<name()
导致分段错误,最可能的情况是:
name_
仍为nullptr name_
是无效指针(例如,如果您复制了您的结构,但不尊重rule of 3)为了使您的代码更可靠,您可以使用char*
更改所有string
,繁琐的内存分配和c样式的字符串操作。
如果不这样做,请确保在复制或复制构造结构时遵守3的规则以避免浅拷贝和指针问题,并确保在使用固定大小的char数组时,您确定不会溢出缓冲区
另请注意,执行此操作可能会溢出缓冲区:
file.getline(n, ',');
name(n);
因为istream::getline()
,当它有两个参数时,将第二个参数作为大小(这里是44,逗号的ascii值)。 file.getline(n, 7, ',')
将是正确的形式。