Javascript | Match3 Fehlersuche - C, C++, C#, Java und andere Programmiersprachen

Javascript | Match3 Fehlersuche

Werbeanzeige

Chodot

Frischling

Beiträge: 14

14.05.2018, 14:51

Javascript | Match3 Fehlersuche

Guten Tag,
Ich arbeite gerade an einem Match3 Spiel und habe folgendes Problem.
Bei der einen Funktion, wo zwei Blöcke miteinander vertauscht werden sollen, ist ein Fehler.
Es werden nicht immer die gewünschten Blöcke vertauscht.

Javascript-Quelltext

function swap(fieldC,row,col,dir){
 var row2,col2,
     fieldCopy=JSON.parse(JSON.stringify(fieldC));
    //get second position
    switch(dir){
      case 't':row2=row-1;col2=col;break;
      case 'b':row2=row+1;col2=col;break;
      case 'l':row2=row;col2=col-1;break;
      case 'r':row2=row;col2=col+1;
    }
    //swap
    var tmp=fieldCopy[row][col];
    fieldCopy[row][col]=fieldCopy[row2][col2];
    fieldCopy[row2][col2]=tmp;

        return fieldCopy;
    }
}

/*** START HERE ***/
var cursorX=3,cursorY=3;
var field=createField(8,5); //creates 8*8 Matrix with 5 different Values
var tmp=swap(field,cursorX,cursorY,'t');
if(tmp){
  field=tmp;
  cursorY--;
}
mode=0;

Ich habe jetzt mal nur einen kleinen unvollständigen Ausschnitt reingepackt.
An sich funktioniert das Programm, nur halt eben nicht bei jeder Eingabe(Pfeiltasten) wie gewünscht.
Ich bin diesen Codeabschnitt bereits mehrere male durchgegangen, auch mit Papier und Bleistift und es müsste auch alles richtig funktionieren, aber das tut es nicht.
Da ich in einem Emulator programmiere, gibt es auch keine Konsole, die Fehler anzeigt.

Falls sich jemand das ganze unsauber geschriebene Programm antun möchte:
- Programmiert wird in einem speziellen Emulator (https://tic.computer)

Inhalt anzeigen/verstecken

Javascript-Quelltext

const clientW= 240,
                        clientH= 136,
                        dias= 5; //Amount of the Diamonds

var cursorX= 3,
                cursorY= 3,
                mode=0,
                pause=0;

var console={
 data: [],
    log: function(txt){
     if(typeof txt === "number")
         txt=txt.toString();
     this.data.push(txt);
    },
    out: function(y){
        var output=this.data.toString();
     print(output,0,y);
    }
}
function createField(len,val){
 var field=[];
 var row,col,tmp=0;
 for(row=0;row<len;row++){
    field[row]=[];
     for(col=0;col<len;col++){
         tmp=Math.floor(Math.random()*val);
            field[row][col]=tmp;
        }
    }
    return field;
}
function checkField(){}

function swap(fieldC,row,col,dir){
 var row2,col2,
        fieldCopy=JSON.parse(JSON.stringify(fieldC));
    //get second position
    switch(dir){
     case 't':row2=row-1;col2=col;break;
        case 'b':row2=row+1;col2=col;break;
        case 'l':row2=row;col2=col-1;break;
        case 'r':row2=row;col2=col+1;
    }
    //swap
    var tmp=fieldCopy[row][col];
    fieldCopy[row][col]=fieldCopy[row2][col2];
    fieldCopy[row2][col2]=tmp;
    
    console.log('r:'+row+'.c:'+col+'\n');
    console.log('r2:'+row2+'.c2:'+col2+'\n');
    
    //if checkSection -> true
    var marked=[];tmp=0;
    tmp=checkSection(fieldCopy,row,col);
    if(tmp)
  marked=marked.concat(tmp);
 tmp=checkSection(fieldCopy,row2,col2);
    if(tmp)
     marked=marked.concat(tmp);
 
    //if marked == 0 -> swap back & return
    if(!marked[0]){
     return;
    }
    //retrun field
    else{
     fieldCopy=replace(fieldCopy,marked,-1);
        return fieldCopy;
    }
}

//TODO
function checkSection(fieldC,row,col){
 var tmp=[],marked=[],counter=0;
    var fieldCopy=JSON.parse(JSON.stringify(fieldC)),
     num=fieldCopy[row][col]; 
    //checkRows
    for(var i=0;i<5;i++){
     //avoid 'Out of Bounds' error
     if((row>1)||(i>1))
        if((row<fieldCopy.length-2)||(i<3))
    
     if(fieldCopy[row+i-2][col]===num){
         counter++;
            if(counter<4)tmp.push([row+i-2,col]);
            switch(counter){
          case 3:
              marked=tmp;
                 break;
            case 4:
              marked.push([row+i-2,col]);
                 break;
             case 5:
              marked.push([row+i-2,col]);
         }
        }
        else{
         counter=0;tmp=[];
        }
    }
    counter=0;tmp=[];//reset 
    //checkCols
    for(var i=0;i<5;i++){
      if(fieldCopy[row][col+i-2]===num){
          counter++;
            if(counter<4)tmp.push([row,col+i-2]);
            switch(counter){
          case 3:
     if(marked[0]){
      if(i===2){
       marked.push([row,col-2]);
       marked.push([row,col-1]);}
      if(i===3){
       marked.push([row,-1]);
       marked.push([row,col+1]);}
      if(i===4){
       marked.push([row,col+1]);
       marked.push([row,col+2]);}
     }else marked=tmp;
                 break;
                case 4:
                case 5:
                 marked.push([row,col+i-2]);
            }
        }
        else{
          counter=0;tmp=[];
        }
    }
  //retrun
    if(marked[0]){
   return marked;}
    else
   return;
}

function replace(fieldC,markedC,val){
 fieldCopy=JSON.parse(JSON.stringify(fieldC));
    markedCopy=JSON.parse(JSON.stringify(markedC));
    
    for(var i=0;i<markedCopy.length;i++){
     fieldCopy[markedCopy[i][0]][markedCopy[i][1]]=val;
    }
    return fieldCopy;
}

function shift(){}

function drawField(fieldC){
 var row,col,num=0,
        startX=50, startY=3,
                    fieldCopy=JSON.parse(JSON.stringify(fieldC));
    //drawField
 for(row=0;row<fieldCopy.length;row++){
     for(col=0;col<fieldCopy[0].length;col++){
            num=fieldCopy[row][col];
            spr(num,startX+col*17,startY+row*16,1,2,0,0,1,1);
        }
    }
    //drawCursor
    var c=6;if(mode)c=8;
    rectb(startX-2+cursorX*17,startY-3+cursorY*16,18,18,c)
}

// ****** Here start ******
cls();var x=0,y=0;
var field=createField(8,dias);
drawField(field);
console.out(clientH-8);

function TIC()
{
 //if interrupt
 for(var i=0;i<6;i++)if(btn(i)&&!pause){pause+=8;
     //update
        switch(i){
         case 0://top
             if(!mode)if(cursorY>0)cursorY--;else;
                else{//if mode
                 var tmp=swap(field,cursorY,cursorX,'t');
                    if(tmp){
                     field=tmp;
                        cursorY--;
                    }
                    mode=0;
                }
             break;
            case 1://bottom
             if(!mode)if(cursorY<7)cursorY++;else;
                else{//if mode
                 var tmp=swap(field,cursorY,cursorX,'b');
                    if(tmp){
                     field=tmp;
                        cursorY++;
                    }
                    mode=0;
                }
             break;
            case 2://left
             if(!mode)if(cursorX>0)cursorX--;else;
                else{//if mode
                 var tmp=swap(field,cursorY,cursorX,'l');
                    if(tmp){
                     field=tmp;
                        cursorX--;
                    }
                    mode=0;
                }
             break;
            case 3://right
             if(!mode)if(cursorX<7)cursorX++;else;
                else{//if mode
                 var tmp=swap(field,cursorY,cursorX,'r');
                    if(tmp){
                     field=tmp;
                        cursorX++;
                    }
                    mode=0;
                }
             break;
            case 5://change mode
             if(!mode)mode=1;else mode=0;
        }
        //draw
        cls();
        drawField(field);
        console.log('x:'+x);
        console.log('y:'+y);
        //console.out();
        print(cursorX,0,100);
        print(cursorY,10,100);
    }
    if(pause>0)pause--;
}

Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von »Chodot« (16.05.2018, 20:43)

Zum Seitenanfang

DNKpp

Alter Hase

Beiträge: 635

14.05.2018, 17:32

Ist mir jetzt persönlich zu wenig Fehlerbeschreibung. Bei welchen Eingaben passiert denn was genau, was du nicht erwartet hast? Ich würde jetzt einfach mal ins Blaue hinein tippen, dass du da irgendwo einen out of bounds access hast.

EDIT: Zumal du auch nur in deinem 't' case swapst. Hast du den Rest vergessen?

Zum Seitenanfang

Chodot

Frischling

Beiträge: 14

14.05.2018, 18:46

Zitat von »DNKpp«

Ist mir jetzt persönlich zu wenig Fehlerbeschreibung.

Du hast vollkommen recht. Ich weiß einfach nicht, wie ich das besser formulieren soll.

Zitat von »DNKpp«

EDIT: Zumal du auch nur in deinem 't' case swapst. Hast du den Rest vergessen?

Vergessen nicht, nein. Wollte ich hinterher nachholen, auch das mit der Out of Bounds Exception.

Aber ich glaube, das Problem liegt in dem folgendem Abschnitt, ich mache es auch ganz kurz:

	Javascript-Quelltext
1 2 3 4	//swap var tmp=fieldCopy[row][col]; fieldCopy[row][col]=fieldCopy[row2][col2]; fieldCopy[row2][col2]=tmp;

Ist damit alles in Ordnung?

Zum Seitenanfang

David Scherfgen

Administrator

Beiträge: 10 382

Wohnort: Hildesheim

Beruf: Wissenschaftlicher Mitarbeiter

14.05.2018, 18:46

Sieht OK aus.

David Scherfgens Website | Konzentrationstest Polizei

Zum Seitenanfang

Chodot

Frischling

Beiträge: 14

14.05.2018, 18:49

Zitat von »David Scherfgen«

Sieht OK aus.

Dann werde ich noch ein weiteres mal den ganzen Quellcode durchgehen.
Ich dachte, dass Problem läge bei der einen Funktion.

Danke für eure Hilfe

EDIT:

Ich habe den Fehler nun endlich entdeckt.
Bei

	Javascript-Quelltext
1	var tmp=swap(field,cursorX,cursorY,'t');

musste ich die beiden Cursor Koordinaten tauschen.

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »Chodot« (14.05.2018, 19:12)

Zum Seitenanfang

Chodot

Frischling

Beiträge: 14

18.05.2018, 13:38

EDIT: Hat sich erledigt

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »Chodot« (18.05.2018, 18:55)

Zum Seitenanfang

TGGC

1x Rätselkönig

Beiträge: 1 799

Beruf: Software Entwickler

22.05.2018, 08:48

Darum ist es immer gut, sowas in eine Point oder Vector Struktur zu kapseln.

Tetris Meeting Mainz | WMK Gamesroom | Schau dem neuen Tetris Großmeister zu

Zum Seitenanfang

Chodot

Frischling

Beiträge: 14

22.05.2018, 21:50

Zitat von »TGGC«

Darum ist es immer gut, sowas in eine Point oder Vector Struktur zu kapseln.

Ist das denn Standard und kannst du das bitte etwas genauer erläutern, also wie ich das mit js umsetzen kann. Für mich hört sich das irgedwie nach ANSI-C an.

Zum Seitenanfang

TGGC

1x Rätselkönig

Beiträge: 1 799

Beruf: Software Entwickler

23.05.2018, 07:32

z.B. sowas: http://paperjs.org/reference/point/

Mit einer bestimmten Sprache hat das nichts zu tun, sowas geht in jeder objektorientierten Sprache.

Tetris Meeting Mainz | WMK Gamesroom | Schau dem neuen Tetris Großmeister zu

Zum Seitenanfang

Werbeanzeige

spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Javascript | Match3 Fehlersuche

Javascript | Match3 Fehlersuche

Javascript-Quelltext

Inhalt anzeigen/verstecken

Javascript-Quelltext

Zitat von »DNKpp«

Zitat von »DNKpp«

Javascript-Quelltext

Zitat von »David Scherfgen«

Javascript-Quelltext

Zitat von »TGGC«

Ähnliche Themen