Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Для новичков > Команда ветвления if


Автор: Mello 10.2.2012, 19:15
Вот кусок программного кода, вводятся название продукции и ее товарооборот, при вводе отрицательного числа происходит закрытие программы
Код

double stat()

    {            

char name1[9], name2[9], name3[9];
float tovb1, tovb2, tovb3;
float tovt1, tovt2, tovt3;
float iq1, iq2, iq3;
float ip1, ip2, ip3;


system ("cls");
int i;

printf(" \n Vvedite nazvaniya produktov: \n"); 
scanf_s("%9s", name1,10);
scanf_s("%9s", name2, 10);
scanf_s("%9s", name3, 10);


    float* fArray[3]={&tovb1,&tovb2,&tovb3};
    fArray[0]=&tovb1;
    printf(" Vvedite bazisnyi tovarooborot \n");

for (i=0; i<3; i++)
{

    scanf_s("%f",fArray[i]);
    
    
    
    if (tovb1<0) 
    {
    
     std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);  if (tovb2<0)
 
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);if (tovb3<0)
    
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);
    
if (tovb3>0)

    goto x;
}

}


x:;
    float* tArr[3]={&tovt1,&tovt2,&tovt3};
    tArr[0]=&tovt1;
    printf(" Vvedite tekushyi tovarooborot \n");
for(i=0; i<3; i++)
{
    scanf_s("%f", tArr[i]);
    
    if (tovt1 <0)
    {
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);  
if (tovt2<0)
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);  
if (tovt3<0)
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);
if (tovt3>0)
goto x1;

}
Дело в том что, если введенное значение отрицаеально, то как и нужно происходит выход из программы, при вводе остальных отрицательных в остальные две переменые, программа не производит выход! Хелп!



Добавлено через 44 секунды
Дело в том что, если введенное значение отрицаеально, то как и нужно происходит выход из программы, при вводе остальных отрицательных в остальные две переменые, программа не производит выход! Хелп!

Автор: feodorv 10.2.2012, 19:38
Цитата(Mello @  10.2.2012,  19:15 Найти цитируемый пост)
    if (tovt1 <0)
    {
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);  
if (tovt2<0)
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);  
if (tovt3<0)
std::cout <<"Otrizatelnye chisla nedopuskayutsya";
exit(EXIT_SUCCESS);
if (tovt3>0)
goto x1;
}


Может быть так:
Код

    if (tovt1 <0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS); 
    }
    if (tovt2<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);  
    }
    if (tovt3<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);
    }
    if (tovt3>0) goto x1;

А то у Вас все exit'ы находятся внутри if (tovt1 <0) { ... }

Автор: vol4ek 10.2.2012, 20:23
Код


 if (tovt1 <0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      Sleep(5000);
      exit(EXIT_SUCCESS); 
    }



прочитать чтоб можно было.  smile 

Автор: Mello 10.2.2012, 22:44
Цитата

Может быть так:код C++
1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
      if (tovt1 <0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS); 
    }
    if (tovt2<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);  
    }
    if (tovt3<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);
    }
    if (tovt3>0) goto x1;




А то у Вас все exit'ы находятся внутри if (tovt1 <0) { ... }

Как раз так я изначально и делал,  но тогда уже и при вводе положительного числа программа почемуто закрывается smile 

Автор: feodorv 11.2.2012, 02:00
Цитата(Mello @  10.2.2012,  22:44 Найти цитируемый пост)
программа почемуто закрывается 

Что-нибудь пишет, или так закрывается? А если отладчиком пошагово посмотреть?

Автор: Mello 11.2.2012, 18:46
Да пишет что отрицательные числа недопускается,  хотя ввожу положительные, и закрывается, это меня больше всего удивило

Автор: feodorv 12.2.2012, 00:36
Цитата(Mello @  11.2.2012,  18:46 Найти цитируемый пост)
Да пишет что отрицательные числа недопускается,  хотя ввожу положительные, и закрывается, это меня больше всего удивило 

Судя по всему, у Вас нарушена логика происходящего в программе. Смотрите:
Код

float tovb1, tovb2, tovb3;
...
float* fArray[3]={&tovb1,&tovb2,&tovb3};
...
fArray[0]=&tovb1; /* это надо выбросить ;) т.к. лишнее */

Пока всё супер, но нужно учесть, что  переменные tovb1, tovb2, tovb3 не проинициализированы. В случае, когда Вы их будете считывать с консоли, это не имеет значения. Но!... что же дальше:
Код

for (i=0; i<3; i++)
{
    scanf_s("%f",fArray[i]);
    ....
}

И тут всё супер. Но!...
Код

for (i=0; i<3; i++)
{
    scanf_s("%f",fArray[i]);

    if (tovt1 <0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS); 
    }
    if (tovt2<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);  
    }
    if (tovt3<0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS);
    }

  ...
}

уже не супер. В теле цикла считывается переменная по адресу fArray[i], а проверка на отрицательность идёт сразу по всем 3-м переменным! На первом шаге цикла (i равное 0) tovb2 и tovb3 вообще ещё не проинициализированы (содержат мусор), а уже делается попытка проверить их значение на отрицательность)))

По-моему, логичнее было бы так:
Код

for (i=0; i<3; i++)
{
    scanf_s("%f",fArray[i]);

    if ( *fArray[i] <0)
    {
      std::cout <<"Otrizatelnye chisla nedopuskayutsya";
      exit(EXIT_SUCCESS); 
    }
}

И операторов меньше, и смысла больше)))

Автор: Mello 14.2.2012, 11:06
Спасибо большое, вы мне помогли, но вот есть подобная конструкция, только она занесена в бесконечный цикл и при вводе отрицательных значений  возвращает к началу цикла, разрушение цикла осуществляется меткой оставленой за пределами тела цикла
Код
int ix1, ix2, ix3, ix4;

unsigned short f1,f2,f3,f4;
unsigned short fi1,fi2,fi3,fi4;
unsigned short m1, m2, m3, m4;
while (1)
{
    

printf(" 1. Vvedite kolichestvo izgotovlenyh detaley\n");  
scanf_s("%i", &ix1);
if(ix1<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (ix1>0)
printf("1.kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 1 brigade\n");
scanf_s("%d", &f1);
if (f1<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (f1>0)
printf("1.kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 2 brigade\n");
scanf_s("%d", &fi1);
if (fi1<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (fi1>0)
m1=f1+fi1;
printf("1.Obshee kolichestvo rabochih imejushih dannuyu proizvoditelnost ravna\n %i" ,m1 );
if (m1<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (m1>0)
system("cls");

printf(" 2. Vvedite kolichestvo izgotovlenyh detaley \n");
scanf_s("%i", &ix2);
if (ix2>0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (ix2<0)
printf("2.kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 1 brigade\n");
scanf_s("%d", &f2);
if (f2>0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (f2<0)
printf("2.kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 2 brigade\n");
scanf_s("%d", &fi2);
if (fi2>0)

m2=f2+fi2;
printf("2.Obshee kolichestvo rabochih imejushih dannuyu prroizvoditelnost ravna\n %i" ,m2 );
if (m2>0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (m2<0)
system("cls");
printf(" 3. Vvedite kolichestvo izgotovlenyh detaley\n"); 
scanf_s("%i", &ix3);
if (ix3<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (ix3>0)
printf("3. Kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 1 brigade \n");
scanf_s("%d", &f3);
if (f3<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (f3>0)
printf("3. Kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 2 brigade \n");
scanf_s("%d", &fi3);
if (fi3<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (fi3>0)
m3=f3+fi3;
printf("3.Obshee kolichestvo rabochih imejushih dannuyu proizvoditelnost \n %i" ,m3);
if (m3<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (m3>0)

system("cls");
printf(" 4. Vvedite kolichestvo izgotovlenyh detaley\n"); 
scanf_s("%i", &ix4);
if (ix4<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (ix4>0)
printf("4. Kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 1 brigade \n");
scanf_s("%d", &f4);
if (f4<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (f4>0)
printf("4. Kol-vo rabochih imeyushih dannuyu proizvoditelnost' v 2 brigade \n");
scanf_s("%d", &fi4);
if (fi4<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim";
continue;
}
if (fi4>0)
goto t;
}


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

Автор: feodorv 14.2.2012, 11:52
Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
разрушение цикла осуществляется меткой оставленой за пределами тела цикла

Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
if (fi4>0)
goto t;

Давайте обходиться без дополнительных goto:
Код

if (fi4>0) break;



Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
Тут подобные симптомы

Так ведь всегда можно напечатать, что именно вызвало "подобные симптомы", есди уж отладчик никак:
Код

if (f1<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim (f1)";
continue;
}
...
if (f2<0)
{
std::cout <<" \n Vvod otricatelnyh chisel nedopustim (f2)";
continue;
}
...



Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
if (fi4>0)
goto t;

Мне не понятно, если я введу fi4 как 0, то цикл по-новой пойдёт? За что? А если m1 есть 0, то экран не будем стирать?


Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
Тут подобные симптомы, при вводе первого отр. значения преходит в начало цикла, потом только через два значения и т.д 

Не понял симптомов. Вы же сами пишете continue, что и означает переход на начало цикла. 


Цитата(Mello @  14.2.2012,  11:06 Найти цитируемый пост)
unsigned short f1,f2,f3,f4;
unsigned short fi1,fi2,fi3,fi4;
unsigned short m1, m2, m3, m4;

Проверять изначально неотрицательные величины (unsigned) на отрицательность???

Ну если у Вас unsigned, то и ввод нужно делать unsigned:
Код
scanf_s("%u", &ix1);


Ну и если у Вас C++, то уж до конца:
Код

cin >> ix1;

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)