Arduino MS5803 library compilation error

Using Arduino Programming Questions
1

Hi everyone,
Im having a problem with compiling a program for my pressure sensor. I use those library for the sensor GitHub - vic320/Arduino-MS5803-14BA: An Arduino library for the MS5803-14BA pressure/temperature sensor. Works with SPI and i2C. and when i try to compile i get those error:

In file included from read.ino:5:
MS5803.h:66: error: ISO C++ forbids initialization of member 'press'
MS5803.h:66: error: making 'press' static
MS5803.h:66: error: ISO C++ forbids in-class initialization of non-const static member 'press'
MS5803.h:67: error: ISO C++ forbids initialization of member 'temp'
MS5803.h:67: error: making 'temp' static
MS5803.h:67: error: ISO C++ forbids in-class initialization of non-const static member 'temp'

Is there a problem with library or with arduino ide ?

2

Or with your sketch, which you didn't post.

3

Those messages look suspiciously like you are not using the IDE to compile your code. If you aren't, this isn't the place to ask for help.

4

The sketch i use is in the github examples folder, and im using an 1.0.5 IDE also tried other versions and result is the same.

5

The sketch i use is in the github examples folder

Post it here. Don't expect us to go hunting.

6

Here is the sketch :

#include <SPI.h>
#include <Wire.h>
#include <MS5803.h>

// Chip Select pin for SPI
#define SENSOR_CS_PIN 9

// Use this constructor for SPI
MS5803 sensor = MS5803(SENSOR_CS_PIN);

// Use this constructor for i2c - Address is set in the library implementation file. Default is 0x76.
//MS5803 sensor = MS5803();

void setup() {
  // Start the serial ports.
  Serial.begin( 115200 );
  delay(3000);

  // Initalize the sensor which resets the sensor, downloads the needed coeffecients, 
  // and does a CRC check on the returned data. This will verify that we are talking to
  // the device and that all is well.
  if ( sensor.initalizeSensor() ) {
    Serial.println( "Sensor CRC check OK." );
  } 
  else {
    Serial.println( "Sensor CRC check FAILED! There is something wrong!" );
  }

}

void loop() {
  // Call read sensor first, which downloads the sensor data and converts it to
  // mBars and Degrees C.
  sensor.readSensor();

  Serial.print("Pressure = ");
  Serial.print(sensor.pressure());
  Serial.println(" mBars");

  Serial.print("Temperature = ");
  Serial.print(sensor.temperature());
  Serial.println("C");

  //Just to make it easier to read. 
  //The sensor can be read as fast as desired.
  delay(100); 
}

But i started to dig in google and those errors says something about initializtion of variables in class , and when i press sendin IDE its jumping to MS5803.h and highligth those lines :

private:
    
    float             press             =0;            // Stores actual pressure in mbars
    float             temp		=0;                   // Stores actual temp in degrees C.

and give error i posted earlier.

7

The error messages are correct. Variables can not be initialized in a constructor. That library is crap.

8

So there is a easy way to move those variables to correct place or its useless and i need to make a new library ?

9

So there is a easy way to move those variables to correct place or its useless and i need to make a new library ?

Edit the header file, Remove the =0 bit from those two statements.
Edit the source file. Initialize press and temp in the constructors (both of them).

10

Hmm im no expert in programming in C i dont know how to do it , but anyway thank you veru much for help. At least i know what is wrong .

11

Hmm im no expert in programming in C i dont know how to do it ,

Changing:

    float             press             =0;            // Stores actual pressure in mbars
    float             temp		=0;                   // Stores actual temp in degrees C.

to

    float             press;            // Stores actual pressure in mbars
    float             temp;                   // Stores actual temp in degrees C.

hardly requires an advanced degree in computer science.

Adding:

   press = 0;
   temp = 0;

to the constructors is similarly not socket science.

12

Ok i give up this library is a crap, i did move variables as you said and now its giving tons of errors saying there are multiple definitions.

13

I deleted the assignments in the header, put them into the constructor, and got the following error message:

Binary sketch size: 8,248 bytes (of a 32,256 byte maximum)
(I can't test, because I don't have the hardware)

14

Can You post modified files u made i will check them with my sensor.

15

Can You post modified files u made i will check them with my sensor.

It works the other way. You post the code you modified, and the errors, and we'll help you figure out why they occur.

16

It works the other way. You post the code you modified, and the errors, and we'll help you figure out why they occur.

The other approach is called "fish-flinging"

17

OK so as you suggested i changed :

float press; // Stores actual pressure in mbars
float temp; // Stores actual temp in degrees C

and added:

press = 0;
temp = 0;

to cpp file and error about : ISO C++ forbids initialization of member has gone but now it give me this :

ms5803\MS5803.cpp.o: In function `MS5803':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:62: multiple definition of `MS5803::MS5803(unsigned char)'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:62: first defined here
ms5803\MS5803.cpp.o: In function `MS5803':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:62: multiple definition of `MS5803::MS5803(unsigned char)'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:62: first defined here
ms5803\MS5803.cpp.o: In function `MS5803':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:69: multiple definition of `MS5803::MS5803()'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:69: first defined here
ms5803\MS5803.cpp.o: In function `MS5803':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:69: multiple definition of `MS5803::MS5803()'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:69: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::ms5803CRC4(unsigned int*)':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:183: multiple definition of `MS5803::ms5803CRC4(unsigned int*)'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:183: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::resetSensor()':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:130: multiple definition of `MS5803::resetSensor()'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:130: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::ms5803CmdAdc(char)':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:217: multiple definition of `MS5803::ms5803CmdAdc(char)'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:217: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::readSensor()':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:110: multiple definition of `MS5803::readSensor()'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:110: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::ms5803ReadCoefficient(unsigned char)':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:149: multiple definition of `MS5803::ms5803ReadCoefficient(unsigned char)'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:149: first defined here
ms5803\MS5803.cpp.o: In function `MS5803::initalizeSensor()':
D:\arduino-1.0.5\libraries\ms5803/MS5803.cpp:72: multiple definition of `MS5803::initalizeSensor()'
MS5803.cpp.o:C:\Users\bartek\AppData\Local\Temp\build1708876184977661276.tmp/MS5803.cpp:72: first defined here
18

and added:

press = 0;

temp = 0;



to cpp file

But, I'm going to leave it to you to guess where I added them.

Well, OK, we'll leave it to you to guess where the problem is.

Is there some part of "Post the damned code" that is beyond your comprehension?

19

I dont understand that thing about constructor , that You wrote to add this in constructor

press = 0;
   temp = 0;

.Sorry im very stupid in C programming.

20

The library has this in it:

// Constructor when using SPI.
MS5803::MS5803(uint8_t cs) {
    _cs = cs;
    interface = true;
}

Your sketch has this in it:

// Use this constructor for SPI
MS5803 sensor = MS5803(SENSOR_CS_PIN);