Почему Config::search у меня возвращает мусор?

Question

Acaunt @Acaunt

C++

Почему Config::search у меня возвращает мусор?

Я - программист-любитель, не программист. Многого не знаю, только основы.

Хотел создать для своей программы конфигурационный файл. Для этого создал класс с функциями: обновление конфига, проверка существования конфига и поиск значений в конфиге.

config.h

#ifndef CONFIG_H_
#define CONFIG_H_

class Config {
private:
	static void update_config();
public:
	static int initialize();
	static const char* search(const char* parametr);
};

#endif /* CONFIG_H_ */

У меня все функции работают нормально кроме поиска значений Config::search. Он все нормально находит но только возвращает мусорные значения. Как это исправить?

Функция должна работает так: Config::search("title") возвращает NAME.

config.cpp

#include "Config.h"

#include <iostream>
#include <fstream>
#include <string>

std::ofstream w_config;
std::ifstream r_config;

void Config::update_config() {
	std::cout << "Create file Config" << std::endl;
	w_config.open("Config.ini");

	w_config << "title = NAME" << std::endl;
	w_config << "window_mode = 0" << std::endl;
	w_config << "width = 800" << std::endl;
	w_config << "hight = 600" << std::endl;

	w_config.close();
}

int Config::initialize() {
	r_config.open("Config.ini");
	if (!r_config.is_open()) {
		std::cout << "Have not file Config.ini" << std::endl;
		r_config.close();
		update_config();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		std::cerr << "Failed open file Config.ini" << std::endl;
		r_config.close();
		return -1;
	}

	else {
		std::cout << "Open file Config.ini" << std::endl;
		r_config.close();
		return 0;
	}
}

const char* Config::search(const char* parametr) {
	std::string str;
	for (int a = 0; a < 2; a++) {
		r_config.open("Config.ini");
		std::cout << "Search in file Config.ini -> " << parametr;

		while (!r_config.eof()) {
			r_config >> str;
			if (parametr == str) {
				std::getline(r_config, str);
				r_config.close();
				str.erase(0, 3);
				a = 2;
				break;
			}
			else {
				str = "";
			}
		}

		if (str == "") {
			r_config.close();
			std::cout << " -> not found" << std::endl;
			std::cout << "Re-";
			update_config();
		}
	}

	std::cout << " = " << str << std::endl;

	return str.c_str();
}

Вопрос задан более года назад
297 просмотров

8 комментариев

Подписаться 1 Простой 8 комментариев

Александр Ананьев @SaNNy32

Почему не возвращать объект std::string вместо const char* ? А мусор возвращается, потому что объект str уничтожается после выхода из функции поиска.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Александр Ананьев, поверьте я пробовал у меня компилятор ругается говорит что нельзя возвращать std::string.

Написано более года назад
Александр Ананьев @SaNNy32

Acaunt, возвращать можно. Покажите ошибку компилятора.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Александр Ананьев, написал return str;
Компилятор пишет: не существует подходящей функции преобразования из "std::string" в "const char*"

Написано более года назад
Евгений Петряев @Gremlin92

Acaunt, ну так описание метода тоже поменяй

Написано более года назад
Acaunt @Acaunt Автор вопроса

Евгений Петряев, если вы имеете ввиду что вместо const char* search сделать std::string search то мне нужно возвращать именно const char* это нужно для библиотеки glfw для передачи в функцию создания окна

Написано более года назад
Александр Ананьев @SaNNy32

Acaunt, возвращайте std::string, а потом из нее получите const char* вызовом c_str() у объекта строки в нужном месте для передачи в glfw.

Написано более года назад

Acaunt @Acaunt Автор вопроса

Александр Ананьев, Евгений Петряев, Wataru,
Спасибо вам всем я бился над этим решением 2 дня и сейчас 31 декабря в 00:05 путем совместных действий ответ был найден. Да я переделал функцию на std::string я пробовал это и раньше просто я не использовал #include string в классе, у меня вылазила ошибка компилятора но после подключения библиотеки ошибка пропала. Ладно всех с наступающим. Пост скриптум я оставлю здесь исправленный код если вдруг кому-то понадобится
Класс:

#ifndef CONFIG_H_
#define CONFIG_H_

#include <string>

class Config {
private:
	static void update_config();
public:
	static int initialize();
	static std::string search_str(std::string parametr);
	static int search_int(std::string parametr);
	static bool search_bool(std::string parametr);
};

#endif /* CONFIG_H_ */

И функции:

#include "Config.h"

#include <iostream>
#include <fstream>

std::ofstream w_config;
std::ifstream r_config;

void Config::update_config() {
	std::cout << "Create file Config" << std::endl;
	w_config.open("Config.ini");

	w_config << "title = NAME" << std::endl;
	w_config << "window_mode = 0" << std::endl;
	w_config << "width = 800" << std::endl;
	w_config << "hight = 600" << std::endl;

	w_config.close();
}

int Config::initialize() {
	r_config.open("Config.ini");
	if (!r_config.is_open()) {
		std::cout << "Have not file Config.ini" << std::endl;
		r_config.close();
		update_config();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		std::cerr << "Failed open file Config.ini" << std::endl;
		r_config.close();
		return -1;
	}

	else {
		std::cout << "Open file Config.ini" << std::endl;
		r_config.close();
		return 0;
	}
}

std::string Config::search_str(std::string parametr) {
	std::string str;
	for (int a = 0; a < 2; a++) {
		r_config.open("Config.ini");
		std::cout << "Search in file Config.ini -> " << parametr;

		while (!r_config.eof()) {
			r_config >> str;
			if (parametr == str) {
				std::getline(r_config, str);
				r_config.close();
				str.erase(0, 3);
				a = 2;
				break;
			}
			else {
				str = "";
			}
		}

		if (str == "") {
			r_config.close();
			std::cout << " -> not found" << std::endl;
			std::cout << "Re-";
			update_config();
		}
	}

	std::cout << " = " << str << std::endl;

	return str;
}

int Config::search_int(std::string parametr) {
	return stoi(search_str(parametr));
}

bool Config::search_bool(std::string parametr) {
	return stoi(search_str(parametr));
}

Написано более года назад

Решения вопроса 1

25 комментариев

Acaunt @Acaunt Автор вопроса

Александр Ананьев, Евгений Петряев, Wataru,
Спасибо вам всем я бился над этим решением 2 дня и сейчас 31 декабря в 00:05 путем совместных действий ответ был найден. Да я переделал функцию на std::string я пробовал это и раньше просто я не использовал #include string в классе, у меня вылазила ошибка компилятора но после подключения библиотеки ошибка пропала. Ладно всех с наступающим. Пост скриптум я оставлю здесь исправленный код если вдруг кому-то понадобится
Класс:

#ifndef CONFIG_H_
#define CONFIG_H_

#include <string>

class Config {
private:
	static void update_config();
public:
	static int initialize();
	static std::string search_str(std::string parametr);
	static int search_int(std::string parametr);
	static bool search_bool(std::string parametr);
};

#endif /* CONFIG_H_ */

И функции:

#include "Config.h"

#include <iostream>
#include <fstream>

std::ofstream w_config;
std::ifstream r_config;

void Config::update_config() {
	std::cout << "Create file Config" << std::endl;
	w_config.open("Config.ini");

	w_config << "title = NAME" << std::endl;
	w_config << "window_mode = 0" << std::endl;
	w_config << "width = 800" << std::endl;
	w_config << "hight = 600" << std::endl;

	w_config.close();
}

int Config::initialize() {
	r_config.open("Config.ini");
	if (!r_config.is_open()) {
		std::cout << "Have not file Config.ini" << std::endl;
		r_config.close();
		update_config();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		std::cerr << "Failed open file Config.ini" << std::endl;
		r_config.close();
		return -1;
	}

	else {
		std::cout << "Open file Config.ini" << std::endl;
		r_config.close();
		return 0;
	}
}

std::string Config::search_str(std::string parametr) {
	std::string str;
	for (int a = 0; a < 2; a++) {
		r_config.open("Config.ini");
		std::cout << "Search in file Config.ini -> " << parametr;

		while (!r_config.eof()) {
			r_config >> str;
			if (parametr == str) {
				std::getline(r_config, str);
				r_config.close();
				str.erase(0, 3);
				a = 2;
				break;
			}
			else {
				str = "";
			}
		}

		if (str == "") {
			r_config.close();
			std::cout << " -> not found" << std::endl;
			std::cout << "Re-";
			update_config();
		}
	}

	std::cout << " = " << str << std::endl;

	return str;
}

int Config::search_int(std::string parametr) {
	return stoi(search_str(parametr));
}

bool Config::search_bool(std::string parametr) {
	return stoi(search_str(parametr));
}

Написано более года назад

mayton2019 @mayton2019

А что будет если эта функция не найдет int?

return stoi(search_str(parametr));

Написано более года назад
Wataru @wataru Куратор тега C++

mayton2019, читайте справку. Будет выброшено исключение.

Написано более года назад
Acaunt @Acaunt Автор вопроса

mayton2019, в функции serch_str идёт поиск параметра по названию и предполагается что все значения указаны правильны. Я также сделал проверку на тот случай если если необходимо параметра нет в конфиге, он просто пересоздается. А после если вызваны serch_int search _bool будут переобразованы в необходимые типы.
А делать проверку на то что вернулся не тот тип я как-то не очень хочу. Как мне кажется большая часть пользователей вряд-ли будет залезать в конфиг и менять значения. Позже я допишу пару функций позволяющих переписывать конкретные значения для настройки самой программы во время её использования

Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, Ну, в конфиге может быть написано с ошибкой. Но если вам нормально, что программа упадет при ошибке там, то сделайте хотя бы проверку на пустую строку. вот search_str вернул, что нет параметра. Тогда в качестве int возвращайте, например 0. Но лучше, конечно, возвращать std::optional<int>. Тогда в вызывающем коде можно проверить, что int нашелся. В идеале, конечно, надо бы и все ошибки обробатвать. Если число не распарсилось, то надо выдать ошибку в лог и вернуть nullopt.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Wataru, спасибо за совет посмотрю позже как сделать эту проверку

Написано более года назад
mayton2019 @mayton2019

Wataru, судя по протоколу автор не ожидает исключений. Он ожидает что будет число типа int на выходе. Если-бы я ожидал опционал (тоесть возможность не-получения результата) - то я-бы декларировал опциональный int на выходе.

Написано более года назад
Евгений Шатунов @MarkusD Куратор тега C++

mayton2019, собственно, да. И в плюсах такое поведение только приветствуется.
В C++23 можно будет написать так: config.search_int("foo").and_then(lambda);. Код выходит довольно наглядный.

Написано более года назад

Acaunt @Acaunt Автор вопроса

Wataru, mayton2019, я переписал много функций. Добавил функцию перезаписи параметра, немного усовершенствовал инициализацию, и упростил систему поиска значений, перенёс все значения параметров в массив строк. И самое важное теперь каждый раз, когда возвращается неправильное значение переписывается только строчка с данным значением (я так и не понял как можно использовать std::optional<int> о которых вы писали, но мне кажется это уже не так важно. В моём коде меня только смущает функция получения количества переменных lot_list() так как она может вызываться несколько раз если несколько переменых неправильных. Я пробовал использовать sizeof(list)/sizeof(list[0]) но компилятор выдает ошибку: std::string []: недопустимый оператор sizeof.
Вот код:
Класс:

#ifndef CONFIG_H_
#define CONFIG_H_

#include <string>

class Config {
private:
	static const int lot_list();
	static std::string list[];
	static void update();
public:
	static int initialize();
	static void rewrite(std::string parametr, std::string value = "");
	static std::string search_str(std::string parametr);
	static int search_int(std::string parametr);
	static bool search_bool(std::string parametr);
};

#endif /* CONFIG_H_ */

Функции:

#include "Config.h"

#include <iostream>
#include <fstream>

std::ofstream w_config;
std::ifstream r_config;

const int Config::lot_list() {
	int i = 0;
	while (!list[i].empty()) {
		++i;
	}
	return i;
}

std::string Config::list[]{
	"title = NAME",
	"window_mode = 0",
	"width = 800",
	"hight = 600"
};

void Config::update() {
	std::cout << "Create file Config.ini" << std::endl;
	w_config.open("Config.ini");
	for (int a = 0; a < lot_list(); a++) {
		w_config << list[a] << std::endl;
	}
	w_config.close();
}

int Config::initialize() {
	r_config.open("Config.ini");
	if (!r_config.is_open()) {
		r_config.close();
		std::cout << "Have not file Config.ini" << std::endl;
		update();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		r_config.close();
		std::cerr << "Failed open file Config.ini" << std::endl;
		return -1;
	}
	else if (r_config.peek() == EOF) {
		r_config.close();
		std::cout << "File Config.ini empty" << std::endl << "Re-";
		update();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		r_config.close();
		std::cerr << "Failed open file Config.ini" << std::endl;
		return -1;
	}
	else {
		r_config.close();
		std::cerr << "Open file Config.ini" << std::endl;
		return 0;
	}
}

void Config::rewrite(std::string parametr, std::string value) {
	std::cout << "Re-Create file Config.ini" << std::endl;
	std::string str;

	if (value == "") {
		for (int a = 0; a < lot_list(); a++) {
			str = list[a].substr(0, list[a].find_first_of(' '));
			if (str == parametr) {
				value = list[a];
				break;
			}
		}
	}
	else {
		value = parametr + " = " + value;
	}

	bool i = 1;
	w_config.open("Config_copy.ini");
	r_config.open("Config.ini");
	while (!r_config.eof()) {
		std::getline(r_config, str);
		if (parametr == str.substr(0, str.find_first_of(' '))) {
			w_config << value << std::endl;
			i = 0;
		}
		else if (str == "") {}
		else {
			w_config << str << std::endl;
		}
	}
	if (i) {
		w_config << value << std::endl;
	}
	w_config.close();
	r_config.close();

	w_config.open("Config.ini");
	r_config.open("Config_copy.ini");
	while (!r_config.eof()) {
		std::getline(r_config, str);
		if (str == "") {}
		else {
			w_config << str << std::endl;
		}
	}
	w_config.close();
	r_config.close();
	std::remove("Config_copy.ini");
}

std::string Config::search_str(std::string parametr) {
	std::string str;

	for (int a = 0; a < 2; a++) {
		r_config.open("Config.ini");
		std::cout << "Search in file Config.ini -> " << parametr;

		while (!r_config.eof()) {
			std::getline(r_config, str);

			if (parametr == str.substr(0, str.find_first_of(' '))) {
				r_config.close();
				str.erase(0, parametr.length() + 3);
				break;
			}
			else {
				str = "";
			}
		}

		if (str == "") {
			r_config.close();
			std::cout << " -> not found" << std::endl;
			rewrite(parametr);
		}
		else {
			break;
		}
	}

	std::cout << " = " << str << std::endl;

	return str;
}

int Config::search_int(std::string parametr) {
	std::string int_str = Config::search_str(parametr);
	char* ch;
	strtol(int_str.c_str(), &ch, 10);

	if (*ch) {
		std::cout << "Not the corect value" << std::endl;
		rewrite(parametr);
		int_str = search_str(parametr);
	}

	return stoi(int_str);
}

bool Config::search_bool(std::string parametr) {
	std::string bool_str = search_str(parametr);

	if (bool_str == "0") {}
	else if (bool_str == "1") {}
	else {
		std::cout << "Not the corect value" << std::endl;
		rewrite(parametr);
		bool_str = search_str(parametr);
	}

	return stoi(bool_str);
}

Написано более года назад

Wataru @wataru Куратор тега C++

Acaunt, list сделайте std::vector<std::string>, а не сишный массив. И тогда у него будет метод size(), который вернет количество элементов.

Написано более года назад
Acaunt @Acaunt Автор вопроса
Wataru, написал как ты сказал
std::vector<std::string> Config::list[]
Библиотеку vector я подключил
После повылазило 6 ошибок:
1) слишком много инициализаторов в list
2) инициализация: невозможно преобразовать "const char[13]" в "
std::vector<std::string,std::allocator<std::string>>
"
3) бинарный "=": не найден оператор, принимающий правый операнд типа "
std::vector<std::string,std::allocator<std::string>>
" (или приемлемое преобразование отсутствует). В функции rewrite строчка value = list[a];
4) бинарный "<<": не найден оператор, принимающий правый операнд типа "
std::vector<std::string,std::allocator<std::string>>
" (или приемлемое преобразование отсутствует). В функции update строчка w_config << list[a] << std::endl;
5) функция "substr": не является членом "
std::vector<std::string,std::allocator<std::string>>
". В функции rewrite строчка str = list[a].substr(0, list[a].find_first_of(' '));
6) "find_first_of": не является членом "
std::vector<std::string,std::allocator<std::string>>
" та же строчка что в предыдущем пункте
Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt,
написал как ты сказал

Нет, я говорил ВМЕСТО массива использовать вектор. Удалите квадратные скобки.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Wataru, извините неправильно понял. Да помогло большое спасибо.
Сделал с sizeof /sizeof возвращает шестнадцатиричное значение
Сделал list.size тоже 16ричное значения

Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, Что? Не может оно 16-ричное значение выдавать. Оно врзвращает size_t, ака unsigned int . Может, вы его так выводите.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Wataru, в функции lot_list написал
return list.size();
Консоль мне вывела значение: 00007FF7D67301

Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, какая консоль? Return не выводит в ничего. Смотрите код, где происходит вызов. Это число похоже на адрес. Вы выводите указатель или массив.

Написано более года назад
Acaunt @Acaunt Автор вопроса

Wataru, я просто написал отдельно вывести переменную в консоль потому что мне нужно было удостовериться что возвращает нужное значение

Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, какую переменную? Каков ее тип?

Написано более года назад
Acaunt @Acaunt Автор вопроса
Wataru,
size_t Config::lot_list() { return list.size; } int main() { std::cout << Config::lot_list()

Я делал int lot_list тоже самое
Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, этот код не скомпилируется. Там точки с запятой не хватает. Приведите код из исходника, а не пишите что-то аналогичное пл памяти.

Написано более года назад
Wataru @wataru Куратор тега C++

Acaunt, ах да, size - это метод. Его надо вызывать. Поставьте там скобки в lot_list.

Написано более года назад

Acaunt @Acaunt Автор вопроса

Wataru, я написал тот код с телефона не обращая на грамматику так как я не могу отправить сразу с компьютера(интернета нет) поэтому мне приходится через юсб передавать текстовый файл на телефон и с телефона сюда вот код

#ifndef CONFIG_H_
#define CONFIG_H_

#include <string>
#include <vector>

class Config {
private:
	
	static std::vector<std::string> list;
	static void update();
public:
	static size_t lot_list();
	static int initialize();
	static void rewrite(std::string parametr, std::string value = "");
	static std::string search_str(std::string parametr);
	static int search_int(std::string parametr);
	static bool search_bool(std::string parametr);
};

#endif /* CONFIG_H_ */

#include "Config.h"

#include <iostream>
#include <fstream>

std::ofstream w_config;
std::ifstream r_config;

size_t Config::lot_list() {
	return list.size();
}

std::vector<std::string> Config::list{
	"title = NAME",
	"window_mode = 0",
	"width = 800",
	"hight = 600"
};

void Config::update() {
	std::cout << "Create file Config.ini" << std::endl;
	w_config.open("Config.ini");
	for (int a = 0; a < lot_list(); a++) {
		w_config << list[a] << std::endl;
	}
	w_config.close();
}

int Config::initialize() {
	r_config.open("Config.ini");
	if (!r_config.is_open()) {
		r_config.close();
		std::cout << "Have not file Config.ini" << std::endl;
		update();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		r_config.close();
		std::cerr << "Failed open file Config.ini" << std::endl;
		return -1;
	}
	else if (r_config.peek() == EOF) {
		r_config.close();
		std::cout << "File Config.ini empty" << std::endl << "Re-";
		update();
		r_config.open("Config.ini");
	}

	if (!r_config.is_open()) {
		r_config.close();
		std::cerr << "Failed open file Config.ini" << std::endl;
		return -1;
	}
	else {
		r_config.close();
		std::cerr << "Open file Config.ini" << std::endl;
		return 0;
	}
}

void Config::rewrite(std::string parametr, std::string value) {
	std::cout << "Rewrite the parameter in file Config.ini" << std::endl;
	std::string str;

	if (value == "") {
		for (int a = 0; a < lot_list(); a++) {
			str = list[a].substr(0, list[a].find_first_of(' '));
			if (str == parametr) {
				value = list[a];
				break;
			}
		}
	}
	else {
		value = parametr + " = " + value;
	}

	bool i = 1;
	w_config.open("Config_copy.ini");
	r_config.open("Config.ini");
	while (!r_config.eof()) {
		std::getline(r_config, str);
		if (parametr == str.substr(0, str.find_first_of(' '))) {
			w_config << value << std::endl;
			i = 0;
		}
		else if (str == "") {}
		else {
			w_config << str << std::endl;
		}
	}
	if (i) {
		w_config << value << std::endl;
	}
	w_config.close();
	r_config.close();

	w_config.open("Config.ini");
	r_config.open("Config_copy.ini");
	while (!r_config.eof()) {
		std::getline(r_config, str);
		if (str == "") {}
		else {
			w_config << str << std::endl;
		}
	}
	w_config.close();
	r_config.close();
	std::remove("Config_copy.ini");
}

std::string Config::search_str(std::string parametr) {
	std::string str;

	for (int a = 0; a < 2; a++) {
		r_config.open("Config.ini");
		std::cout << "Search in file Config.ini -> " << parametr;

		while (!r_config.eof()) {
			std::getline(r_config, str);

			if (parametr == str.substr(0, str.find_first_of(' '))) {
				r_config.close();
				str.erase(0, parametr.length() + 3);
				break;
			}
			else {
				str = "";
			}
		}

		if (str == "") {
			r_config.close();
			std::cout << " -> not found" << std::endl;
			rewrite(parametr);
		}
		else {
			break;
		}
	}

	std::cout << " = " << str << std::endl;

	return str;
}

int Config::search_int(std::string parametr) {
	std::string int_str = Config::search_str(parametr);
	char* ch;
	strtol(int_str.c_str(), &ch, 10);

	if (*ch) {
		std::cout << "Not the corect value" << std::endl;
		rewrite(parametr);
		int_str = search_str(parametr);
	}

	return stoi(int_str);
}

bool Config::search_bool(std::string parametr) {
	std::string bool_str = search_str(parametr);

	if (bool_str == "0") {}
	else if (bool_str == "1") {}
	else {
		std::cout << "Not the corect value" << std::endl;
		rewrite(parametr);
		bool_str = search_str(parametr);
	}

	return stoi(bool_str);
}

Написано более года назад

Wataru @wataru Куратор тега C++

Acaunt, и тут вы его нигде не выводите же.

Предполагаю, что вы там тоже без скобок написали. Тогда вы выводили бы адрес функции, а адреса cout выводит в 16-ричном виде.

Написано более года назад

Acaunt @Acaunt Автор вопроса

Wataru,

#include <iostream>
#include "Config.h"
int main() {
std::cout << Config::lot_list() << std::endl;
return 0;
}

Написано более года назад

Acaunt @Acaunt Автор вопроса

Wataru, всё, я проверил ошибок нет. Я просто хотел узнать то ли число выводится и немного испугался, когда увидел шестнадцатиричное значение. Ладно, большое спасибо. Надеюсь больше не побеспокою.

Написано более года назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации

Похожие вопросы

Программирование

+1 ещё

Простой
Как запустить mp4 файл из ресурсов в проигрывателе C++?
- 1 подписчик
- час назад
- 18 просмотров
1

ответ
C++

+1 ещё

Простой
Как избавить от заданных размеров количества строк и столбцов при считывании из файла?
- 1 подписчик
- 5 часов назад
- 28 просмотров
1

ответ
C++

Простой
Как создать свой модуль в С++?
- 2 подписчика
- 22 часа назад
- 124 просмотра
0

ответов
C++

+1 ещё

Простой
Как исправить ошибку undefined reference to при коспиляции кода с++ в VS?
- 1 подписчик
- вчера
- 84 просмотра
1

ответ
C++

+1 ещё

Простой
Как поменять кодировку так что бы после сохранения файла оставались русские буквы?
- 1 подписчик
- вчера
- 62 просмотра
1

ответ
C++

+2 ещё

Простой
Почему перемещение в игре в терминале ведет себя странно?
- 1 подписчик
- вчера
- 79 просмотров
0

ответов
C++

+1 ещё

Простой
Правильно ли в C++ использовать директиву препроцессора define?
- 1 подписчик
- вчера
- 123 просмотра
4

ответа
C++

Простой
Как реализовать добавление и удаление элементов вектора в классe?
- 1 подписчик
- 06 мая
- 46 просмотров
0

ответов
C++

Простой
Как создать структуру с вектором, который хранит экземпляры классов?
- 1 подписчик
- 06 мая
- 87 просмотров
1

ответ
C++

+1 ещё

Простой
Какие книги подойдут для изучения c++?
- 1 подписчик
- 06 мая
- 72 просмотра
1

ответ
Показать ещё Загружается…

Android-разработчик / Senior Android Developer

Ciliz • Санкт-Петербург

от 300 000 ₽

Системный аналитик

Концепт-разработка • Москва

от 180 000 до 250 000 ₽

С++ разработчик в блокчейн проект

Zano

от 120 000 до 450 000 ₽

Доработка игры PacMan GYM ATARI PYTHON

08 мая 2024, в 17:54

1000 руб./за проект

Таргетолог

08 мая 2024, в 17:35

5000 руб./за проект

Написать скрипт на Го для Gate.io

08 мая 2024, в 17:32

2000 руб./за проект

Почему не возвращать объект std::string вместо const char* ? А мусор возвращается, потому что объект str уничтожается после выхода из функции поиска.
Александр Ананьев, поверьте я пробовал у меня компилятор ругается говорит что нельзя возвращать std::string.
Acaunt, возвращать можно. Покажите ошибку компилятора.
Александр Ананьев, написал return str;
Компилятор пишет: не существует подходящей функции преобразования из "std::string" в "const char*"
Acaunt, ну так описание метода тоже поменяй
Евгений Петряев, если вы имеете ввиду что вместо const char* search сделать std::string search то мне нужно возвращать именно const char* это нужно для библиотеки glfw для передачи в функцию создания окна
Acaunt, возвращайте std::string, а потом из нее получите const char* вызовом c_str() у объекта строки в нужном месте для передачи в glfw.

Answer 1 · 2022-12-30 23:30:35

Проблема вот в этой строчке:
return str.c_str();

Тут вы возвращаете указатель на внутренние данные у str. Но при выходе из функуции str уничтожается - это же локальная переменная. В итоге у вас получается висячий указатель (указатель на память, которой вы уже не владеете). Эту память какая-то другая часть вашей программы переиспользует и там остается что-то не ваше, выглядещее для вас, как мусор.

Вообще, это undefined behavior - доступ к висячему указателю. Программа вполне может и аварийно завершится.

Для решения этой проблемы возвращайте std::string. Или выделяйте char* вручную, через new[] (только не забудьте указатель потом удалить в вызывающем коде). Но лучше, конечно, возвращать string и не мучатся с ручным управлением указателями.

Почему Config::search у меня возвращает мусор?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт