Notifications
Clear all
Arduino
57
Posts
9
Users
59
Likes
2,474
Views
Topic starter
2022-03-22 1:08 pm
Hello all, I am having trouble modifying an Adafruit ina260 example program. I am using a NANO and Adafruit ina260 current sensor, to operate a 5VDC normally closed relay that in turn will switch off current to a motor in case of overload. The motor armature can be stalled in some conditions where it must operate leading to a massive overload and motor destruction. The goal is that the components once assembled can act as an very fast acting, auto reset electronic fuse with a preset cool-down period when amp tolerance presets are exceeded. It seem simple project to work on "if" statements and the Adafruit ina260 performs perfect. It can be read on the serial monitor. But after several hours I cant get to the bottom of my code issues. I am very inexperienced, this is the first time I have tried to combine an example with modifications to the code to achieve the desired result. I hope the error is simple but I can't find it. I keep getting the error message on the second if statement, any help would be appreciated, Thanks in advance, Quince
#include <Wire.h>
#include <Adafruit_INA260.h>
int mypinout = 9;
int mypinin = 4;
float val = 0;
Adafruit_INA260 ina260 = Adafruit_INA260();
void setup() {
Serial.begin(115200);
// Wait until serial port is opened
while (!Serial) {
delay(10);
pinMode(mypinout, OUTPUT);
pinMode(mypinin, INPUT);
}
Serial.println("Adafruit INA260 Test");
if (!ina260.begin()) {
Serial.println("Couldn't find INA260 chip");
while (1);
}
Serial.println("Found INA260 chip");
}
void loop() {
val = analogRead(mypinin);
digitalWrite(mypinout, val);
Serial.print("Current: ");
Serial.print(ina260.readCurrent());
Serial.println(" mA");
Serial.print("Bus Voltage: ");
Serial.print(ina260.readBusVoltage());
Serial.println(" mV");
Serial.print("Power: ");
Serial.print(ina260.readPower());
Serial.println(" mW");
if ()) {
(val,> 2000,) digitalWrite, mypinout, HIGH);
};
delay(25000);
if ()) {
(val, < 2000) digitalWrite, mypinout, LOW);
}
Serial.println();
delay(100);
}
C In function 'void loop()':
ina260_modifed:47:7: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:47:8: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:51:7: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:51:8: error: expected primary-expression before ')' token
if ()) {
^
exit status 1
expected primary-expression before ')' token
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.
C:\Users\James\Documents\Arduino\ina260_modifed\ina260_modifed.ino: In function 'void loop()':
ina260_modifed:47:7: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:47:8: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:51:7: error: expected primary-expression before ')' token
if ()) {
^
ina260_modifed:51:8: error: expected primary-expression before ')' token
if ()) {
^
exit status 1
expected primary-expression before ')' token
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.
2022-03-22 1:34 pm
The IF statement isn't checking anything, it needs a condition to check:
if ()) {
The condition can't be empty, e.g.
if (myFlag == true)) {
ron bentley and Inst-Tech reacted
2022-03-22 1:54 pm
[snip]
It seem simple project to work on "if" statements and the Adafruit ina260 performs perfect. It can be read on the serial monitor. But after several hours I cant get to the bottom of my code issues. I am very inexperienced, this is the first time I have tried to combine an example with modifications to the code to achieve the desired result. I hope the error is simple but I can't find it. I keep getting the error message on the second if statement, any help would be appreciated, Thanks in advance, Quince
[snip]
if ()) {
(val,> 2000,) digitalWrite, mypinout, HIGH);
};
Your first one looked ok, but as @yurkshirelad explained, your conditional statement
had the incorrect syntax - This is how it should look like:
if (condition) { //statement(s) }
...where 'condition' evaluates to either 'true' or 'false'
Cheers.
ron bentley reacted
2022-03-22 1:59 pm
The condition can't be empty, e.g.
if (myFlag == true)) {
You might still have time to edit out that extra bracket?
Cheers
ron bentley reacted
2022-03-22 2:00 pm
Hi @quince, As @yurkshirelad has already commented, I'll just clarify the comment..
change your if statement.
if ()) {
(val,> 2000,) digitalWrite, mypinout, HIGH);
};
to this:
if (val > 2000){
digitalWrite(mypinout, HIGH);
}
and similarly, change your other "if" statement as well..
regards,
LouisR
LouisR
ron bentley and Ron reacted
2022-03-22 2:10 pm
Just a tip when posting code, for you and other members viewing... When in reply mode, firstly, in your Arduino IDE, select the code that you want to copy, then right click on your highlighted code selection and choose "Copy as HTML", and then paste that HTML code into the source code window, which you can find by clicking on the following button on your reply screen as shown below:
Cheers.
Inst-Tech and ron bentley reacted
2022-03-22 2:38 pm
Remove the extra right paren, you have in multiple places if()) it should be if().
Try the auto format menu option. Tru using arduino2, I think it will catch this or better still platformIO
Arduino says and I agree, in general, the const keyword is preferred for defining constants and should be used instead of #define
"Never wrestle with a pig....the pig loves it and you end up covered in mud..." anon
My experience hours are >75,000 and I stopped counting in 2004.
Major Languages - 360 Macro Assembler, Intel Assembler, PLI/1, Pascal, C plus numerous job control and scripting
ron bentley reacted
2022-03-22 3:14 pm
@yurkshirelad The empty condition is fine. The problem is that there are one left paren and two right parens. The second should not be there. That is exactly what the compiler is saying.
ron bentley reacted
2022-03-22 3:31 pm
@codeslinger BINGO
Arduino says and I agree, in general, the const keyword is preferred for defining constants and should be used instead of #define
"Never wrestle with a pig....the pig loves it and you end up covered in mud..." anon
My experience hours are >75,000 and I stopped counting in 2004.
Major Languages - 360 Macro Assembler, Intel Assembler, PLI/1, Pascal, C plus numerous job control and scripting
2022-03-22 3:34 pm
@yurkshirelad The empty condition is fine. The problem is that there are one left paren and two right parens. The second should not be there. That is exactly what the compiler is saying.
Ah - I wasn't wearing my reading glasses. 🤣
Topic starter
2022-03-22 4:09 pm
Hello All, Once again the problem is solved. Thank you, Quince
Topic starter
2022-03-22 4:11 pm
@inst-tech Thanks Louis, Notation seem to be my issue. You made the solution very clear. Quince
Inst-Tech reacted
2022-03-22 4:13 pm
@frogandtoad NEVER compare a value or experssion to true or false. If X is a value or expression then the proper form is
if( X ) // check if X is true
or
if( !X ) to check if X is false.
This is expecially problematic for comparisons to true. C/C++ defines any value not equal to zero as true and any equal to zero as false. The constants true and false are defined as zero and some non-zero value. For instance if, true is defined as 1 but X evaluates to 2, then X==true is false and would be false for any X not equal to 1. This is counter to the above definition of true and false.
Ron reacted
2022-03-22 4:26 pm
@codeslinger I just recently read an in-depth article about the traps of TRUE and FALSE. I learned/relearned how compiler's deal with non-false. Some compilers are more demanding than others, and most/many programmers/coders/copiers are woefully undereducated. My best advice is to use your compiler supplied TRUE and FALSE if provided BUT yes, do NOT test in the if, just do as @codeslinger says. Thanks for the reminder!
Arduino says and I agree, in general, the const keyword is preferred for defining constants and should be used instead of #define
"Never wrestle with a pig....the pig loves it and you end up covered in mud..." anon
My experience hours are >75,000 and I stopped counting in 2004.
Major Languages - 360 Macro Assembler, Intel Assembler, PLI/1, Pascal, C plus numerous job control and scripting
2022-03-22 6:30 pm
I would usually declare my variable as bool(ean), so it would be 0 or 1, false or true, hence should not be an issue and if (x== trrue) , for eg, is a perfectly valid syntax and semantic.
Ron Bentley
Creativity is an input to innovation and change is the output from innovation. Braden Kelley
A computer is a machine for constructing mappings from input to output. Michael Kirby
Through great input you get great output. RZA
Gauss is great but Euler rocks!!
frogandtoad and Inst-Tech reacted
Page 1 / 4
Next