trjhtr

I am making an IOT weather station based on a particle photon that sends data via webhook to one of my other projects. I am very new to c++ and programming in general.

What can I do better with my code? What can be optimised? And what are industry best practices that I can put into place?

Here's my code:

#include <Adafruit_DHT/Adafruit_DHT.h>

#define DHTPIN 2 

#define DHTTYPE DHT11

DHT dht(DHTPIN, DHTTYPE);

void setup() 
Serial.begin(9600);
dht.begin();


void loop() 

delay(2000);

//Variables
//DHT11 sensor floats
float h = dht.getHumidity();
float t = dht.getTempCelcius();
float f = dht.getTempFarenheit();
float hi = dht.getHeatIndex();
//UV sensor floats
float sensorValue = analogRead(A0);
int UVLevel = 0;

//Error Retry
if (isnan(h) 

For reference these are the sensors I am using:

DHT 11 Temp/Humidity sensor

UV sensor

I'll assume the indentation is an artifact of pasting the code here.

The big problem here is that long series of if statements. A simple improvement is to make use of the else clause, along with knowledge of what values you've tested for already, it can be simplified to

if (sensorValue <= 10)
 UVLevel = 0;
else if (sensorValue <= 46)
 UVLevel = 1;
// etc.

But we can do better. Since the range of values you're checking with is contiguous, and the result (UVLevel) is linear, we can set up an array to hold the data, then just use a loop to find the right place.

static values = 10, 46, 65, /* ... */, 240 ;
UVLevel = std::size(values); // 12
for (int i = 0; i < std::size(values); ++i)
 if (values[i] <= sensorValue) 
 UVLevel = i;
 break;
 
}

We initially set UVLevel the the maximum value so that if the sensor value is greater than 240 we get the correct result without having to check after the loop.

For a larger range of possible values, the for loop used here can be replaced with a more complicated binary search.