Найдите ошибку!

@khan01 · Регистрация: 20.11.2016

Author24 — интернет-сервис помощи студентам

C++

#include<iostream>
#include <string>
 
using namespace std;
int main(){
/*Дана строка подсчитать количество цифр содержащейся в нем*/ 
    int a = 0;
    string s;
    cin >> s;
    for (int i = 0; i < s.size(); i++) {
        if (48 <= s[i] <= 57);
            a++;
 
        
    }
 
    cout << a;
    system("pause");
    }

Правильно я использовал АSCII КОД???

@FreeYourMind · 14.01.2017, 10:49

Сообщение от khan01

if (48 <= s[i] <= 57);

для начала тут по моему должно быть 48<=s[i] && s[i]<=57 + к этому компиляторбудет проверять элемент строки на наличие в ней цифр из диапазона от 48 до 57.. тут мне кажется даже не пахнет ASCII

@Bretbas · 14.01.2017, 11:06

Если нужно в строке найти количество цифр, содержащихся в ней, то лучше будет написать код вот так:

C++

#include <iostream>
#include <algorithm>
#include <string>
 
using namespace std;
 
int main() {
    string str;
    cin >> str;
    
    int num = std::count_if(str.begin(), str.end(), []( const auto& c ) { return isdigit(c); });
    cout << num;
}

@TheCalligrapher · 14.01.2017, 11:10

Сообщение от khan01

C++

1	/Дана строка подсчитать количество цифр содержащейся в нем/

Нашел еще ошибки!!!11 Строка - она, цифры - они, поэтому "количество цифр, содержащихся в ней".

@DevAlone · 14.01.2017, 12:00

Сообщение от khan01

if (48 <= s[i] <= 57); a++;

меняйте не

C++

1	if(s[i] >= 48 && s[i] <= 57)

это вам не питон

@FreeYourMind 147 / 147 / 104 Регистрация: 13.11.2016 Сообщений: 557
	14.01.2017, 10:49	2
	Сообщение от khan01 if (48 <= s[i] <= 57); для начала тут по моему должно быть 48<=s[i] && s[i]<=57 + к этому компиляторбудет проверять элемент строки на наличие в ней цифр из диапазона от 48 до 57.. тут мне кажется даже не пахнет ASCII 0

	14.01.2017, 12:00

Опции темы