samples: sensor: pressure_polling: Log altitude only if the sensor supports it

[ttmut]

Most pressure sensors in the Zephyr tree do not provide altitude data. This change adds a check to ensure altitude is printed only when the sensor supports it. A delay is also added between reads to avoid flooding the console.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[kartben]

Thanks for keeping the doc in sync!
Wasn't quite sure initially why you dropped logging and switched to printk's but I think I get it now.

[ttmut]

Thanks for keeping the doc in sync! Wasn't quite sure initially why you dropped logging and switched to printk's but I think I get it now.

I wanted to avoid code duplication but now I am not sure if it is actually worth it. This was the inital change:

if (ret == 0) {
    LOG_INF("temp %.2f Cel, pressure %f kPa, altitude %f m",
	   	   sensor_value_to_double(&temperature),
	   	   sensor_value_to_double(&pressure),
	   	   sensor_value_to_double(&altitude));
} else {
    LOG_INF("temp %.2f Cel, pressure %f kPa",
	   	   sensor_value_to_double(&temperature),
	   	   sensor_value_to_double(&pressure));
}

I can revert it to this if you wish.

[kartben]

Thanks for keeping the doc in sync! Wasn't quite sure initially why you dropped logging and switched to printk's but I think I get it now.

I wanted to avoid code duplication but now I am not sure if it is actually worth it. This was the inital change:

if (ret == 0) {
    LOG_INF("temp %.2f Cel, pressure %f kPa, altitude %f m",
	   	   sensor_value_to_double(&temperature),
	   	   sensor_value_to_double(&pressure),
	   	   sensor_value_to_double(&altitude));
} else {
    LOG_INF("temp %.2f Cel, pressure %f kPa",
	   	   sensor_value_to_double(&temperature),
	   	   sensor_value_to_double(&pressure));
}

I can revert it to this if you wish.

no it's alright :) Thanks!