主页 / 问题 / 1056591
Accepted
Павел Ериков
Asked:2020-12-10 04:33:47 +0000 UTC2020-12-10 04:33:47 +0000 UTC

C++中的回调函数

  • 772

我想确保我正确和充分地编写了所有内容。请告诉我代码中存在哪些错误，最好是关于如何正确处理的建议。

#include <iostream>
using namespace std;


class Point {
private:
    int xPos, yPos;
public:
    Point(int x, int y) : xPos(x), yPos(y) { }
    Point() : Point(0,0) { }
    int x() { return xPos; }
    int y() { return yPos; }
    void setX(int x) { this->xPos = x; }
    void setY(int y) { this->yPos = y; }
};

class Square {
private:
    void(*callBackFunc)(Point point);
    Point posLeftUp;
public:
    Square(){}
    Square(Point pos) : posLeftUp(pos) {}
    //Имитация движения квадрата по окну
    void Move(char key) {
        if (key == 'A')
            posLeftUp.setX(posLeftUp.x() - 2);
        else if (key == 'D')
            posLeftUp.setX(posLeftUp.x() + 2);
        else if (key == 'W')
            posLeftUp.setY(posLeftUp.y() + 2);
        else if (key == 'S')
            posLeftUp.setY(posLeftUp.y() - 2);
        callBackFunc(posLeftUp);
    }
    void setCallbackFunc(void(*fn)(Point point)) {
        callBackFunc = fn;
    }
};
//Имитация окна
class MainWindow {
private:
    static void getPosition(Point point);
    Square square;
public:
    MainWindow() {
        Point pos(10, 10);
        square = Square(pos);
        square.setCallbackFunc(getPosition);
        square.Move('A');
        square.Move('D');
        square.Move('W');
        square.Move('S');
    }
};

void MainWindow::getPosition(Point point){
        cout << point.x() << " : " << point.y() << endl;
}

int main() {
    MainWindow test;
    return 0;
}
c++

1 个回答

  1. Best Answer

    以下是我可以评论的几点。

    在类定义中，我将首先指定public，因为这是类的接口。

    class Point {
public:
    Point(int x, int y) : xPos(x), yPos(y) { }
    Point() : Point(0,0) { }
    int x() { return xPos; }
    int y() { return yPos; }
    void setX(int x) { xPos = x; }
    void setY(int y) { yPos = y; }
private:
    int xPos;
    int yPos;
};

    不要在一行上合并声明：

        // плохо:
    // int xPos, yPos; 

    // хорошо:
    int xPos;
    int yPos;

    在setX并且setY不使用this：

    void setX(int x) { xPos = x; }
void setY(int y) { yPos = y; }

    switch中使用Move()。他非常适合这里。

    void Move(char key) {
    switch(key)
    {
    case 'A': posLeftUp.setX(posLeftUp.x() - 2); break;
    case 'D': posLeftUp.setX(posLeftUp.x() + 2); break;
    case 'W': posLeftUp.setY(posLeftUp.y() + 2); break;
    case 'S': posLeftUp.setY(posLeftUp.y() - 2); break;
    }

    callBackFunc(posLeftUp);
}

    const在不应该改变的地方使用变量。编译器将帮助您避免愚蠢的错误：

    void Move(const char key) {

    在main()可以删除return 0;。当它main()到达结尾并且return丢失时，它将return 0;自动被暗示。

    正如@Andrey 所提到的，通过“将声明与类中的定义分开”，这意味着创建单独的文件来声明类和定义其方法。以你的为例class Square：

    让我们创建两个文件：square.hpp和square.cpp.

    square.hpp：

    #ifndef SQUARE_HPP
#define SQUARE_HPP

#include <functional>
#include "Point.hpp"   // Файл, содержащий объявление класса Point

class Square {
public:
    Square();
    Square(const Point pos);
    void Move(const char key);
    void setCallbackFunc(std::function<void(const Point p)>);

private:
    std::function<void(const Point p)> callBackFunc;
    Point posLeftUp;
};

#endif //SQUARE_HPP

    在这个文件中，我们声明了类Square，即我们列出了它的方法和成员。这足以知道如何在其他地方使用这个类。

    这种设计被称为 Header guard（“header guard” -所以它会是俄语？）

    #ifndef SQUARE_HPP
#define SQUARE_HPP

// ...

#endif //SQUARE_HPP

    它可以防止该文件在编译期间多次hpp嵌套在同一个文件中。cpp

    这些方法如何实现的细节将放在第二个文件中

    square.cpp：

    #include "square.hpp"

Square::Square() : callBackFunc(nullptr)
{}

Square::Square(const Point pos) :
    callBackFunc(nullptr),
    posLeftUp(pos)
{}

void Square::Move(const char key)
{
    switch(key)
    {
    case 'A': posLeftUp.setX(posLeftUp.x() - 2); break;
    case 'D': posLeftUp.setX(posLeftUp.x() + 2); break;
    case 'W': posLeftUp.setY(posLeftUp.y() + 2); break;
    case 'S': posLeftUp.setY(posLeftUp.y() - 2); break;
    }

    if(callBackFunc)
    {
        callBackFunc(posLeftUp);
    }
}

void Square::setCallbackFunc(std::function<void(const Point p)> fn)
{
    callBackFunc = fn;
}

    因此，我们将类接口与其方法的实现分开。

    顺便说一句，一个类Point实际上可能只是一个结构，因为它的私有成员通过您定义的方法xPos是yPos完全可读和可写的。setX()并且setY()不做额外的工作，即隐藏为私人成员x也没有任何好处：y

    struct Point{
    Point() : x(0), y(0) {};
    Point(const int x, const int y) : x(x), y(y) {};
    int x;
    int y;
};

    如果我们有义务检查传递的值，那是另一回事setX()，然后将其setY()隐藏为私有是合理的：xy

    void Point::setX(const int val)
{
    constexpr int min_value = -100;
    constexpr int max_value = 100;

    if (val < min_value || val > max_value)
    {
        // Ошибка. Переданное значение неприемлемо.
        throw std::runtime_error("setX(): значение вне диапазона");
    }

    x = val;
}

    关于回调。使用标题中的工具<functional>：

    std::function<>- 可以包含函数的包装类。

    std::bind()- 将一个函数的执行绑定到另一个函数的函数，向它传递可以预定义的参数。

    #include <functional>
#include <iostream>
using namespace std;


class Point {
public:
    Point(int x, int y) : xPos(x), yPos(y) { }
    Point() : Point(0,0) { }
    int x() { return xPos; }
    int y() { return yPos; }
    void setX(int x) { xPos = x; }
    void setY(int y) { yPos = y; }

private:
    int xPos;
    int yPos;

};

class Square {
public:
    Square(){}
    Square(Point pos) : posLeftUp(pos) {}
    //Имитация движения квадрата по окну
    void Move(const char key) {
        switch(key)
        {
        case 'A': posLeftUp.setX(posLeftUp.x() - 2); break;
        case 'D': posLeftUp.setX(posLeftUp.x() + 2); break;
        case 'W': posLeftUp.setY(posLeftUp.y() + 2); break;
        case 'S': posLeftUp.setY(posLeftUp.y() - 2); break;
        }

        if(callBackFunc)
        {
            callBackFunc(posLeftUp);
        }
    }

    void setCallbackFunc(std::function<void(const Point p)> fn) {
        callBackFunc = fn;
    }

private:
    std::function<void(const Point p)> callBackFunc = nullptr;
    Point posLeftUp;
};
//Имитация окна
class MainWindow {
private:
    static void getPosition(Point point);
    Square square;
public:
    MainWindow() {
        using namespace std::placeholders;

        Point pos(10, 10);
        square = Square(pos);
        square.setCallbackFunc(std::bind(&MainWindow::getPosition, _1));
        square.Move('A');
        square.Move('D');
        square.Move('W');
        square.Move('S');
    }
};

void MainWindow::getPosition(Point point){
        cout << point.x() << " : " << point.y() << endl;
}

int main() {
    MainWindow test;
}
    • 4

相关问题