3 ms·
Nitpick: this looks like Python. You don't need + to concatenate string literal. This is the type of thing a linter can catch.
by maleldil 1y ago
Nitpick: this looks like Python. You don't need + to concatenate string literal. This is the type of thing a linter can catch.
- layer8 1y agoIt’s actually Java, where the “+” is necessary.
- Sohcahtoa82 1y agoIMO, implicit string concatenation is a bug, not a feature. I once made a stupid mistake of having a list of directories to delete: directories_to_delete = ( "/some/dir" "/some/other/dir" ) for dir in directories_to_delete: shutil.rmtree(dir) Can you spot the error? I somehow forgot the comma in the list. That meant that rather than creating a tuple of directories, I created a single string. So when the `for` loop ran, it iterated on individual characters of the string. What was the first character? "/" of course. I essentially did an `rm -rf /` because of the implicit concatenation.