search

Nagisa-AC / Flixter

Movie based app that allows users to get information on current playing movies
0 stars 0 forks source link

Remove debugging statements from final code #2

Open tisganitis-fb opened 2 years ago

tisganitis-fb commented 2 years ago

When working in a large codebase with many collaborators, extra debugging statements can cause confusion, clutter, and potentially impact the experience for other developers (e.g. if someone adds a TON of debug logs that crowd out other useful info). There's also a chance of leaking sensitive information into logs in production. So, we delete (not just comment out) debug logging statements like this before submitting final code.

https://github.com/Nagisa-AC/Flixter_4/blob/55b210bc6a267e674b58ec5d2b526b8ea0013a8e/Flixer_4/DetailsViewController.m#L23

tisganitis-fb commented 2 years ago

Some other examples: https://github.com/Nagisa-AC/Flixter_4/blob/55b210bc6a267e674b58ec5d2b526b8ea0013a8e/Flixer_4/MovieViewController.m#L47 https://github.com/Nagisa-AC/Flixter_4/blob/55b210bc6a267e674b58ec5d2b526b8ea0013a8e/Flixer_4/MovieViewController.m#L63-L65 https://github.com/Nagisa-AC/Flixter_4/blob/55b210bc6a267e674b58ec5d2b526b8ea0013a8e/Flixer_4/MovieViewController.m#L88-L89