Заполнение массива целыми числами

Question

Я пытаюсь заполнить массив номерами от 1111 до 8888, причем каждое целое число от числа от 1 до 8 в C++. Однако, когда я запускаю его, он выводит только большие отрицательные числа, указывающие на ошибку. Я честно понимаю, что такое ошибка, поэтому было бы полезно, если бы вы могли мне помочь. Благодаря!

int fillArray() 
    { 
    int arrayPosition; 
    int guesses[4096]; 

arrayPosition = 0; 
for (int i = 1; i <= 8; i++) 
     for (int j = 1; j <= 8; j++) 
     for (int k = 1; k <= 8; k++) 
      for (int m = 1; m <= 8; m++) 
      { 
      guesses[arrayPosition] = ((i * 1000) + (j * 100) + (k *10) + m); 
      cout << guesses[arrayPosition]; 
        arrayPosition++; 
      } 

return guesses[4096]; 
    } 

Answer 1

Ваш тип возврата неверен. int fillArray(), но вы пытаетесь вернуть int[4096], который был объявлен в стеке ... То, что вы на самом деле делаете с return guesses[4096];, возвращает ячейку памяти после вашего массива в памяти, что, вероятно, просто мусор, следовательно, ваш выпуск с большими отрицательными номерами.

Вы можете это исправить, выделив ваш массив в куче, и возвращает указатель на начало этого массива:

int * fillArray() 
{ 
    int arrayPosition; 
    int * guesses = new int[4096]; 

    // other stuff stays the same... 

    return guesses; 
} 

Однако, так как ваша функция называется fillArray, было бы больше смысла передать в массив и заполнить его, а не создавать массив в функции. (Если вы хотите это сделать, вместо этого вы можете называть его чем-то вроде make_1_to_8_array, чтобы было более ясно, что вы создаете что-то, что нужно будет удалить позже.) Предоставление int* в качестве первого аргумента позволит вам пройти в базовый адрес вашего массива, который вы хотите насытиться

void fillArray(int * guesses) 
{ 
    int arrayPosition; 

    // other stuff stays the same... 

} 

Или, если вы хотите, чтобы убедиться, что вы используете массив точного размера:

void fillArray(int (&guesses)[4096]) 
{ 
    int arrayPosition; 

    // other stuff stays the same... 

} 

Обратите внимание, что функция теперь возвращает void, так как вы просто обновляете массив, который был передан, и вам не нужно возвращать что-либо новое.

Answer 2

Вы создаете свой массив локально, а затем пытаетесь его вернуть. Если вы попробуете распечатать (отладить) результат вашего массива до возвращения, вы увидите, что все в порядке. Однако, как только вы вернетесь, массив не будет задерживаться. Попробуйте передать массив в свою функцию.

Answer 3

Ваш for -loops выглядят правильно, но обработка вашего массива выключена, как подчеркнуто другими ответами.

Как правило, в C++ используется std::vector и передать это в по ссылке в качестве аргумента. Это избавит вас от необходимости использовать выделение памяти и освобождение памяти. Вот пример, в том числе вывод в for -loops:

#include <iostream> 
#include <vector> 

int fillArray(std::vector<int>& guesses) 
{ 
    for (int i = 1; i <= 8; i++) 
     for (int j = 1; j <= 8; j++) 
      for (int k = 1; k <= 8; k++) 
       for (int m = 1; m <= 8; m++) 
       { 
        guesses.push_back((i * 1000) + (j * 100) + (k * 10) + m); 
        std::cout << guesses.back() << std::endl; 
       } 

    return guesses.back(); 
} 

int main() 
{ 
    std::vector<int> guesses; 
    std::cout << fillArray(guesses) << std::endl; 
}