Почему не работает часть программы?

Question

[iamgrootyhuuu]

Я недавно начала изучать С++ и еще не очень во всем разбираюсь, поэтому помогите понять как исправить программу, чтобы она работала.
Тема: оператор выбора
Задание: Ввести день, месяц и год, проверить правильность введенной даты и вывести дату
следующего дня.

#include <iostream>
#include <conio.h>
#include <cmath>

using namespace std;
int d, m, g, S;

int main()
{
setlocale(LC_ALL, "Russian_Russia.1251");

cout « "Введите дату"; cin » d, m, g;
if (g >= 1, g <= 12);
{
cout « "g=" « endl;

}

else
{
cout « "Такого года нет" « endl;
}
switch (d, m, g)
{
if (m >= 1, m <= 12)
{
cout « "m=" « endl;
}
else
{
cout « " Такого месяца нет" « endl;
}
if (d >= 1, d <= 31)
{
case 1:
case 3:
case 5:
case 7:
case 8:
case 10:
case 12 :

cout « "Правильно" « endl;
}
else {
cout « "Неправильно" « endl;
}
if (d >= 1, d <= 30)
{
case 4:
case 6:
case 9:
case 11:
cout « "Правильно" « endl;

}
else {
cout « "Неправильно" « endl;
}
if (d >= 1, d <= 28)
{
case 2:
cout « "Правильно" « endl;

}
else {
cout « "Неправильно" « endl;
}
S = (d + 1, m + 1, g + 1);
cout « "S=" « endl; cin » S;
}

return 0;
}

Comments

[Василий Банников]

Можно отступы нормально расставить? И какая часть не работает?

[Василий Банников]

У вас программа не работает, тк вместо << у вас французские кавычки.

[iamgrootyhuuu]

Василий Банников, там получается запускаешь программу и выполняется все до switch, но и дело еще в том, что когда пытаешься ввести год, условие не выполняется и он на все выдает "такого года нет"

[pfemidi]

Давно ли внутри switch можно делать if? Не внутри блока case {}, а прямо внутри switch? Это какой стандарт C++?

[iamgrootyhuuu]

pfemidi, это С++ да, то есть внутри switch нельзя ставить if? А где тогда поставить условие?

[Василий Банников]

iamgrootyhuuu, я хз зачем тут вообще свич. Дайте полную формулировку алгоритма.

[iamgrootyhuuu]

Василий Банников, нужно с помощью switch case сделать программу алгоритма по вышеизложенному заданию. Думаю тут нет смысла использовать if else , но я не знаю, как правильно без них написать, чтобы условия все выполнялись

[Василий Банников]

iamgrootyhuuu, сначала алгоритм опиши.

[iamgrootyhuuu]

Василий Банников, я не очень понимаю, что именно я должна описать в алгоритме

[pfemidi]

У вас программа не работает, тк вместо << у вас французские кавычки.

Василий Банников, да это вообще не программа, а какой-то инвалид. Причём инвалид не по возрасту и не в результате травмы, а прям с рождения, хуже однорукого и одноногого слепоглухонемого дауна с ДЦП.

Блин, "Почему не работает часть программы?". Да тут ВСЁ не может работать! Даже компилироваться не будет, до работы и дело не дойдёт!

[Евгений Шатунов]

pfemidi,

Давно ли внутри switch можно делать if?

Всегда можно было. Это стандартный синтаксис. case - это просто метка, на которую исполнение прыгает при соответствии условия. Локальность на все case едина - это локальность switch. Далее, после каждого case уже можно организовать свою локальность. Собственно, именно поэтому определение локальных переменных просто под case и сулит проблемами.
Возможно это тебя шокировало, но таков синтаксис C++. :)

[Евгений Шатунов]

iamgrootyhuuu, давай я задам несколько вопросов чтобы помочь тебе. Для тебя будет полезно ответить на них хотя бы самой себе. Но лучше, конечно, чтобы я мог прочитать твои ответы и у нас получился бы диалог.

int d, m, g, S;
Какой смысл в существовании каждой из этих переменных? Если у тебя не получается сформулировать смысл существования какой-то переменной, тебе нужно стереть эту переменную. Если у тебя получается сформулировать смысл существования переменной, то почему переменная названа настолько бессмысленно?

switch (d, m, g)
Тут два раза применен int operator , ( int, int ), какой у него эффект по умолчанию?

Насколько тебе удобно читать свой код, когда все фигурные скобки и код всех областей видимости находится на одном уровне?

[pfemidi]

Евгений Шатунов, что, можно прямо

switch(some)
    {
        if (some == other)
            doOther();
        case 1:
            doCaseOne();
            break;
        default:
            doDefault();
            break;
    }

а не

switch(some)
    {
        case 1:
            doCaseOne();
            break;
        default:
            if (some == other)
                doOther();

            doDefault();
            break;
    }

[iamgrootyhuuu]

Евгений Шатунов, по заданию нужно ввести день, месяц, год и получается вывести дату следующего дня, соответственно, d-день, m-месяц,g-год,S-следующий день. int - он же обозначает, что переменная целая, а switch получается показывает, какие переменные нужно проверять в условии, которое написано в case. На счет расположения кода, мне вроде удобно читать, но, видимо, такое расположение не совсем верно

[Евгений Шатунов]

iamgrootyhuuu,

d-день, m-месяц,g-год,S-следующий день

А что тебе будет понятнее прочитать: d или day_number?
Соответственно: m или month_number?
И еще: S или next_day_number?
Ты ссылаешься на задание, но ты не всегда будешь его помнить. В следующем году, когда ты уже все забудешь, ты откроешь этот код и точно так же, как и все остальные участники текущего обсуждения, не будешь видеть никакого смысла в этих однобуквенных названиях.
Моим советом, который дальше только поможет тебе, будет: всегда выбирай осмысленные имена. Никогда не ссылайся на задания или что-то еще, передавай суть заданий прямо в тексте программы. Для этого всегда выбирай только осмысленные имена, которые отражают смысл существования в рамках твоей задачи.
Пока у тебя получается вот такое. Там человек сперва пишет одно, а потом имеет в виду вообще другое. Твой код, точно так же, пока только вводит в заблуждение.
Твоя стратегия мышления должна быть схожей вот с этим. Посмотри там сам вопрос, а потом цепочку комментариев от обозначенного и ниже.

а switch получается показывает, какие переменные нужно проверять в условии, которое написано в case

Тут нужно сходить в документацию оператора switch [?] и увидеть что проверка идет только для одного выражения. Я не просто так написал про int operator , ( int, int ), по умолчанию comma operator возвращает только свой правый аргумент.
Код switch (d, m, g) эквивалентен switch (g).
На самом деле другие люди тебе мало чем смогут помочь по изучению языка. Даже я вряд ли смогу тебе полноценно помочь. Тебе будет лучше обратиться к известным общедоступным источникам информации по языку. Сперва все будет очень непонятно, но ты просто читай, на начальных этапах всегда важно собирать информацию.
Между тем, тебе всегда стоит помнить что люди будут только рады дать исчерпывающий ответ на твой конкретный вопрос. "Помочь с синтаксисом" - довольно широкий вопрос, на который довольно тяжело ответить. "Как правильно пользоваться конструкцией switch-case" - хороший, конкретный вопрос, на который любой квалифицированный специалист будет рад дать ответ.

мне вроде удобно читать

На самом деле неудобно даже тебе и даже сейчас. Пока у тебя не больше 20 строк кода, ты можешь позволить себе такое форматирование. Но посмотри на это. Там сложность зашкаливает. Без горизонтального блочного форматирования код был бы абсолютно нечитаемым. И, на самом деле, твой код является нечитаемым абсолютно для всех разработчиков потому что люди уже привыкли код видеть блоками. Тебе стоит просто попробовать и ты поймешь насколько это облегчает понимание.
Подобные статьи помогут тебе с этим вопросом: [1], [2], [3], [4], [5].

Возвращаясь к твоему коду. У твоего кода должна быть структура, которая отражает ход решения задачи.

Ввести день, месяц и год, проверить правильность введенной даты и вывести дату следующего дня.

Задача у тебя довольно большая, на самом деле. Решать ее целиком - идея плохая. Твою задачу нужно поделить на этапы. Минимально тут выделяется три больших тапа: ввод данных, проверки и вычисления, вывод результата.
Каждый из этих этапов тоже нужно разделить.
На какие более мелкие этапы ты бы разделила каждый из этих больших этапов?

[Евгений Шатунов]

pfemidi, я так понимаю, это вопрос?
Да, можно так, как у тебя в первом блоке кода. И это нормально, ты ведь пишешь всякий код после case. Чем в этом случае обделен if?
Более того, существуют ли какие-либо ограничения для установки меток [?] внутри локальных областей функций?

Единственное что, в твоем первом блоке кода doOther(); никогда не выполнится потому что исполнение прыгает только на метки case, но не перед ними. Исполнение дальше switch не пойдет, а всегда только куда-нибудь прыгнет.

[pfemidi]

Евгений Шатунов, я ничего не имею против if после case, но у ТС if расположен не после case, а сразу же после switch. И это IMHO не только работать, это даже компилироваться не будет.

[Евгений Шатунов]

pfemidi,

И это IMHO не только работать, это даже компилироваться не будет.

Это лишнее. Транслятор работает согласно грамматике языка, а грамматика именно такая, что все это позволяет.
Не надо играть в нострадамуса и пытаться что-то предсказать. Никаких частных мнений в подобной ситуации существовать может. Все твои знания и выводы должны опираться только на стандарт и документацию.
Вместо предположений тебе стоит делать проверки. Как видишь, трансляцию этот код проходит спокойно.

[pfemidi]

Евгений Шатунов, да... Согласен. Что называется "до седых дожил..." Я просто за все свои 30 лет практики никогда не видел чтобы if был сразу же после switch, вне всякого case. И был уверен что компилятор подобный бред (а это ну явно бред, это же 100% unreachable code) не пропустит. Ошибался.

[Евгений Шатунов]

pfemidi, я за 20 лет своей практики видел всякое. Такой код в коммерческих продуктах тоже видел. Весь мой опыт и позиция относительно подобных вопросов позволяют мне определять ошибки трансляторов, чем я последние 5 лет периодически и занимаюсь.
Иногда транслятор может сказать что в твоем коде ошибка просто потому что ошибка на самом деле в трансляторе. И хороший специалист должен уметь определять такие ситуации. А значит, хороший специалист должен лучше транслятора знать стандарт языка.

Answer 1

[Wataru]

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

Comments

[iamgrootyhuuu]

конечно у меня нет понимания базового синтаксиса, мне только начали все это объяснять и пока не очень заходит, я поэтому и прошу здесь помощи , чтобы исправить ошибки и программа заработала

[Wataru]

iamgrootyhuuu, Перечитайте учебник. Погуглите "синтаксис си++". Там будет куча примеров. Учить вас с нуля программировать я не хочу.

[iamgrootyhuuu]

Wataru, я не прошу вас учить меня с нуля, мне нужен ответ на вопрос который я задала

[Wataru]

Ответ дан - везде, где вы используете запятые, происходит не то, что вы думаете.

Answer 2

[Homemade]

Я бы заменил свич-ы на иф-ы. И отступы правильно нужно ставить, а то так тяжело читать.

Comments

[iamgrootyhuuu]

но в операторе выбора там же должен быть switch

[Homemade]

iamgrootyhuuu, имею ввиду полностью написать на ифе, без свитча. Разумеется, если того допускает условие задания.

[iamgrootyhuuu]

Homemade, мне наоборот нужен switch case , чтобы программу написать

Answer 3

[Aveal]

Вкратце, в таком ключе:

if ((g >= 0) && (g <= 2021))
{
	cout « "year: ok" « endl;
}
else
{
	cout « "year: error" « endl;
}

if ((m >= 1) && (m <= 12))
{
	cout « "month: ok" « endl;
}
else
{
	cout « "month: error" « endl;
}

switch(m)
{
	case 1:
		cout « "january" « endl;
		break;
		
	case 2:
		cout « "february" « endl;
		break;
		
	// etc...
}

// to be continued...

А вообще рекомендую, например, Герберт Шилдт - "C++", точное название не помню.