C++ удаления дубликатов, почему не работает?

Question

[Flysink]

Помогите понять почему в visual studio работает , а в qt нет
Я учусь не кидайте какашки

void Wid::test() {
vector<string> stp;
    stp.push_back(text->toPlainText().toStdString());
    sort(stp.begin(),stp.end());
    stp.erase(unique(stp.begin(),stp.end()),stp.end());
    ofstream filesave(pp);
    copy(stp.begin(),stp.end(), ostream_iterator<string>(filesave,"\n"));
}

Вводится текст и после нажатие кнопки собственно вызиваеться через connect() функция test() которая должна удалять дубликаты в QPlainTextEdit и записывать в файл

Comments

[Антон Жилин]

Выдаёт ошибку или просто файл не создаётся?

[Flysink]

Антон Жилин: файл создается и записывается все что ввел без удаления дублей, если попробовать удалить весь вектор, все работает в файл ничего не поступает stp.erase(stp.begin(),stp.end()); а в от с unique не ясно почему так

[MiiNiPaa]

Flysink: А может дублей там нет? Нужно проверить все символы в строках на идентичность на всякий случай. Можете файл выложить, тот самый, с дублями (Как он создался, не вручную копировать, чтобы информацию не потерять)

[Flysink]

MiiNiPaa: конечно, если сможете подправить буду очень благодарен, часть кода

#include "port.h"
#include "ui_port.h"
#include <QString>
#include <string>
#include <vector>
#include <fstream>
#include <iostream>

using namespace std;
    const char *pp = ".//text2.txt";

Port::Port(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::Port)
{
    ui->setupUi(this);

    but = new QPushButton;
    but->resize(200,50);
    but->setText("Click");
    text = new QPlainTextEdit;
    text->resize(640,480);
    text->show();
    but->show();

    connect(but,SIGNAL(clicked()),this,SLOT(clearp()));

}

Port::~Port()
{
    delete ui;
}

void Port::clearp() {

    vector<string> stp;
    stp.push_back(text->toPlainText().toStdString());
    sort(stp.begin(),stp.end());
    stp.erase(unique(stp.begin(),stp.end()),stp.end());

    ofstream filesave(pp);
    copy(stp.begin(),stp.end(), ostream_iterator<string>(filesave,"\n"));
   }

Файл создаю в директории где хранится сам проект, можно автом. создавать либо указать другой путь не важно, через окно qplaintextedit я ввожу текст, содержания которого примерно
111
111
222
222
333
444

На выходе в файле text2.txt я должен получить

111
222
333
444
но как вы успели заметить выходит не так как было задумано

Answer 1

[Mercury13]

void Wid::test() {
    vector<string> stp;
    stp.push_back(text->toPlainText().toStdString());
    // В stp одна штука
    sort(stp.begin(),stp.end());
    // Ну что ей будет, этой штуке?
    stp.erase(unique(stp.begin(),stp.end()),stp.end());
    // unique даст end; поведение vector при этом не определено.
    ofstream filesave(pp);
    copy(stp.begin(),stp.end(), ostream_iterator<string>(filesave,"\n"));
}

Comments

[MiiNiPaa]

> поведение vector при этом не определено
Вообзе-то определено. erase(end, end) не сделает ничего

[Flysink]

Точно как я сам не заметил, а как text->toPlaintText() в цикле передать в вектор ? то есть берем строку до \n и заносим и идем к следующей и т.д

[Flysink]

всем спасибо решил.

QStringList edit;
    edit = text->toPlainText().split("\n");

    foreach(QString vv, edit) {
       stp.push_back(vv.toStdString());
    }

Но может есть проще и быстрее вариант ?

Answer 2

[MiiNiPaa]

Как правильно заметил Mercury13 , у вас в векторе один элемент — всё содержимое QPlainTextEdit вместе с переносами строки и т.п.

Чтобы работало, как вам хочется, нужно взять это содержимое, разбить его на строки, запихнуть в вектор и потом работать с этим.