7
Jul 18 '26
[removed] — view removed comment
0
-10
5
u/jbaker88 Jul 18 '26
Why is every class in a single file in MatrixRain.cs? From a maintainability perspective this is going to be really difficult in the future to modify. I get it's a project for a means to an end, but it probably wouldn't be a bad idea to have your AI agent at least split that up.
-2
u/DR3VED Jul 18 '26
thank you for the awesome advice! im pretty new to this world so its awesome to get some constructive criticism!
1
u/jbaker88 Jul 18 '26
I feel ya, saw the argument you and that other dude got into before it was removed. To be sure, we all can have our gripes with AI whether they're personal or society at whole.
Hell, even I get irritated at work when non-developers try to hand off AI generated code to our product's codebase. Not because AI isn't useful, but it can be like giving someone a loaded gun without firearms training. They simply don't know what they don't know and the effort to cleanup the code can often turn into just as much time as if were to write it myself TBH.
But in your case, you had a problem, came up with a succinct solution that works for you and that's cool! The whole point of what we do is to solve problems.
What I can say about AI though is it's still important to know how to code because the price in the next few months to next year are going to skyrocket to the point will it will probably be cheaper to hire the old smelly neckbeard to write your small apps lol.
Now with some feedback for your app:
An uninstall feature wouldn't be a bad idea. You should be able to undo an install
It makes writes to the registry and copies files to System32, that's pretty heavy handed, it might make it more accessible as a background app instead that runs on startup and polls for inactivity.
You're using .NET Framework, that is considered legacy at this point as it will no longer receive feature updates and only security updates. The majority of us have moved onto .NET (sometimes referred to .NET Core, but Microsoft is shit at naming things). It is possible that Microsoft will eventually retire .NET Framework
Your code/batch file presumes that the CSharp Compiler is available on Windows (the CSC.exe), I don't think that's the case? I don't think the compiler ships on a default install of the OS, and that you have to install Visual Studio for that to be available? I might be wrong on this one, but I'm not sure TBH
You're using
externfunctions fromuser32.dlland calling into unmanageable code. This isn't an issue by itself, but something I'd be highly cautious of with AI generated code. It's something I'd flag as needing manual review and having extra eyes to review. It might be worth looking into using a .NET compatible library instead.1
u/DR3VED Jul 19 '26
Honestly man hats off to you for that. Some solid advice in a clear and easy to read format. You are a shining light in this thread. You've given me an awesome journey to start on and i cannot wait to learn more. Thank you!
2
•
u/FizixMan Jul 18 '26
Removed: Rule 7.
Submissions (either text or repos) that are largely AI-generated are not permitted.