Как получить двумерный массив для записи из файла, в котором в качестве разделителей используются только пробелы

Question

Я пытаюсь взять файл и поместить его в 3 разных массива.Два из этих массивов являются одномерными, а другой - 2D.Текстовый файл выглядит следующим образом:

Bill Hansley 1 1 1 1 1
Todd Howard 2 3 1 0 0
Sam Duke 0 1 1 0 0
Danny Martin 1 0 2 0 1

Я пытаюсь взять этот текстовый файл и вставить его первые имена в массив с именем firstNames [], затем еще один массив с последними именами с именем lastNames [],и, наконец, для чисел, я хочу их в массиве с именем Productsorders [] [].Мой код выглядит следующим образом.

bool loadOrderFile(string orderFN,
    string firstNames[], string lastNames[],
    int productsOrders[MAX_ORDERS][MAX_PRODS],
    int &namesCount, int &prodCount, string &menuName) 
{
    ifstream File;
    File.open(orderFN.c_str());
    if (File.is_open()) {
        cout << "Order file opened..." << endl;
    }
    int i = 0;
    getline(File, menuName);
    (File >> prodCount);
    while (File) {
        File.get();
        (File >> firstNames[i]);
        (File >> lastNames[i]);
        (File >> productsOrders[i][i]);
        (File >> productsOrders[i + 1][i + 1]);
        (File >> productsOrders[i + 2][i + 2]);
        (File >> productsOrders[i + 3][i + 3]);
        (File >> productsOrders[i + 4][i + 4]);

        (i++);
    }
    cout << "Menu name: " << menuName << endl;
    cout << "Product Count: " << prodCount << endl;
    cout << "There were " << (prodCount - 1) << " orders read in." << endl;
    for (int i = 0; i < 10; i++) {
        cout << productsOrders[i][i] << endl;
    }
    for (int i = 0; i < 10; i++) {
        cout << firstNames[i] << lastNames[i] << endl;
    }
    return true;
}

Кажется, что массивы имен работают так, как они выводят имена, как и должно, но двумерный массив выводит

1
2
0
1
0
2
0
1
0
0

, когда должно быть

1 1 1 1 1
2 3 1 0 0
0 1 1 0 0
1 0 2 0 1

Буду признателен за любую помощь.

Answer 1

Честно говоря при разборе текстовых файлов;Я бы разделил логику на отдельные функции в соответствии с их уникальными обязанностями.

Ваш код выглядел бы примерно так:

#include <vector>
#include <string>
#include <sstream> 
#include <iostream>
#include <fstream>
#include <exception>

// Structure of your data type...
struct Order {
    std::string firstName;
    std::string lastName;
    std::vector<int> productOrders; // maybe use vector instead of array...
    // any other variable(s) or container(s) you may need...
};

// Simple ostream operator<< overload to print your Order Struct
// in a nice readable format.
std::ostream& operator<<(std::ostream& os, const Order& order ); 

// Function to split a string based on a single character delimiter
std::vector<std::string> splitString( const std::string& s, char delimiter );

// Function that will get each line text from a file and stores it into a vector
// of strings. Then closes the file handle once the entire file has been read.
void getAllLinesFromFile(const char* filename, std::vector<std::string>& output);

// This function will parse a single line of text or string that is contained in 
// the vector of strings. The declaration of this can vary; if you have other
// information such as header information before the actual data structures
// you would have to modify this function's declaration-definition to accommodate
// for those variables. 
void parseLine( const std::vector<std::string>& fileContents, std::vector<Order>& orders );

// Simple and clean looking main function.
int main() {
    try {
        std::vector<std::string> fileContents;
        getAllLinesFromFile( "filename.txt", fileContents );
        std::vector<Order> orders;
        parseLine( fileContents, orders );

        for ( auto& o : orders )
            std::cout << o << '\n';           

    } catch( std::runtime_error& e ) {
        std::cerr << e.what() << std::endl;
        return EXIT_FAILURE;
    }
    return EXIT_SUCCESS;

}

std::ostream& operator<<(std::ostream& os, const Order& order ) {
    os << order.firstName << " " << order.lastName << '\n';    
    for (auto& p : order.productOrders)
        os << p << " ";
    os << '\n';
    return os;
}


std::vector<std::string> splitString( const std::string& s, char delimiter ) {
    std::vector<std::string> tokens;
    std::string token;
    std::istringstream tokenStream( s );
    while( std::getline( tokenStream, token, delimiter ) ) {
        tokens.push_back( token );
    }

    return tokens;
}

void getAllLinesFromFile(const char* filename, std::vector<std::string>& output) {
    std::ifstream file(filename);
    if (!file) {
        std::stringstream stream;
        stream << "failed to open file " << filename << '\n';
        throw std::runtime_error(stream.str());
    } else {
        std::cout << "File " << filename << " opened successfully.\n";
    }

    std::string line;
    while (std::getline(file, line)) {
        if (line.size() > 0)
            output.push_back(line);
    }
    file.close();
}

void parseLine( const std::vector<std::string>& fileContents, std::vector<Order>& orders ) {
    // Here is where you would do the logic to parse the vector of strings
    // this function may vary based on your file structure. If there is any
    // header information you would have to extract that first from the 
    // vector's index. 

    // Once you get to the index in the vector that describes your data structure
    // it is hear that you would want to call `splitString()` using the current
    // current index of that vector and the space character as your delimiter.
    // This will create a vector of strings that are now considered to be tokens.

    // On each pass of the loop for each line of contents you will want to
    // create an instance of the Order Structure, then use that to populate
    // the vector of Orders that was passed in by reference.

    // Once all of the contents are done being parsed the function will exit
    // and your vector of Orders will have the appropriate data.    
}

Наличие такой структуры облегчило бы отладку при разборетекстовые файлы.Я считаю, что более элегантно извлечь все данные из файла и сохранить их в каком-либо контейнере;проще всего было бы либо string, либо несколько stream buffer, например stringstream.Затем закройте файл, когда закончите получать все содержимое.После того, как все прочитано из текстового файла и сохранено в контейнер строк.Здесь вы хотите проанализировать strings или streams и проверить, является ли информация действительной или нет.Многократно открывать и закрывать дескриптор файла неэффективно и медленно, и многое может пойти не так.Я думаю, что легче извлечь содержимое из файла, сохранить его, закрыть и закончить с файлом, а затем перейти к реальной работе.

Answer 2

Ваша проблема в том, что вы неправильно обращаетесь к своим двумерным массивам.

Например, в массиве 3x3 2D у вас есть два индекса [a][b], двумерное представлениеэтого массива будет выглядеть так:

[0][0] [0][1] [0][2]

[1][0] [1][1] [1][2]

[2][0] [2][1] [2][2]

Так что, когда вы выводите, например:

for (int i = 0; i < 10; i++) {
    cout << productsOrders[i][i] << endl;
}

Вы можете видеть, что вы получите только диагональную линию через массив, ачем все предметы ([0][0], [1][1], [2][2]).Чтобы напечатать весь массив, вам нужно использовать два цикла.

У вас есть похожая проблема с вашим вводом, когда вы увеличиваете оба индекса одновременно.