Remix.run Logo
Remix clone Hacker News
new | show | ask | jobsGithub
▲ maleldil 4 days ago

Nitpick: this looks like Python. You don't need + to concatenate string literal. This is the type of thing a linter can catch.

▲Sohcahtoa82 3 days ago | parent | next [-]

IMO, 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.

▲layer8 3 days ago | parent | prev [-]

It’s actually Java, where the “+” is necessary.