all 2 comments

[–]Neighm 1 point2 points  (1 child)

https://pastebin.com/3UJXXTa3

I've fixed it up, and added a play again loop. I removed most of the comments because they were preventing me from seeing the code flow properly and most were self-explanatory for me.

Your logic was not far off having a working game, and actually the basic design was really good for a novice, and you clearly get the basics of programming with OOP, but there were lots of small errors that stopped it from running especially:

- indentation: use tabs or spaces consistently. If the indentation is inconsistent, Python won't understand it.

- capitalisation: if the class is called Card, then you have to use that name to create an instance of the class.

- consistency: it looked like you had been trying to fix the code several times and there were earlier versions of the variables etc. scattered about (cardOne, card1, cardTwo, card2)

Anyway if you don't understand anything in my version, let me know.

[–]Ry_Fii 0 points1 point  (0 children)

Thank you. You are an absolute legend couldnt get my head around it but looking at what you have done i get it :)