Project Euler # 8 Code - Do While цикл не выполняет весь код более одного раза - [C++]

Question

Я пытаюсь решить проблему Project Euler № 8. Я написал код, который, как я думаю, сможет решить его здесь. Вероятно, это не самый эффективный алгоритм, но я думаю, что он должен работать. Что происходит на выходе, так это то, что он выполняет все циклы for и оператор if в первый раз, а затем при последующих проходах через цикл do while он только выполняет итерацию переменной «position», но не выполняет ни один из других кодов. Вот что у меня есть. Любая помощь будет оценена по достоинству.

Задача № 8 состоит в том, чтобы найти, какие 13 последовательных цифр данного 1000-значного числа имеют наибольший продукт. На данный момент мой код отлично умножает первые 13 цифр, а затем не может двигаться дальше. Для меня это очень странное поведение.

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

int main(){ 
    char num[1001] = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450"; 
    int position = 0; 
    unsigned int product_holder = 1; 
    unsigned int product = 1; 
    do{ 
     int holder[13] = {}; 
     for (int i = 0; i < 13; i++){ 
      holder[i] = num[i+position] - '0'; 
     } 
     for (int j = 0; j < 13; j++){ 
      product_holder *= holder[j]; 
     } 
     if (product < product_holder){ 
      product = product_holder; 
      std::cout << product << std::endl; 
     } 
     position++; 
     std::cout << "The position is currently " << position << std::endl; 
    }while(position != 9999); 

    std::cout << product << std::endl; 
    return 0; 
} 

Answer 1

Ваша программа ведет к неопределенному поведению.

Линия

holder[i] = num[i+position] - '0'; 

вызовет проблему, потому что num определяется как char num[1001];, но i + position может идти более чем 10000. Таким образом, вы читаете за пределами массива num, что является неопределенным поведением.

Вот пример вывода программы:

... 
The position is currently 7283 
The position is currently 7284 
Segmentation fault (core dumped) 

Примечание: выход недетерминированно из-за неопределенного поведения.

И вот обижая часть, анализируется с помощью GDB:

Core was generated by `./a.out'. 
Program terminated with signal SIGSEGV, Segmentation fault. 
#0 0x0000000000400a07 in main() at eul.cpp:12 12 
      holder[i] = num[i+position] - '0'; 
(gdb) print i 
$1 = 12 
(gdb) print position 
$2 = 7284 
(gdb) print num[i + position] 
Cannot access memory at address 0x7fff27c11000