Clojure:java互操作,为什么这段代码不起作用?

时间:2016-07-07 12:44:54

标签: java clojure hashmap java-interop

我使用Apache Maths编写了一个类(无论如何),我有一个奇怪的故障(或者我的Java代码中可能存在问题?)。

这是代码

package distributions;

import java.util.HashMap;
import java.util.Map;
import java.util.Random;
import org.apache.commons.math3.distribution.AbstractIntegerDistribution;
import org.apache.commons.math3.util.FastMath;

public class CustomDistribution extends AbstractIntegerDistribution {

    long[] v;
    double[] p;
    double[] cp;
    Map<Long, Double> mp = new HashMap<Long, Double>();
    Map<Long, Double> mcp = new HashMap<Long, Double>();

    public CustomDistribution (long[] values, double[] probabilities) {
        v = values;
        p = probabilities;

        int len = values.length;

        for (int i = 0; i < len; i++) {
            mp.put(v[i], p[i]);
        }

        cp = new double[len];
        cp[0] = p[0];
        for (int i = 1; i < len; i++) {
            cp[i] = cp[i-1] + p[i];
        }

        for (int i = 0; i < len; i++) {
            mcp.put(v[i], cp[i]);
        }
    }

      public Map<Long, Double> getMCP () {return mcp;}

    @Override
    public double cumulativeProbability(int v) {
        return mcp.get(v);
    }

    // Not implemented, can be done in Clojure code
    @Override
    public double getNumericalMean() {
        return 0;
    }

    // Not implemented, can be done in Clojure code
    @Override
    public double getNumericalVariance() {
        return 0;
    }

    @Override
    public int getSupportLowerBound() {
        return 0;
    }

    @Override
    public int getSupportUpperBound() {
        return 0;
    }

    @Override
    public boolean isSupportConnected() {
        return false;
    }

    @Override
    public double probability(int v) {
        return mp.get(v);
    }

    // Uses a naive search implementation, should be ok due to data size
    public int sample () {

        double r = FastMath.random();
        int len = p.length;
        Boolean flag = false;
        int i = 0;
        int result = -1;

        while (i < len && flag == false) {
            if (cp[i] < r) {
                i = i + 1;
            }
            else {
                result = (int) v[i];
                flag = true;
            }
        }
        return result;
    }
    }

Clojure包装:

(defn create-distribution
  "Creates a distribution from empirical data"
  [data]
  (let [values (long-array (data :values))
        probabilities (double-array (data :probabilities))]
    (CustomDistribution. values probabilities)))

(create-distribution {:values [1 2 3 4] :probabilities [0.3 0.2 0.2 0.3]})

基本上它是一个包含数据的分布,有两种形式:HashMaps和Arrays。

在Clojure中,我尝试过:

(.probability dist 4)
(.probability dist (int 4))

两者都返回NullPointerException,NativeMethodAccessorImpl..blabla .cumulativeProbabilityMethod也是如此 (另一方面,.sample工作正常,因此可能是HashMap的问题)

我发现可能在构造函数中mcp和mp(即HashMaps)在构造函数中没有正确计算但是当我尝试时:

(.get (.getMCP dist) 4) 

它会返回正确的结果。 MCP hashmap实际上是我期望的结果。

那为什么我的代码不起作用?我的两行是&#34;功能&#34;相同。 如果我输错了,我就看不到了。 也许是因为不变性的事情?

谢谢!

1 个答案:

答案 0 :(得分:1)

好的,所以我想出了问题,我忘了Java没有像Clojure那样灵活的数字原语。

此代码有效:

@Override
public double cumulativeProbability(int v) {
    return mcp.get((long) v);
}

@Override
public double probability(int v) {
    return mp.get((long) v);
}

在Clojure代码中,double的翻译是自动的。 在Java中,我传递了一个Integer,其中HasMap期望Long