jugant amb clang-tidy

source/sprite.h

@@ -10,7 +10,7 @@ class Texture;
 class Sprite {
    public:
     // --- Constructores y destructor ---
-    Sprite(std::shared_ptr<Texture> texture, float x, float y, float w, float h);
+    Sprite(std::shared_ptr<Texture> texture, float pos_x, float pos_y, float width, float height);
     Sprite(std::shared_ptr<Texture> texture, SDL_FRect rect);
     explicit Sprite(std::shared_ptr<Texture> texture);
     virtual ~Sprite() = default;
@@ -20,21 +20,21 @@ class Sprite {
     virtual void clear();   // Reinicia las variables a cero
 
     // --- Getters de posición y tamaño ---
-    float getX() const { return pos_.x; }
-    float getY() const { return pos_.y; }
-    float getWidth() const { return pos_.w; }
-    float getHeight() const { return pos_.h; }
-    SDL_FRect getPosition() const { return pos_; }
-    SDL_FRect &getRect() { return pos_; }
+    [[nodiscard]] auto getX() const -> float { return pos_.x; }
+    [[nodiscard]] auto getY() const -> float { return pos_.y; }
+    [[nodiscard]] auto getWidth() const -> float { return pos_.w; }
+    [[nodiscard]] auto getHeight() const -> float { return pos_.h; }
+    [[nodiscard]] auto getPosition() const -> SDL_FRect { return pos_; }
+    auto getRect() -> SDL_FRect& { return pos_; }
 
     // --- Setters de posición y tamaño ---
-    void setX(float x) { pos_.x = x; }
-    void setY(float y) { pos_.y = y; }
-    void setWidth(float w) { pos_.w = w; }
-    void setHeight(float h) { pos_.h = h; }
-    void setPosition(float x, float y);
-    void setPosition(SDL_FPoint p);
-    void setPosition(SDL_FRect r) { pos_ = r; }
+    void setX(float pos_x) { pos_.x = pos_x; }
+    void setY(float pos_y) { pos_.y = pos_y; }
+    void setWidth(float width) { pos_.w = width; }
+    void setHeight(float height) { pos_.h = height; }
+    void setPosition(float pos_x, float pos_y);
+    void setPosition(SDL_FPoint point);
+    void setPosition(SDL_FRect rect) { pos_ = rect; }
 
     // --- Zoom ---
     void setZoom(float zoom) { zoom_ = zoom; }
@@ -44,12 +44,12 @@ class Sprite {
     void incY(float value) { pos_.y += value; }
 
     // --- Sprite clip ---
-    SDL_FRect getSpriteClip() const { return sprite_clip_; }
+    [[nodiscard]] auto getSpriteClip() const -> SDL_FRect { return sprite_clip_; }
     void setSpriteClip(SDL_FRect rect) { sprite_clip_ = rect; }
-    void setSpriteClip(float x, float y, float w, float h) { sprite_clip_ = SDL_FRect{x, y, w, h}; }
+    void setSpriteClip(float pos_x, float pos_y, float width, float height) { sprite_clip_ = SDL_FRect{pos_x, pos_y, width, height}; }
 
     // --- Textura ---
-    std::shared_ptr<Texture> getTexture() const { return texture_; }
+    [[nodiscard]] auto getTexture() const -> std::shared_ptr<Texture> { return texture_; }
     void setTexture(std::shared_ptr<Texture> texture) { texture_ = texture; }
 
    protected:
@@ -57,5 +57,5 @@ class Sprite {
     std::shared_ptr<Texture> texture_;  // Textura donde están todos los dibujos del sprite
     SDL_FRect pos_;                     // Posición y tamaño donde dibujar el sprite
     SDL_FRect sprite_clip_;             // Rectángulo de origen de la textura que se dibujará en pantalla
-    double zoom_ = 1.0f;                // Zoom aplicado a la textura
+    double zoom_ = 1.0F;                // Zoom aplicado a la textura
 };