GeezerVison

Hi @catilac, I have finished the updates. Added the print() declaration to the PMatrix interface and implemented formatted toString() methods in both PMatrix2D and PMatrix3D. This makes the solution simpler and polymorphic as suggested. Ready for your review!

@Sumamasonia Hi! Can you help me understand the changes you made in PMatrix2d.java and PMatrix3D.java? I don't think you need to make any changes to the implementations there.

Hi @catilac! The reason I updated the implementations is to leverage polymorphism properly as you suggested.

Previously, both classes had their own independent print() logic. By adding public void print() to the PMatrix interface, I added the @OverRide annotation to the existing methods in PMatrix2D and PMatrix3D to explicitly link them.

Also, I refactored the formatting logic into toString(). This makes the code cleaner and allows developers to use System.out.println(matrix) as well as matrix.print(), with both using the same formatting logic. Let me know if you’d prefer me to revert the implementation changes and keep the original print() methods as they were!

@Sumamasonia ohhhh I see what you've done. That's really cool. Sorry I didn't understand it

Thank you @catilac! I'm glad you liked the approach. I wanted to make sure the fix follows Java's best practices for polymorphism. Looking forward to seeing it merged! 🙌

@all-contributors please add @Sumamasonia for Code

@catilac

This project's configuration file has malformed JSON: .all-contributorsrc. Error:: Unexpected token : in JSON at position 45082

https://github.com/all-contributors please add @Sumamasonia for Code

@all-contributors please add @Sumamasonia for Code

@catilac

I've put up a pull request to add @Sumamasonia! 🎉

Thank you so much @catilac for the merge and for adding me as a contributor! It was a great learning experience. Looking forward to contributing more to Processing in the future! 🎉