4 ms·
The AI is adding comments like these: private void makeAllWarningsFatal(Project project) { /** * Makes all warnings fatal for the given project. *
by ripjaygn 3y ago
The AI is adding comments like these:
private void makeAllWarningsFatal(Project project) {
/**
* Makes all warnings fatal for the given project.
*
* @param project the project for which to make warnings fatal
*/
What does this achieve? The code already has readable function names. It just creates clutter and makes things worse by forcing people to scroll more by making less code fit in a window.
What would be useful is commenting why a setting to make all warnings fatal is a use case. For example, a comment stating that this is used to improve code quality and reduce risk by forcing the developer to fix all warnings which could be hiding errors. This AI is utterly useless at coming up with that.
- Freedom2 3y agoPlaying devil's (GPT's?) advocate here - they can be somewhat useful for autogenerated documentation where you don't have access to the code and only some form of browser or other text documentation. Otherwise I agree with you.
- stevage 3y agoYeah I think there is potentially value in generating comments at the time of generating documentation - but not actually storing them in the code.
- tivert 3y ago> What does this achieve? State-of-the-art digital solutions to liberate humanity from the tyranny of work.
- stevage 3y agoI'm not sure how you read my comment as somehow defending the current implementation or the comments it has generated so far.