如何读取infile中的指令，然后使用infile中的参数将函数形状写入outfile

问题描述 投票：1回答：1

所以我只需要一些关于如何修复这个程序的指导，所以我需要读一个标题为“infile.txt”的文件，文件里面是描述应该绘制的形状的指令（单个大写字符） ，即R，T，D，S，E）然后它给char表示应填充形状然后int中的列数和行数。 txt文件如下所示：

T & 4
S @ 6
T x 5
R * 5 7
D $ 7
D + 5
R = 4 3
E

现在，我甚至不确定我是否可以使用switch语句完成此操作，因为infile被读作字符串。但后来我对如何改变switch语句感到困惑。无论如何，在读取infile之后，我必须输出绘制到outfile的形状。希望这是有道理的，我是超级入门级，几乎不知道我在做什么。所以这是我的代码：

#include <iostream>
#include <fstream>
#include <cstdlib>
using namespace std;


void draw_rect (char out_char, int rows, int columns); // Draws a rectangle shape

void draw_square (char out_char, int rows); //Draws a square shape

void draw_triangle (char out_char, int rows);// Draws a triangle shape

void draw_diamond (char out_char, int rows); // Draws a diamond shape

//void dimension_instructions(char value);

int main()
{
    ofstream outfile;
    ifstream infile;
    int row, col;
    bool exit = false;
    char value;
    char code;
    infile.open("infile.txt");
    outfile.open("outfile.txt");
    if(!infile.good())
    {
        cout << "failed to open\n";
    }else
    {
        string buffer;
        while(!infile.eof())
        {
            getline(infile, buffer);
            cout << buffer << endl;
        }

        while(!exit)
        {
            cout << "Enter your shape R for rectangle, T for triangle, D for diamond, S for square, and E to exit" << endl;
            cin >> code;
            switch(code)
            {
            case 'R':
               dimension_instructions(code);
               cin >> value >> row >> col;
               draw_rect(value, row, col);
               break;
            case 'T':
                dimension_instructions(code);
                cin >> value >> row;
                draw_triangle(value, row);
                break;
            case 'D':
                dimension_instructions(code);
                cin >> value >> row;
                draw_diamond(value, row);
                break;
            case 'S':
                dimension_instructions(code);
                cin >> value >> row;
                draw_square(value, row);
                break;
            case 'E':
                cout << "Exiting";
                exit = true;
                break;
            default:
                cout << "Invalid input, try again" << endl;
            }
        }
     infile.close();
    }
    outfile.close();

    return  0;

}

/*void dimension_instructions(char value)
{
    if (value == 'R')
    {
        cout << "Enter your character rows and columns values." << endl;
    }else
    {
        cout << "Enter your character and row values" << endl;
    }
}*/

void draw_diamond (char out_char, int rows)
{
    int space = 1;
    space = rows - 1;
    for (int i = 1; i <= rows; i++)
    {
        for (int k = 1; k <= space; k++)
        {
            cout << " ";
        }
        space--;
        for( int k = 1; k <= 2*i-1; k++)
        {
            cout << out_char;
        }
        cout << endl;
    }
    space = 1;
    for (int i = 1; i <= rows; i++)
    {
       for(int k = 1; k <= space; k++)
       {
           cout << " ";
       }
       space++;
       for(int k = 1; k <= 2*(rows-i)-1; k++)
       {
           cout << out_char;
       }
       cout << endl;
    }
}

void draw_triangle (char out_char, int rows)
{
     for (int i = 0; i < rows; i++)
    {
        for (int j = 0; j <= i; j++)
        {
            cout << out_char;
        }
        cout << endl;
}
}

void draw_square (char out_char, int rows)
{
    for (int i = 0; i < rows; i++)
    {
        for (int i = 0; i < rows; i++)
        {
            cout << out_char;
        }
        cout << endl;
    }
}

void draw_rect (char out_char, int rows, int columns)
{
    for (int i = 0; i < rows; i++)
    {
        for (int i = 0; i < columns; i++)
        {
            cout << out_char;
        }
        cout << endl;
    }
}
c++ file-io shapes
1个回答
1
投票

好的，根据你的评论我看到你被困在哪里以及为什么。 （如果你还没有完成它，你还需要在draw_square和draw_rect中修复你的循环变量）。

您的主要问题是不了解如何处理每行不同数量的输入。当你遇到这个问题时，你已经正确地选择了getline来读取每一行到buffer，但是那么呢？这就是stringstream让一切变得不同的地方。

为什么？有两个原因，（1）它允许你用基本的iostream buffer逐字解析>>的内容。（2）它在需要时允许你循环直到流尾读数尽可能多（或者很少） ）存在的标记，当你到达行的末尾时停止（这在文件流本身使用>>是不可能的，因为>>消耗空白并且很乐意跳过每个'\n'）

通过这种方式，您的代码实际上只需要一些重构（用于修复混乱逻辑的一个奇特的词）。

要启动Do not Hardcode Filenames或在代码中使用magic-numbers。使用main()的参数将文件名传递给程序并在需要的地方声明常量。还要避免使用不会消耗前导空格的char。 cin >> a_char;阅读' '（空间）同样正在读取其他内容。

还适当地调整变量的范围。您不需要声明所有变量，因此它们在整个main()中都可见。在适当的范围内声明/定义它们。

例如：

...
#include <sstream>
...
int main (int argc, char **argv) {  /* don't hardcode filenames */

    ifstream infile;    /* infile and buffer are the only variables */
    string buffer;      /* that need to be scoped at main() */

传递文件名作为参数时，只需验证用户提供的文件名或在提示之前向他提供使用信息。

    if (argc < 2) {     /* validate at least 1 argument is provided */
        cerr << "error: insufficient input.\n"
                "usage: " << argv[0] << " filename.\n";
        return 1;
    }

你有你的论点，现在验证你打开你的文件阅读：

    infile.open (argv[1]);  /* open filename provided as 1st argument */

    if(!infile.good()) {    /* validate file is open for reading */
        cerr << "failed to open infile\n";
        return 1;
    }

现在，您控制读取循环的重要变化。 getline提供您所需要的一切。简单地循环，而getline为buffer提供了良好的输入，例如

    while (getline(infile, buffer)) {   /* loop reading each line */
        int row, col;               /* remaining variables scoped inside */
        string value, code;         /* your read loop, use strings */

        stringstream ss(buffer);    /* create stringstream from buffer */

现在你正在阅读每一行，你已经从stringstream创建了一个buffer来解析你的角色 - 除了注意value, code如何被声明为string而不是char--它提供了一种简单的方法来跳过只读取非空白字符的前导空格。然后，您只需访问所需的字符，例如value[0]。

验证你对code有一个很好的解读

        if (!(ss >> code)) {        /* validate code read into string */
            cerr << "error: ss >> code.\n";
            break;
        }

然后，只需要重复读取所需数据的相同验证，即在每个switch() case:中调用正确的函数，例如，

        switch (code[0])    /* switch on 1st char of code */
        {
            case 'R':
                if ((ss >> value >> row >> col))    /* validate read */
                    draw_rect (value[0], row, col); /* draw rect */
                else    /* or handle error */
                    cerr << "error: 'R' invalid format '" << buffer << "'\n'";
                break;
            case 'T':
                if ((ss >> value >> row))   /* ditto for rest of shapes */
                    draw_triangle(value[0], row);
                else
                    cerr << "error: 'T' invalid format '" << buffer << "'\n'";
                break;
            case 'D':
                if ((ss >> value >> row))
                    draw_diamond(value[0], row);
                else
                    cerr << "error: 'D' invalid format '" << buffer << "'\n'";
                break;
            case 'S':
                if ((ss >> value >> row))
                    draw_square(value[0], row);
                else
                    cerr << "error: 'S' invalid format '" << buffer << "'\n'";
                break;
            case 'E':
                cout << "Exiting\n";
                goto exitE;     /* goto to break nested loops / scopes */
                break;
            default:
                cout << "Invalid input, try again" << endl;
        }
    }
    exitE:;     /* the lowly goto provides a simple exit */

除了关闭infile（这将自动发生，但手动显示你对关闭的考虑没有伤害）。

但请注意使用goto而不是exit的旗帜。虽然goto没有太多的压力，但它有一个非常宝贵的目的 - 能够彻底打破嵌套循环和范围。不要用它来跳出函数（longjmp是技术限制），但它可以大大简化你打破嵌套循环和跳过几行的逻辑。 （在错误条件下，跳过通常在循环结束时执行的代码，在同一设置中也很有用）

所以了解它的用途。您可以自由使用旗帜，但您可以在许多设置中找到goto清洁剂。

有了这个，你可以完全放弃（暂时忽略outfile）与类似的东西：

#include <iostream>
#include <fstream>
#include <sstream>
#include <cstdlib>

using namespace std;

void draw_rect (char out_char, int rows, int columns); // Draws a rectangle shape
void draw_square (char out_char, int rows); //Draws a square shape
void draw_triangle (char out_char, int rows);// Draws a triangle shape
void draw_diamond (char out_char, int rows); // Draws a diamond shape

int main (int argc, char **argv) {  /* don't hardcode filenames */

    ifstream infile;    /* infile and buffer are the only variables */
    string buffer;      /* that need to be scoped at main() */

    if (argc < 2) {     /* validate at least 1 argument is provided */
        cerr << "error: insufficient input.\n"
                "usage: " << argv[0] << " filename.\n";
        return 1;
    }
    infile.open (argv[1]);  /* open filename provided as 1st argument */

    if(!infile.good()) {    /* validate file is open for reading */
        cerr << "failed to open infile\n";
        return 1;
    }

    while (getline(infile, buffer)) {   /* loop reading each line */
        int row, col;               /* remaining variables scoped inside */
        string value, code;         /* your read loop, use strings */

        stringstream ss(buffer);    /* create stringstream from buffer */

        if (!(ss >> code)) {        /* validate code read into string */
            cerr << "error: ss >> code.\n";
            break;
        }

        switch (code[0])    /* switch on 1st char of code */
        {
            case 'R':
                if ((ss >> value >> row >> col))    /* validate read */
                    draw_rect (value[0], row, col); /* draw rect */
                else    /* or handle error */
                    cerr << "error: 'R' invalid format '" << buffer << "'\n'";
                break;
            case 'T':
                if ((ss >> value >> row))   /* ditto for rest of shapes */
                    draw_triangle(value[0], row);
                else
                    cerr << "error: 'T' invalid format '" << buffer << "'\n'";
                break;
            case 'D':
                if ((ss >> value >> row))
                    draw_diamond(value[0], row);
                else
                    cerr << "error: 'D' invalid format '" << buffer << "'\n'";
                break;
            case 'S':
                if ((ss >> value >> row))
                    draw_square(value[0], row);
                else
                    cerr << "error: 'S' invalid format '" << buffer << "'\n'";
                break;
            case 'E':
                cout << "Exiting\n";
                goto exitE;     /* goto to break nested loops / scopes */
                break;
            default:
                cout << "Invalid input, try again" << endl;
        }
    }
    exitE:;     /* the lowly goto provides a simple exit */

    infile.close();

    return  0;

}

void draw_diamond (char out_char, int rows)
{
    int space = 1;
    space = rows - 1;
    for (int i = 1; i <= rows; i++)
    {
        for (int k = 1; k <= space; k++)
        {
            cout << " ";
        }
        space--;
        for( int k = 1; k <= 2*i-1; k++)
        {
            cout << out_char;
        }
        cout << endl;
    }
    space = 1;
    for (int i = 1; i <= rows; i++)
    {
    for(int k = 1; k <= space; k++)
    {
        cout << " ";
    }
    space++;
    for(int k = 1; k <= 2*(rows-i)-1; k++)
    {
        cout << out_char;
    }
    cout << endl;
    }
}

void draw_triangle (char out_char, int rows)
{
    for (int i = 0; i < rows; i++)
    {
        for (int j = 0; j <= i; j++)
        {
            cout << out_char;
        }
        cout << endl;
    }
}

void draw_square (char out_char, int rows)
{
    for (int i = 0; i < rows; i++)
    {
        for (int j = 0; j < rows; j++)
        {
            cout << out_char;
        }
        cout << endl;
    }
}

void draw_rect (char out_char, int rows, int columns)
{
    for (int i = 0; i < rows; i++)
    {
        for (int j = 0; j < columns; j++)
        {
            cout << out_char;
        }
        cout << endl;
    }
}

（注意：循环变量i, j修复了draw_square和draw_rect，这似乎是复制/粘贴错误 - 除此之外，没有对您的形状函数进行任何更改）

示例使用/输出

$ ./bin/drawshapes dat/drawshapes.txt
&
&&
&&&
&&&&
@@@@@@
@@@@@@
@@@@@@
@@@@@@
@@@@@@
@@@@@@
x
xx
xxx
xxxx
xxxxx
*******
*******
*******
*******
*******
      $
     $$$
    $$$$$
   $$$$$$$
  $$$$$$$$$
 $$$$$$$$$$$
$$$$$$$$$$$$$
 $$$$$$$$$$$
  $$$$$$$$$
   $$$$$$$
    $$$$$
     $$$
      $

    +
   +++
  +++++
 +++++++
+++++++++
 +++++++
  +++++
   +++
    +

===
===
===
===
Exiting

仔细看看，如果您有其他问题，请告诉我。

最新问题
© www.soinside.com 2019 - 2024. All rights reserved.