如何优化C++坐标写入文件代码？实现自定义函数复用逻辑

你的自定义函数coordinate_syntax存在几个关键错误，导致无法正常工作：

• 返回类型错误：map << x_coordinate << ...的返回值是ofstream&（输出流引用），并非double，函数声明的返回类型不匹配
• 变量作用域问题：函数中引用的map是build_environment_map函数内的局部变量，外部函数无法直接访问
• 函数设计冗余：该函数仅需完成写入操作，不需要返回值

修正后的代码 #

#include <iostream>
#include <fstream>
#include <math.h>

using namespace std;
#define PI 3.14159265

// 修正后的坐标写入函数：接收输出流引用和坐标参数
void coordinate_syntax(ofstream& out, double x_coordinate, double y_coordinate){
    out << x_coordinate << ' ' << y_coordinate << endl;
}

int build_environment_map(){

    double x0[5]={0.8, 0.8, 1.7, 2.2, 2.25};
    double y0[5]={1.5, 2.25, 1.0, 1.3, 0.25};
    double r=0.125, wallLength=2.5;
    double RD=PI /180;
    ofstream map;
    map.open("EnvironmentMap.txt");

    // 调用封装后的函数
    for (int i=0; i<360; i=i+20) {
        coordinate_syntax(map, x0[4] + r*cos(i*RD), y0[4] + r*sin(i*RD));
    }
    for (int i=0; i<360; i=i+10) {
        coordinate_syntax(map, x0[0]+ r*cos(i*RD), y0[0]+ r*sin(i*RD));
    }
    for (double j=0 ; j<wallLength; j=j+0.01){
        coordinate_syntax(map, j, 0);
    }
    for (double x=0.3; x<0.55; x=x+0.01){
        coordinate_syntax(map, x, 0.3);
    }
    map.close();
    return 0;
}

int main(int argc, char **argv){
    build_environment_map();
    return 0;
}

关键修改说明 #

1. 函数参数调整：新增ofstream& out参数，将输出流对象传入函数，解决作用域问题
2. 返回类型修正：改为void，函数仅需完成写入操作，无需返回值
3. 函数内部实现：使用传入的out流对象完成坐标写入
4. 调用方式更新：循环中调用函数时，传入map流对象和对应坐标参数

修改后代码可正常复用坐标写入逻辑，同时保留原有功能。

内容的提问来源于stack exchange，提问作者user20398295