- Forum
- General C++ Programming
- delete please, didnt know I couldnt post
delete please, didnt know I couldnt post completed code here
This is very likely in vain. Beginners usually don't consider other posts apart from their own.
1) Wow, a single nonimportant warning even on my paranoidal setup! Good.
3) Bug on line 26: does that semicolon need to be here?
2)
Those
unsigned stuff in loops creates signed/unsigned mismatch. You should use signed type as size is signed too.
Applicable to everything: check your indentation.
3) Bug on line 26: does that semicolon need to be here?
2)
Those
'/n's... a) they are not newline symbols, b) they have no effect. It looks like they are here to separate blocks of code, but comments or empty space will do that better. unsigned stuff in loops creates signed/unsigned mismatch. You should use signed type as size is signed too.
for (unsigned col = 1; col <= size; col*2, col++){ marked part does not do anything at all.Applicable to everything: check your indentation.
Not clear what the OP intended with line 26.
As is, the
If the OP intended to conditionally increment round as a method of detecting that the cin failed, that would work on the first round, but not subsequent rounds. The OP does check for the success of the cin at line 29, so relying on round to detect a cin failure is unnecessary also.
As is, the
if statement is unnecessary.If the OP intended to conditionally increment round as a method of detecting that the cin failed, that would work on the first round, but not subsequent rounds. The OP does check for the success of the cin at line 29, so relying on round to detect a cin failure is unnecessary also.
| closed account wrote: |
|---|
| delete please, didnt know I couldnt post completed code here |
I think it more likely OP's teacher found out he was asking for help here and yelled at him for it.
Topic archived. No new replies allowed.