Программа написано верно, но почему-то не работает (на Си)

[Дмитрий Огнев]

Написал прогу, кампилятор на нее не ругается. Запускаю, но правильно она не работает.

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

#include "stdafx.h"
#include <conio.h>
#include <iostream>
using namespace std;

int flag(char s)
{
	int k=0;
	if (s == 'б' || s == 'в' || s == 'г' || s == 'д' || s == 'ж' || s == 'з' || s == 'к' || s == 'л' || s == 'м' || s == 'н' || s == 'п' 
        || s == 'р' || s == 'с' || s == 'т' || s == 'ф' || s == 'х' || s == 'ц' || s == 'ч' || s == 'ш' || s == 'щ') k = 1;
	return k; 
}
int main(int argc, char* argv[])
{
	char s;
	cout << "Vvod" << endl;
	cin >> s;
	if (flag(s)==1) cout << "True";
	else cout << "False";
	system("PAUSE");
	return 0;
}

[BDA]

Кодировка текста в исходнике и при вводе в консоли не совпадают. Исходник, скорее всего, имеет кодировку CP1251, а ввод в консоли происходит в CP866. Забейте коды букв - кодировка. Используйте unsigned char на всякий случай. Или попробуйте поиграться с setlocale и chartooem.

[Вадим Мошев]

1. Операдны логических выражений должны заключаться в скобки:

if ((a == b) || (c == d))

2.
вот этого уродства:

if (s == 'б' || s == 'в' || s == 'г' || s == 'д' || s == 'ж' || s == 'з' || s == 'к' || s == 'л' || s == 'м' || s == 'н' || s == 'п'
          || s == 'р' || s == 'с' || s == 'т' || s == 'ф' || s == 'х' || s == 'ц' || s == 'ч' || s == 'ш' || s == 'щ') k = 1;

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

3. Пусть ваша функция возвращает логическое значение (bool)

[ATL]

И проще перебрать гласные. Их поменьше.

[BDA]

Вадим Мошев, "==" имеет больший приоритет, чем "||".

[Вадим Мошев]

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

И проще перебрать гласные. Их поменьше.

А если в качестве аргумента передана не буква? Например, число или символ?

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

Вадим Мошев, "==" имеет больший приоритет, чем "||".

спасибо

[type_Oleg]

Кстати, гласных гораздо меньше, чем согласных.

Если бы задача была " TRUE , если буква не Ы , FALSE - если Ы " - стали бы перечислять 32 буквы ?

[Вадим Мошев]

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

Кстати, гласных гораздо меньше, чем согласных.

Если бы задача была " TRUE , если буква не Ы , FALSE - если Ы " - стали бы перечислять 32 буквы ?

Здесь я бы вывел на экран результат операции сравнения: letter != 'Ы'