не понимаю ошибку

@Troo · Регистрация: 09.10.2013

Author24 — интернет-сервис помощи студентам

Написать программу, решающую задачу линейного поиска элемента в заданном
вещественном массиве.
_____
при вводе единицы выводит 0,
а при других значениях ''no"

C++

#include <iostream>
#include <stdlib.h>
#include <time.h>
 
using namespace std;
 
void double1 (double *mas, int const n);
void double2(double *mas, int const n);
int linsearch(double *mas, const int n,double el);
 
int main()
{
    srand((unsigned)time(0));
    int const n=5;
    double masd[n];
    double1(masd, n);
    double2(masd, n);
    cout << '\n';
 
    double el;
    cin>>el;
    if (el==true)
        cout << linsearch (masd,n,el) << endl;
    else
        cout << "no" <<endl;
 
    return 0;
}
void double1 (double *mas, int const n)
{
    for(int i = 0; i < n; i++)
            mas[i] = 10*double(rand())/RAND_MAX;
}
void double2(double *mas, int const n)
{
    for(int i = 0; i < n; i++)
    cout << mas[i] << ' ';
    cout << endl;
 
}
int linsearch(double *mas, const int n,double el)
{
    int i;
    for (i=0; i<n; i++)
if (mas[i]==el)
{return el ; break; }
 
}

@Max Dark · 17.11.2013, 23:46

строки 22- 23 замените на

C++

int num;
num = linsearch(mas, n , el);
if(num<n)
    cout<<num<<endl;

и подправьте функцию поиска

C++

int linsearch(double *mas, const int n,double el)
{
    int i;
    for (i=0; i<n; i++)
if (mas[i]==el)
{ break; }
 return i;
}

alsav22 · 17.11.2013, 23:49

Вот это измените (не нужно смешивать dooble и bool ):

C++

1
2
3

double el;
cin>>el;
if (el==true)

Добавлено через 2 минуты

C++

int linsearch(double *mas, const int n,double el)
{
    int i;
    for (i=0; i<n; i++)
        if (mas[i]==el)
        {
            return el ; 
            break; 
        }
 
}

el - double, возвращаете int. Если if не выполнится, то что вернётся? И зачем break после return?

@Troo · 18.11.2013, 21:44 **[ТС]**

Cra3y, у меня в присваивании значения num выдает ошибку

C++

1	num = linsearch(mas, n , el);

alsav22, писала что вернется -1,
но тогда программа при любом значении выводила -1

Добавлено через 3 минуты
Cra3y, все поняла ошибку, но он все равно выводи что значения нет

C++

int main()
{
    srand((unsigned)time(0));
    int const n=5;
    double masd[n];
    double1(masd, n);
    double2(masd, n);
    cout << '\n';
 
    double el;
    cin>>el;
    int num;
    num = linsearch(masd, n , el);
    if(num<n)
    cout<<num<<endl;
    else
        cout << "no" <<endl;
 
    return 0;
}
void double1 (double *mas, int const n)
{
    for(int i = 0; i < n; i++)
            mas[i] = 10*double(rand())/RAND_MAX;
}
void double2(double *mas, int const n)
{
    for(int i = 0; i < n; i++)
    cout << mas[i] << ' ';
    cout << endl;
 
}
 
int linsearch(double *mas, const int n,double el)
{
    int i;
    for (i=0; i<n; i++)
if (mas[i]==el)
{ break; }
 return i;
}

@zss · 18.11.2013, 21:47

C++

int linsearch(double *mas, const int n,double el)
{
    int i;
    for (i=0; i<n; i++)
        if (mas[i]==el)
            return i; 
    return -1;
}

-1 будет означать, что такой элемент не найден.

@Troo · 18.11.2013, 22:04 **[ТС]**

zss, так у меня в любом случае почему-то выдает, что элемент не найден.

@Max Dark · 18.11.2013, 22:07

дело в том что вещественные числа выводятся на экран не полностью, а только до некоторой цифры после запятой
Вот немного переделанный генератор массива

C++

void double1 (double *mas, int const n) {
    for(int i = 0; i < n; i++)
        mas[i] = round(1000.0*double(rand())/RAND_MAX)/100.0;
}

он гарантирует, что после запятой будет не более 2 цифр

Bash

./a.out 
1.96 0.44 0.72 9.23 1.9 
 
0.44
1

еще допишите #include <cmath> в начало программы

@Troo · 18.11.2013, 22:14 **[ТС]**

Cra3y, спасибо большое

Добавлено через 2 минуты
Cra3y, помогите пожалуйста
почему вот тут ошибка:
не был объявлен "round"

C++

1	mas[i] = round(1000.0*double(rand())/RAND_MAX)/100.0;

@Max Dark · 18.11.2013, 22:16

C++

1	#include <cmath>

в начало программы

@Troo · 18.11.2013, 22:18 **[ТС]**

Cra3y,
спасибо большое помогли очень)

@Max Dark · 18.11.2013, 22:24

Troo, Видимо ваш компилятор не поддерживает эту функцию
попробуйте заменить round на floor или cell

@zss · 19.11.2013, 10:22

C++

1	#include <math.h>

@Troo 1 / 1 / 1 Регистрация: 09.10.2013 Сообщений: 73
	18.11.2013, 22:04 [ТС]	6
	zss, так у меня в любом случае почему-то выдает, что элемент не найден. 0

@Troo 1 / 1 / 1 Регистрация: 09.10.2013 Сообщений: 73
	18.11.2013, 22:18 [ТС]	10
	Cra3y, спасибо большое помогли очень) 0

@Max Dark шКодер самоучка 2281 / 1957 / 945 Регистрация: 09.10.2013 Сообщений: 4,357 Записей в блоге: 7
	18.11.2013, 22:24	11
	Troo, Видимо ваш компилятор не поддерживает эту функцию попробуйте заменить round на floor или cell 1