How to use multiple conditions in a for loop in a function?

Question

I'm still new to python and have just started learning. The task given is to find the amount of punctuation, vowels, and constants in a given text. But whenever I run the code it just gives me a 0.

def getInfo(text):

    pun = [".", ",", " ", "\'", "\"", "!"]
    vowels = ["a", "e", "i", "o", "u"]

    count = 0
    count2 = 0
    count3 = 0
    for char in text:
        if char in pun:
           count += 1
        return count
    
        if char.lower() in vowels:
           count2 += 1
        return count2
        
        if (not char.lower() in vowels) and (not char.lower() in pun):      
            count3 += 1
        return count3

ashraful16 · Accepted Answer · 2020-08-25 20:33:45Z

1

You are retuning value after checking punctuation, rest of them are ignored. So you are getting 0. Your code is valid for punctuation check.

answered Aug 25, 2020 at 20:33

ashraful16

2,7923 gold badges13 silver badges34 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

score 1 · Accepted Answer · 2020-08-25 20:41:03Z

1

When program reaches return it exits from loop.

def getInfo(text):

    pun = [".", ",", " ", "'", '"', "!"]
    vowels = ["a", "e", "i", "o", "u"]

    count = 0
    count2 = 0
    count3 = 0
    for char in text:
        if char in pun:
            count += 1

        if char.lower() in vowels:
            count2 += 1

        if (not char.lower() in vowels) and (not char.lower() in pun):
            count3 += 1
    return "count: {0}, count2: {1}, count3: {2}".format(count, count2, count3)

print(getInfo("We are in 2020."))

Output:

count: 4, count2: 4, count3: 7

edited Aug 25, 2020 at 20:41

answered Aug 25, 2020 at 20:33

user13959036

Comments

Pygirl · Accepted Answer · 2020-08-25 20:35:49Z

0

Return should be outside the loop.

It should be :

def getInfo(text):

    pun = [".", ",", " ", "\'", "\"", "!"]
    vowels = ["a", "e", "i", "o", "u"]

    count = 0
    count2 = 0
    count3 = 0
    for char in list(text):
        if char in pun:
           count += 1
   
        if char.lower() in vowels:
           count2 += 1
        
        if (not char.lower() in vowels) and (not char.lower() in pun):      
            count3 += 1
    return (count, count2, count3)

getInfo('myname.is hello world!')

# (4, 6, 12)

answered Aug 25, 2020 at 20:35

Pygirl

13.4k6 gold badges36 silver badges48 bronze badges

Comments

José Braz · Accepted Answer · 2020-08-25 20:36:20Z

0

You are using return keyword, that's, all the code below will not run and the function will returns the variable count.

Please, verify this.

answered Aug 25, 2020 at 20:36

José Braz

6957 silver badges10 bronze badges

Collectives™ on Stack Overflow

How to use multiple conditions in a for loop in a function?

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related