не удалось переместить элемент между? - PullRequest
Новая
       5

не удалось переместить элемент между?

1 голос
/ 23 марта 2020

Я пытался решить судоку с помощью Blacktracking al go, все хорошо, вызывается canvar, и я могу видеть число, но все равно число не движется, т.е. логика c не работает current.i === 0; - вот где я получаю ошибку! даже я объявил sperate переменную для num, также проблема не решена. только если я удаляю .num current == 0, тогда он не показывает никаких ошибок, но номер все еще не движется

введите описание изображения здесь 

var cell = [];
var stack = [];
var sudoku =  [2,3,0,9,4,0,6,7,0,
               8,0,0,3,2,5,9,1,4,
               9,0,0,7,6,0,3,2,0,
               1,0,0,0,0,0,7,9,2,
               5,0,3,2,1,0,4,8,6,
               4,0,0,6,8,0,5,3,1,
               7,0,0,1,0,0,0,0,9,
               6,5,9,8,7,2,1,4,3,
               3,0,0,0,9,0,0,0,7];
var current;
var number = 1;

function setup(){
    createCanvas(450,450);
    var a=0;
    var b=0;
    for(var i=0;i<81;i++){
        if(a%9==0 && i!=0){
            b = b+50;
            a = 0;
        }
        each[i] = new each(a,b,i,sudoku[i]);
        a = a+50;
    }
    current = cell[0];
}

function draw(){
    background(10);
    for(var i=0;i<81;i++){
        each[i].show();
    }

    if(current.num === 0){ //the error is typeerror can't read the property of num 

        if(! sameColumn(current.i,number) && ! sameRow(current.i,number) && ! sameSquare(current.i,number) && number<(10)){
            current.num = number;
            stack.push(current);
            number = 0;
            current.each[current.i+1];
        }
        else {
            if(number > 8){
            current.num = 0;
            current = stack.pop();
            number = current.num;
            current.num = 0;
        }
        } 
    }
    else{
        current = each[current+1];
        number = 0;
    }
    number++;
}

function each(a,b,i,num){
        this.a = a;
        this.b = b;
        this.i = i;
        this.num = num;
        this.show = function(){
            noFill();
            stroke(255);
            rect(this.a,this.b,50,50);
            textSize(32);
            text(this.num,a+12,b+40);
        }
    }

1 Ответ

2 голосов
/ 23 марта 2020

Ошибка довольно прямолинейна. current = cell[0]; становится неопределенным, поскольку вы определили cell как пустой массив и не манипулировали им после этого.

Из того, что я наблюдал до сих пор, многие части вашего кода логически не работают, например, 

same Column(current.i,number) && ! sameRow(current.i,number) && ! sameSquare(current.i,number)

определенно выдаст вам ошибку, если она выполнена (это не так поскольку выполнение не доходит до этой строки), если только у вас нет отдельного файла js, содержащего эти функции.

Еще один 

current = cell[current+1];

, если переменная current чтобы сохранить объект ячейки, не имеет смысла добавлять 1 к нему, и наоборот.

Теперь я считаю, что так должна выглядеть функция установки: 

function setup(){
    createCanvas(450,450);
    var a=0;
    var b=0;
    for(var i=0;i<81;i++){
        if(a%9==0 && i!=0){
            b = b+50;
            a = 0;
        }
        cell[i] = new Cell(a,b,i,sudoku[i]); //changed each[i] to cell[i], also renamed the 'each' class
        a = a+50;
    }
    current = cell[0];
}

Если возможно, пожалуйста, отредактируйте немного больше информации о том, что именно делает ваш код. Ура:)

...