Почему прога зацикливаеться?

[BrutalSnowflake]

Юппи! решил проблему. Кароче, заметил, что зацикливание начинаеться если вводишь букву или слово, а не цифру. Т.е. если я введу 5, то оно будет всё делать правильно - выбивать ошибку и просить ввести заново. Взял изменил тип вводимого oper с int на char, и в свиткейсе в одинарные кавычки всё взял естественно. И теперь всё норм работает))) ааааа, фак еааах! ]=>

[Haarhus]

Сейчас я покажу Вам переделанную версию Вашего программного кода под Visual Studio C++ (MSDEV)


/././././././././././././././././././././././././././././././././././././././././././././././././
Вы вечно молитесь своим богам,
и компилятор всё прощает вам.

#include "stdafx.h"
#include <iostream>
#include <conio.h>
using namespace std;
void main() {
        int x=0,operation_nomber;
        cout << "Viberite operciu: \n 1. Slogenie \n 2. Vichitanie \n 3. Umnogenie \n 4. Delenie \n\n";
        cin >> operation_nomber;
        while(!x) {
                if (operation_nomber>= 1 && operation_nomber<=4) break;
                else {
                       cin >> operation_nomber;
                       cout << "Oshibka vvoda! Vvedite chislo ot 1 do 4";
                }
        }
        system("pause");
}

[BrutalSnowflake]

cin >> operation_nomber;
cout << "Oshibka vvoda! Vvedite chislo ot 1 do 4";

Эти строки надо местами поменять

[Obey-Kun]

void main запрещено стандартом, сколько ж повторять)
Я правильный код ещё на первой странице дал:

#include <iostream>

int main()
{
    int operation_type;
    std::cout << "Viberite operciu: \n 1. Slogenie \n 2. Vichitanie \n 3. Umnogenie \n 4. Delenie \n\n";
    std::cin >> operation_type;
    while ( operation_type >= 1 && operation_type <= 4 ) {
        std::cout << "Oshibka vvoda! Vvedite chislo ot 1 do 4" << std::endl;
        std::cin >> operation_type;
    }
    std::cin.get();
}

[BrutalSnowflake]

Haarhus, не катит твой вариант. При введении допустим вместо числа допустим буквы, прога опять зациклиться, чтоб собственно и было раньше.

на int main() уже заменил.

Obey-Kun, с этим кодом происходит то что я написал Haarhus'у в предыдущем сообщении

Точнее даже тут код вообще делает что-то обратное от нужного. Оно выбивает ошибку при введении правильных чисел. А при введении не правильны просто прерывает прогу О_о

Кстати, дополниетельный вопрос. Для чего в этом куске кода использовать *(звёздочку). Это указатель, но зачем он тут? (спрашиваю так как писал этот кусок не я, а на форуме подсказали):

int Error(char *a)

   {  bool error=false;

   	int strl=strlen(a), i, j;

      for (i=0;i<strl;i++)



      {	switch (a[i]){

      		case 'M':case 'D': case 'C': case 'L': case 'X':case 'V': case 'I': case ' ': break;

            default: error=true;

            break;}

[Obey-Kun]

Да, обратное. А ты читать не умеешь? Ладно я не заметил ошибку, но ты же заметил... Разве не очевидно, где именно надо сделать исправление?

Было:

#include <iostream>

int main()
{
    int operation_type;
    std::cout << "Viberite operciu: \n 1. Slogenie \n 2. Vichitanie \n 3. Umnogenie \n 4. Delenie \n\n";
    std::cin >> operation_type;
    while ( operation_type >= 1 && operation_type <= 4 ) {
        std::cout << "Oshibka vvoda! Vvedite chislo ot 1 do 4" << std::endl;
        std::cin >> operation_type;
    }
    std::cin.get();
}

Стало

#include <iostream>

int main()
{
    int operation_type;
    std::cout << "Viberite operciu: \n 1. Slogenie \n 2. Vichitanie \n 3. Umnogenie \n 4. Delenie \n\n";
    std::cin >> operation_type;
    while ( operation_type < 1 || operation_type > 4 ) {
        std::cout << "Oshibka vvoda! Vvedite chislo ot 1 do 4" << std::endl;
        std::cin >> operation_type;
    }
    std::cin.get();
}

Кстати, дополниетельный вопрос. Для чего в этом куске кода использовать *(звёздочку). Это указатель, но зачем он тут? (спрашиваю так как писал этот кусок не я, а на форуме подсказали):

Передача массива по указателю на первый его элемент.

[BrutalSnowflake]

Не, ну эт я понял. Но проблема зацикливание в этом коде будет.

Передача массива по указателю на первый его элемент.

Т.е. если перед именем массива будет стоять звёздочка, и данному указателю другого адреса не присвоено, то он указывает на первый элемент? т.е. равносильно тому что записать char a[1]?

[kaljan775]

имя массива - указатель на его первый элемент
a[1]- разименованный указатель
проще говоря
если вы сделаете с int-овым массивом

cout<<a<<endl<<a[1]

;
выведется
<адрес в памяти>
0//ну или не ноль, как повезет :D

[BrutalSnowflake]

Хм...так вроде ж при вводе cout<<a[1] должна вывестись инфа хранящаяся в файле данной ячейке а не её адрес О_о

[Obey-Kun]

Не, ну эт я понял. Но проблема зацикливание в этом коде будет.

В каком? где?