А что возвращает now = f.read()?
Update.
Вы, наверное, хотели что-то вроде такого сделать:
def deadhosts():
f = os.popen('curl \'http://xxxxxxx')
now = f.read()
if now:
l = now.split('\'')
m = 1
result_list = list()
while m < len(l):
res = l[m]
result_list.append(res)
print(res)
m += 4
return result_list
else:
up = "All devices is up"
print(up)
Если что-то пришло (строка не пуста), делим строку по разделителю, и начиная со второго элемента (m=1), каждый четвёртый нужно вывести, если ничего не пришло, то все хорошо.
Правильно?
К сожалению код возвращает только первый элемент списка - 10.177.0.9, хотя должно быть два - [('10.177.0.9', 5, '3'), ('10.177.0.7', 5, '1')]. Да, согласен, вид в котором пребывает эта строка очень неудобен, но так возвращает результат система, которую я опрашиваю. – psiisyapk
Мой код вернул 2 ip адреса. return должен быть после цикла идти, а не внутри его, тогда проблем нет:
10.177.0.9
10.177.0.7
['10.177.0.9', '10.177.0.7']
И я бы на вашем месте все-таки преобразовал строку в объект питона или разобрал регуляркой, пример:
С регулярками:
import re
pattern_ip = re.compile(r'(\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})')
def deadhosts():
# f = os.popen('curl \'http://xxxxxxx')
# now = f.read()
now = "[('10.177.0.3', 5, '3'), ('10.177.0.7', 5, '1')]"
match = pattern_ip.findall(now)
if match:
print('Match: {}'.format(match))
return match
up = "All devices is up"
print(up)
Через преобразование в объект питона:
import ast
def deadhosts():
# f = os.popen('curl \'http://xxxxxxx')
# now = f.read()
now = "[('10.177.0.3', 5, '3'), ('10.177.0.7', 5, '1')]"
if now:
obj = ast.literal_eval(now)
print(obj)
ip_list = [val[0] for val in obj]
print('IP list: {}'.format(ip_list))
return ip_list
up = "All devices is up"
print(up)