Форум программистов [Powered by Invision Power Board]

Код


#include <iostream>

const unsigned int CLASSROOMS = 20;

int main( int argc, char* argv[] )
{

    unsigned int currentClassRoomNum;

    std::cout << "Number of students fo class 1?" << std::endl;
    std::cin >> currentClassRoomNum;

    unsigned int maxClassRoomNum = currentClassRoomNum;
    unsigned int minClassRoomNum = currentClassRoomNum;

    int currentClassRoom = 1;

    do
    {
        std::cout << "Number of students fo class " << currentClassRoom + 1 << "?" << std::endl;
        std::cin >> currentClassRoomNum;

        if( currentClassRoomNum > maxClassRoomNum )
        {
            maxClassRoomNum = currentClassRoomNum;
        }
        else if( currentClassRoomNum < minClassRoomNum )
        {
            minClassRoomNum = currentClassRoomNum;
        }
        ++currentClassRoom;
    }
    while( currentClassRoom < CLASSROOMS );

    std::cout << "Result = ( " << maxClassRoomNum << " - " << minClassRoomNum << " ) = " << maxClassRoomNum - minClassRoomNum << std::endl;

    ::system( "pause" );

    return 0;
}

Цитата(darkart @ 29.1.2012, 21:30

)

Код


++currentClassRoom;    }    while( currentClassRoom < CLASSROOMS )

;

Код


}
while( currentClassRoom++ < CLASSROOMS )

Цитата(Mirkes @ 30.1.2012, 09:49)

Я в С не силен, но по моему в цикле ошибка, обработано будет на 1 класс меньше, чем требовалось. Скорее всего нужно заменить фрагмент

Цитата(darkart @ 29.1.2012, 21:30

)

Код


++currentClassRoom;    }    while( currentClassRoom < CLASSROOMS )

;

на

Код


}
while( currentClassRoom++ < CLASSROOMS )

Цитата(darkart @ 29.1.2012, 21:30)

Код


const unsigned int CLASSROOMS = 20;

int main( int argc, char* argv[] )
{
    int currentClassRoom = 1;

    do
    {
        ++currentClassRoom;
    }
    while( currentClassRoom < CLASSROOMS );
}

Цитата

А помещать операцию изменения счетчика в проверку условия цикла вообще плохая практика.

Цитата(Mirkes @ 31.1.2012, 05:44)

Цитата(darkart @ 29.1.2012, 21:30)

Код


const unsigned int CLASSROOMS = 20;

int main( int argc, char* argv[] )
{
    int currentClassRoom = 1;

    do
    {
        ++currentClassRoom;
    }
    while( currentClassRoom < CLASSROOMS );
}

Я оставил в цикле только необходимое для разбора.
Итак до начала цикла currentClassRoom = 1
после первого прохода 2 и т.д.

То есть после 19-го прохода currentClassRoom =20
условие While не выполнилось и обработка 20-го класса не будет произведена.
В чем я ошибся?

Код


    std::cout << "Number of students fo class 1?" << std::endl; // 0
    std::cin >> currentClassRoomNum;
    unsigned int maxClassRoomNum = currentClassRoomNum;
    unsigned int minClassRoomNum = currentClassRoomNum;

    int currentClassRoom = 1; // соответственно теперь нам осталось проверить с 1 по 19
    do { ...

Цитата(Mirkes @ 31.1.2012, 10:13)

Верно, недосмотрел. Хотя ввод данных в двух местах тоже странная практика.
Но все верно.

Цитата(k0rvin @ 31.1.2012, 11:26

)

Не всегда, в данном случае вполне уместна.

Цитата(Zotec @ 29.1.2012, 20:10

)

20 классов школы

Цитата(darkart @ 1.2.2012, 01:42

)

Подсказать как исправить?

Цитата(darkart @ 1.2.2012, 01:42

)

volatile, Подсказать как исправить?

Цитата(Mirkes @ 31.1.2012, 10:13

)

Хотя ввод данных в двух местах тоже странная практика.

Цитата(volatile @ 1.2.2012, 01:03)

Уместно то оно уместно, но только в данном учебном случае, с жестко заданным количеством

Код


#include <iostream>

const unsigned char CHOICE_YES = 'y';

int main( int argc, char* argv[] )
{

    int currentClassRoomNum;
    int classRooms;

    setlocale( LC_ALL, "Russian" );

    do
    {
        std::cout << "Количество классов?" << std::endl;
        std::cin >> classRooms;
        if( classRooms < 0 )
        {
            unsigned char choice;
            std::cout << "Количество классов не может быть отрицательным." << std::endl << "Прервать выполнение( y ), переввести данные( иначе ) )?" << std::endl;
            std::cin >> choice;
            choice = static_cast< unsigned char >( tolower( choice ) );
            if( choice == CHOICE_YES )
            {
                std::cout << "Выход..." << std::endl;
                system( "pause" );
                return 0;
            }
        }
    }
    while( classRooms < 0 );

    

    if( classRooms > 0 )
    {
        do
        {
            std::cout << "Количество учеников в классе №1?" << std::endl;
            std::cin >> currentClassRoomNum;
            if( currentClassRoomNum < 0 )
            {
                unsigned char choice;
                std::cout << "Количество учеников не может быть отрицательным." << std::endl << "Прервать выполнение( y ), переввести данные( иначе ) )?" << std::endl;
                std::cin >> choice;
                choice = static_cast< unsigned char >( tolower( choice ) );
                if( choice == CHOICE_YES )
                {
                    std::cout << "Выход..." << std::endl;
                    system( "pause" );
                    return 0;
                }
            }
        }
        while( currentClassRoomNum < 0 );

        int maxClassRoomNum = currentClassRoomNum;
        int minClassRoomNum = currentClassRoomNum;

        if( classRooms > 1 )
        {
            int currentClassRoom = 1;

            do
            {    
                do
                {
                    std::cout << "Количество учеников в классе №" << currentClassRoom + 1 << "?" << std::endl;
                    std::cin >> currentClassRoomNum;
                    if( currentClassRoomNum < 0 )
                    {
                        unsigned char choice;
                        std::cout << "Количество учеников не может быть отрицательным." << std::endl << "Прервать выполнение( y ), переввести данные( иначе ) )?" << std::endl;
                        std::cin >> choice;
                        choice = static_cast< unsigned char >( tolower( choice ) );
                        if( choice == CHOICE_YES )
                        {
                            std::cout << "Выход..." << std::endl;
                            system( "pause" );
                            return 0;
                        }
                    }
                }
                while( currentClassRoomNum < 0 );


                if( currentClassRoomNum > maxClassRoomNum )
                {
                    maxClassRoomNum = currentClassRoomNum;
                }
                else if( currentClassRoomNum < minClassRoomNum )
                {
                    minClassRoomNum = currentClassRoomNum;
                }
                ++currentClassRoom;
            }
            while( currentClassRoom < classRooms );
        }    

        std::cout << "Результат = ( " << maxClassRoomNum << " - " << minClassRoomNum << " ) = " << maxClassRoomNum - minClassRoomNum << std::endl;
    }
    else
        std::cout << "В школе нет классов..." << std::endl << "Печаль..." << std::endl << "Нет пути..." << std::endl << "Не результат." << std::endl;
    

    ::system( "pause" );

    return 0;
}