0

Написал функцию которая возвращает только уникальные символы строки. Когда пытаюсь вывести в терминал – показывает какую-то дичь. Можете подсказать чо не так?

Я в C++ дня 4 от силы, и если функция полностью кривая то так и скажите. 👌

#include <iostream>
#include <set>


std::string *get_unique(std::string *str){
    const int str_size = (str->size()) - 1;
    std::string unique_str;    
    std::set<char> unique_set;
    for (int i = 0; i < str_size; i++){        
        unique_set.emplace((*str)[i]);
    }
    const int unique_set_size = unique_set.size();
    for (int i = 0; i < unique_set_size - 1; i++){
        unique_str += *std::next(unique_set.begin(),i);      
    }
    std::string *punique_str = &unique_str;
    return punique_str;
}

int main(){
    std::string hello = "Hello, World";
    std::cout << *get_unique(&hello);
    return 0;
}
3
  • 1
    Да, кривая. unique_str будет уничтожена при выходе из функции, а вы возвращаете на нее указатель. Вам не нужно возвращать указатель, просто верните объект строки. 6 мар 2020 в 17:49
  • Странно, вроде читал что можно вернуть локальный. static в этом как-то замешан?) Спасибо!
    – Non
    6 мар 2020 в 17:51
  • Croessmah, спасибо, понял.
    – Non
    6 мар 2020 в 17:53

1 ответ 1

2

Кроме того, что нельзя возвращать указатель на локальные переменные, вы выбрали несколько сложноватый путь решения своей задачи. Вот так будет попроще:

#include <iostream>
#include <string>
#include <algorithm>

using namespace std;

string get_unique(const string& str)
{
    string res = str;
    sort(res.begin(),res.end());
    res.erase(unique(res.begin(),res.end()),res.end());
    return res;
}

int main(){
    string hello = "Hello, World";
    cout << get_unique(hello);
}
0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.