Python3 Function with for loop

Question

I am trying to make a function to iterate over the list. Can anyone let me know and explain that what t is that I am doing wrong?

iplist = ['1.1.1.1', '2.2.2.2', '3.3.3.3', '4.4.4.4']

def ip_addr(addr_list):
    for ip_addresses in addr_list:
        return ip_addresses


test = ip_addr(iplist)

print(test)

I am expecting output:

1.1.1.1
2.2.2.2
3.3.3.3
4.4.4.4

However, I am getting the following output:

1.1.1.1

Thanks,

A return ends the function, and therefore breaks the for-loop — Energya
– Energya, Commented Sep 28, 2019 at 21:45
Replace the return ip_address with print(ip_address)? All your function does is loop over the list anyway — Energya
– Energya, Commented Sep 28, 2019 at 21:52

Bill the Lizard · Accepted Answer · 2019-09-28 21:54:07Z

2

Your function returns the value it encounters in the first iteration of the for loop, causing the loop to terminate immediately. Instead of a loop, join the values of the list together and return them all in one string.

def ip_addr(addr_list):
    return "\n".join(addr_list)

answered Sep 28, 2019 at 21:54

Bill the Lizard

407k213 gold badges579 silver badges892 bronze badges

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

2 Comments

M__ Over a year ago

This is more pythonic

Sherry Over a year ago

This is also a valid solution!!

Timothée Dautrême · Accepted Answer · 2019-09-28 21:59:24Z

0

Your problem is you return at the first iteration of the loop, then you get only the first iteration. Try that:

iplist = ['1.1.1.1', '2.2.2.2', '3.3.3.3', '4.4.4.4']

def ip_addr(addr_list):
    addresses = ""
    for ip_addresses in addr_list:
        addresses += ip_addresses + "\n"
    return addresses  

test = ip_addr(iplist)
print(test)

answered Sep 28, 2019 at 21:59

Timothée Dautrême

3753 silver badges13 bronze badges

Comments

phylogram · Accepted Answer · 2019-09-28 21:53:46Z

0

I am not sure, what you are trying to do, but here are some solutions:

iplist = ['1.1.1.1', '2.2.2.2', '3.3.3.3', '4.4.4.4']

def ip_addr(addr_list):
    for ip_addresses in addr_list:
        yield ip_addresses


test = ip_addr(iplist)

print(*[ip for ip in test])

or just

iplist = ['1.1.1.1', '2.2.2.2', '3.3.3.3', '4.4.4.4']
print('\n'.join(iplist))

answered Sep 28, 2019 at 21:53

phylogram

1331 silver badge7 bronze badges

2 Comments

M__ Over a year ago

Yield is cool, but doesn't help here. The list goes in as a list and yield returns the list. Nothing has changed

phylogram Over a year ago

@MichaelG. I do not know, what he/she is trying to do. Maybe there is more behind it, like some string manipulation?

Physicing · Accepted Answer · 2019-09-28 22:00:49Z

-1

Above to all valid answers, you can also use a dummy empty list and append inside the values and return that dummy list too.

answered Sep 28, 2019 at 22:00

Physicing

5425 silver badges17 bronze badges

Collectives™ on Stack Overflow

Python3 Function with for loop

4 Answers 4

2 Comments

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related