Should be:Code:if (surname[1] != 1 || surname[1] != 2 || surname[1] != 3 || surname[1] != 4 || surname[1] != 5 || surname[1] != 6 || surname[1] != 0){ surname[1] = '0'; }Code:if (surname[1] != '1' || surname[1] != '2' || surname[1] != '3' || surname[1] != '4' || surname[1] != '5' || surname[1] != '6' || surname[1] != '0'){ surname[1] = '0'; }
still doesn't seem to work, whatever the value of the character in the array is it replaces it with a zero.
Edit:
although I did think of doing it like this, but I get an error in compiling as it doesn't seem to like the else command.
Code:if (surname[1] == '1' || surname[1] == '2' || surname[1] == '3' || surname[1] == '4' || surname[1] == '5' || surname[1] == '6' || surname[1] == '0'){ else surname[1] = '0'; }
Last edited by manutdfan; 11-13-2006 at 05:12 PM.
Try actually looking at your code, specifically your braces.Originally Posted by manutdfan
What exactly is that else doing in there? What if does it belong to?Code:if ( ){ else surname[1] = '0'; }
Quzah.
Hope is the first step on the road to disappointment.
thanks a lot it is working now properly.
hmm, that wasn't in the code I looked at, was it?
no it was in the code I posted in post 17, although I couldn't get the other piece to work correctly.
Weird.
This will always set surname[1] to '0'. If surname is equal to '1', then (surname[1] != '2') will be true and surname[1] will be set to '0'. If surname[1] is anything but '1', (surname[1] != '1') will be true, and again, surname[1] will be set to '0'.Code:if (surname[1] != '1' || surname[1] != '2' || surname[1] != '3' || surname[1] != '4' || surname[1] != '5' || surname[1] != '6' || surname[1] != '0'){ surname[1] = '0'; }
Code:#include <stdio.h> void J(char*a){int f,i=0,c='1';for(;a[i]!='0';++i)if(i==81){ puts(a);return;}for(;c<='9';++c){for(f=0;f<9;++f)if(a[i-i%27+i%9 /3*3+f/3*9+f%3]==c||a[i%9+f*9]==c||a[i-i%9+f]==c)goto e;a[i]=c;J(a);a[i] ='0';e:;}}int main(int c,char**v){int t=0;if(c>1){for(;v[1][ t];++t);if(t==81){J(v[1]);return 0;}}puts("sudoku [0-9]{81}");return 1;}
I think the OP's figured that out; see the latest version of that code.
Presumably modified as Quzah said.Code:if (surname[1] == '1' || surname[1] == '2' || surname[1] == '3' || surname[1] == '4' || surname[1] == '5' || surname[1] == '6' || surname[1] == '0'){ else surname[1] = '0'; }
In any case, it would be easier to use < and >:
I think that's what the OP intended . . . .Code:if(surname[1] < '0' && surname[1] > '6') surname[1] = '0';
dwk
Seek and ye shall find. quaere et invenies.
"Simplicity does not precede complexity, but follows it." -- Alan Perlis
"Testing can only prove the presence of bugs, not their absence." -- Edsger Dijkstra
"The only real mistake is the one from which we learn nothing." -- John Powell
Other boards: DaniWeb, TPS
Unofficial Wiki FAQ: cpwiki.sf.net
My website: http://dwks.theprogrammingsite.com/
Projects: codeform, xuni, atlantis, nort, etc.
