Regex pattern выборка совпадений

Question

Некоторые шаблоны строк, которые я использую, не дают мне совпадения во время выполнения, в то время как когда я использую средство проверки регулярных выражений в сети, например Regular Expression Tester, мне показывают, что мой шаблон получает желаемый результат.

public class RegexMatches 
{ 
    public static void main(String args[]){ 
     String input = "[1463895254]PING www.andi.dz (213.179.181.44) 100(128) bytes of data.[1463895254]108 bytes from 213.179.181.44: icmp_seq=1 ttl=54 time=195 ms[1463895255]108 bytes from 213.179.181.44: icmp_seq=2 ttl=54 time=202 ms[1463895256]108 bytes from 213.179.181.44: icmp_seq=3 ttl=54 time=180 ms[1463895257]108 bytes from 213.179.181.44: icmp_seq=4 ttl=54 time=200 ms[1463895258]108 bytes from 213.179.181.44: icmp_seq=5 ttl=54 time=206 ms[1463895259]108 bytes from 213.179.181.44: icmp_seq=6 ttl=54 time=188 ms[1463895260]108 bytes from 213.179.181.44: icmp_seq=7 ttl=54 time=182 ms[1463895261]108 bytes from 213.179.181.44: icmp_seq=8 ttl=54 time=223 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=9 ttl=54 time=187 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=10 ttl=54 time=199 ms"; 

     try{ 
      String op[] = parseIndividualIP(input); 
      System.out.println(op[1]);    
      System.out.println(op[8]);     
     } 
     catch (Exception e){ 
      e.printStackTrace(); 
     } 
    } 

// Doesn't Work... 
    public static String[] parseIndividualIP(String input) throws IPAddressNotFoundException { 
     // Capture the ip-address after 'x byes from' 
     Pattern p = Pattern.compile("bytes\\s+from\\s+([\\d,\\.]+):"); 
     Matcher m = p.matcher(input); 
     if (m.find()){ 
      int i = 0; 
      String[] s = new String[10]; 
      while(m.find()){ 
       s[i] = m.group(++i); 
      } 
      return s; 
     } 
     else 
      throw new IPAddressNotFoundException(); 
    } 

} 

Я не знаю, почему во время выполнения у меня нет совпадений и как я должен отлаживать эту проблему. Поскольку шаблон был перепроверен до и после запуска.

Строка ввода -

[1463895254]PING www.andi.dz (213.179.181.44) 100(128) bytes of data.[1463895254]108 bytes from 213.179.181.44: icmp_seq=1 ttl=54 time=195 ms[1463895255]108 bytes from 213.179.181.44: icmp_seq=2 ttl=54 time=202 ms[1463895256]108 bytes from 213.179.181.44: icmp_seq=3 ttl=54 time=180 ms[1463895257]108 bytes from 213.179.181.44: icmp_seq=4 ttl=54 time=200 ms[1463895258]108 bytes from 213.179.181.44: icmp_seq=5 ttl=54 time=206 ms[1463895259]108 bytes from 213.179.181.44: icmp_seq=6 ttl=54 time=188 ms[1463895260]108 bytes from 213.179.181.44: icmp_seq=7 ttl=54 time=182 ms[1463895261]108 bytes from 213.179.181.44: icmp_seq=8 ttl=54 time=223 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=9 ttl=54 time=187 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=10 ttl=54 time=199 ms 

Regex Pattern Used -

bytes\\s+from\\s+([\\d,\\.]+): 

Answer 1

Основной проблемой является использование m.group(++i). В вашем регулярном выражении у вас есть одна группа захвата, которая захватывает IP-адрес (([\\d,\\.]+)), что означает, что вы должны вызывать m.group(1), так как это возвращает строку, захваченную первой группой.

Объект Pattern является потокобезопасным, поэтому его можно скомпилировать один раз и повторить один и тот же экземпляр.

Следующий код содержит групповое исправление, а также некоторые изменения для удобочитаемости. Изменено использование массивов в связанном списке и вместо исключения исключения возвращает метод и пустой список, если совпадений не найдено.

import java.util.LinkedList; 
import java.util.List; 
import java.util.regex.Matcher; 
import java.util.regex.Pattern; 

public class RegexMatches { 
    private static final Pattern PATTERN = Pattern.compile("bytes\\s+from\\s+([\\d,\\.]+):"); 

    public static void main(String args[]) { 
     String input = "[1463895254]PING www.andi.dz (213.179.181.44) 100(128) bytes of data.[1463895254]108 bytes from 213.179.181.44: icmp_seq=1 ttl=54 time=195 ms[1463895255]108 bytes from 213.179.181.44: icmp_seq=2 ttl=54 time=202 ms[1463895256]108 bytes from 213.179.181.44: icmp_seq=3 ttl=54 time=180 ms[1463895257]108 bytes from 213.179.181.44: icmp_seq=4 ttl=54 time=200 ms[1463895258]108 bytes from 213.179.181.44: icmp_seq=5 ttl=54 time=206 ms[1463895259]108 bytes from 213.179.181.44: icmp_seq=6 ttl=54 time=188 ms[1463895260]108 bytes from 213.179.181.44: icmp_seq=7 ttl=54 time=182 ms[1463895261]108 bytes from 213.179.181.44: icmp_seq=8 ttl=54 time=223 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=9 ttl=54 time=187 ms[1463895263]108 bytes from 213.179.181.44: icmp_seq=10 ttl=54 time=199 ms"; 

     List<String> op = parseIndividualIP(input); 
     op.forEach(System.out::println); 
    } 

    public static List<String> parseIndividualIP(String input) { 
     Matcher m = PATTERN.matcher(input); 
     List<String> ips = new LinkedList<>(); 
     while (m.find()) { 
      ips.add(m.group(1)); 
     } 
     return ips; 
    } 
}