Отладка этого кода для нахождения наибольшего произведения из 13 соседних цифр в числе 1000 di git - PullRequest
0 голосов
/ 13 апреля 2020

Я пытался создать программу на C ++ для решения 8-й задачи Project Euler . Тем не менее, моя программа не выводит правильный ответ. Я пытался отследить его, но не могу найти свою логическую ошибку.

Восьмая проблема проекта Эйлера заключается в поиске значения наибольшего произведения из тринадцати смежных цифр в числе из 1000 цифр. Мой код выводит 9223372019204292608 в качестве ответа. Правильный ответ, я считаю, 23514624000. Мой код следующий:

    #include <iostream>
    #include <cassert>
    using namespace std;

    int main() {

       //Declare input number as a string. Declare variables
       string input = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450";

       char initialArray[1000];
       int finalArray[1000];
       long products[987];
       long highestProduct = 0L;

       //Divide input string into an array of single characters
       for(int i = 0; i < 1000 ; i++){
            initialArray[i] = input.at(i);
       }

       //Translate the array of characters into an array of single-digit integers
       for(int i = 0; i < 1000 ; i++){
           finalArray[i] = input.at(i) - '0';
       }

       //Initialize each element in the products array to 1
       for (int i = 0; i < 100 ; i++){
           products[i] = 1L;
       }


       //Perform multiplications
       for(int i = 0; i < 987; i++){
           for( int j = 0; j < 13; j++){
               products[i] = products[i] * finalArray[j+i];
           }
       }

       for(int i = 0; i < 987; i++){
           if (products[i] > highestProduct){
               highestProduct = products[i];
           }
       }

       cout << "The highest product of 13 adjacent digits from the input is: " <<                            highestProduct << endl;

       return 0;
   }

   //Correct answer is supposed to be 23514624000

Ответы [ 2 ]

2 голосов
/ 13 апреля 2020

Вы не инициализируете весь массив products. В идеале вам не следует инициализировать их отдельно.

//Perform multiplications
for(int i = 0; i < 987; i++)
{
  products[i] = 1L;  // initialize
  for( int j = 0; j < 13; j++)
  {
     products[i] = products[i] * finalArray[j+i];
  }
}

Таким образом, вы не можете go ошибиться с инициализацией всех элементов.

1 голос
/ 13 апреля 2020

Смотрите здесь:

// Инициализируйте каждый элемент в массиве продуктов как 1
для (int i = 0; i <<strong> 100 ; i ++) {


Также обратите внимание, что вы можете использовать только массив l oop through, используя обновление продукта на каждом шаге следующим образом:

product = (product / finalArray[i - 13]) * finalArray[i]
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...