22
EgzysT
6y

Computer Science student here, and it looks like my group partners have no idea what keeping classes short and straight to the point means.

The project is in C++, and I'm trying to understand this mess that they made, when they have a class called "Manager" that basically did EVERYTHING, whose header file has only the declarations of functions. 50+ functions. Only half of then documented, and most of them with apparently random names. The file has more than 200 lines of reading.

I've never worked with so much Spaghetti in my life yet.

Worst part: I spent time and effort organising some other classes, breaking down methods, untangling code and all that tedious stuff. But one week before delivery, they decide to delete all of my work, because they "didn't understand it" and didn't even think of asking me to explain the changes.
And if that wasn't enough, they refused to give me some percentage of the grade due to that code not being in the delivery.

I am so freaking done with those guys -_-

Comments
  • 0
    Thank you! 😊
    I just had no idea how to expand upon that monolithic beast 😅
  • 1
    "short and straight to the point" / C++... Puns intended?
  • 1
    Every class finishing with "Manager" is a code smell. It most of time break the single responsability principle.

    Good luck with the project.
  • 1
    Thank you @Etnath !
  • 2
    oh... you know you're working with amateurs when they can't even name things sensibly.

    i mean, everyone knows Managers don't do anything, it's the worst name for your god class =D
  • 0
    @Midnigh-shcode I can see so many god-related name puns for a god classs xD
  • 1
    @EgzysT based on what you wrote, the most appropriate would probably be

    namespace TheChurchOf {
    public class SpaghettiMonster {
    //omnipotence implementation here
    }
    }

    btw is it also static?
  • 0
    @cabrasm one of the things that they try to promote is teamwork, and they do this also by applying consequences to everyone if a group member fails.
    In real life if your group is failing you all need to solve your problems or the project will suffer.
    It's unfair, but I can see why... :/
  • 0
    @cabrasm No in the last 3 days we managed to salvage it with a lot of band aids and mom's spaghetti but it worked and we had a good grade.
    But I sure as hell am not working with those people again :/
Add Comment