Нахождение и подсчет прописных букв

@vonorf · Регистрация: 05.10.2009

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

Возникла проблема с подсчетом прописных букв, вот условие задачи:

Написать функцию, которая получает в качестве параметров две строки и возвращает в вызывающую программу количество прописных букв, одновременно содержащихся в заданных строках.

В главной программе прочесть с клавиатуры две строки, содержащие текст, и вывести на экран количество общих прописных букв во введенных строках.

Мой код в VS 2008:

C++

#include "stdafx.h"
#include <iostream>
#include <ctype.h>
#include <stdlib.h>
using namespace std;
int f1(const char*,const char*);
int main(int argc, char* argv[])
{
    char string1[20],string2[20]; 
    cout<<"vvedi dannie:";  
    cin>>string1>>string2;
    cout<<"kolichestvo obshchih propisnyh bukv: "<<f1<<"\n";
    return 0;
}
int f1(const char*s1,const char*s2)
{
    for (;*s1!='\0'&&*s2!='\0';s1++,s2++)
    return (isupper(*s1)+isupper(*s2));
}

В ответе программа выводит вроде бы шестнадцатеричный код, вот пример ее работы:

vvedi dannye:Hello World!
kolichestvo obshchih propisnyh bukv: 01091037

Ошибка в коде программы или надо дополнительно переводить числа в десятичный код? Если да, то как?

@XuTPbIu_MuHTAu · 05.10.2009, 19:19

vonorf, ты распечатал адрес функции. Кстати,она еще и неправильно написана

@vonorf · 05.10.2009, 19:23 **[ТС]**

Учу этот C++ меньше месяца, очень, очень, очень много еще не знаю. Можешь ли ты подсказать, как правильно написать?

@XuTPbIu_MuHTAu · 05.10.2009, 19:25

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

Учу этот C++ меньше месяца, очень, очень, очень много еще не знаю. Можешь ли ты подсказать, как правильно написать?

Могу.Функцию надо вызывать,чтобы получить ее результат.

@vonorf · 05.10.2009, 19:26 **[ТС]**

А если показать на моем примере?

@XuTPbIu_MuHTAu · 05.10.2009, 19:27

f1(string1,string2);

@TanT · 05.10.2009, 19:31

это тока самое начало. STL конечно ещё не известен. то есть нужен вариант с совсем простыми и всем известными элементами языка, я так понимаю
щас поем и напишу

если XuTPbIu_MuHTAu не опередит

@vonorf · 05.10.2009, 19:39 **[ТС]**

Огромное спасибо за помощь, все получилось.

C++

#include "stdafx.h"
#include <iostream>
#include <ctype.h>
#include <stdlib.h>
using namespace std;
int f1(const char*,const char*);
int main(int argc, char* argv[])
{
    char string1[20],string2[20]; 
    cout<<"vvedi dannie:";  
    cin>>string1>>string2;
    cout<<"kolichestvo obshchih propisnyh bukv: "<<f1(string1,string2)<<"\n";
    return 0;
}
int f1(const char*s1,const char*s2)
{
    for (;*s1!='\0'&&*s2!='\0';s1++,s2++)
    return (isupper(*s1)+isupper(*s2));
}

При написании Hello World выдает 2 прописные буквы, но если строчные буквы находятся в середине слова он их не воспринимает...

@TanT · 05.10.2009, 19:48

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

int f1(const char*s1,const char*s2)
{
for (;*s1!='\0'&&*s2!='\0';s1++,s2++)
return (isupper(*s1)+isupper(*s2));
}[/CPP]

При написании Hello World выдает 2 прописные буквы, но если строчные буквы находятся в середине слова он их не воспринимает...

правильно, ты возвращаешь сумму значений двух результатов isupper(*s1) и isupper(*s2). где isupper(*s1) - возвращает один, если первый символ строки s1 вверхнем регистре, и ноль, если в нижнем

@vonorf · 05.10.2009, 19:52 **[ТС]**

Понятно, а есть возможность сделать так, чтобы рассматривался не только первый символ?

@XuTPbIu_MuHTAu · 05.10.2009, 19:56

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

Понятно, а есть возможность сделать так, чтобы рассматривался не только первый символ?

Есть.Рассматривай каждый в цикле и только после этого делай return.

P.S, Tant, готовый код я все равно в эту тему не буду постить

@TanT · 05.10.2009, 20:10

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

Есть.Рассматривай каждый в цикле и только после этого делай return.

P.S, Tant, готовый код я все равно в эту тему не буду постить

причина: воспитание или комерческая тайна?
P.S. вопрос вполне серьёзный.

@XuTPbIu_MuHTAu · 05.10.2009, 20:17

TanT, Это скорее подход к проблеме. Можно помогать,а можно решить проблему вместо автора. Исходя из сегодняшнего расположения звезд я выбрал первый вариант

vonorf, return осуществляет выход из функции,поэтому цикла фактически не получается. Заведи внутри отдельную переменную,в которой посчитаешь количество,а потом ее верни ретёрном.

@TanT · 05.10.2009, 20:22

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

TanT, Это скорее подход к проблеме. Можно помогать,а можно решить проблему вместо автора. Исходя из сегодняшнего расположения звезд я выбрал первый вариант

хитро, хотя глядя на ник, думаю не один я так думаю

@vonorf · 05.10.2009, 20:29 **[ТС]**

C++

int f1(const char*s1,const char*s2)
{
    float S;
    for (;*s1!='\0'&&*s2!='\0';s1++,s2++)
    S=(isupper(*s1)+isupper(*s2));
    return S;
}

Если вы имели ввиду это, то он вообще ничего не считает

@XuTPbIu_MuHTAu · 05.10.2009, 20:32

vonorf, интересно,какой логикой ты пользовался,
1. Объявляя S как float.Думаешь,там может получиться дробное число?
2. не присваивая S никакого начального значения? Он считает начиная непонятно с какого числа.
3. ничего не суммируя,а заново перезаписывая что-то в S?

@TanT · 05.10.2009, 20:33

покрайней мере так

C++

int f1(const char*s1,const char*s2)
{
        float S=0; 
        for (;*s1!='\0'&&*s2!='\0';s1++,s2++)
            S+=isupper(*s1)+isupper(*s2);
        return S;
}

хотя достаточно int

@vonorf · 05.10.2009, 20:34 **[ТС]**

а такой вариант верен?

C++

int f1(const char*s1,const char*s2)
{
    int S;
    for (S=0;*s1!='\0'&&*s2!='\0';s1++,s2++)
    S=(isupper(*s1)+isupper(*s2));
    return S;
}

@TanT · 05.10.2009, 20:37

нет, S - неопределено, точнее не обнулено

@vonorf · 05.10.2009, 20:39 **[ТС]**

TanT, XuTPbIu_MuHTAu, большое вам спасибо. Все понял, все работает.

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 19:56	11
	Сообщение от vonorf Понятно, а есть возможность сделать так, чтобы рассматривался не только первый символ? Есть.Рассматривай каждый в цикле и только после этого делай return. P.S, Tant, готовый код я все равно в эту тему не буду постить 0

@TanT эволюционирую потихоньку 468 / 466 / 91 Регистрация: 30.06.2009 Сообщений: 1,401
	05.10.2009, 20:10	12
	Сообщение от XuTPbIu_MuHTAu Есть.Рассматривай каждый в цикле и только после этого делай return. P.S, Tant, готовый код я все равно в эту тему не буду постить причина: воспитание или комерческая тайна? P.S. вопрос вполне серьёзный. 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 20:17	13
	TanT, Это скорее подход к проблеме. Можно помогать,а можно решить проблему вместо автора. Исходя из сегодняшнего расположения звезд я выбрал первый вариант vonorf, return осуществляет выход из функции,поэтому цикла фактически не получается. Заведи внутри отдельную переменную,в которой посчитаешь количество,а потом ее верни ретёрном. 0

@TanT эволюционирую потихоньку 468 / 466 / 91 Регистрация: 30.06.2009 Сообщений: 1,401
	05.10.2009, 20:22	14
	Сообщение от XuTPbIu_MuHTAu TanT, Это скорее подход к проблеме. Можно помогать,а можно решить проблему вместо автора. Исходя из сегодняшнего расположения звезд я выбрал первый вариант хитро, хотя глядя на ник, думаю не один я так думаю 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 20:32	16
	vonorf, интересно,какой логикой ты пользовался, 1. Объявляя S как float.Думаешь,там может получиться дробное число? 2. не присваивая S никакого начального значения? Он считает начиная непонятно с какого числа. 3. ничего не суммируя,а заново перезаписывая что-то в S? 1

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 19:19	2
	vonorf, ты распечатал адрес функции. Кстати,она еще и неправильно написана 0

@vonorf 7 / 7 / 0 Регистрация: 05.10.2009 Сообщений: 31
	05.10.2009, 19:23 [ТС]	3
	Учу этот C++ меньше месяца, очень, очень, очень много еще не знаю. Можешь ли ты подсказать, как правильно написать? 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 19:25	4
	Сообщение от vonorf Учу этот C++ меньше месяца, очень, очень, очень много еще не знаю. Можешь ли ты подсказать, как правильно написать? Могу.Функцию надо вызывать,чтобы получить ее результат. 0

@vonorf 7 / 7 / 0 Регистрация: 05.10.2009 Сообщений: 31
	05.10.2009, 19:26 [ТС]	5
	А если показать на моем примере? 0

@XuTPbIu_MuHTAu 2255 / 770 / 25 Регистрация: 27.05.2008 Сообщений: 1,496
	05.10.2009, 19:27	6
	f1(string1,string2); 1

@TanT эволюционирую потихоньку 468 / 466 / 91 Регистрация: 30.06.2009 Сообщений: 1,401
	05.10.2009, 19:31	7
	это тока самое начало. STL конечно ещё не известен. то есть нужен вариант с совсем простыми и всем известными элементами языка, я так понимаю щас поем и напишу если XuTPbIu_MuHTAu не опередит 0

@TanT эволюционирую потихоньку 468 / 466 / 91 Регистрация: 30.06.2009 Сообщений: 1,401
	05.10.2009, 19:48	9
	Сообщение от vonorf int f1(const chars1,const chars2) { for (;s1!='\0'&&s2!='\0';s1++,s2++) return (isupper(s1)+isupper(s2)); }[/CPP] При написании Hello World выдает 2 прописные буквы, но если строчные буквы находятся в середине слова он их не воспринимает... правильно, ты возвращаешь сумму значений двух результатов isupper(s1) и isupper(s2). где isupper(*s1) - возвращает один, если первый символ строки s1 вверхнем регистре, и ноль, если в нижнем 0

@vonorf 7 / 7 / 0 Регистрация: 05.10.2009 Сообщений: 31
	05.10.2009, 19:52 [ТС]	10
	Понятно, а есть возможность сделать так, чтобы рассматривался не только первый символ? 0

@TanT эволюционирую потихоньку 468 / 466 / 91 Регистрация: 30.06.2009 Сообщений: 1,401
	05.10.2009, 20:37	19
	нет, S - неопределено, точнее не обнулено 1

@vonorf 7 / 7 / 0 Регистрация: 05.10.2009 Сообщений: 31
	05.10.2009, 20:39 [ТС]	20
	TanT, XuTPbIu_MuHTAu, большое вам спасибо. Все понял, все работает. 0