MSVC Debugger me dit que mon vecteur a plus de positions que celles spécifiées

Question

J'ai déclaré ce qui suit:

const int NUMBER_OF_DIGITS = 16; 
std::vector<int> digits(NUMBER_OF_DIGITS); 

Cependant quand j'ouvre le débogueur MSVC il montre les éléments suivants:

Cette est comment j'ai ajouté des valeurs au vecteur:

for (int i = 0; i < NUMBER_OF_DIGITS; ++i) { 
     scanf("%d", &digits[i]); 
    } 

Est normal? Puis-je juste ignorer ça? Ou est-ce un problème?

code complet (le programme est toujours pas encore prêt):

#include <iostream> 
#include <vector> 
#include "stdio.h" 

const int NUMBER_OF_DIGITS = 16; 

int checksum, tmp1, tmp2, tmp3, sum = 0; 

std::vector<int> digits(NUMBER_OF_DIGITS); 
std::vector<int> new_digits(NUMBER_OF_DIGITS); 

int luhn_checksum(std::vector<int> cardnumber[NUMBER_OF_DIGITS]) { 
    //step 1: duouble every second number 
    for (int i = 1; i < NUMBER_OF_DIGITS; i += 2) { 
     new_digits[i] = digits[i] * 2; 
     if (new_digits[i] > 9) { 
      //if the product is larger than 9 we will add the two numbers together 
      //example: 9 * 2 = 18 so we will add 1 + 8 to get 9 
      tmp1 += new_digits[i] % 10; 
      new_digits[i] /= 10; 
      tmp1 = 0; 
     } 
    } 

    //step 2: sum all the values 
    for (int i = 0; i < NUMBER_OF_DIGITS; ++i) { 
     checksum += new_digits[i]; 
    } 

    return checksum; 
} 

void get_card_numbers(void) { 
    for (int i = 0; i < NUMBER_OF_DIGITS; ++i) { 
     scanf("%d", &digits[i]); 
    } 
} 

void initialize(void) { 
    for (int i = 0; i < NUMBER_OF_DIGITS; ++i) { 
     digits.push_back(0); 
     new_digits.push_back(0); 
    } 
} 


int main() { 
    initialize(); 

    get_card_numbers(); 

    printf("checksum is: %d\n", luhn_checksum(&digits)); 


    std::cout << digits.size() << std::endl; 


    int x; scanf("%d", &x); 
    return 0; 
} 

Answer 1

Le constructeur que vous utilisez pour digits est Réglage de la taille par spécifiant le compte. Donc, après avoir appelé push_back, vous venez d'ajouter un autre 16 au vecteur. Utilisez un constructeur différent qui ne définit pas le nombre.

int _tmain(int argc, _TCHAR* argv[]) 
{ 
    const int NUMBER_OF_DIGITS = 16; 

    std::vector<int> digits(NUMBER_OF_DIGITS); 
    //std::vector<int> digits; 

    int digitsLen = digits.size(); 
    // Here is 16 

    for (int i = 0; i < NUMBER_OF_DIGITS; ++i) { 
     digits.push_back(0); 
    } 

    int digitsLen2 = digits.size(); 
    // Here is 32 

    return 0; 
} 

Answer 2

dépollués votre code un peu:

#include <iostream> 
#include <vector> 
#include <algorithm> 

using namespace std; 

static const size_t NUMBER_OF_DIGITS = 16; 

class cards { 
public: 
    cards(); 
    void read(); 
    int luhnChecksum() const; 
private: 
    vector<int> digits; 
}; 

cards::cards() : digits(NUMBER_OF_DIGITS, 0) 
{ 
} 

void cards::read() { 
    for_each(digits.begin(), digits.end(), [](int& i) { cin >> i; }); 
} 

int cards::luhnChecksum() const { 
    vector<int> newdigits(digits.begin(), digits.end()); 
    for (size_t i=1; i<NUMBER_OF_DIGITS; i += 2) { 
     newdigits[i] = digits[i] * 2; 
     if (newdigits[i] > 9) { 
    int tmp1 = newdigits[i] % 10; 
    newdigits[i] /= 10; 
    newdigits[i] += tmp1; 
     } 
    } 

    return accumulate(newdigits.begin(), newdigits.end(), 0); 
} 

int main() { 
    cards c; 
    c.read(); 
    cout << "checksum = " << c.luhnChecksum() << endl; 
    return 0; 
}