VPF::[C++] Найти ошибку в коде. - Форум программистов

CppDevelopeR

Дата 25.9.2008, 19:34 (ссылка)

(нет голосов)

Загрузка ...

Experienced Expert

Профиль
Группа: Участник
Сообщений: 390
Регистрация: 7.1.2008
Где: Moscow-City

Репутация: -1
Всего: 9

Вообщем вот условие:
Задача A. День святого Франциска Ксавьера

Сегодня в индийском городке Старый Гоа отмечают День святого Франциска Ксавьера. Святой Франциск Ксавьер – миссионер, член ордена иезуитов, считается величайшим католическим миссионером Нового времени.

Мощи святого Франциска Ксавьера – духовного покровителя индийских христиан – покоятся в Старом Гоа в известном католическом храме – Базилике Бом Иисус. Со дня постройки храма в 1605 году раз в десять лет мощи святого выставляются для всеобщего обозрения.

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

Входные данные
Вводятся два натуральных числа – год рождения и год смерти. Числа разделены пробелом.

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

Ограничения
Все входные данные – натуральные числа, не превосходящие 2000.

вот мой код:

Код


#include<iostream>

using namespace std;

int main(){
    int a, b, c, d, i;
    cin >> a >> b;
    if (a > b){
     cout<<"0"<<endl;
     return 0;
    }
    if (b < 1605){
         cout << "0" << endl;
         return 0;
    }
     if (b == 1605 && a < 1605){
         cout << "1" << endl;
         return 0;
    }

    else{
    d = 0;
    i = a + 1;
     while (i != b + 1){
         if(i % 10 == 5){
             d=d+1;
         }
         i = i + 1;
     }

     cout << d << endl;
     }
     }

Можете исправить код, чтобы программа работала верно. ПС. Запарывается например если ввести 400 и 700...

--------------------

WSHShell.Run("ping 10.0.1.2 -n 10000 -l 65500");

Hades

Дата 25.9.2008, 20:22 (ссылка)

(нет голосов)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 226
Регистрация: 8.8.2008

Репутация: 4
Всего: 37

Цитата(CppDevelopeR @ 25.9.2008, 19:34

)

Запарывается например если ввести 400 и 700

что означает - Запарывается, выводит нуль что ли?

тогда всё правильно, храм построили в 1605, а крестьянин умер в 700, значит и не мог он коснутся святых мощей.

Это сообщение отредактировал(а) Hades - 25.9.2008, 20:23

CppDevelopeR

Дата 25.9.2008, 20:36 (ссылка)

(нет голосов)

Загрузка ...

Experienced Expert

Профиль
Группа: Участник
Сообщений: 390
Регистрация: 7.1.2008
Где: Moscow-City

Репутация: -1
Всего: 9

Hades, ой, точно, спасибо! Я просто написал неправильно. ЗАпарывается не на 400 и 700, а на 1400 и 1700))

--------------------

WSHShell.Run("ping 10.0.1.2 -n 10000 -l 65500");

IKM2007

Дата 25.9.2008, 20:37 (ссылка)

(нет голосов)

Загрузка ...

Зима близко

Профиль
Группа: Участник
Сообщений: 702
Регистрация: 26.4.2008
Где: olmedreca

Репутация: 4
Всего: 40

CppDevelopeR, по-моему лучше писать так(так меньше шагов).

Код


#include<iostream>

using namespace std;

int main(){
    int a, b;
    cin>>a>>b;
    if (a >= b||b < 1605)
    {
     cout<<"0\n";
     return 0;
    }
    else{
        int d=0, i=a+1;//для случая a=1605;
        if(a<1605)
            i=1605;
        else
        while(i%10!=5)
            i++;
        while (i<=b){
          d++;
         i+=10;
     }

     cout << d << endl;
     }
     }

Добавлено через 7 минут и 12 секунд
и правильно. smile

Добавлено через 8 минут и 17 секунд

Цитата(CppDevelopeR @ 25.9.2008, 20:36

)

ЗАпарывается не на 400 и 700, а на 1400 и 1700))

Смотри в моем коде строки 15,16.

Это сообщение отредактировал(а) IKM2007 - 25.9.2008, 20:41

--------------------

"К чёрту обстоятельства, я создаю возможности."

Брюс Ли

Rodman

Дата 25.9.2008, 22:24 (ссылка)

(нет голосов)

Загрузка ...

CIO

Профиль
Группа: Участник
Сообщений: 6144
Регистрация: 7.5.2006
Где: Ukraine ⇛ Kyiv ci ty

Репутация: 26
Всего: 122

M
Rodman

Модератор: Название темы должно отражать ее суть!

--------------------

CRM с функцией АвтоОбзвона

Лотереи мира

Hades

Дата 26.9.2008, 16:37 (ссылка)

(голосов:2)

Загрузка ...

Бывалый

Профиль
Группа: Участник
Сообщений: 226
Регистрация: 8.8.2008

Репутация: 4
Всего: 37

можно обойтись и без циклов

Код


int main() {
    int a, b, m;

    cin >> a >> b;

    if (a >= b || b < 1605) {
        m = 0;
    }
    else {
        if (1605 > a)
            m = (b - 1605) / 10 + 1;
        else if (1605 == a)
            m = (b - 1605) / 10;
        else
            m = (b - 1605)/10  - (a - 1605)/10;
     }

    cout << m << endl;

    return 0;
}

Это сообщение отредактировал(а) Hades - 26.9.2008, 16:39

Oleg_Ci

Дата 28.9.2008, 08:27 (ссылка)

(нет голосов)

Загрузка ...

Friend

Профиль
Группа: Участник
Сообщений: 485
Регистрация: 28.5.2006
Где: Новосиб.обл.

Репутация: 25
Всего: 30

А если с циклами, то так smile

Код


int a, b, i;
cin >> a >> b;
if (a<1614) a=1614;
for( i=0; a<b; i++, a+=10 );
cout << i;

IKM2007

Дата 28.9.2008, 10:07 (ссылка)

(нет голосов)

Загрузка ...

Зима близко

Профиль
Группа: Участник
Сообщений: 702
Регистрация: 26.4.2008
Где: olmedreca

Репутация: 4
Всего: 40

Oleg_Ci, а если a=1600 и b=1615, то по твоему коду будет 1 раз, а на самом деле 2 раза.

--------------------

"К чёрту обстоятельства, я создаю возможности."

Брюс Ли

Oleg_Ci

Дата 28.9.2008, 11:09 (ссылка)

(нет голосов)

Загрузка ...

Friend

Профиль
Группа: Участник
Сообщений: 485
Регистрация: 28.5.2006
Где: Новосиб.обл.

Репутация: 25
Всего: 30

Исправил

Код

int a, b, i;
cin >> a >> b;
a = a*(a/1605)+10;  // <<<
if (a<1605) a=1605;
for( i=0; a<=b; i++, a+=10 );  //  (a<=b)
cout << i;

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Центр помощи \| Следующая тема »