Не могу распарсить результат работы программы, запутался с алгоритмом?

Question

[User .]

Я не очень силен в программировании, в python тем более.

Моя задача:
Есть txt файл в котором содержится результат работы masscan
Требуется вывести каждый ip со всеми найденными его открытыми портами :

res.txt
Код:

# Masscan 1.0.3 scan initiated Sun Dec 23 23:00:31 2018
# Ports scanned: TCP(1;80-80) UDP(0;) SCTP(0;) PROTOCOLS(0;)
Host: 192.168.1.1 () Ports: 80/open/tcp////
Host: 192.168.1.1 () Ports: 801/open/tcp////
Host: 192.168.1.2 () Ports: 801/open/tcp////
Host: 192.168.1.2 () Ports: 445/open/tcp////
Host: 192.168.1.3 () Ports: 80/open/tcp////
Host: 192.168.1.3 () Ports: 8080/open/tcp////
Host: 192.168.1.3 () Ports: 21/open/tcp////
# Masscan done at Sun Dec 23 23:00:45 2018

Моя задача получить данные в таком формате.

192.168.1.1 80, 801
192.168.1.2 801, 445
192.168.1.3 80, 8080, 21

#!/usr/bin/env python
import sys, re, os

ports = []
ip = None

with open('mres.txt','r') as f:

    for elem in f.readlines():

        if not elem.startswith('#'):
              if ip != None:
                  print('1')
                  if elem.split(' ')[1] == ip:
                      ports.append(elem.split(' ')[3].split('/')[0])
                      continue
                  else:
                      print(ports)
                      print('nmap '+ip +' ports: '+str(ports))
                      ports=[]
              else:
                  print('2')
                  #print('Unic: '+str(ip) + ' port: '+ str(elem.split(' ')[3].split('/')[0]))
                  ip = elem.split(' ')[1]
                  ports.append(elem.split(' ')[3].split('/')[0])

Answer 1

[AWEme]

Набросал чутка, пробуйте.

import re
from collections import defaultdict

ips = defaultdict(list)
regular = re.compile(r'Host: ([\d\.]+).+?Ports: (\d+)/')

with open('res.txt', 'r') as f:
    for line in f:
        line = line.strip()
        if not line.startswith('#'):
            ip, port = regular.search(line).groups()
            ips[ip].append(port)

for k, v in ips.items():        # Выведет:
    print(k, ', '.join(v))      # 192.168.1.1 80, 801
                                # 192.168.1.2 801, 445
                                # 192.168.1.3 80, 8080, 21

with open('outputfile.txt', 'w') as f: # Запишет тоже самое
    for k, v in ips.items():
        f.write('{} {}\n'.format(k, ', '.join(v)))

Comments

[longclaps]

хехе, аналогично

import re
from collections import defaultdict

d = defaultdict(list)
p = re.compile(r'^\s*Host: (?P<host>\d{1,3}\.\d{1,3}\.\d{1,3}'
               r'\.\d{1,3}) \(\) Ports: (?P<port>\d+)/.*').match

with open('data.txt', 'r') as f:
    for s in f:
        m = p(s)
        if m is not None:
            d[m.group('host')].append(m.group('port'))

for host, ports in d.items():
    print(host, end=':')
    print(*ports, sep=',')

[sim3x]

longclaps, if m:

[longclaps]

sim3x, привычка: проверка на is/is_not чуть быстрее.

[sim3x]

longclaps,

In [1]: x = None

In [2]: %timeit if not x: pass
10000000 loops, best of 3: 42.4 ns per loop

In [3]: %timeit if x is None: pass
10000000 loops, best of 3: 52.5 ns per loop

In [4]: %timeit if x: pass
10000000 loops, best of 3: 39.1 ns per loop

In [5]: %timeit if x is not None: pass
10000000 loops, best of 3: 51.3 ns per loop

?

[longclaps]

sim3x, да, я стал жертвой умозрительных построений (

[User .]

Большое всем спасибо!!!!

[sim3x]

AWEme, ваш ответ не есть решение
Комент longclaps - решение

[AWEme]

sim3x, с чего вы это взяли?
Моё решение работает не верно/не рабоатет?

[sim3x]

AWEme, сравните ваше решение с тем, что ниже

Подумайте, почему ваше вариант можете не сработать на большом файле

[AWEme]

sim3x, да, читать файл целиком - не лучшая практика, но в данном случае я не вижу в этом проблемы, да и вопрос не о том, как считывать большие файлы.
А маэстро longclaps может оформить своё решение как полноценный ответ и получить всё, что ему причетается.