C++で重複する数字が含まれているか判定するプログラムを作る

Question

前提・実現したいこと

整数に重複する数字が含まれているか判定するプログラムを作りたいのですがうまくいきません。

発生している問題・エラーメッセージ

どんな数字をいれても"重複する数字が含まれているになってしまう"

該当のソースコード

#include<iostream>
bool has_duplicate_digits(int x);
int main(void)
{
    int x,c;
    std::cout<<"整数を入力してください";
    std::cin>>x;
    //has_duplicate_digits(x);//関数
    if(has_duplicate_digits(x)){
        std::cout <<"重複する数字が含まれている\n";
    }else{
        std:: cout <<"重複する数字は含まれていない\n";
    }
    return 0;
}
bool has_duplicate_digits(int x)
{
    int a,i,b;
    int counts[10]={};
    for(i=0;i<10;i++){
        if(x!=0){
        a=x%10;
        //printf("%d",a);
        x/=10;
    }else{
        break;
    }
    }
    if(a==0){        //出現した文字数をカウントする
        counts[0]++; 
        if(counts[0]>=2){
            b++;
        } //文字数が2以上だったらtrue
    }else if(a==1){
        counts[1]++;
        if(counts[1]>=2){
            b++;
        }
    }else if(a==2){
        counts[2]++;
        if(counts[2]>=2){
            b++;
        }
    }else if(a==3){
        counts[3]++;
        if(counts[3]>=2){
            b++;
        }
    }else if(a==4){
        counts[4]++;
        if(counts[4]>=2){
            b++;
        }
    }else if(a==5){
        counts[5]++;
        if(counts[5]>=2){
            b++;
        }
    }else if(a==6){
        counts[6]++;
        if(counts[6]>=2){
            b++;
        }
    }else if(a==7){
        counts[7]++;
        if(counts[7]>=2){
            b++;
        }
    }else if(a==8){
        counts[8]++;
        if(counts[8]>=2){
            b++;
        }
    }else if(a==9){
        counts[8]++;
        if(counts[9]>=2){
            b++;
        }
    }
    if(b>=1){
        return true;
    }else{
        return false;
    }
}

Answer 1

コンパイラは何ですか？
もし g++ なら -Wallオプションを付けてみると警告が出ます。
b が初期化されていません。

次に x がある値の時にどうなるかコードを追いかけてみてください。
例えば x が 314 のとき、a の値は 4 1 3 と変化します。
x は 314 31 4 0 と変化します。

forループを抜けると a には最上位の桁の数 4 が入っています。
これだけを見て重複を判断できるはずがありません。

bool has_duplicate_digits(int x)
{
	int counts[10] = { };
	do {
		if (++count[x % 10] > 1) return true;
	} while (x /= 10);
	return false;
}

Answer 2

bool has_duplicate_digits(int x) {
  int counts[10] = {0};
  // 各数の出現頻度をcountsに求め、
  while ( x != 0 ) {
    counts[x % 10]++;
    x /= 10;
  }
  // counts 内に2以上のものがあれば重複あり
  for ( int i = 0; i < 10; ++i ) {
    if ( counts[i] > 1 ) return true;
  }
  return false;
}

あるいは

#include <string>
#include <set>

bool has_duplicate_digits(int x) {
  std::string digits = std::to_string(x);
  return std::set<char>(digits.begin(), digits.end()).size() != digits.size();
}

Comments

[episteme]

コレ↓で十分かな。

bool has_duplicate_digits(int x) {
int counts[10] = {0};
while ( x != 0 ) {
if ( ++counts[x % 10] > 1 ) return true;
x /= 10;
}
return false;
}

Answer 3

bを0で初期化してください。
あとforループの閉じる位置間違えていると思います。