Error invalid operands of types float and int to binary operator

#include <iostream>
#include <cstdio>
#include <cstdlib>
#include <cmath>


using namespace std;
float f(float x);

int main()
{

    float a;
    float b;
    int n;
    float h;
    float x;
    float area;

    cout << "Please input the first limit: ";
    cin >> a;
    cout << "Please input the second limit: ";
    cin >> b;
    cout << "How many rectangles do you want to use? ";
    cin >> n;

    h = (b-a)/n;

    area = (f(a)+f(b))/2;

    for (int i=1;i<n;i++) {
        area += f(a+i*h);
    }

    area = area*h;
    cout << "The area under the curve of f(x) = (2/sqrt(3.14))(exp(-x^2)) is ";
    cout << area;
}

float f(float x){
     return (exp(-x^2))(2/sqrt(3.14));
}

1

#include <iostream>
#include <cmath>

using namespace std;

int main()

{
float pprice;
cout << "Enter the purchase price =>" << endl;
cin >> pprice;
float cash = 10000%pprice;
return 0;
}

1

Error: invalid operands of types 'int' and 'float*' to binary'operator/'
int *average = new int((num) / data);

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

float *data;
int num;
int mode;
double average, median;

cout << "How many students were surveyed?  ";
cin >> num;
data = makeArray(num);

float getAverage(float *data, int num)
{
int *average = new int((data) / num);
return *average;
}

1

Это означает, что вы сравниваете два несовместимых типа вместе. Один из num а также data является intа другой float*, В зависимости от поведения, которое вы хотите, вы захотите

1. Разыменуйте указатель, как в *x для того, что x это указатель
   2а. Вы хотите разыграть int к float за floating point divisionгде результат преобразуется обратно в int
   2b. Вы хотите разыграть float для int, за integer division, который затем будет преобразован обратно в int,

Обновить
Поскольку вы обновили свой код, я укажу на большую проблему; у тебя сейчас утечка памяти.

Я бы посоветовал вам вместо этого возвратить ваше целое число по значению и, возможно, передать по ссылке или константе ссылки и избегать указателей целиком, но более того, я бы предложил некоторую симметрию в ваших входных параметрах, а также правильность констант:

//your code:
float getAverage( float *data, int sum )
{
//data is a float* and needs to be de-ref'd and casted to int for float but isnt
int *average = new int( (data) / num );
//note that now average is a pointer to a newly constructed int that will never be free'd
return *average;  //because you return it here, where the value will then be implicily converted to a float and be mostly worthless.
}

// In both suggestions I will use symmetric types, I will not introduce dynamic memory and I will use floating point division for maximal accuracy.

// Suggestion one use Const References
float getAverage( const float &data, const int &num)
{
float result = data / (float) num;
return result;
}

// Suggestion two use pointers to constants
float getAverage( const float *data, const int *num )
{
float result = (*data) / float(*num);
return result;
}

// Suggestion three pass by value since primitives are cheap
float getAverage( float data, int num)
{
float result = data / (float) num;
return result;
}

1

• Forum
• General C++ Programming
• What does» invalid operands of types `in

What does» invalid operands of types `int’ and `double’ to binary `operator%’ «

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46

	//Header file
#include <iostream>

using namespace std;

	//Named constants
const int HALFDOLLAR = 50;
const int QUARTER = 25;
const int DIME = 10;
const int NICKEL = 5;

int main()
{
		//Declare variable
	int change;

	cout << "Enter change in cents: ";		
	cin >> change;							
	cout << endl;

	cout << "The change you entered is " << change << endl;		

	cout << "The number of half-dollars to be returned is " << change / HALFDOLLAR << endl;		

	change = change % HALFDOLLAR;		

	cout << "The number of quarters to be returned is " << change / QUARTER << endl;		

	change = change % QUARTER;		

	cout << "The number of dimes to be returned is " << change / DIME << endl;		

	change = change % DIME;		

	cout << "The number of nickels to be returned is " << change / NICKEL << endl;		

	change = change % NICKEL;		

	cout << "The number of pennies to be returned is " << change << endl;		

	cout << endl << "Please press enter to exit the program...";

	cin.get();

	return 0;
}

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50

//this program calculates the number of quarters,
// dimes and nickles and pennies are nessasary to 
//generate the number of cents imputed
#include <iostream.h>
#include<string.h>

main() 
{
     double toBeChanged; 
     int numberOfDimes, numberOfQuarters, numberOfNickels, numberOfPennies;
     cout << " I will show you how many quarters, dimes, nickels, and npennies needed to make the amount. nHow much money do you have? (i.e. 1.25)" << 'n';
     cin >> toBeChanged;
     
cout << "For "<< toBeChanged << " you will need:"<<'n';  
   
     toBeChanged = toBeChanged * 100;
          
      if(toBeChanged >= 25)
{
      numberOfQuarters =  (toBeChanged / 25);
      toBeChanged = (int)toBeChanged % 25;
}

      if(toBeChanged >= 10)
{
               numberOfDimes = (toBeChanged / 10);
               toBeChanged = (int)toBeChanged % 10;
}

               if(toBeChanged >= 5)
{
               numberOfNickels =  (toBeChanged / 5);
               toBeChanged = (int)toBeChanged % 5;
}

               if(toBeChanged >= 1)
{
               numberOfPennies = (toBeChanged  / 1);
               toBeChanged = (int)toBeChanged % 1;
}


cout << numberOfQuarters<< " quarters,"<<'n';
cout << numberOfDimes << " dimes, "<< 'n';
cout << numberOfNickels << " nickels,"<< 'n';
cout << numberOfPennies << " pennies."<< 'n';

system("pause"); 
return 0;
}

Topic archived. No new replies allowed.

Не знаю, почему я получаю эту ошибку. Это похоже на то, как a*a обрабатывает a* как указатель (float* на float* — это то же самое, что float на float). По-видимому, проблема решается, когда я объявляю a в подклассах, но дело в том, что я хочу, чтобы мои подклассы автоматически получали это от моего родительского класса.

Еще одна вещь, когда я добавляю «float a;» для каждого подкласса, который может запускать программа. Я сделал это, чтобы проверить, будет ли это работать, но нет. Поплавок a не получает значения в setA. Я только что добавил cout << a; после a = b ;.

#include"Oblik.h"
#include<cmath>

using namespace std;

class Jednakostranicni : public GeometrijskaFigura{

    float P(){
        return (a*a*sqrt(3))/4; //12    13  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'float*' and 'float*' to binary 'operator*'
    }
    float O(){
        return a+a+a; //15  12  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'float*' and 'float*' to binary 'operator+'
    }
    
};

class Kvadrat : public GeometrijskaFigura{
    
    float P(){
        return a*a;//23 12  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'float*' and 'float*' to binary 'operator*'
    }
    float O(){
        return a+a+a+a;//26 12  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'float*' and 'float*' to binary 'operator+'
    }
    
};

class Krug : public GeometrijskaFigura{
    
    float P(){
        return a*a*3.14;//34    12  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'float*' and 'float*' to binary 'operator*'
    }
    float O(){
        return 2*a*3.14;//37    12  C:UsersNameDesktopprvi.cpp  [Error] invalid operands of types 'int' and 'float*' to binary 'operator*'
    }
    
};

int main(){
    
    GeometrijskaFigura *f;
    int x;
    
    cin>>x;
    
    f->setA(x);
    
    f=new Jednakostranicni;
    cout<<"Jednakostranicni-P: "<<f->P()<<" O: "<<f->O()<<endl;
    f=new Kvadrat;
    cout<<"Kvadrat-P: "<<f->P()<<" O: "<<f->O()<<endl;
    f=new Krug;
    cout<<"Krug-P: "<<f->P()<<" O: "<<f->O()<<endl;
    
    
    
    return 1;
    
    
}

// this is Oblik.h file code
#include<iostream>

using namespace std;

class GeometrijskaFigura{
    protected: float a;
    public:
        
        void setA(float b){
         a=b;
        }
        
        virtual float P()=0;
        virtual float O()=0;
};

У вас неопределенное поведение, поскольку f не инициализирован:

GeometrijskaFigura *f;
int x;
    
std::cin >> x;
    
f->setA(x); // UB: use uninitialized f.

Должно быть

int a;

std::cin >> a;

Jednakostranicni j;
j.setA(a);
std::cout << "Jednakostranicni-P: " << a.P() << " O: " << a.O() << std::endl;

Kvadrat k;
k.setA(a);
std::cout << "Kvadrat-P: " << k.P() << " O: " << k.O() << std::endl;

Krug krug;
krug.setA(a);
std::cout << "Krug-P: " << krug.P() << " O: " << krug.O() << std::endl;

1. Уважаемые гуру пожалуйста подскажите.
   Я с ардуино знаком на весьма любительском уровне.
   Я взял скетч из примеров из библиотеки Modbus-Master-Slave-for-Arduino-master.
   В этом примере данные из регистров мдбас слейв устройств считываются в массив
   uint16_t au16data[];
   В Модбас слейв в двух регистрах по 16 бит лежит одно значение float размером 32 бит
   то есть в двух ячейках uint16_t массива у меня сохраняться одно значение float размером 32 бит.
   Я планировал в скетче из двух uint16_t побитово перенести в переменную float 32 бита.
   Примерно так:

    uint16_t au16data[16] ;
     float volts  ;

    
     bitWrite(volts, 0, bitRead(au16data [1], 0));
     bitWrite(volts, 1, bitRead(au16data [1], 1));
     bitWrite(volts, 2, bitRead(au16data [1], 2));
     bitWrite(volts, 3, bitRead(au16data [1], 3));

    
     bitWrite(volts, 4, bitRead(au16data [2], 0));
     bitWrite(volts, 5, bitRead(au16data [2], 1));
     bitWrite(volts, 6, bitRead(au16data [2], 2));
     bitWrite(volts, 7, bitRead(au16data [2], 3));

   Но компилятор ругается.

   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:112:37: error:   in evaluation of ‘operator|=(float, long unsigned int)’
   #define bitSet(value, bit) ((value) |= (1UL << (bit)))
   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:114:52: note: in expansion of macro ‘bitSet’
   #define bitWrite(value, bit, bitvalue) (bitvalue ? bitSet(value, bit) : bitClear(value, bit))
   C:UsersVitalyAppDataLocalTemparduino_modified_sketch_712994sketch_jun25c.ino:121:5: note: in expansion of macro ‘bitWrite’
        bitWrite(volts, 0, bitRead(au16data [1], 0));
   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:113:39: error: invalid operands of types ‘float’ and ‘long unsigned int’ to binary ‘operator&’
   #define bitClear(value, bit) ((value) &= ~(1UL << (bit)))
   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:114:73: note: in expansion of macro ‘bitClear’
   #define bitWrite(value, bit, bitvalue) (bitvalue ? bitSet(value, bit) : bitClear(value, bit))
   C:UsersVitalyAppDataLocalTemparduino_modified_sketch_712994sketch_jun25c.ino:121:5: note: in expansion of macro ‘bitWrite’
        bitWrite(volts, 0, bitRead(au16data [1], 0));
   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:113:39: error:   in evaluation of ‘operator&=(float, long unsigned int)’
   #define bitClear(value, bit) ((value) &= ~(1UL << (bit)))
   C:UsersVitalyAppDataLocalArduino15packagesarduinohardwareavr1.6.21coresarduino/Arduino.h:114:73: note: in expansion of macro ‘bitClear’
   #define bitWrite(value, bit, bitvalue) (bitvalue ? bitSet(value, bit) : bitClear(value, bit))
   C:UsersVitalyAppDataLocalTemparduino_modified_sketch_712994sketch_jun25c.ino:121:5: note: in expansion of macro ‘bitWrite’
        bitWrite(volts, 0, bitRead(au16data [1], 0));

   Встречал в инете обьяснение этому, но моих знаний не хватило понять это, там все была на СИ++.
   Пожалуйста подскажите, как в моем случае должен выглядеть код.
   Просьба сильно не пинать.

2. bitwrite не работает с плавающей точкой.
   тебе надо использовать union сначала, чтоб конвертировать float в 2 int-а.

   union IntConvert {
     int   IntValues[2];
     float FloatValue;
   }

   и потом засовывать int-ы в массив, а забирать оттуда float целиком. И наоборот. И массив и float физически находятся по одному и тому же адресу памяти, разное только обращение к ней.
   Прочитай за union чонить.

   
   Andrey12 и Mitrandir нравится это.
   

3. Последнее редактирование: 27 июн 2018

4. Про union уже сказали. Ещё один путь — прямое копирование памяти:

   uint16_t from[2];
   float to;

   uint8_t* fromRead = (uint8_t*) from;
   uint8_t* toWrite = (uint8_t*) &to;

   memcpy(toWrite,fromRead,4); // копируем 4 байта

   Но! Это не всегда корректно, хотя бы из-за порядка следования байт. И вообще — желание из двух целых скопировать во float — кмк, сигнализирует о том, что ТС не совсем понимает, что нужно. На примере: допустим, в одном uint16_t содержится значение температуры в целых, во втором — в десятитысячных долях. Тогда конвертируется это во float довольно просто:

   uint16_t tempWhole = 12;
   uint16_t tempFract = 1234;

   float result = tempWhole;
   result *= 10000.0;
   result += tempFract;
   result /= 10000.0;

5. Вот, для демонстрации безумства первого подхода (прямое копирование) и корректности работы второго — набросал тут: http://cpp.sh/ — небольшую программку:

   // Example program
   #include <iostream>
   #include <string>
   #include <cstring>

   int main()
   {
       uint16_t from[2] = {56,1234};
       float to;

       uint8_t* fromRead = (uint8_t*) from;
       uint8_t* toWrite = (uint8_t*) &to;

       memcpy(toWrite,fromRead,4); // копируем 4 байта  

      
     std::cout << to << «n«;

    
     uint16_t tempWhole = 56;
     uint16_t tempFract = 1234;

    
     float result = tempWhole;
     result *= 10000.0;
     result += tempFract;
     result /= 10000.0;

    
     std::cout << result << «n«;
   }

   Вывод программы:

6. А были какие-то сомнения ?

7. а нельзя ли так:

   uint16_t a,b; //2 половинки числа
   uint32_t x; //временная переменная
   float32_t y; //результат
   x=a;
   x=x<<16;
   x=x+b;
   y=(float32_t)x;

   Чую, я не понял задачу.

8. Нет, это для демонстрации ТС

9. Вот код:

   // Example program
   #include <iostream>
   #include <string>

   int main()
   {
   uint16_t a = 56,b=1234; //2 половинки числа
   uint32_t x; //временная переменная
   float y; //результат
   x=a;
   x=x<<16;
   x=x+b;
   y=(float)x;
     std::cout << y;

   }

   Тестировал здесь: http://cpp.sh/ Вывод:

   Не очень похоже на 56.1234, не находите?

10. На 8-ми битке такое делать нельзя. Да и зачем?

11. В каком месте такое нельзя сделать на восьмибитке? Вывод в поток — дык это не основа алгоритма, это просто демонстрация результатов, не более того. Остальное — вполне валидный код на С++, который ОБЯЗАН компилироваться любым современным компилятором.

12. Огромное спасибо всем откликнувшимся за подсказки, кажется победил:

    int16_t au16data[]={17260, 0, 17259, 52429 };// 236.00 and 235.8
    float* ptr_recData;
    uint8_t numPar =0;   // номер параметра из массива
    float volts =0 ;
    int16_t temp[2];

    void setup() {
    Serial.begin(9600);
    ptr_recData = (float*)(&temp[0]) ;  // получаем адрес переменной
    }

    void loop() {

    temp [1] = au16data[(numPar*2)];
    temp [0] = au16data[(numPar*2)+1];
    volts =*ptr_recData;

    Serial.println(temp [0]);
    Serial.println(temp [1]);
    Serial.println(volts,8);

    Serial.println(«_»);

    ++numPar*2;
    temp [1] = au16data[(numPar*2)];
    temp [0] = au16data[(numPar*2)+1];
    volts =*ptr_recData;

    Serial.println(temp [0]);
    Serial.println(temp [1]);
    Serial.println(volts,8);
    Serial.println(«__________»);
    delay(2000);
    }

13. плавающая запятая для чего? Вывод на дисплей? Можно отдельно выводить целую и отдельно дробную. АЛУ АВР-ок и так убогое, а вы ему ещё и плавающую запятую….

14. Ты болеешь? Я ТС отвечал, и приводил только как ПРИМЕР. Что ТС там делает — хз, и мне пох. Не болей.

15. Ну он же у него одни флоаты, да ещё и глобальные зачем-то. Ну ты прав — конкурентов не надо растить — без работы останешься.
    Ыыыы с такими горе программистами конкуренции не будет НИКОГДА. Так что я спокоен)))