search

ChrisViral / HydroTech

RCS docking autopilot/docking came for KSP
2 stars 0 forks source link

Spring cleaning! #2

Closed ChrisViral closed 8 years ago

ChrisViral commented 8 years ago

This code is very old, and very... od formatted. I need to understand what happens first and foremost before I can actually put any work into this. Gonna go through every single class and reformat, simplify, and merge files when possible. Not a fan of partial classes, rather would organize my stuff in #region blocks.

Already went through three files, and merged two of those together. Let's hope this keeps on going faster.

ChrisViral commented 8 years ago

Got Resharper and did some bulk cleaning in e8d2c6a and 1893b2f then finished cleaning the Autopilot folder, went much, much faster. Gonna use Resharper to try and merge some of the constants into the same file.

ChrisViral commented 8 years ago

Managed to merge all the constants into four different files, all in the same namespace, this should declutter a whole fucking lot. I'll probably want to get rid of those classes in the future however.

Also, finished clean the Panels and UI classes, might go over and convert more string concatenation madness into string formats

ChrisViral commented 8 years ago

HydroTech_RCS.csproj is now completely cleaned, the only errors remaining are from HydroTech_FC not compiling, which is the next step

ChrisViral commented 8 years ago

Went through every single file and cleaned a whole lot of code, seems good enough for now, should start working on #3