web-dev-qa-db-ja.com

C ++で2D配列を印刷する方法は？

配列を使用してテキストファイルを画面に印刷しようとしていますが、テキストファイルのように表示されない理由がわかりません。

テキストファイル：

1 2 3 4
5 6 7 8

破棄機能を適用すると、画面に次のように表示されます。

1
2
3
4
5
6
7
8

コード：

#include <iostream>
#include <fstream>
#include <stdlib.h>
#include <string>

using namespace std;

const int MAX_SIZE = 20;
const int TOTAL_AID = 4;

void discard_line(ifstream &in);
void print(int print[][4] , int size);

int main()
{
    //string evnt_id[MAX_SIZE]; //stores event id
    int athlete_id[MAX_SIZE][TOTAL_AID]; //stores columns for athelete id
    int total_records;
    char c; 
    ifstream reg;
    reg.open("C:\\result.txt");

    discard_line(reg);
    total_records = 0;

    while( !reg.eof() )
    {
        for (int i = 0; i < TOTAL_AID; i++)
        {
            reg >> athlete_id[total_records][i] ;//read aid coloumns
        }
        total_records++;
        reg.get(c);
    }

    reg.close();

    print(athlete_id, total_records);

    system("pause");
    return 0;
}

void discard_line(ifstream &in)
{
    char c;

    do
        in.get(c);
    while (c!='\n');
}

void print(int print[][4] , int size)
{    
    cout << " \tID \t AID " << endl;
    for (int i = 0; i < size; i++)
    {
        for (int j = 0; j < TOTAL_AID; j++)
        {
            cout << print[i][j] << endl;
        }           
    }
}
c++arraysmultidimensional-arrayrevision
6
Nick

各番号の後にstd::endlを印刷しています。 1行に1行にしたい場合は、各行の後にstd::endlを出力する必要があります。例：

#include <iostream>

int main(void)
{
    int myArray[][4] = { {1,2,3,4}, {5,6,7,8} };
    int width = 4, height = 2;

    for (int i = 0; i < height; ++i)
    {
        for (int j = 0; j < width; ++j)
        {
            std::cout << myArray[i][j] << ' ';
        }
        std::cout << std::endl;
    }
}

また、ファイルの先頭にusing namespace std;を書き込むと、（型、関数などの）ユーザー定義名の一部があいまいになるため、悪い習慣と見なされることに注意してください。 std::の接頭辞を使い果たしたくない場合は、他の関数や他のファイルが影響を受けないように、小さなスコープ内でusing namespace std;を使用してください。

13
LihO

「endl」を見逃すのは間違いだけではありません。また、関数discard_line（reg）を呼び出すため、プログラムはソースファイルの最初の行をスキップするため、他のデータ（5 6 7 8）のみを取得できます。この機能を使用する必要はまったくありません。さらに、配列を初期化し、MAX_SIZEなどの配列の境界をチェックして、入力データが配列をオーバーフローしないようにします。

1
MichaelGD