错误 - =和网络工作者

时间:2018-04-30 21:22:04

标签: javascript web-worker

我正在用JS开发一个小游戏,我正在使用很多网络工作来管理我的敌人。 这是我的ennemis.js代码(这是我的网络工作者):

var xEnnemis, nbrEnnemis;
var x=0;
//Seul objectif : calculer les positions des ennemis.
onmessage = function(e) {
    var idEnnemis = String(e.data[0]);
    var recXEnn = String(e.data[1]);




    xEnnemis = recXEnn;
    var recSpeedEnn = String(e.data[2]);
    vitesseEnnemis = recSpeedEnn;

    animationEnnemisInterval = setInterval(calculateEnnemisPos, 120); //60

    function calculateEnnemisPos(){
            if (xEnnemis>0){
                xEnnemis -= vitesseEnnemis;
            }else{
                xEnnemis=0;
            }
        //console.log("Send datas + "+ idEnnemis);
        postMessage([idEnnemis, xEnnemis]);
    } 
}

这是我的主要脚本

if (typeof(Worker) !== "undefined") {console.log("Tu supportes les workers  !")} else {console.log("Sorry bro...");} 
var canvas = document.getElementById("canvas");
var ctx = canvas.getContext('2d');
//ENEMIE
var nbrnnemis = 3; //Math.floor(Math.random() * (3 - 5 + 1) ) + 5;
var ennemie = new Image();
ennemie.src = "media/zombie.png";
var xEnnemisGen = [];
var vitesseEnnemis = [];
var posEnnemisrecue = [];
var ennemisRecu = 0;
var animZombie=0;
for (i=0;i<nbrEnnemis; i++){
    var w = new Worker('ennemis.js');
    xEnnemisGen[i] = 500;
    //xEnnemisGen[i] = Math.floor(Math.random() * (1401 - 1290 + 1) ) + 1281;
    vitesseEnnemis[i] = Math.floor(Math.random() * (4 - 10 + 1) ) + 4;
    //console.log("Envoyé au worker : " +i + " " + xEnnemisGen[i] + " " + vitesseEnnemis[i] );
    w.onmessage = test;
    w.postMessage([i, xEnnemisGen[i], vitesseEnnemis[i]]);
}
console.log("Nombres d'ennemis : " + nbrEnnemis);
//console.log(xEnnemis);
function test(e){
    var id = String(e.data[0]);
    var position = String(e.data[1]);
    //console.log("Just received "+e.data+ " from Worker !"); //fait ramer de ouf
    console.log("Recu la position: "+position+" avec l'id: " +id);
    posEnnemisrecue.push(position);
    ennemisRecu++;
    if (ennemisRecu == nbrEnnemis){
        if (animZombie==23){animZombie=0;}else{animZombie+=1;}
        for (i=0;i<nbrEnnemis;i++){
            ctx.drawImage(ennemie, animZombie*118,0, 118,177, posEnnemisrecue[i], 500, 118,177);
            console.log("c'est dessiné + "+ posEnnemisrecue[i]);
        }
        posEnnemisrecue=[];
        ennemisRecu = 0;
        //ctx.clearRect(0,0,1280, 720);
        ctx.drawImage(ennemie, animZombie*118,0, 118,177, 0, 0, 118,177);
        console.log(animZombie);
        console.log("dessiné " + animZombie);
    }
}

但有时我的网络工作者发送的xEnnemis的值不会减少;它甚至随着我们在这个日志中看到而增加:

dessiné 6
Recu la position: 500 avec l'id: 2
Recu la position: 0 avec l'id: 1
Recu la position: 0 avec l'id: 0
c'est dessiné + 500
c'est dessiné + 0
7
dessiné 7
Recu la position: 500 avec l'id: 2
Recu la position: 0 avec l'id: 1
Recu la position: 0 avec l'id: 0
c'est dessiné + 500
c'est dessiné + 0

你知道为什么吗? 谢谢!

编辑:可能是经过C.Champagne版本测试后的第四次

好的,刚刚测试过,似乎没有用。字符串不是答案。无论如何,谢谢你的帮助!

新的日志:

dessiné 15
Recu la position: 0 avec l'id: 0
Recu la position: 2100 avec l'id: 1
Recu la position: 0 avec l'id: 2
c'est dessiné + 0
c'est dessiné + 2100
c'est dessiné + 0
16
dessiné 16
Recu la position: 0 avec l'id: 0
Recu la position: 2101 avec l'id: 1
Recu la position: 0 avec l'id: 2
c'est dessiné + 0
c'est dessiné + 2101
c'est dessiné + 0
17
dessiné 17
Recu la position: 0 avec l'id: 0
Recu la position: 2102 avec l'id: 1
Recu la position: 0 avec l'id: 2
c'est dessiné + 0
c'est dessiné + 2102
c'est dessiné + 0
18

正如你所看到的,它甚至在增加。 我想知道这不是我的主脚本的test(e)函数的错误,这个函数被一次调用太多次并做了什么。

刚刚在Chrome上测试过,看看它是不是导航器错误,而且它也在做同样的事情。如果你有任何想法,因为我被卡住了。 提前谢谢!

编辑:2018年5月7日

好的伙计们,刚发现!这是因为

vitesseEnnemis[i] = Math.floor(Math.random() * (4 - 10 + 1) ) + 4;

有时会返回错误的值!我不知道为什么,但这里有一些结果

Array [ -1, 0, -1 ]

如你所见,我会继续调查。 如果你有同样的问题,请评论这篇文章!

提前致谢!

Escatrag

2 个答案:

答案 0 :(得分:1)

正如我(和其他人)所怀疑的那样,你使用字符串代替数字似乎会产生问题。

使用字符串进行计算是一个坏主意,您可能会遇到意外行为(连接而不是添加...)。

我更改了Worker

中的初始化初始化
var idEnnemis = String(e.data[0]);
var recXEnn = String(e.data[1]);

xEnnemis = recXEnn;
var recSpeedEnn = String(e.data[2]);

...通过

var idEnnemis = e.data[0];
var recXEnn = e.data[1];

xEnnemis = recXEnn;
var recSpeedEnn = e.data[2];

......事情似乎好转了。

答案 1 :(得分:0)

Sooooo,在更准确地看待这条线之后

vitesseEnnemis[i] = Math.floor(Math.random() * (4 - 10 + 1) ) + 4;

我决定检查语法是否正确。 MDN link在这里拼写正确的语法。我的代码应该是

而不是我的行
vitesseEnnemis[i] = Math.floor(Math.random() * (4 - 1 + 1) ) + 1;

现在一切正常,我把问题解决了。

感谢您的帮助!