記載の問題に対して書いたプログラムに出たエラーが自己解決できませんでした

問題文
整数 X を十進法で表したときの各桁の数字の和を f(X) としたとき、X が f(X) で割り切れる場合、X はハーシャッド数である。
整数 N が与えられるので、ハーシャッド数かどうか判定せよ。

制約
1≦N≦10^8
入力は全て整数
/////////////////////////////////////////////////////////////////
上記の問題において、

int型で整数入力を受け取り、string型のSに変換、Sの要素をひとつづつ調べてf(x)を求める、という方針を立て、

#include <bits/stdc++.h>
using namespace std;

int main() {
int N, x = 0;
string S;

cin >> N;

S = to_string(N);

for(int i = 0;i < sizeof(S);i++){
for(int j = 0;j < 10;j++){
if(S[i] == j){
x += j;
}
}
}

if(N % x == 0){
cout << "Yes" << endl;
}
else{
cout << "No" << endl;
}

}

というプログラムを書きました。

結果として、実行時エラーとして

./Main.cpp: In function ‘int main()’:
./Main.cpp:12:19: warning: comparison of integer expressions of different signedness: ‘int’ and ‘long unsigned int’ [-Wsign-compare]
12 | for(int i = 0;i < sizeof(S);i++){
| ~~^~~~~~~~~~~

となってしまいました。

調べたところ、「符号の異なる整数式の比較をしている」と分かったのですが、iと配列の大きさを比べる式において、0≦iとしているのにこのエラーになってしまった原因がわかりませんでした。

また、そもそもこの問題に対してこのプログラムではできない、このプログラムでは間違っているという場合は改善点などを教えていただけるとありがたいです。

よろしくお願いします。

maisumakun

2022/03/19 03:06

> 実行時エラーとしてこれは「コンパイル時」の「ワーニング」です。実行時エラーではありません。

行動規範の内容に同意します

回答2件

んで、それはエラーではありません。
ワーニングですよ

投稿2022/03/19 03:29

y_waiwai

総合スコア87836

AKIRA

2022/03/19 05:06

実行時エラーみたいな文言が出てきたためエラーかと思ってしまっていました。回答ありがとうございます！

episteme

2022/03/19 06:33

> ./Main.cpp:12:19: warning: comparison of integer expressions... "warning"って書いてある。

行動規範の内容に同意します

ベストアンサー

sizeof(S) は文字列Sの長さを返しません。

※ C++っぽく書いてみた

C++
1#include <iostream>
2#include <string>
3#include <numeric>
4using namespace std;
5
6int main() {
7  int N;
8  cin >> N;
9
10  string S = to_string(N);
11  int x = transform_reduce(S.begin(), S.end(), 0,
12                           [](int x, int y) -> int { return x+y; },
13                           [](char c) -> int { return c-'0'; });
14  cout << ((N % x == 0) ? "Yes" : "No") << endl;
15
16}