vectorクラスで4桁の数字で奇数の数を判別したい。

Question

vectorクラスを使用して4桁内にある奇数の数を判定するものを作りたいです。
例えば、1342であれば2を、5761であれば3を返すものです。
以下のコードでは正しく判定されないです。

vector<int> t;
int cnt = 0;
int u[4];
int value = 0;
	cin >> value;
t.push_back(value);
cnt += count(t.begin(), t.end(), 1);
cnt += count(t.begin(), t.end(), 3);
cnt += count(t.begin(), t.end(), 5);
cnt += count(t.begin(), t.end(), 7);
cnt += count(t.begin(), t.end(), 9);

cout << cnt << endl;

引用テキスト

Answer 1

このようなことをしたいのでしょうか？

int value;
std::cin >> value;

int numOfOdd = 0;
for(int i = 0; i < 4; i++) {
    if(value % 2 == 1) {
        numOfOdd++;
    }

    value /= 10;
}

std::cout << numOfOdd << std::endl;

もうちょっと柔軟にすると、次のような感じになるでしょう。

for(int val = value; val != 0; val /= 10) {
    if(val % 2 == 1) {
        numOfOdd++;
    }
}

---

vectorクラスは確かに配列のように複数の値を保管できます。
しかし、自動的に一桁一桁分解してくれるわけではありません。
どうしてもvectorを使いたいなら、こんな風な関数が必要かと。

// 整数を一桁一桁vectorにpush
void pushEachDigit(std::vector<int> &dst, int num) {
    dst.clear();
    dst.shrink_to_fit();

    for(; num != 0; num /= 10) {
        dst.push_back(num % 10);
    }
}

inline bool isOdd(int num) {
    return num % 2 == 1;
}

// vectorに格納された『奇数』の個数を返す
int countOddDigit(const std::vector<int> &vec) {
    int numOfOdd = 0;
    for(auto elem: vec) {
        if(isOdd(elem)) {
            numOfOdd++;
        }
    }
    return numOfOdd;
}

そうすると、質問者様の書きたいコードはこんな構造になりますね。

int tsuika::argk(std::vector<int>& vec) {
    int value = 0;
    std::cin >> value;

    puchEachDigit(vec, value);
    int numOfOdd = countOddDigit(vec);

    return numOfOdd;
}

Comments

[退会済みユーザー]

とても苦戦していたのでこんなに簡単に記述できるなんて驚きました！
ありがとうございます。
また機会があったら回答よろしくお願いします。

(以下が変更後のコードです)

int tuika::argk(vector<int>& val)
{
int value=0;
cin >> value;
int numOfOdd = 0;
for (int val = value; val != 0; val /= 10) {
if (val % 2 == 1) {
numOfOdd++;
}
}
cout << numOfOdd << endl;

[LouiS0616]

引数が全く使われていませんが、想定内でしょうか？

[退会済みユーザー]

すみません…
"return 0;"記入忘れてました。

[LouiS0616]

いえ、そっちじゃなくて、vector<int> &valのことです。
それにどうせreturnするならnumOfOddを返した方がよいのでは。

[退会済みユーザー]

ベストアンサーにさせていただきます。

[退会済みユーザー]

自分なにか勘違いしていたようです。
"vector<int> &val"にすることによってvectorを実現できる
と解釈していました。
"return num;"は分かるのですが、こちらのコードをどのように
してvectorにするのでしょうか？

[退会済みユーザー]

はやとちりで
t.push_back(val);
付けてみるの忘れました。自分の範囲内では
push_backどまりだと思います。

[LouiS0616]

追記しました。
しかし、本当にvectorを使う必要があるのか？という疑問は残ります。

[退会済みユーザー]

コード
う～…

分からないです。もうちょっと勉強してみます。

[LouiS0616]

どのあたりが分かりませんか？

[退会済みユーザー]

"/= 10"と"puchEachDigit(vec, value)"・"countOddDigit(vec)"(関数かと思い、
検索をかけたのですがどちらもヒットしません)がよくわかりません。
isOddの式も分からないです。

[LouiS0616]

num /= 10; は、num = num / 10; と同じ意味です。
二つの関数はヒットしなくて当然です。この場で定義していますから。そういう意味では、情報はすべてこのページにあります。

[LouiS0616]

isOddに関しては、次のように読み替えればいいかもしれません。処理の結果は同じです。
bool isOdd(int num) {
....if(num % 2 == 1) {
........return true;
....}
....else {
........return false;
....}
}
(空白を.で代替しています)

[退会済みユーザー]

遅れた返信になりすみません。

すみません　やっぱりわかりません、pushEachDigit
ではどのように値がpushされているのでしょうか？
またclear()はvectorの初期化ということでしょうか？

[LouiS0616]

clear()とshrink_to_fit()はここではセットで用いていて、目的は推測の通りvectorの要素の削除です。
ただ厳密に言うなら、初期化とは呼べないんですけどね。

[LouiS0616]

『どのように値がpushされているか』ロジックの話ですね。
これに関しましては、実際に紙かなにかに書いてみるとわかるかと思います。
for文を用いてやっていることは、以下のような感じです。
____1. 1の位を取り出してベクトルにプッシュ。
____2. 10で割って小数点以下を切り捨てる。
____3. もしnumが0になっていたらfor文を終了。

[退会済みユーザー]

実際に区切って実行して
おおかた分かることができた思います、助かりました
ありがとうございます。

また良かったら回答よろしくお願いします！