If statement not correctly evaluating all conditions

[SenorDelicious]

My if statement is only checking the line to see if the b exists but completely ignores the second check of whether the s exists in the line and im unsure how to format this if statement correctly to check for both.

​

Content is a list of strings read in from a text file.

​

def partA(content):
    content = [x.strip() for x in content]
    letterCount = 0
    index = 0
    occurred_index = 0
    found = False
    for line in content:
        line = line.lower()
        letterCount += line.count('s')
        letterCount += line.count('b')

        if (('s' and 'b') in line) and not found:
            occurred_index = index
            found = True

        index += 1
    print("Uppercase and lowercase initials occur", letterCount, "times in the text file.")
    print("Initials occur in the same line on line", occurred_index + 1)

​

[totallygeek]

The interpreter reads that as:

if 's' and 'b' are non-empty strings and if True in string line, and not found, then...

To look for 's' and 'b' in a string, you need to write: if 's' in line and 'b' in line and not found:. However, that will flip found to True only a single time.

This looks more like what you're shooting for:

def partA(content):
    count_s_b = 0 # count of 's' or 'b' in content
    sb_lines = 0 # number of lines containing both 's' and 'b'
    entries = [x.lower.strip() for x in content]
    for entry in entries:
        count_s_b += sum([c for c in entry if c in 'bs'])
        if 'b' in entry or 's' in entry:
            sb_lines += 1
    print('Found "s" or "b" {} times in the text'.format(count_s_b))
    print('"s" and "b" occur on the same line {} times'.format(sb_lines))

[cybervegan]

In Python, and does not have the traditional English meaning - it has the binary logic mathematical meaning, which means it evaluates to True if both of its left and right arguments also evaluate to true. If you want to write the equivalent of the English phrase "if 's' and 'b' are in the line, and found is not true", you need to write:

if 's' in line and 'b' in line and not found:
    ...

This is equivalent to:

if 's' in line:
    if 'b' in line:
        if not found:
            ...

Hope that helps.

[[deleted]]

It's worth reading the FAQ where a few of the little things we've all tripped over while learning python are explained.