解析用逗号分隔的文件，并等号

Question

我有一个看起来像如下

name=helium,sign=he,number=2 
name=lithium,sign=li 
name=gold,sign=au,number=79 

名称，标志和数量都在地图中密钥的文件。如果某个键没有列出，它应该变成“未识别”值。现在我想解析这个逗号/等号分隔的文件，但是我没有成为我需要的结果，也找不到和改进错误。

这是我到目前为止有：

static Map<Table, String> periodicTable = new LinkedHashMap<Table, String>(); 
String[] list = fileToString(args[0]); // fileToString is to become the file content as a String array, where in the first field of the array is saved the first line of the file, in the second field - the second line and so on. 
for (int i = 0; i < list.length; i++) { 
    String[] element = list[i].split(","); 

    for (int j = 0; j < element.length; j++) { 
     String[] feature = element[j].split("="); 
     if (feature[0].equals("name") 
      || feature[0].equals("sign") || feature[0].equals("number")) { 
      feature[1] = periodicTable.get(feature[0]); 
     } else { 
      feature[1] = "unspecified"; 
     } 
    } 
} 


public class Table { 
private String name; 
private String number; 
private String sign; 

public Table(String name, String number, String sign) { 
    this.name = name; 
    if (Integer.parseInt(number) >= 1 && Integer.parseInt(number) <= 118) { 
     this.number = number; 
    } 
    this.sign = sign; 
    } 
} 

这将是巨大的，如果有人可以帮助我这一个。提前致谢！

Answer 1

不是一个解决方案，但你的代码的一些意见：

1. 您解析文件中的代码是从主要的方法，我认为你错过了它，而不是分裂的地方

2. 使用的StringTokenizer ：

   StringTokenizer tokenizer = new StringTokenizer（line，“，”）; （tokenizer.hasMoreTokens（））{ String element = tokenizer.nextToken（）; }

3. 使用的foreach的风格，而不是编入索引：

   为（串行：list）{上线 }

4. 尝试使用 //富 “等于” 常量字符串的方法代替变量：

   如果（ “姓名” .equals（特征[0]））{}

   因为变量可以为空，瓦特这可能会破坏你的代码。

Answer 2

使您的方法更简洁的一个相对标准的方法是将元素属性封装在enum中。这样你可以很容易地扩展你正在存储的属性。它也避免了你的丑陋if陈述。我已将Table更改为Element，因为对我而言，Table将成为Element的列表。

public class Element { 
    enum Attribute { 
     NAME ("name", Element::setName), 
     ATOMIC_NUMBER("number", Element::setNumber), 
     SIGN("sign", Element::setSign); 

     private final String attributeName; 
     private final ValueSetter valueSetter; 

     Attribute(String elementName, Element.ValueSetter valueSetter) { 
      this.elementName = elementName; 
      this.valueSetter = valueSetter; 
     } 

     public static void setElementValue(Element element, String attributeValue) { 
      String[] attributeValuePair = attributeValue.split("="); 
      String attributeName = attributeValuePair[0]; 
      String value = attributeValuePair[1]; 
      values().stream().filter(attributeName::equals) 
       .forEach(attr -> attr.valueSetter.setValue(element, value)); 
     } 
    } 

    interface ValueSetter { 
     void setValue(Element element, String value); 
    } 

    private Element() { 
     // set default vaues 
    } 

    private static setName(Element element, String name) { 
     element.name = name; 
    } 

    private static setSign(Element element, String sign) { 
     element.sign = sign; 
    } 

    private static setNumber(Element element, String number) { 
     element.number = Integer.parseInt(number); 
    } 

    public static Element readFromLine(String line) { 
     Element element = new Element(); 
     line.split(",").stream() 
      .forEach(attrVal -> Attribute.setElementValue(element, attrVal)); 
    } 

    // same for setNumber, setSign 
} 

那么你的阅读行代码可以是相当多的整洁：

List<Element> table = lines.stream() 
    .map(Element::readFromLine) 
    .collect(Collectors.toList()); 

我已经跳过了很多错误检查在这里，但希望你的想法。