错误访问冲突写入位置0x00229C20。当试图在控制台中输入一个字符串时

时间:2016-10-08 18:57:03

标签: c++

我正在尝试在我的代码中用c ++输入一个字符串,当我运行时,我总是得到以下错误:在赋值-1.exe中0x0F5023F5(msvcp140d.dll)抛出异常:0xC0000005:访问冲突写入位置0x00229C20。我会在下面发布我的代码,如果有人可以帮助我那将是伟大的。请注意,我已经知道我试图访问你无法访问的内存位置的问题,我只是不知道如何解决它

HEADER FILE:
#ifndef item_H
#define item_h

class item
{
private:
    //attributes
    int itemID;
    char itemName[20];
    float itemcost;
    float itemprice;
    //utility function
    float calcPrice();

public:
    //constructor
    item(int = 000, char[] = "itemUnknown", float = 0,float = 0);
    //destructor
    ~item();
    //set functions
    void setAll(int, char[], float, float);
    void setID(int);
    void setName(char[]);
    void setCost(float);
    //get function
    int getID();
    float getcost();
    float getprice();
    void getname();
    //print function
    void print();
};
#endif

CPP:

#include "Dariush.h"
#include <iostream>
#include <iomanip>
#include<string>
using namespace std;
//constructor will set attributes
item::item(int ID, char n[] , float c,float p)
{


        setID(ID);
        setName(n);
        setCost(c);
        setAll(ID, n, c, p);

}
//destructor will print destroing two objects
item::~item()
{
    cout << "destroing two objects : " << " " << itemName << " "
        << " & " << itemName << endl;
}
//set functions :
void item::setID(int ID)
{
    cout << "please enter the item's ID  :  " << endl;
    cin >> ID;

}
void item::setName(char n[])
{
    cout << "please enter the item's name" << endl;
    cin.ignore();
    cin.getline(n, 20);


}
void item::setCost(float c)
{
    cout << "please enter the item's cost : " << endl;
    cin >> c;
}
void item::setAll(int ID, char n[], float c, float p)
{
    itemID = (ID > 0 && ID < 999) ? ID : 0;
    strcpy_s(itemName, n);
    itemcost = (c > 0) ? c : 0;
    calcPrice();

}
//get functions : 
int item::getID()
{
    return itemID;
}
float item::getcost()
{
    return itemcost;
}
float item::getprice()
{
    return itemprice;
}
void item::getname()
{
    cout << itemName << endl;
}
//print function :
void item::print()
{
    cout << "ID : " << itemID << endl
        << "Name : " << itemName << endl
        << "cost : " << itemcost << endl
        << "price : " << itemprice << endl;
}
// utility function for price callculation : 
float item::calcPrice()
{
    if (itemcost < 1000)
    {
        itemprice = itemcost + (itemcost*0.1);
    }
    else
        itemprice = itemcost + (itemcost*0.2);
    return itemprice;
}

MAIN.CPP:
#include "Dariush.h"
#include <iostream>
#include<string>
using namespace std ;
void main()
{
    item i1;
    item i2;
    i1.print();
    i2.print();
}

感谢您的帮助。

2 个答案:

答案 0 :(得分:1)

让我们仔细看看这三个函数声明:

item(int = 000, char[] = "itemUnknown", float = 0,float = 0);
void setAll(int, char[], float, float);
void setName(char[]);

这里的事情是你声明的字符“数组”参数根本不是数组。相反,它们是指针。在声明参数时,例如char n[]实际上由编译器翻译为char *n

构造函数声明使指针指向常量字符串文字""。关于常量字符串文字的重要之处在于它们确实是常量。尝试修改字符串文字会导致未定义的行为。并且您正在尝试使用cin.getline(n, 20)函数中的setName调用来更改此字面值。不仅如此,你还告诉cin.getline函数读取多于字符串文字的内容。

简单的解决方案是让setName读入成员变量itemName

答案 1 :(得分:0)

此代码存在许多问题,但导致访问冲突的问题是:

void item::setName(char n[])
{
    cout << "please enter the item's name" << endl;
    cin.ignore();
    cin.getline(n, 20); //here
}

您应该使用cin.getline(itemName, 20);代替。

另外,为防止将来出现此类错误,请将参数声明为char const n[]而不是char n[] - 当您使用带有非常量指针的字符串文字作为参数时,良好的编译器应显示警告。