0
If statements break the code
I was working on the Fisherman code challenge and, after writing the whole thing and trying to test it, nothing would show up. After commenting almost the whole code, it was finally able to output the number you entered. I am providing the code and you can try to uncomment the if statements. If anyone has any fix/explanation, it would be appreciated. https://code.sololearn.com/c3IUT43BklHC/?ref=app
7 Respuestas
+ 4
I agree you should use brackets, not just to improve readability, it'll bite you one day, but your choice.
But that's not the problem today... probably, haven't read through the code too much :)
for(int i=0; i*i<=a; i++)
if(a%i==0)
...
See it yet?
a%0 is a division by 0 which results in an error.
0
please us {} in if statements and use meaningful names for your variables.
To work use isFisher(a)==0 or 1 in the if statement.
Why you give b a value when you input it in the next line?
0
Borislav Kalinski In C++ you don't need to use brackets whenever you only have 1 line inside and if the output is eother 0 or 1 you don't need to check, because 0 means false and 1 means true.
0
Borislav Kalinski I do that incase there's only 1 input, also, if you want to add more suggestions please write a new comment, as it's hard to check if you edit your own
0
https://github.com/borislavvv00
Check my acount to see my works if you want.
Yes you can write without brackets but is it not readable i mean check the standarts.
0
Borislav Kalinski Sorry, I'm really not experienced with working with other people, so I don't really know how github works. As for the readability, I find it easier on the eyes without the brackets when the indentation is right.
0
Dennis Wow, I can't believe I was that dumb. Anyways, thank you so much for the help.