csharp10 lines
6.4/10
GROSS
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
C# code, judged.
14 C# posts rated by Ewwsers · average 7.3 EWW. Beautiful, cursed and questionable C#, scored 0 to 10 and roasted in the Roast Pit.
csharp12 lines
8.6/10
HAUNTED
AI ROASTFired at 15 ratings - once per post
You catch the exception, throw it in the trash, and immediately re-run the exact thing that just blew up. `while(true)` plus an empty catch is a slot machine with your user's data as the payout.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp10 lines
7.0/10
NASTY
AI ROASTFired at 17 ratings - once per post
async void and an empty catch is you personally guaranteeing every save failure gets swallowed and then blamed on the database. You named it SaveAsync, which is a lot like naming a grenade cuddle.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp6 lines
8.3/10
HAUNTED
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp8 lines
7.1/10
NASTY
AI ROASTFired at 10 ratings - once per post
A static Dictionary with zero locking and you're calling it a cache. Two threads hit that ContainsKey at the same time and you get to enjoy an exception or a duplicated load, dealer's choice. This is a race condition wearing a name tag.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp10 lines
0.7/10
NEARLY BEAUTIFUL
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp3 lines
7.5/10
CURSED
AI ROASTFired at 14 ratings - once per post
`retries` calls `.ToString()` on a value that came out of a config dictionary, and `enabled` uses a ternary to convert a bool into the same bool. Every string in here is one null away from a runtime explosion, and you clearly know it because `timeout` is the only one you bothered to guard.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp10 lines
1.0/10
HONESTLY GREAT
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp6 lines
6.9/10
NASTY
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp8 lines
9.1/10
EWW
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp3 lines
8.8/10
EWW
AI ROASTFired at 16 ratings - once per post
`Config["enabled"].ToLower() == "true" ? true : false` — a boolean, a string comparison and a ternary, all to compute something C#'s `bool.Parse` already does in plain sight. Congratulations on writing the world's longest road to two possible answers.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp10 lines
6.5/10
GROSS
AI ROASTFired at 9 ratings - once per post
async void on a save method, so nobody can await it, nobody can catch it, and nobody will ever know if it worked. But hey, at least the empty catch makes the silence official.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
csharp13 lines
8.2/10
DEEPLY CURSED
The AI roast is still asleep - it fires once, automatically, when enough judgments land.
csharp11 lines
9.0/10
EWW
AI ROASTFired at 5 ratings - once per post
Catching `Exception` is bad enough, but returning `Ok(null)` means every database outage, timeout, and null reference gets served to the client as a cheerful HTTP 200. You built a status code that lies, and it's giving bad news a fake smile.
AI-generated - based on this post's Eww Score and ratings - roasts code, never people.
Post your own C#: drop some code. More languages: JavaScript, TypeScript, Python, Java, C++, C, Go, Rust.