VPF::[C++] Неправильно работает программа

lrad

Дата 20.1.2007, 23:59 (ссылка)

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

Загрузка ...

Новичок

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

Репутация: нет
Всего: 2

Помогите найти ошибку

Код


#include<iostream.h>
#include<iomanip.h>
#include<conio.h>
//Прототипы функций
void inputmas(float[15],char);
float raschet(float [15]);
void printrez(float [15],char);

   void main()
   {
    cout <<"Задача № 1  \n";
    cout <<"Даны вещественные массивы А[15]F[15] \n";
    cout <<"Для каждого массива определить наибольшее \n";
    cout <<"из отрицательных значений" << endl;
    cout<<"\nДля вывода результатов вам нужно самостоятельно заполнить массив!\n";

   // Объявление массива и переменных
   float a[15],f[15];
   // Обращение
   inputmas(a,'a'); // в функцию в качестве аргумента
   inputmas(f,'f'); // передается массив и его имя
                       // вызов функции, возращающей 1 значение

   printrez(a,'a');
   printrez(f,'f');
   cout<<"Нажмите любую клавишу...";
   getch();

   return;
   }

// Определение функции ввода массива inputmas()

   void inputmas(float x[15],char name)
   {

    cout <<"Вводим массив  " << name << endl;
  int i;
  for ( i=0;i<15;i++)
  {
  cout <<"Введите элемент массива  " << name <<"["<<i<<"]=";
  cin >>x[i];
  }
  }
// Определение расчетной функции raschet()
float raschet(float m[15])
{
float max=0;

for (int i=0;i<15;i++)
if (m[i]>max && m[i] <0)
max=m[i];

return max;
}

 //     //Вывод массива и результата

void printrez(float x[15],char name)
{
cout<<"Исходный массив "<<name<<":"<<endl;
for (int i=0;i<15;i++)
cout<<setw(5)<<x[i];

float res=raschet(x);
if(res!=0)
cout<<endl<<"Наибольшее отрицательное значение в массиве "<<name<<" равно " <<raschet(x)<< endl;
else
cout<<endl<<"Отрицательных значений нет";
}

nworm

Дата 21.1.2007, 00:03 (ссылка)

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

Загрузка ...

Опытный

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

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

Пробуй пошаговую отладку F7, F8, F4.

Хорошо ещё написать, что выдаёт компилятор. А то не очень хочетсся 70 строк смотреть из-за какойни-будь запятой.

zkv

Дата 21.1.2007, 00:06 (ссылка)

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

Загрузка ...

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

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

Код


float raschet(float m[15])
{
float max=0;
for (int i=0;i<15;i++)
if (m[i]>max && m[i] <0)
max=m[i];
return max;
}

Цитата(lrad @ 20.1.2007, 23:59

)

cout <<"Для каждого массива определить наибольшее \n";
cout <<"из отрицательных значений" << endl;

float max=0; - max теперь больше любого отрицательного значения, естественно в массиве ничего не находится так как условие (m[i]>max && m[i] <0) никогда не выполнится, в следующий раз пиши, что именно не получается

Это сообщение отредактировал(а) zkv - 21.1.2007, 00:07

PoloS

Дата 21.1.2007, 00:06 (ссылка)

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

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 89
Регистрация: 29.12.2006
Где: МО, г. Одинцово

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

Код


max = 0;

ты ищешь нибольшее из отрицательных, и как 1 значение берешь заведомо большее число.

надо так

Код


max = m[0];
for (inti i = 1; i < 15; i++)...
...
...

lrad

Дата 21.1.2007, 00:07 (ссылка)

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

Загрузка ...

Программа эта под dos
Она компилируется и компонуется нормально!!!
Результат работы самой программы неправильный
Тоесть ответ выдает неправильный

PoloS

Дата 21.1.2007, 00:07 (ссылка)

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

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 89
Регистрация: 29.12.2006
Где: МО, г. Одинцово

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

Цитата(zkv @ 21.1.2007, 00:06)

в одну секунду с тобой ответили smile

lrad

Дата 21.1.2007, 00:09 (ссылка)

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

Загрузка ...

Пробовал и так
а где гарантия что 1 элемент будет отрицательным?

Код



#include<iostream.h>
#include<iomanip.h>
#include<conio.h>
//Прототипы функций
void inputmas(float[15],char);
int raschet(float [15]);
void printrez(float [15],char);

   void main()
   {
    cout <<"Задача № 1  \n";
    cout <<"Даны вещественные массивы А[15]F[15] \n";
    cout <<"Для каждого массива определить наибольшее \n";
    cout <<"из отрицательных значений" << endl;
    cout<<"\nДля вывода результатов вам нужно самостоятельно заполнить массив!\n";

   // Объявление массива и переменных
   float a[15],f[15];
   // Обращение
   inputmas(a,'a'); // в функцию в качестве аргумента
   inputmas(f,'f'); // передается массив и его имя
   // вызов функции, возращающей 1 значение

   printrez(a,'a');
   printrez(f,'f');
   cout<<"Нажмите любую клавишу...";
   getch();

   return;
   }

// Определение функции ввода массива inputmas()

   void inputmas(float x[15],char name)
   {

    cout <<"Вводим массив  " << name << endl;
  int i;
  for ( i=0;i<15;i++)
  {
  cout <<"Введите элемент массива  " << name <<"["<<i<<"]=";
  cin >>x[i];
  }
  }
// Определение расчетной функции raschet()
 int raschet(float m[15])
  {
      int i,max;
      max=m[0];

    for (i=0;i<15;i++)
    if (m[i] >max&& m[i] <0)
    max=m[i];

 return max;

  }

 //     //Вывод массива и результата

 void printrez(float x[15],char name)
 {

 cout<<"Исходный массив  "<<name<<":"<<endl;
 for (int i=0;i<15;i++)
 cout<<setw(5)<<x[i];
 cout<<endl<<"Наибольшее отрицательное значение в массиве  "<<name<<" равно " <<raschet(x)<< endl;


 }

PoloS

Дата 21.1.2007, 00:11 (ссылка)

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

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 89
Регистрация: 29.12.2006
Где: МО, г. Одинцово

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

точно! тогда те нужно завести флаг
1)first = false;
2)нашел отрицательный
3) смотришь first == false, если да то max = этому числу и first = true;
иначе сравниваешь с max и максимальное заносишь в max

Добавлено @ 00:13

Код


// Определение расчетной функции raschet()
 int raschet(float m[15])
  {
      int i,max;
bool f = false;
    for (i=0;i<15;i++)
    if (m[i] < 0)
   {
    if (f == false) max = m[i], f = true;
    else 
         {if (max < m[i]) max = m[i];}
   }
 return max;
  }

Это сообщение отредактировал(а) PoloS - 21.1.2007, 00:14

zkv

Дата 21.1.2007, 00:13 (ссылка)

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

Загрузка ...

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

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

Цитата(lrad @ 21.1.2007, 00:09

)

а где гарантия что 1 элемент будет отрицательным?

конечно нет гарантий, так что теперь писать заведомо неработающие варианты, можно например в каждой итерации проверять max == 0 и если равно и текущий элемент отрицательный, то писать его значение в max

lrad

Дата 21.1.2007, 00:23 (ссылка)

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

Загрузка ...

Цитата

Код


// Определение расчетной функции raschet()
 int raschet(float m[15])
  {
      int i,max;
bool f = false;
    for (i=0;i<15;i++)
    if (m[i] < 0)
   {
    if (f == false) max = m[i], f = true;
    else 
         {if (max < m[i]) max = m[i];}
   }
 return max;
  }

Это не то
надо более простое решение
P.S. слишком заумно для студента 1 курса smile

PoloS

Дата 21.1.2007, 00:26 (ссылка)

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

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 89
Регистрация: 29.12.2006
Где: МО, г. Одинцово

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

Цитата(lrad @ 21.1.2007, 00:23)

Это не то
надо более простое решение
P.S. слишком заумно для студента 1 курса smile

я сам первокурсник smile

и ничего замутного тут не вижу... можешь сделать как zkv предложил, смысл такой же и доп. переменную не надо вводить:

Код


// Определение расчетной функции raschet()
 int raschet(float m[15])
  {
      int i,max = 0;
    for (i=0;i<15;i++)
    if (m[i] < 0)
   {
    if (max) 
         if (max < m[i]) max = m[i];
    else 
         {max = m[i];}
   }
 return max;
  }

тока при воводе рез-та, рассматривай случай, когда функция возвращает 0. Это значит отриц. чисел нет

Это сообщение отредактировал(а) PoloS - 21.1.2007, 00:29

zkv

Дата 21.1.2007, 00:34 (ссылка)

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

Загрузка ...

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

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

PoloS,

Цитата(PoloS @ 21.1.2007, 00:26

)

тока при воводе рез-та, рассматривай случай, когда функция возвращает 0

а у вас кстати функция в таком случае работать будет некорректно:

Цитата(PoloS @ 21.1.2007, 00:11

)

int raschet(float m[15])
  {
      int i,max;
bool f = false;
    for (i=0;i<15;i++)
    if (m[i] < 0)
   {
    if (f == false) max = m[i], f = true;
    else
         {if (max < m[i]) max = m[i];}
   }
return max; //если не встретится отрицательных чисел, то в max - мусор
  }

PoloS

Дата 21.1.2007, 00:37 (ссылка)

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

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 89
Регистрация: 29.12.2006
Где: МО, г. Одинцово

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

Цитата(zkv @ 21.1.2007, 00:34)

PoloS,

Цитата(PoloS @ 21.1.2007, 00:26

)

тока при воводе рез-та, рассматривай случай, когда функция возвращает 0

а у вас кстати функция в таком случае работать будет некорректно:

Цитата(PoloS @ 21.1.2007, 00:11

)

согласен, накатал на скорую руку, и тока потом понял что мусор в max и в результате невозможно определить нашелся рез-т или нет...

Rockie

Дата 21.1.2007, 00:40 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1143
Регистрация: 23.4.2006

Репутация: 8
Всего: 31

lrad, здесь

Код


float max=0;

тебе нужно указывать не ноль и не первый эл-т, а первый найденный отрицательный элемент. Вот тебе ссылка в тему:

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

p.s.: первая страница была куда забавнее smile

Добавлено @ 00:42
все-таки первая. решения аналогичной задачи на второй =)

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

Чтобы иметь большой гардероб - надо иметь большой гардероб.

lrad

Дата 21.1.2007, 00:59 (ссылка)

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

Загрузка ...

Rockie, Спасибо!
теперь то я уже разберусь smile

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