Question

Here is some code:

foo = "Bears"
"Lions, Tigers and %(foo)s" % locals()

My PEP8 linter (SublimeLinter) complains about this, because foo is “unreferenced”. My question is whether PEP8 should count this type of string interpolation as “referenced”, or if there is a good reason to consider this “bad style”.

Answer 1

Well, it isn’t referenced. The part that’s questionable style is using locals() to access variables instead of just accessing them by name. See this previous question for why that’s a dubious idea. It’s not a terrible thing, but it’s not good style for a program that you want to maintain in the long term.

Edit: It’s true that when you use a literal format string, it seems more explicit. But part of the point of the previous post is that in a larger program, you will probably wind up not using a literal format string. If it’s a small program and you don’t care, go ahead and use it. But warning about things that are likely to cause maintainability problems later is also part of what style guides and linters are for.

Also, locals isn’t a canonical representation of names that are explicitly referenced in the literal. It’s a canonical representation of all names in the local namespace. You can still do it if you like, but it’s basically a loose/sloppy alternative to explicitly using the names you’re using, which is again, exactly the sort of thing linters are supposed to warn you about.