for文を使っての狙った出力にならない

前提

AtCoderの問題（下）での質問です。
「 ABC 081 A - Placing Marbles (100 点)
0 と 1 のみから成る 3 桁の番号 s が与えられます。1 が何個含まれるかを求めてください。」
でfor文を使って問題を解こうとしてわからないところがありました。

int main(void){
    
    int N;
    int s[10];
    int ans=0;
    
     cin>>N;
     for(int i=1;i<N;++i) cin>>s[i];
     for(int i=1;i<N;++i){
     if(s[i]==1){
         ans+=1;
    }
}
     cout<<ans<<endl;
}

というコードを作って
入力は
３
１０１
と入れたら
出力が０になってしまいます。
実現したいのは
１の数だけ数えたいです。今回の例だと”１０１”と１が２つあるので２と出力したいです。
どこが間違っているのかが分かりません。
ご教授お願いします。

補足情報（FW/ツールのバージョンなど）

C++
ここにより詳細な情報を記載してください。

1T2R3M4

2023/01/16 04:56

提示のコードでコンパイルできたのですか。

int32_t

2023/01/16 04:56

人にコードを見せるときはインデントをきちんとしていただけると読みやすいです。

can110

2023/01/16 04:58

提示コードはコンパイルエラーが発生します（d[i]とは？）。また問題文とコードに相違があるようですが意図通りでしょうか（Nとは？）

BeatStar

2023/01/16 05:10

一つ目。そもそもコンパイルエラーになるし、インデントも汚いです。二つ目。問題文では「1,2,3 の番号がついた 3 つのマスからなるマス目を持っています」と「3マスであること」を明示されているのになぜかコードではint s[10];のように要素数10で確保している。これ必要ですか? 三つ目。問題文が省略されているためわかりづらいです。AtCoderの番号等が書かれているため探すことは可能ですが、この手の問題だと「そもそも回答を公開したり質問したりすることを禁じている」場合があります。Paizaスキルチェックはまさにそうです。AtCoderでも出題されて24時間だったか…は同じく質問することも禁じられているようです。四つ目。入力値とコードでの入力部のあれが合わないです。五つ目。デバッグぐらいはしましょう。デバッグのやり方は確かに初心者にはハードルが高いですが、ほぼ必須の能力です。六つ目。データ型を理解しましょう。int型とか。

回答4件

自己解決

みなさんフィードバックありがとうございました。
int main(void){
int s[3];
int ans=0;
for(int i=0;i<3;++i) cin>>s[i];
for(int i=0;i<3;++i){
if(s[i]==1){
ans+=1;
}
}
cout<<ans<<endl;
}
上のコードでできました。
入力
1
0
1
出力
２

投稿2023/01/16 05:54

jaogjig

総合スコア21

回答へのコメント

BeatStar

2023/01/16 08:00

質問本文では、 ``` ３１０１ ``` という入力値で、本来の問題の入力値は ``` 101 ``` で、この自己解決では ``` 1 0 1 ``` という入力値となっています。それはルール違反ですよ。そもそも仕様にないことを勝手にやっているのですから。

fana

2023/01/16 09:56

変に考えずに，単純に char c; for( int i=0; i<3; ++i ){ std::cin>>c; if(c=='1')++ans; } とかで良いのでは．

Zuishin

2023/01/17 01:08

AtCoder の出力例を取得して自動的にテストするようなツールがあったような気がします。インデントの乱れからもちゃんとした編集ツールを使っていないことがわかりますし、そんな雑なやり方では学習効率も低いでしょう。ただしいコードができたかどうかは自分で判断するのではなくツールを使ってチェックしましょう。