[Resource] Little RichText

[Xafi]

HI today i did a class to handle sf::text.



I wait tips to optimize

RichText.hpp

/* AUTHOR: XAFI */
#ifndef RICHTEXT_H
#define RICHTEXT_H
#include <SFML/Graphics.hpp>
#include <string>
#include <vector>
class RichText : public sf::Drawable,public sf::Transformable
{
public:
        RichText();
        RichText & operator << (const std::string&);
        RichText & operator << (const sf::Color&);
        RichText & operator << (const sf::Uint32);

        void setFont(const sf::Font &);
        void setCharacterSize(int);
        void setColor(int, sf::Color);
        void setString(int, std::string);
        void Clear();
       
        sf::Vector2f getSize() const;
        const std::string getString();
        int getCharacterSize();

private:
        void draw(sf::RenderTarget& target, sf::RenderStates states) const;
        void UpdatePos() const;
        mutable std::vector<sf::Text> Text;
        sf::Color LastColor;
        sf::Uint32 LastStyle;
        mutable sf::Vector2f Size;



};

#endif //RICHTECT_H
 

RichText.cpp

/* AUTHOR: XAFI */

#include "RichText.h"

RichText::RichText() :
        LastColor(sf::Color::White),
        LastStyle(sf::Text::Style::Regular)


{
}

RichText & RichText::operator << (const std::string& string)
{
        sf::Text t;
        t.setString(string);
        t.setColor(LastColor);
        t.setStyle(LastStyle);
        Text.push_back( t );
        return *this;
}
RichText & RichText::operator << (const sf::Color &color)
{
        LastColor = color;
        return *this;
}

RichText & RichText::operator << (const sf::Uint32 style)
{
        LastStyle = style;
        return *this;
}

void RichText::setFont(const sf::Font& font)
{
        for (std::vector<sf::Text>::iterator it = Text.begin(); it != Text.end(); ++it)
        {
                it->setFont(font);
        }
}

void RichText::draw(sf::RenderTarget &target, sf::RenderStates states) const
{
        UpdatePos();
        states.transform *= getTransform();
        for (std::vector<sf::Text>::const_iterator it = Text.begin(); it != Text.end(); ++it)
        {
                target.draw(*it, states);
        }
}
void RichText::setCharacterSize(int size)
{
        for (std::vector<sf::Text>::iterator it = Text.begin(); it != Text.end(); ++it)
        {
                it->setCharacterSize(size);
        }
}
const std::string RichText::getString()
{
        std::string tmp;
        for (std::vector<sf::Text>::iterator it = Text.begin(); it != Text.end(); ++it)
        {
                tmp += it->getString().toAnsiString();
        }
        return tmp;
}
void RichText::UpdatePos() const
{
        int x = 0;
        for (std::vector<sf::Text>::iterator it = Text.begin(); it != Text.end(); ++it)
        {
                x += it->getLocalBounds().width;
                it->setPosition(x - it->getLocalBounds().width, 0);
        }
        Size.x = x;
}
void RichText::setColor(int index, sf::Color color)
{
        Text[index].setColor(color);
}
void RichText::setString(int index, std::string string)
{
Text[index].setString(string);
}
void RichText::Clear()
{
        LastColor = sf::Color::White;
        LastStyle = sf::Text::Style::Regular;
        Text.erase(Text.begin(), Text.end());
}
sf::Vector2f RichText::getSize() const
{
        return Size;
}

int RichText::getCharacterSize()
{
        return Text[0].getCharacterSize();
}
 

[Tank]

"BIG BOOBIES" and "big tits"?

[AlexAUT]

First, I'm not a c++ expert  . For a high quality answer you have to wait for some other guys, e.g. Nexus.

        const std::string getString();
        int getCharacterSize();

        void UpdatePos() const;
 

Here you could inprove the const correctness. I would write:

        std::string getString() const;
        //Or maybe const sf::String &getString() const;
        int getCharacterSize() const;

        void UpdatePos(); //Why const here? It changes something inside the class
 

And you could improve the performance a tiny bit:

    //your version:
     void setString(int, std::string);
   // new:
     void setString(int, const std::string &);

//Also here
    //your version
    sf::Vector2f getSize() const;
    //new:
    const sf::Vector2f &getSize() const;
 

And you should use ONE style for the names of functions. e.g. you use getString() but UpdatePosition()...


But the result in your video looks very nice!


AlexAUT

[Xafi]

OK right now i revise the code
Thank you!.

"BIG BOOBIES" and "big tits"?

I could not think otherwise 

[eXpl0it3r]

void RichText::setCharacterSize(int size)

should probably be

void RichText::setCharacterSize(unsigned int size)

to match the SFML signature and since negative font sizes don't make sense.

void RichText::setString(int index, std::string string)

should use std::size_t instead of int. Negative indices don't make sense and can cause crashes, plus the index type is std::size_t.
The question here is though, whether it makes sense to let the user access the text with a random index. If you do allow it, you'll need to make sure that the index exists, otherwise your application will crash.

Does it support multiple lines?
Btw have you ever seen this Richt Text class?

[Xafi]

Does it support multiple lines?

still not, i am thinking the best way implement it.
Thank you!

---------------------------------------------------

should use std::size_t instead of int. Negative indices don't make sense and can cause crashes, plus the index type is std::size_t.
The question here is though, whether it makes sense to let the user access the text with a random index. If you do allow it, you'll need to make sure that the index exists, otherwise your application will crash.

Something like that?
 

void RichText::setString(std::size_t index, std::string string)
{
        if (index < Text.size())
        {
                Text[index].setString(string);
        }
}