search

Student26533841 / Card_Shuffle_Project

0 stars 0 forks source link

26300524 William Ashfield #1

Open WillAshfield opened 1 year ago

WillAshfield commented 1 year ago

It seems you have more classes than necessary in your main branch. Is it supposed to be like this or do a few of the classes need to be moved / removed.

WillAshfield commented 1 year ago

There also isn't much error handling in the code, could this be added into some of the methods of the code to prevent crashing by an error?

WillAshfield commented 1 year ago

Overall, the code seems to work well and does what it should when prompted in the console, however the classes 'Testing' and 'Card' could do with more comments explaining each of your methods which would help others in completely understanding your code.