Почему 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();
}

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

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

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

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

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

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

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

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

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

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

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

Написано более года назад
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, всё, я проверил ошибок нет. Я просто хотел узнать то ли число выводится и немного испугался, когда увидел шестнадцатиричное значение. Ладно, большое спасибо. Надеюсь больше не побеспокою.

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

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

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

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

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

C++

+1 ещё

Простой
Правильно ли реализован код с использованием Box2d и SFML?
- 1 подписчик
- вчера
- 96 просмотров
0

ответов
C++

+1 ещё

Средний
Почему после статической линковки на другом пк все равно запрагивает dll?
- 1 подписчик
- 19 дек.
- 109 просмотров
1

ответ
C++

Простой
Почему не выводится в консоль символ ╔?
- 1 подписчик
- 18 дек.
- 92 просмотра
0

ответов
C++

+2 ещё

Простой
Как обучить акустическую модель?
- 2 подписчика
- 18 дек.
- 93 просмотра
0

ответов
C++

+1 ещё

Средний
Как сделать линковку в проекте на с++ Visual Studio?
- 1 подписчик
- 17 дек.
- 116 просмотров
1

ответ
C++

Простой
Можно ли в шаблоне через requires ограничить не typename?
- 1 подписчик
- 17 дек.
- 44 просмотра
1

ответ
Windows

+2 ещё

Простой
Как правильно изменять размер левого края окна с winapi?
- 1 подписчик
- 15 дек.
- 154 просмотра
1

ответ
C++

+1 ещё

Простой
Как решить проблему с uwebsockets в С++ Visual Studio2022?
- 1 подписчик
- 12 дек.
- 87 просмотров
1

ответ
C++

+1 ещё

Простой
Каким образом можно установить новый Libcurl в Visual Studio2022 C++?
- 1 подписчик
- 11 дек.
- 59 просмотров
1

ответ
C++

+3 ещё

Простой
Как мне получать файлы от пользователей?
- 1 подписчик
- 10 дек.
- 215 просмотров
2

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

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

ИТ-Холдинг Т1

от 350 000 ₽

Архитектор ИС

ИТ-Холдинг Т1

от 400 000 ₽

Преподаватель курса по информационной безопасности

Eltex • Новосибирск

от 130 000 ₽

Игра С# MVC console + wpf

22 дек. 2024, в 10:44

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

Извлечь данные из архива

22 дек. 2024, в 10:12

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

Разработать Android приложение Kotlin

22 дек. 2024, в 09:51

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

Почему не возвращать объект std::string вместо const char* ? А мусор возвращается, потому что объект str уничтожается после выхода из функции поиска.
Александр Ананьев, поверьте я пробовал у меня компилятор ругается говорит что нельзя возвращать std::string.
Acaunt, возвращать можно. Покажите ошибку компилятора.
Александр Ананьев, написал return str;
Компилятор пишет: не существует подходящей функции преобразования из "std::string" в "const char*"
Евгений Петряев, если вы имеете ввиду что вместо 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 у меня возвращает мусор?

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

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

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