我正在尝试创建一个余弦相似性函数,然后在HTML元素中显示结果。我写了以下内容:
function cosinesim(A,B){
var dotproduct=0;
var mA=0;
var mB=0;
for(i = 0; i < A.length;){
dotproduct += (A[i] * B[i]);
mA += (A[i]*A[i]);
mB += (B[i]*B[i]);
}
mA = Math.sqrt(mA);
mB = Math.sqrt(mB);
var similarity = (dotproduct)/(mA)*(mB)
return similarity;
}
//.....
var array1 = [1,0,0,1];
var array2 = [1,0,0,0];
var p = cosinesim(array1,array2);
document.getElementById("sim").innerHTML = String(p);
我已经测试过了,我输入的两个数组都是相同的长度,但是当我的代码运行到这个位置时,它崩溃了,我似乎找不到问题所在。
感谢您的帮助。
function cosinesim(A, B) {
var dotproduct = 0;
var mA = 0;
var mB = 0;
for(var i = 0; i < A.length; i++) {
dotproduct += A[i] * B[i];
mA += A[i] * A[i];
mB += B[i] * B[i];
}
mA = Math.sqrt(mA);
mB = Math.sqrt(mB);
var similarity = dotproduct / (mA * mB);
return similarity;
}
var array1 = [1, 0, 0, 1];
var array2 = [1, 0, 0, 0];
var p = cosinesim(array1, array2);
console.log(p);
这应该给出实际的余弦相似性。你失踪了:
for
循环中的var i
和i++
,如前所述- 这一行中
(mA * mB)
周围的额外括号:var similarity = dotproduct / (mA * mB);
如果没有括号,除法在乘法之前完成
使用映射并减少javascript 的功能
function dotp(x, y) {
function dotp_sum(a, b) {
return a + b;
}
function dotp_times(a, i) {
return x[i] * y[i];
}
return x.map(dotp_times).reduce(dotp_sum, 0);
}
function cosineSimilarity(A,B){
var similarity = dotp(A, B) / (Math.sqrt(dotp(A,A)) * Math.sqrt(dotp(B,B)));
return similarity;
}
var array1 = [1,2,2,1];
var array2 = [1,3,2,0];
var p = cosineSimilarity(array1,array2);
console.log(p);
希望这能有所帮助!!编码快乐!!
您在循环中错过了i++
,这导致了一个无休止的
将for(i = 0; i < A.length;)
替换为for(i = 0; i < A.length;i++)
修复了问题
function cosinesim(A,B){
var dotproduct=0;
var mA=0;
var mB=0;
for(i = 0; i < A.length; i++){
dotproduct += (A[i] * B[i]);
mA += (A[i]*A[i]);
mB += (B[i]*B[i]);
}
mA = Math.sqrt(mA);
mB = Math.sqrt(mB);
var similarity = (dotproduct)/(mA)*(mB)
return similarity;
}
var array1 = [1,0,0,1];
var array2 = [1,0,0,0];
var p = cosinesim(array1,array2);
console.log(p);