Question

我的代码已经可以使用了，请看：http://pastebin.com/mekKRQkG

现在，我的功能正常运行，但我正在使用我已声明globally的信息，我想，我希望将它们转换为lines 11-15所示的格式，但我我不确定如何转换他们这样做。简单地说，我正在尝试转换我的

功能

"void add_county_election_file"

格式为

"void add_county_election_file(const string, const vector &, const vector &, const vector &, const vector &)"

我不知道从哪里开始或者如何开始。

有人可以帮助我，并告诉我如何为第一个功能执行此操作，以便我可以全面实施吗？

谢谢你们！

Answer 1

您的函数声明应如下所示：

void add_county_election_file(const string, vector<int>&, vector<string>..);

确保矢量模板的参数列表正确（这是您在＆lt;＆gt;之间放置的类型）

然后将您的函数的实现与声明匹配：

   void add_county_election_file(const string, vector<int>&, vector<string>..){...}

现在使用main中适当的arguemtns调用你的函数：

string s;
vector<int> arg;
vector<string> sv;
void someFunction (s, arg, sv ...);

Answer 2

我认为你所宣称的功能是正确的 void add_county_election_file(const string, vector<int>&, vector<int>&,..);

所以你只需要用必需的参数调用上面的函数，因为你现在没有传递参数，你的当前定义也不接受任何参数。

作为一种良好做法，在int main()函数中，您可以使用switch而不是if else。

Answer 3

将变量和函数存储在类中，重载运算符并创建函数以访问这些变量。
声明int main()中的所有变量并设置要传递到每个函数的参数，例如

void print_results()被修改为

void print_results(std::vector<int> vec, int nCount, etc..)
与第一个类似，创建一个结构来保存所有数据成员，然后将结构（通过ref）传递给每个函数。

 struct CountryTracker
    {
        std::vector<int> ID;
        std::string name;
        //etc...
    }

`void print_results(CountryTracker& Obj) //pass single struct into functions`

Answer 4

OOP的方法是创建一个名为ElectionInfo的类，其中：

这些将是其成员字段：

vector <string> countyNameVector;
vector <int> countyNCount;
vector <int> countyFCount;
vector <int> countyOCount;
int NCount;
int FCount;
int OCount;
int NTotal;
int FTotal;
int OTotal;

这些将是其成员函数：

void add_county_election_file(const string);
void search_county(const string);
void print_results();

这样你就不必将引用传递给周围的向量，而是你可以这样做：

ElectionInfo an_elect_info;
char selection = get_menu_choice();

// some if-statements to decide which of the following to call:

an_elect_info.add_county_election_file(county_name);
an_elect_info.search_county(county_name);
an_elect_info.print_results();

但如果您更愿意继续使用当前的功能方法：

在main方法中声明并初始化以下内容：

vector <string> countyNameVector;
vector <int> countyNCount;
vector <int> countyFCount;
vector <int> countyOCount;
int NCount;
int FCount;
int OCount;
int NTotal;
int FTotal;
int OTotal;

注释掉的函数声明的语法应该调整为：

void add_county_election_file(const string, vector<string>&, vector<int>&, vector<int&, vector<int>&);

（当然，定义也应该效仿）

您可以这样调用它：

add_county_election_file(countyname, countyNameVector, countyNCount, countyFCount, countyOCount);

对象自动按引用传递。

Answer 5

重构的基本过程应该在第一步只涉及代码分组和放置，并且应该只涉及编写新逻辑。以此为原则，您可以先按照以下方式修改代码。

string ReadInputString(const char* title)
{
    string s

    cout << title;
    cin >> s;

}

void add_county_election_file(const std::string& filename
    , std::vector<string>& countyNameVector
    , std::vector<int>& countyNCount
    , std::vector<int>& countyFCount
    , std::vector<int>& countyOCount
    )
{
        int NCount = 0;
        int FCount = 0;
        int OCount = 0;
        int NTotal = 0;
        int FTotal = 0;
        int OTotal = 0;
        char vote;
    std::ifstream input((filename).c_str());
    string countyName;
    if(input.is_open())
        {
        input >> countyName;
        countyNameVector.push_back(countyName);
        while(input >> vote)
                        {
                        if(vote == 'N' || vote == 'n')
                                {
                NCount = NCount + 1;
                }
                                else if(vote == 'F' || vote == 'f')
                                        {
                                        FCount = FCount + 1;
                                        }
                                        else
                                                {
                                                OCount = OCount + 1;
                                                }
                        }
            countyNCount.push_back(NCount);
            countyFCount.push_back(FCount);
            countyOCount.push_back(OCount);
        }
        cout << countyName << endl;
}

void add_county_election_file()
{
  string fn = ReadInputString("Enter the county file to process: ");
  add_county_election_file(fn,g_countyNameVector,g_countyNCount,g_countyFCount,g_countyOCount);
}

正如您所看到的，我刚刚提取了您的代码并将它们移动到单个函数并更改了名称以使其具有一定的意义。与函数ReadInputString一样 - 行“cin＆gt;＆gt; s”最初是“cin＆gt;＆gt; filename”。抽象名称“s”表示ReadInputString不知道或不关心它从控制台读取的字符串的语义含义。

为了不改变你的主要功能 - 我添加了一个重载的add_county_election_file，它调用一个函数，然后调用另一个函数。这个想法是你应该保持一些不变的东西并改变其他的（好的），然后在需要时交替。我已经更改了全局变量的名称，以便使用“g_”将它们与局部变量区分开来 - 重点是“g_”只能在代码中的很少位置找到。

C ++实现不使用全局声明的函数

5 个答案: