как сделать проще

[геннадий522]

помогите как сделать проще?

double  ost(double x)
{
    while (x > 6.283)
    {
        x=x-6.283;
    }

    return x;
}

[8Observer8]

геннадий522, а куда же проще?
Можно запись упростить:

x -= 6.283;

А ещё константу, как-то осмысленно попробовать обозвать:

#define LIMIT 6.283

double  ost(double x)
{
    while (x > LIMIT)
    {
        x -= LIMIT;
    }

    return x;
}

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

[геннадий522]

да мне нужно поменять

#include "StdAfx.h"
#include <iostream>
#include <stdio.h>
#include <math.h>
using namespace std;
double  ost(double x)
{
    while (x > 6.283)
    {
        x=x-6.283;
    }

    return x;
}


int main()
{
 double x,a,b,dx,i,f=1,dx1;

cout<<"Vvedite а"<<endl;
 cin>>a;

 cout<<"Vvedite b"<<endl;
 cin>>b;

 dx=(b-a)/1000;
 cout<<"Vvedite shag dx1"<<endl;
 cin>>dx1;

 for(;dx1<=dx;)
 {
 cout<<"Vvedite drugoe znachenie dx1"<<endl;
 cin>>dx1;
 
 }
 for(i=a;i<=b;i=i+dx1)
 {
if (ost(exp(i)) <= 3.1415)
 f=f*((log(cos(pow(2.0,i)+3.14)))/log(5.0));
else (ost(exp(i)) >= 3.1415);
 }
 cout<<"Znachenie proizvedeniya:"<<f;
 system("pause");
	return 0;
}

программа работает но мне нужно узнать как по-другому еще можно ее сделать.

[8Observer8]

геннадий522, для начала код записать по-человечески

и заменить:

for(;dx1<=dx;)

на while:

while (dx1<=dx)

Вот так можно оформить:

//#include "StdAfx.h"
#include <iostream>
#include <stdio.h>
#include <math.h>
using namespace std;

double ost(double x) {
    while (x > 6.283) {
        x = x - 6.283;
    }
    return x;
}

int main() {
    double x, a, b, dx, i, f = 1, dx1;

    cout << "Vvedite а" << endl;
    cin >> a;

    cout << "Vvedite b" << endl;
    cin >> b;

    dx=(b-a)/1000;
    cout << "Vvedite shag dx1" << endl;
    cin >> dx1;

    while (dx1 <= dx) {
        cout << "Vvedite drugoe znachenie dx1"<<endl;
        cin >> dx1;
    }

    for(i = a; i <= b; i = i + dx1) {
        if (ost(exp(i)) <= 3.1415)
            f=f*((log(cos(pow(2.0,i)+3.14)))/log(5.0));
        else (ost(exp(i)) >= 3.1415);
    }

    cout<<"Znachenie proizvedeniya:"<<f;

    system("pause");
    return 0;
}

P.S. А задание какое?

[геннадий522]

Разработать алгоритм вычисления функции из задания в Приложении 1 в(т.е вот для этого f=f*((log(cos(pow(2.0,i)+3.14)))/log(5.0)) )
соответствии с номером бригады. Значения a и b, а так же шага вычисления
вводятся пользователем.
2. Проверить вводимые значения (a, b, шаг) с условием, что должно производиться не
более 1000 итераций вычисления функции. Если условие неверно, повторить
заново ввод значений и проверку.
3. Представить разработанный алгоритм в виде блок-схемы.
4. Произвести вычисление 2-х контрольных примеров для заданной функции.
5. Написать программу на языке С для вычисления функции по разработанному
алгоритму.
6. Произвести проверку работы программы и сравнить результаты с полученными в
п.4.

Содержание отчета:

[Blade]

Сообщение от 8Observer8

А ещё константу, как-то осмысленно попробовать обозвать:

Для объявления констант есть const, define - плохо

[_Bers]

Сообщение от Blade

Для объявления констант есть const, define - плохо

Аргументируйте.

[Пепел Феникса]

препроцессор это в принципе плохо.
для const идет проверка типов, а для препроцессора нет(ибо это тупо замена текста)

[_Bers]

Сообщение от Пепел Феникса

препроцессор это в принципе плохо.
для const идет проверка типов, а для препроцессора нет(ибо это тупо замена текста)

1. Какие проблемы могут быть с проверкой типа в ниже представленном коде?

#define KOL 10

void Foo(int val) { if( val==KOL) { std::cout<< "Урра!\n"; } }
int main()
{
Foo(10);
Foo(KOL);
}

2. Если имеет место быть множеству единиц трансляции, каждой из которых требуется работать с едиными глобальными константами,
что тогда? Предлагаете утонуть в extern`нах?

3. Почему препроцессор - это в принципе плохо?

[8Observer8]

Сообщение от Blade

Для объявления констант есть const, define - плохо

Согласен, спасибо!