zadanie Modyfikowanie tekstu i sprawdzanie jego długości c++

pytanie zadane 1 lipca 2018 w C i C++ przez Nowacx02 Obywatel (1,060 p.)

Witam wykonuje zadanie z kursu programowanie na stronie cpp0x które brzmi następująco:

Zadanie domowe

Napisz program, który z podanego łańcucha znaków usunie powtarzające się spacje występujące obok siebie i zastąpi wszystkie znaczniki oraz znacznikami [b] oraz [/b]. Tekst, który powstanie w wyniku przeprowadzenia wspomnianych operacji wypisz na ekran. Przykładowy fragment aplikacji, który możesz wykorzystać:

#include <iostream>
#include <string>

std::string konwertuj( std::string & sTekst )
{
    std::string sWynik;
    //TODO: tu należy napisać kod realizujący zadanie
    return sWynik;
}
int main()
{
    std::string tekst = "<b>to jest </b> testowy       napis     <b>:)";
    std::cout << konwertuj( tekst ) << std::endl;
    return 0;
}
//Oczekiwane wyjście programu:
//[b]to jest [/b] testowy napis [b]:

Napisałem następujący kod i pytanie czy da się zrobić to jakoś krócej/lepiej?

//Napisz program, który z podanego łańcucha znaków usunie powtarzające się spacje
//występujące obok siebie i zastąpi wszystkie znaczniki <b> oraz </b> znacznikami [b] oraz [/b].
//Tekst, który powstanie w wyniku przeprowadzenia wspomnianych operacji wypisz na ekran. Przykładowy fragment aplikacji, który możesz wykorzystać:
#include <iostream>
#include <string>

using namespace std;

void szukajspacji(string &stekst,string fraza)
   {while (string::npos)
    {
    size_t pozycja=stekst.find(fraza);
    if (pozycja==string::npos)
        return;
    else
       {stekst.erase(pozycja,1); }}}

void szukajb1(string &stekst,string fraza)
{while (string::npos)
    {
    size_t pozycja=stekst.find(fraza);
    if (pozycja==string::npos)
        return;
    else
        {if(fraza=="<b>")
        {stekst.erase(pozycja,3);
        stekst.insert(pozycja,"[b]");}
        else if(fraza=="</b>")
        {stekst.erase(pozycja,4);
        stekst.insert(pozycja,"[/b]");} }}}

int main()
{
    string tekst = "<b>to jest </b> testowy       napis     <b>:)";
    szukajspacji(tekst,"  ");
    szukajb1(tekst,"<b>");
    szukajb1(tekst,"</b>");
    cout<<tekst;

    return 0;
}

komentarz 1 lipca 2018 przez Michał Gibas Pasjonat (19,610 p.)

Polecame poprawić formatowanie kodu bo czyta się to okropnie ;/

2 odpowiedzi

odpowiedź 1 lipca 2018 przez 10kw10 Pasjonat (22,880 p.)

ja bym przelecial petla po stringu i kazdy znak '<' zastapil '[' i analogicznie '>' na ']'

komentarz 1 lipca 2018 przez JAKUBW Nałogowiec (33,470 p.)

To jest źle rozwiązanie bo gdy dam string <><> to otrzymam [[i][b]][] a powinienem otrzymać <[b]><>

odpowiedź 1 lipca 2018 przez niezalogowany

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

void removeMultipliedCharacter(std::string &str, char c)
{
	str.erase(
		std::unique(str.begin(), str.end(), [&c](char left, char right) {return left == right && right == c; }),
		str.end()
	);
}

void replaceAll(std::string& str, const std::string& from, const std::string& to)
{
	std::string::size_type findPos;
	while ((findPos = str.find(from)) != std::string::npos)
	{
		str.replace(findPos, from.length(), to);
	}
}

void prepare(std::string& text)
{
	removeMultipliedCharacter(text, ' ');
	replaceAll(text, "<b>", "[b]");
	replaceAll(text, "</b>", "[/b]");
}

int main()
{
	std::string text = "<b>to jest </b> testowy       napis     <b>:)";
	std::cout << text << "\n";
	prepare(text);
	std::cout << text << "\n";
}