2016-08-13 33 views
0

我写了一个计算卖出货币交易过程(ASK)的平均值和标准差的方法,但我认为这不是定义这种行为的最简单方法,所以我需要一个帮助。从解析的XML文件中计算标准偏差

  1. 在我看来有太多的创建Bigdecimal值。

  2. 我也怀疑使用Arraylist获取值以计算标准偏差,也许这是更好的数据结构来实现该目标。

该方法还包含购买货币兑换课程的平均值(BID在解析文件中),但请不要关注它。

package pl.parser.nbp; 

import java.math.BigDecimal; 
import java.math.RoundingMode; 
import java.util.ArrayList; 
import java.util.List; 

import javax.xml.parsers.DocumentBuilder; 
import javax.xml.parsers.DocumentBuilderFactory; 

import org.w3c.dom.Document; 
import org.w3c.dom.Element; 
import org.w3c.dom.Node; 
import org.w3c.dom.NodeList; 

public class Counting { 
public void countAverageAndStandartDeviaton(String address){ 
    try 
    { 
     DocumentBuilderFactory df = DocumentBuilderFactory.newInstance(); 
     DocumentBuilder db = df.newDocumentBuilder(); 
     Document doc = db.parse(address); 
     doc.getDocumentElement().normalize(); 

     NodeList nList = doc.getElementsByTagName("Rate"); 

     List<BigDecimal> listForStandartDeviation = new ArrayList<>(); 
     BigDecimal averageOfBid = new BigDecimal("0"); 
     BigDecimal averageOfAsk = new BigDecimal("0"); 
     BigDecimal divisor = new BigDecimal(nList.getLength()); 

     for (int temp = 0; temp < nList.getLength(); temp++) { 

      Node nNode = nList.item(temp); 
      if (nNode.getNodeType() == Node.ELEMENT_NODE) { 

       Element eElement = (Element) nNode; 

       BigDecimal valueForBidAverage = new BigDecimal(eElement.getElementsByTagName("Bid").item(0).getTextContent()); 
       BigDecimal valueForStandartDeviation = new BigDecimal(eElement.getElementsByTagName("Ask").item(0).getTextContent()); 
       averageOfBid = averageOfBid.add(valueForBidAverage); 
       averageOfAsk = averageOfAsk.add(valueForStandartDeviation); 
       listForStandartDeviation.add(valueForStandartDeviation); 


      } 
     } 
     averageOfBid = new BigDecimal(averageOfBid.divide(divisor).toString()).setScale(4, RoundingMode.HALF_UP); 
     averageOfAsk = new BigDecimal(averageOfAsk.divide(divisor).toString()); 
     System.out.println(averageOfBid + " - BID Average"); 

     BigDecimal sumStandartDeviation = new BigDecimal("0"); 
     for(int i = 0 ; i<listForStandartDeviation.size(); i++){ 
      BigDecimal valueFromList = new BigDecimal(listForStandartDeviation.get(i).toString()); 
      sumStandartDeviation = sumStandartDeviation.add((valueFromList.subtract(averageOfAsk)).pow(2)); 
     } 
     sumStandartDeviation = sumStandartDeviation.divide(divisor); 
     sumStandartDeviation = new BigDecimal(Math.sqrt(sumStandartDeviation.doubleValue())) 
       .setScale(4, RoundingMode.HALF_UP); 
     System.out.println(sumStandartDeviation + " - ASK Standart Deviation"); 

    } 
    catch (Exception e) 
    { 
     e.printStackTrace(); 
    } 
} 
} 
+2

这个问题可能会更恰当地贴在[代码审查堆栈交易所网站(HTTP://代码审查。 stackexchange.com/)。 –

+0

谢谢你,我也在codereview网站发布 – glinczux

回答

0

你为什么要做这么多的使用BigDecimal的,而不是双重算术计算的,当你最终回复到一倍的运算来计算平方根反正这不是我清楚。

这里是一个XPath 3.1解决方案(这当然可以从Java方便地调用):

let $rates := //Rate 
return (avg($rates/Bid), math:sqrt(sum($rates/(Ask*Ask))))