0

私は次のことを宣言しました。

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

ただし、MSVC デバッガーを開くと、次のように表示されます。

ここに画像の説明を入力

これは、ベクトルに値を追加する方法です。

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

これは正常ですか？これを無視してもいいですか？それともこれは問題ですか？

完全なコード (プログラムはまだ準備ができていません):

#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;
}
4

2 に答える 2

1

使用しているコンストラクターはdigits、カウントを指定してサイズを設定しています。したがって、呼び出した後push_back、ベクトルにさらに 16 を追加したことになります。カウントを設定しない別のコンストラクターを使用してください。

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;
}
于 2015-04-03T00:06:19.380 に答える
0

コードを少しクリーンアップしました：

#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;
}
于 2015-04-02T21:03:59.620 に答える