ошибка доступа нарушение записи местоположения 0x00229C20. при попытке ввода строки в консоли
я пытаюсь ввести строку в моем коде на c ++, и при запуске я всегда получаю следующую ошибку: Исключение, выданное в 0x0F5023F5 (msvcp140d.dll) в assignment-1.exe: 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();
}
спасибо за помощь
Решение
Давайте внимательнее посмотрим на эти три объявления функций:
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 вместо.
Другие решения
Есть много проблем с этим кодом, но та, которая вызывает нарушение доступа:
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[] — хороший компилятор должен отображать предупреждение, когда вы используете строковые литералы с неконстантным указателем в качестве аргумента.
detector