Error reading characters of string c

for( CarsPool::CarRecord &record : recs->GetRecords())
{
  LVITEM item;
  item.mask = LVIF_TEXT;
  item.cchTextMax = 6;

  item.iSubItem = 0;
  item.pszText = (LPSTR)(record.getCarName().c_str()); //breakpoint on this line.
  item.iItem = 0;
  ListView_InsertItem(CarsListView, &item);

  item.iSubItem = 1; 
  item.pszText = TEXT("Available");
  ListView_SetItem(CarsListView, &item);

  item.iSubItem = 2;
  item.pszText = (LPSTR)CarsPool::EncodeCarType(record.getCarType());
  ListView_SetItem(CarsListView, &item);
}

MessageBox(hWnd, (LPSTR)(record.getCarName().c_str()), "Test", MB_OK);

3

• Forum
• General C++ Programming
• Error Reading Characters of String

Error Reading Characters of String

Topic archived. No new replies allowed.

Hello.

I’m a CS student and taking C++ class.

This is my first question and I hope this put me on a good start in MSDN community.

I’m doing a homework experimenting with class hierarchy and data structures of bag and set.

I finally don’t have compile errors but runtime errors.

The error details are as follows:

Unhandled exception at 0x00DB6577 in Assignment06.exe: 0xC0000005: Access violation reading location 0xDDDDDDDD.

As I debug the break indicated this part of the code in the file Container.cpp :

Bag::~Bag() {
	cout << "deleting the bag" << endl;
	delete[] container;
}

with following details:

+ container
0x00ea946c {name=0xdddddddd <Error reading characters of string.> age=-572662307 }
Person *
+ this
0x00d7fbdc {size=1 capacity=10 container=0x00ea946c {name=0xdddddddd <Error reading characters of string.> …} }
Bag *

And the following is the whole code associated with this run.

// Container.h
#include "Person.h"
#define DEFAULT_CAP 50

class Bag
{
public:
	Bag(int capacity = DEFAULT_CAP);
	~Bag();
	void Add(Person p);
	Person* Delete();
	bool isEmpty() const;
	bool isFull() const;
	int currentSize() const;
	int Capacity() const;
	void Full();
	void Empty();

protected:
	int size;
	int capacity;
	Person* container;
};

class Set : public Bag
{
public:
	Set(int capacity = DEFAULT_CAP);
	~Set();
	void Add(Person p);
};

// Container.cpp

#include "Container.h"
#include <iostream>
#include <stdlib.h>
using namespace std;

// Bag class methods

Bag::Bag(int max_capacity) {
	cout << "constructing the bag" << endl;
	capacity = max_capacity;
	size = 0;
	container = new Person[capacity];
}

Bag::~Bag() {
	cout << "deleting the bag" << endl;
	delete[] container;
}

void Bag::Add(Person p) {
	if (isFull())
		Full();
	else 
		container[size++] = p;
}

Person* Bag::Delete() {
	if (isEmpty()) {
		Empty();
		return NULL;
	}
	else {
		//generate a random number between 0 and (size - 1)
		int index = (int) rand() % size;
		Person* p = &container[index];
		container[index] = container[size - 1];
		size--;
		return p;
	}
}

bool Bag::isEmpty() const { return (size == 0); }

bool Bag::isFull() const { return (size == capacity); }

int Bag::currentSize() const { return size; }

int Bag::Capacity() const { return capacity; }

void Bag::Full()
{
	cout << "The container is full" << endl;
}

void Bag::Empty()
{
	cout << "The container is empty" << endl;
}


// Set class methods


Set::Set(int max_capacity) : Bag(max_capacity) {
	cout << "constructing the set" << endl;
}

Set::~Set() {
	cout << "deleting the set" << endl;
	delete[] container;
}

void Set::Add(Person p) {
	for (int i = 0; i < size; i++){
		if (container[i] == p){
			cout << "This person is already in the set" << endl;
			return;
		}
	}
	
	if (isFull())
		Full();
	else
		container[size++] = p;
}

// Person.h
#include <iostream>
#include <string>
using namespace std;

class Person
{
protected:
	char* name;
	int age;
public:
	Person() {}
	Person(char* n, int a) : name(n), age(a) 
	{
		cout << "I'm " << name << ", aged " << age << "." << endl;
	}
	virtual ~Person() {}
	char* Name() { return name; }
	int Age() { return age; }
	bool Person::operator == (const Person& other) const {
		if (strcmp(name, other.name) == 0 && age == other.age)
			return true;
		return false;
	}
};

class Employee : public Person
{
protected:
	char* position;
	int pay;
public:
	Employee() {}
	Employee(char* pos, int p) : position(pos), pay(p) {}
	virtual ~Employee() {}
	char* Position() { return position; }
	int Pay() { return pay; }
	void setPosition(char* c) { position = c; }
	void setPay(int i) { pay = i; }
};

class Student : public Person
{
protected:
	int s_num;
	char* major;
public:
	Student() {}
	Student(int s, char* m) : s_num(s), major(m) 
	{
		cout << "Student number : " << s_num << " ; major : " << major << endl;
	}
	virtual ~Student() {}
	int sNum() { return s_num; }
	char* Major() { return major; }
	void setSNum(int i) { s_num = i; }
	void setMajor(char* c) { major = c; }

	bool Student::operator == (const Student& other) const {
		if (s_num == other.s_num)
			return true;
		return false;
	}
};

class gradStudent : public Student 
{
protected:
	char* concentration;
public:
	gradStudent() {}
	gradStudent(char* c) : concentration(c) {}
	virtual ~gradStudent() {}
	char* Concentration() { return concentration; }
	void setConcentration(char* c) { concentration = c; }
};

class StudentWorker : public gradStudent, Employee
{
protected:
	int regist_num;
public:
	StudentWorker() {}
	virtual ~StudentWorker() {}
	int RegistNum() { return regist_num; }
	void setRegistNum(int i) { regist_num = i; }
};

class phdStudent : public gradStudent
{
public:
	phdStudent() {}
	phdStudent(Person(char* name, int age), Student(int sn, char* m), char* c) 
	{ 
		concentration = c; 
		cout << "I'm a PhD. Student concentrating on " << concentration << endl;
	}
	~phdStudent() {}

};

class phdWorker : public StudentWorker
{
protected:
	char* company;
public:
	phdWorker(Person(char* n, int a), Student(int sn, char* m), char* c)
	{
		company = c;
		cout << "I'm a PhD. student working for " << company << endl;
	}
	~phdWorker() {}
};

and the main

#include <iostream>
#include "Container.h"
using namespace std;

void main()
{
	Bag phdStds(10);
	Set phdWorks(10);

	cout << "Bag capacity : " << phdStds.Capacity() << endl;
	cout << "Set capacity : " << phdWorks.Capacity() << endl;

	phdStds.Add(Person("John", 11));
	phdWorks.Add(Person("Scala", 27));

}

As I give several variations to the codes, now I’m almost sure that the problem is happening when destructors are called. 

I learned some new things as follows:

• 0xDDDDDDDD Used
  by MicroQuill’s SmartHeap and Microsoft’s C++ debugging heap to mark freed heap memory

But…. still this puzzle is out of my reach.

Please give me suggestions, concerns, and corrections as they come to you.

Thank you very much.

• Moved by

  Saturday, June 7, 2014 5:37 PM

• Moved by
  Dave PatrickMVP
  Saturday, June 7, 2014 7:14 PM

#include <string>
#include <iostream>

int main()

{
    std::string myLittlePonyOne = «Rarity»;
    std::string myLittlePonyTwo = «Twilight Sparkle»;

    std::cout << «My favorite My Little Pony Characters are:n»
                  << myLittlePonyOne << » n»
                  << myLittlePonyTwo << » n»;

    std::cin.ignore();
    return 0;
}

Я пытаюсь сохранить значение из

vector<vector<string>> data;

в переменную const char *, объявленную в файле .h, следующим образом:

heightmapName = data[0][1].c_str();

Когда я отлаживаю программу, я замечаю, что переменная heightmapName вернуть это

heightmapName 0xcdcdcdcd <Error reading characters of string.>  const char *

Однако, если я объявлю новый const char* и инициализировать это так:

    const char* what = data[0][1].c_str();
heightmapName = data[0][1].c_str();

what переменная хранит данные просто отлично, в то время как heightmapNameне делает.

Это функция:

void Configuration::fileParser(string fileName)
{
vector<vector<string>> data;
string line;
string delimiter = " ";
ifstream ss(fileName);
if (ss)
{
while (getline(ss, line))
{
vector<string> dataLine;
string token = line.substr(0, line.find(delimiter));
string value = line.substr(line.find(delimiter) +1);
dataLine.push_back(token);
dataLine.push_back(value);
data.push_back(dataLine);
}
ss.close();
}
//storeData(data);
const char* ahah = data[0][1].c_str();
heightmapName =    data[0][1].c_str();
}

Почему это происходит? и как я могу решить это?

пс. Я использую Visual Studio 2017

-1

Независимо от проблемы или реализации, предполагая, что тип heightmapName это действительно const char *это не сработает.

Время жизни data ограничен временем жизни fileParser. Увидеть Что такое время жизни std :: string :: c_str ()?

Следовательно, в конце этой функции данные, указанные data[0][1].c_str() станет недействительным.

Попробуйте скопировать данные, если это необходимо. Или сделайте heightmapName как std :: string.

(Дополнительные советы: если это указатель, подумайте о применении правила пяти: Как на самом деле реализовать правило пяти? — еще одна причина избегать ручного управления памятью)

В общем, я избегаю необработанных указателей в классах C ++, используя умные указатели или структуры (такие как std :: string), которые управляют памятью для меня, это означает, что мне не нужно беспокоиться о правиле 3 или правиле 5, так как я не нужно вручную управлять этими ресурсами.

обновление: вы упомянули, что он «работает» для вас в (теперь удален) сущности.

Такой доступ к памяти после окончания жизни не определен. Одним поведением вполне может быть то, что оно волшебным образом «работает». Скорее всего, эта память просто еще не была перезаписана.

1