0
0

[–] Cuddlefluff ago 

They also needs to take a serious look at their use of try-catch because it is wrong :P

try
{
    ....
}
catch (Exception) <-- BAD!
{
    ....
}

This is not how exceptions are supposed to be handled. If you have an unhandled exception, there's an entire functionality in ASP.NET to handle that eventuality. Removing the "catch-all" clauses will reduce code duplication and improve debugging and logging.

0
0

[–] ptd ago 

Yep, sure. I mean, in all you can find yourself refactoring forever. There's always room to improve something, ya know? I think they're doing a great job of fighting off the huge influx of users, bots, and attackers, though.

0
0

[–] Cuddlefluff ago 

Yeah, I think they're doing a great job as well.. Small infrastructure, few employees.. This kind of pressure completely murdered Reddit in the start for days on end. Hell, even now after being online for a decade Reddit is still completely unstable and unreliable. So yeah, definitely.