Форум программистов, компьютерный форум CyberForum.ru

По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы - C++

Восстановить пароль Регистрация
 
Fox_ua
Сообщений: n/a
01.07.2014, 18:42     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы #1
Помогите отладить программу задача которой :

В восточном календаре принят 60-летний цикл, состоящий из 12-летних подциклов, обозначаемых названиями цвета: зеленый, красный, желтый, белый и черный. В каждом подцикле года носят названия животных: крысы, коровы, тигра, зайца, дракона, змеи, лошади, овцы, обезьяны, курицы, собаки и свиньи. По номеру года вывести его название, если 1984 был началом цикла - годом зеленой крысы. Использовав оператор switch / case.

Вот что у меня вышло :

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
#include "stdafx.h"
#include <math.h>
#include <iostream>
#include <conio.h>
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
{
    int x;
m3: cout<<" Enter year: ";
m1: cin>>x;
    if((x>=1984)&&(x<=3000))
    {
        goto m2;
    }
    else
    if(x<1984)
    {
        cout<<"\n Enter the correct year: ";
        goto m1;
    }
    else
    if(x>3000)
    {
        cout<<"\n Enter the correct year: ";
        goto m1;
    }
    
m2: while(x>10)
    {
        x=x-10;
    }
    switch(x)
    {
        case 1: cout<<"\n White"; break;
        case 2: cout<<"\n Black"; break;
        case 3: cout<<"\n Black"; break;
        case 4: cout<<"\n Green"; break;
        case 5: cout<<"\n Green"; break;
        case 6: cout<<"\n Red"; break;
        case 7: cout<<"\n Red"; break;
        case 8: cout<<"\n Yellow"; break;
        case 9: cout<<"\n Yellow"; break;
        case 10: cout<<"\n White"; break;
    }
 
    while(x>12)
    {
        x=x-12;
    }
    switch(x)
    {
        case 1: cout<<" chicken \n\n"; break;
        case 2: cout<<" dog \n\n"; break;
        case 3: cout<<" pig  \n\n"; break;
        case 4: cout<<" rat \n\n"; break;
        case 5: cout<<" cow \n\n"; break;
        case 6: cout<<" tiger \n\n"; break;
        case 7: cout<<" rabbit \n\n"; break;
        case 8: cout<<" dragon \n\n"; break;
        case 9: cout<<" snake \n\n"; break;
        case 10: cout<<" horse \n\n"; break;
        case 11: cout<<" sheep \n\n"; break;
        case 12: cout<<" monkey \n\n"; break;
    }
    goto m3;
    getch();
    return 0;
}
Программа работает исправно только с 1984 до 1990 дальше возникает проблема с циклом. Как можно её решить ?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
01.07.2014, 18:42     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы
Посмотрите здесь:

Вывести название дня недели по его номеру C++
По номеру месяца определить его название и время года к которому он относится C++
Как выглядеть оператор, если по номеру дня недели необходимо написать его название C++
C++ По введенному номеру года получить его название по старояпонскому календарю.
Вывести время года по номеру месяца C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
nmcf
4318 / 3739 / 1260
Регистрация: 14.04.2014
Сообщений: 14,664
02.07.2014, 09:59     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы #2
Вот всё что нужно после ввода x (c 29-й строки):
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
switch(((x - 1984) % 60) / 12)
    {
        case 0: cout<<"\n Green"; break;
        case 1: cout<<"\n Red"; break;
        case 2: cout<<"\n Yellow"; break;
        case 3: cout<<"\n White"; break;
        case 4: cout<<"\n Black"; break;
    }
switch(((x - 1984) % 60) % 12)
    {
        case 0: cout<<" chicken \n\n"; break;
        case 1: cout<<" dog \n\n"; break;
        case 2: cout<<" pig  \n\n"; break;
        case 3: cout<<" rat \n\n"; break;
        case 4: cout<<" cow \n\n"; break;
        case 5: cout<<" tiger \n\n"; break;
        case 6: cout<<" rabbit \n\n"; break;
        case 7: cout<<" dragon \n\n"; break;
        case 8: cout<<" snake \n\n"; break;
        case 9: cout<<" horse \n\n"; break;
        case 10: cout<<" sheep \n\n"; break;
        case 11: cout<<" monkey \n\n"; break;
    }
Гром
 Аватар для Гром
199 / 118 / 10
Регистрация: 20.03.2009
Сообщений: 1,075
Записей в блоге: 15
02.07.2014, 15:37     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы #3
Ну и я бы повыкидывал все эти goto куда подальше... И еще пара плюшек к коду:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
#include <cmath>  // <cXXX> вместо устаревшего <XXX.h>
#include <iostream>
#include <conio.h>
using namespace std;
 
int main(int argc, char* argv[]) // Обычный стандартный main вместо локального майкрософтского изврата
{
    int x;
    bool goodYear = false;
    while (! goodYear)  // Вместо возни с goto просто используем цикл
        {
        cout<<" Enter year: ";
        cin>>x;
        if (x < 1984 || x > 3000)  // Условие в if можно объединить с помощью ИЛИ, поскольку результат одинаков
            cout<<"\n Enter the correct year: ";
        else
            goodYear = true;
        }
//---
// Code by [nick]nmcf[/nick]
switch(((x - 1984) % 60) / 12)
    {
        case 0: cout<<"\n Green"; break;
        case 1: cout<<"\n Red"; break;
        case 2: cout<<"\n Yellow"; break;
        case 3: cout<<"\n White"; break;
        case 4: cout<<"\n Black"; break;
    }
switch(((x - 1984) % 60) % 12)
    {
        case 0: cout<<" chicken \n\n"; break;
        case 1: cout<<" dog \n\n"; break;
        case 2: cout<<" pig  \n\n"; break;
        case 3: cout<<" rat \n\n"; break;
        case 4: cout<<" cow \n\n"; break;
        case 5: cout<<" tiger \n\n"; break;
        case 6: cout<<" rabbit \n\n"; break;
        case 7: cout<<" dragon \n\n"; break;
        case 8: cout<<" snake \n\n"; break;
        case 9: cout<<" horse \n\n"; break;
        case 10: cout<<" sheep \n\n"; break;
        case 11: cout<<" monkey \n\n"; break;
    }
//---
 
    getch();
    // return 0; - в функции main (и только в ней!) подразумевается по умолчанию, можно не писать
}
ValeryS
Модератор
6377 / 4843 / 442
Регистрация: 14.02.2011
Сообщений: 16,058
02.07.2014, 16:15     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы #4
Цитата Сообщение от Fox_ua Посмотреть сообщение
рограмма работает исправно только с 1984 до 1990
а знаешь почему???
Цитата Сообщение от Fox_ua Посмотреть сообщение
C++
1
2
3
4
5
while(x>10)
    {
        x=x-10;
    }
    switch(x)
все x испортили он уже стал в диапазоне 1-10
и следовательно
этот участок кода
Цитата Сообщение от Fox_ua Посмотреть сообщение
C++
1
2
3
4
5
6
while(x>12)
    {
        x=x-12;
    }
    switch(x)
    {
работает неправильно
обрати внимания на решения, которые тебе предложили
узнай что такое остаток от деления %
в частности
C++
1
2
3
4
while(x>10)
    {
        x=x-10;
    }
можно поменять на
C++
1
2
x%=10;
x+=1;
Добавлено через 26 минут
Цитата Сообщение от ValeryS Посмотреть сообщение
все x испортили он уже стал в диапазоне 1-10
пардон, погнал от 0 до 10
соответственно
Цитата Сообщение от ValeryS Посмотреть сообщение
x%=10;
x+=1;
будет
C++
1
x%=11;
Цитата Сообщение от Fox_ua Посмотреть сообщение
до 1990
при 1990 после первого цикла x будет равен 0
а ни в одном свитче ветки 0 нет
UriahHeep
 Аватар для UriahHeep
222 / 217 / 41
Регистрация: 24.09.2012
Сообщений: 509
Записей в блоге: 2
02.07.2014, 16:52     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы #5
Раз у вас начало отсчета – год крысы, то в switch нужно соответственно:
C++
switch(((x - 1984) % 60) % 12)
    {
        case 0: cout<<" rat \n\n"; break;
        case 1: cout<<" cow \n\n"; break;
        case 2: cout<<" tiger \n\n"; break;
        case 3: cout<<" rabbit \n\n"; break;
        case 4: cout<<" dragon \n\n"; break;
        case 5: cout<<" snake \n\n"; break;
        case 6: cout<<" horse \n\n"; break;
        case 7: cout<<" sheep \n\n"; break;
        case 8: cout<<" monkey \n\n"; break;
        case 9: cout<<" chicken \n\n"; break;
        case 10: cout<<" dog \n\n"; break;
        case 11: cout<<" pig  \n\n"; break;
    }
А то 2014 год у вас получается годом кролика.
Yandex
Объявления
02.07.2014, 16:52     По номеру года вывести его название, если 1984 был началом цикла — годом зеленой крысы
Ответ Создать тему
Опции темы

Текущее время: 16:32. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru