Hey guys, i've been making a little super mario clone in Java but I need help,

you see, if you download it and play it, you will notice it is quite stutter-y, and that really shouldn't be the case since it's such a small program.

So i assume it's 2 things, either my game loop is faulty (i'll provide the code below) or it's simply that Java just sucks at this, so I'm gonna go with the first one cause i love Java.


Anyways, here's my game loop.

/**

* Contains the main flow of the game.

*/

private void run() {



timer.schedule(new TimerTask() {

private final BufferStrategy bf = frame.getBufferStrategy();

private long startTime = System.currentTimeMillis();

private long currTime = startTime;

@Override

public void run() {

elapsedTime = System.currentTimeMillis() - currTime;

currTime += elapsedTime;

try {

//get graphics

g = bf.getDrawGraphics();

//get input

inputGame();

//update the game

updateGame(elapsedTime);

//draw the game

drawGame(g, elapsedTime);



} finally {

g.dispose();

}



// Shows the contents of the backbuffer on the screen.

bf.show();



Toolkit.getDefaultToolkit().sync();

}

//start this loop every 17 milliseconds, which is roughly 58 frames per second

}, 10, 17);

}

the variable timer, is of class Timer in Java.util.Timer, not the swing Timer, which i heard is unreliable, but this one is too clearly.

at the moment, I'm not really doing much with elapsedTime, but it's there just in case, i figured i didn't need it since that loop will
run every 17 milliseconds, and my update and rendering methods only take 2-5 milliseconds to finish.

Do you guys know of very effective game loops that provide very smooth rendering?

I can provide more code if necessary, Thanks in advance!

I've never had a problem with jittery ness, here are pieces of my code to show you what I do in java.

For one I create a class called widget that handle any kind of 2D art work I place on it and extends canvase


public class widget extends Canvas
{
private Renderable context = null;
public void init()
{
GraphicsEnvironment ge = GraphicsEnvironment.getLocalGraphicsEnvironment();
GraphicsDevice gs = ge.getDefaultScreenDevice();
GraphicsConfiguration gc = gs.getDefaultConfiguration();
this.createBufferStrategy(2);
}

@Override public void paint(Graphics g)
{
if(context != null)
g.drawImage(context.fullrender(), 0, 0, this);
else
{
g.setColor(Color.white);
g.fillRect(0, 0, this.getWidth(), this.getHeight());
}
}
public void updateContainer()
{
BufferStrategy strat = this.getStrategy();
Graphics2D g2 = (Graphics2D)strat.getDrawGraphics();
g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
g2.drawImage(context.fullrender(), 0, 0, null);
g2.dispose();
strat.show();
}

public void contentUpdate()
{
this.validate();
this.repaint();
}
}



and with each widget i give it a context, which implements another class I created called Renderable.

This Renderable class contains the actual information that needs to be drawn, so in its fullrender() method itll have all the sprites and tiles and where they need to be rendered to on the widget class, and the widget handles the double buffering and actual rendering to screen itself. That way i easily seperate the stuff I need to render from some of the logic I dont really need to see while setting up a scene to render.