1

Подскажите пожалуйста, как правильно сделать. Мне нужен массив Char Field[20][20] заполненные символами из файла.

Я сделал вот такую функцию:

void Spiel::readFile(std::string fileName) {
    std::ifstream file(fileName.c_str());
    if (!file.is_open()) {
        std::cerr << "Error opening file: " << fileName << std::endl;
        exit(1);
    }
    std::string line;
    int a = 0;
    while (true) {
        if (a == 21)
            break;
        std::getline(file, line);
        for (int i = 0; i < 20; i ++) {
            if (line[i] == std::string::npos) {
                Field[a][i] = '#';
            }
            else {
                Field[a][i] = line[i];
            }
        }
        a++;
        // weiter machen if, wenn Field [i][j] == "B", bx = i, by = j
    }
    std::cerr << "Done" << std::endl;
} 



int main(int argc, char** argv) {
    Spiel spiel;
    spiel.readFile(std::string(argv[1]));
};

По gdb print Field или print line выдает непонятные числа и знаки, но ни как не буквы которые стоят в считываемом файле. Не могу понять что не так

7
  • 1
    Вы бы воспользовались отладчиком, было бы скорее.
    – VladD
    10 июл 2016 в 18:35
  • каким отладчиком ? 10 июл 2016 в 18:35
  • Какой-то странный алгоритм заполнения массива... Начиная с while(true) { if(a==21) .... Читая этот мозголомный код сложно уловить суть.
    – PinkTux
    10 июл 2016 в 18:37
  • @PinkTux замысел был таков, что по строчке все считаются буквы из файла и сохраняются в Field. Переменная а считает количество строк. Когда было сосчитано 20 строк, то break 10 июл 2016 в 18:44
  • 1
    @Владимир: Любым, собственно. Обычно в комплекте с компилятором идёт отладчик.
    – VladD
    10 июл 2016 в 18:47

1 ответ 1

4

Один вопрос: что вы подразумевали под этой строкой line[i] == std::string::npos ? Если я правильно понял, если строка короче 20 символов, то вы ее добиваете решетками, я прав?

В целом сразу бросается в глаза, что вы заполняете массив на 20 строк, но читаете вы 21 раз, да и вообще, зачем мучаться с while(true), если по факту все равно у вас цикл с заранее известным количеством шагов, надо заменить на for .

Теперь, длину строки имеет смысл проверять как i >= line.length() .

В итоге получается примерно так:

void Spiel::readFile(std::string fileName) {
    std::ifstream file(fileName.c_str());
    if (!file.is_open()) {
        std::cerr << "Error opening file: " << fileName << std::endl;
        exit(1);
    }
    std::string line;
    for(int a = 0; a < 20; a++) {
        std::getline(file, line);
        for (int i = 0; i < 20; i ++) {
            if (i >= line.length()) {
                Field[a][i] = '#';
            }
            else {
                Field[a][i] = line[i];
            }
        }
    }
    std::cerr << "Done" << std::endl;
} 

Я проверил этот код на файле с этой самой функцией, получилось так:

#include <iostream>#
#include <string>###
#include <fstream>##
####################
void readFile(std::s
    char Field[20][2
    std::ifstream fi
    if (!file.is_ope
        std::cerr <<
        exit(1);####
    }###############
    std::string line
    int a = 0;######
    while (true) {##
        if (a == 21)
            break;##
        std::getline
        for (int i =
            if (line
                Fiel
0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.