Kiwi's Blog

It looks really amazing nice graphics

You have a lot of duplicated code in that routine when the only things that affect it are flicker and i. It would be more compact to write :-

 

    if (flicker==0)
        i=0;
    else
        i=10;
    flicker^=1;
    for(ID=0;ID!=6;ID++)
        {
        sprites[16+i].y = 207;
        sprites[17+i].y = 207;
        sprites[0+i].y = objecty[ID];
        sprites[0+i].x = objectx[ID];
        sprites[0+i].colour =objectc1[ID];
        sprites[0+i].pattern =objectp1[ID];
        sprites[1+i].y = objecty[ID];
        sprites[1+i].x = objectx[ID];
        sprites[1+i].colour =objectc2[ID];
        sprites[1+i].pattern =objectp2[ID];
        i+=2;
        }
    if(bosson==4){boss4();}
    if(bosson==3){boss3();}
    if(bosson==1){boss();}

To improve code generation efficiency further I'd generate a pointer to sprites before the loop and use that to access the structure elements.

That would work if:

for(ID=0;ID!=6;ID++){
        sprites[16+i].y = objecty[ID];
        sprites[16+i].x = objectx[ID];
        sprites[16+i].colour =objectc1[ID];
        sprites[16+i].pattern =objectp1[ID];
        sprites[17+i].y = objecty[ID];
        sprites[17+i].x = objectx[ID];
        sprites[17+i].colour =objectc2[ID];
        sprites[17+i].pattern =objectp2[ID];
        sprites[0+i].y = objecty[ID];
        sprites[0+i].x = objectx[ID];
        sprites[0+i].colour =objectc1[ID];
        sprites[0+i].pattern =objectp1[ID];
        sprites[1+i].y = objecty[ID];
        sprites[1+i].x = objectx[ID];
        sprites[1+i].colour =objectc2[ID];
        sprites[1+i].pattern =objectp2[ID];
        if(flicker==0){i+=2;}else{i-=2;}
}

Usually when the objects are not activate, they are moved to objecty[iD]=207. if flicker is 1, then it reverse the table from 10 to 0. So if there's 4 objects on a line, they'll flicker instead of disappear. I'll try your code out. Thank you for the advice.

Ah, I missed that. If you define some sprite structure pointers in the "if (flicker)" statement then the same code could work for both.

    if (flicker==0){i=0;} else{i=10;}
    flicker^=1;
    for(ID=0;ID!=6;ID++)
        {

        sprites[16+i].x = objectx[ID];
        sprites[16+i].colour =objectc1[ID];
        sprites[16+i].pattern =objectp1[ID];

        sprites[17+i].x = objectx[ID];
        sprites[17+i].colour =objectc2[ID];
        sprites[17+i].pattern =objectp2[ID];
		if(flicker==0){
        sprites[16+i].y = 207;
        sprites[17+i].y = 207;}
else{		
        sprites[16+i].y = objecty[ID];
        sprites[17+i].y = objecty[ID];		
}
		
        sprites[0+i].x = objectx[ID];
        sprites[0+i].colour =objectc1[ID];
        sprites[0+i].pattern =objectp1[ID];

        sprites[1+i].x = objectx[ID];
        sprites[1+i].colour =objectc2[ID];
        sprites[1+i].pattern =objectp2[ID];
if(flicker==0){
        sprites[0+i].y = objecty[ID];
        sprites[1+i].y = objecty[ID];}
else{		
        sprites[0+i].y = 207;
        sprites[1+i].y = 207;		
}
        if(flicker==1){i+=2;}else{i-=2;}
}

This is the code as right now that works. It does lag the game when it get crowded with a boss, Rockies and the white rocks, Megaman 3 style because it have to now add data to the x color and patterns. However, I can have the enemy vs Quinn collusion detection occur every other frame or check for the first 3 object one frame, and then the last 3 objects in another frame. I may revert to what I have right now.

It can be optimised for size quite a bit more with structure pointers .