Iterable can be either a string or a sequence¶
ID: py/iteration-string-and-sequence Kind: problem Severity: error Precision: high Tags: - reliability - maintainability - non-local Query suites: - python-security-and-quality.qls
for statement is designed to allow you to iterate over the elements of a sequence or other iterable object. Strings in Python are iterable, and often used as such. However, they are also often considered, not as sequences of characters, but as atomic entities.
One source of defects in Python is mistakenly iterating over a non-iterable object such as an integer. This sort of defect is easily detected as a
TypeError will be raised. However, if a string is mistakenly used as the iterable in a
for statement, which also receives other sequences (such as lists) then the code will iterate over the string one character at a time. This is probably not what the programmer intended and results in errors that are hard to find.
Since this defect usually indicates a logical error, it is not possible to give a general method for addressing the defect. However, adding a guard that checks that the iterator is not a string could be worthwhile.
In this example, the loop may iterate over
"Hello" producing one character per line, as well as over
[ "Hello", "World" ] It is likely that the programmer forgot to wrap the
"Hello" in brackets.
#Mistakenly mixed list and string def greeting(): if is_global(): greet = [ "Hello", "World" ] else: greet = "Hello" for word in greet: print(word) #Only use list def fixed_greeting(): if is_global(): greet = [ "Hello", "World" ] else: greet = [ "Hello" ] for word in greet: print(word)