作为类方法的内联函数

Question

我开发了自己的 Matrix 类。构造函数从文件中读取矩阵。矩阵有自由单元和“墙”。构造函数还读取广度优先搜索的起点和终点（以找到从 Start_point 到 Finish_Point 的最短路径）。 这是标题代码：

//MyMatrix.h 文件

#ifndef __MYMATRIX_H__
#define __MYMATRIX_H__

#include <tchar.h>
#include <iostream>
#include <deque>

//using namespace std;
#define MAX_MATRIX_SIZE 1000

#define FREE_CELL_SIGNIFICATION '0'
#define BALL_SIGNIFICATION 'B'
#define UP_SIGNIFICATION 'U'
#define DOWN_SIGNIFICATION 'D'
#define LEFT_SIGNIFICATION 'L'
#define RIGHT_SIGNIFICATION 'R'
#define START_POINT_SIGNIFICATION 'S'
#define FINISH_POINT_SIGNIFICATION 'F'

typedef std::pair<int,int> Field_Point_Type;

//#define IS_RIGHT_NEIGHBOUR_REACHABLE(Current_Point) (((Current_Point.second+1) <= Column_Count)&&((Matrix_Field[Current_Point.first][Current_Point.second+1]==FREE_CELL_SIGNIFICATION)||(Matrix_Field[Current_Point.first][Current_Point.second+1]==FINISH_POINT_SIGNIFICATION))) ? true : false;



class Matrix {
    private:
        int Column_Count; //Cols
        int Row_Count;//Rows
        char** Matrix_Field;
        Field_Point_Type Start_Point;
        Field_Point_Type Finish_Point;
        bool Matrix_Is_Correct;
    public:
        Matrix(_TCHAR* Input_File_Name);
        int Breadth_first_search(unsigned int Start_X,unsigned int Start_Y,unsigned int Finish_X,unsigned int Finish_Y);
        ~Matrix();
        inline int IS_RIGHT_NEIGHBOUR_REACHABLE(Field_Point_Type Current_Point);
};

//MyMatrix.cpp 文件

...

inline int Matrix::IS_RIGHT_NEIGHBOUR_REACHABLE(Field_Point_Type Current_Point)
{      
    return  (((Current_Point.second+1) <= Column_Count)&&((Matrix_Field[Current_Point.first][Current_Point.second+1]==FREE_CELL_SIGNIFICATION)||(Matrix_Field[Current_Point.first][Current_Point.second+1]==FINISH_POINT_SIGNIFICATION))) ? true : false;
}

...

我想定义的是可供下一步算法使用的相邻单元格。 当然，我可以使用这样的代码：

if (((Current_Point.second+1) <= Column_Count)&&((Matrix_Field[Current_Point.first][Current_Point.second+1]==FREE_CELL_SIGNIFICATION)||(Matrix_Field[Current_Point.first][Current_Point.second+1]==FINISH_POINT_SIGNIFICATION))) {
    //Adding of right cell to deque...
    ... 
}

但它看起来很难看。我将为左、上、下单元格添加这样的检查。 我想实现内联函数（如下所示：inline int IS_RIGHT_NEIGHBOUR_REACHABLE(Field_Point_Type Current_Point);）。

if (IS_RIGHT_NEIGHBOUR_REACHABLE(Current_Point)) {
    //Adding of right cell to deque...
    ... 
}

看起来好多了！ 但我以前没有使用过这样的内联函数定义，是偶然发现的。 这是好的编程风格吗？ 在我的类中开发简单的 int IS_RIGHT_NEIGHBOUR_REACHABLE(Field_Point_Type Current_Point); 方法是否更好？ 是否最好留下这样的支票：

if (((Current_Point.second+1) <= Column_Count)&&((Matrix_Field[Current_Point.first][Current_Point.second+1]==FREE_CELL_SIGNIFICATION)||(Matrix_Field[Current_Point.first][Current_Point.second+1]==FINISH_POINT_SIGNIFICATION))) {
    //Adding of right cell to deque...
    ... 
}

Answer 1

我认为我们还没有确定的“良好风格”。能够从单独编译的 .cpp 文件内联函数的编译器是最流行编译器的最新模型。

直到几年前，您还必须将所有内联函数放在 .h 文件中，以便编译器在编译调用时可以看到它。如果您的编译器不是最新型号，这可能仍然是规则。

Answer 2

inline 函数需要在头文件中实现。如果它确实提高了您的性能，您需要通过基准进行检查。

然而，好的编译器可能会自动内联函数（希望如此）。

对于你的问题，我希望有很多小功能。它们通常更容易维护，并且可以单独检查它们是否正确。