search

YUNEEC / Yuneec-SDK-Android-Example

Android example app based on the Yuneec SDK
BSD 3-Clause "New" or "Revised" License
11 stars 9 forks source link

Removed video streaming library and fragment UI #18

Closed Sushma7785 closed 7 years ago

Sushma7785 commented 7 years ago
JonasVautherin commented 7 years ago

Why not removing the code, instead of commenting it out? It will stay in the git history anyway...

Just to quote Uncle Bob in "Clean Code" :-):

It makes me crazy to see stretches of code that are commented out. Who knows how old it is? Who knows whether or not it’s meaningful? Yet no one will delete it because everyone assumes someone else needs it or has plans for it.

That code sits there and rots, getting less and less relevant with every passing day. It calls functions that no longer exist. It uses variables whose names have changed. It follows conventions that are long obsolete. It pollutes the modules that contain it and distracts the people who try to read it. Commented-out code is an abomination.

When you see commented-out code, delete it! Don’t worry; the source code control system still remembers it. If anyone really needs it, he or she can go back and check out a previous version. Don’t suffer commented-out code to survive.