Thread: Need help with repaint();

Hi guys and gals, i've been trying to write this code but i keep getting something wrong and can't figure it out.
I appreciate any help/advice you can give me, thanks!
public class Saxophone extends JFrame implements KeyListener{
char c;
boolean n, b, m, o, k, h = false;
private Graphics g;
public void paint(Graphics g)
{
super.paint(g);
if (c == 'n'){
n = true;}
if (c == 'm'){
m = true;}
if (c == 'b'){
b = true;}
if (c == 'o'){
o = true;}
if (c == 'k'){
k = true;}
if (c == 'h'){
h = true;}
}

--- Update ---

Here's the rest of the code

@Override
public void keyPressed(KeyEvent e) {
// TODO Auto-generated method stub
}
@Override
public void keyReleased(KeyEvent e) {
// TODO Auto-generated method stub
}
@Override
public void keyTyped(KeyEvent e) {
c = e.getKeyChar();
if (n == true){
/*Neck and Bell*/
g.fillOval(406, 410, 80, 60);
g.fillRoundRect(271, 260, 120, 30, 10, 10);
}
if (m == true){
/*Mouthpiece*/
g.setColor(Color.BLACK);
g.fillOval(217, 259, 70, 32);
g.fillRect(238, 260, 50, 30);
}
if (b == true){
/*Body of Saxophone*/
g.setColor(Color.ORANGE);
g.fillArc(350, 560, 120, 120, 180, 180);
g.fillRoundRect(351, 260, 40, 380, 10, 10);
g.fillRoundRect(415, 423, 55, 200, 10, 10);
}
if (o == true){
/*Octave Key*/
g.setColor(Color.ORANGE);
g.drawArc(294, 249, 95, 30, 180, -180);
g.drawArc(294, 248, 95, 30, 180, -180);
g.drawArc(294, 247, 95, 30, 180, -180);
g.setColor(Color.BLACK);
g.drawArc(294, 250, 95, 30, 180, -180);
g.drawArc(294, 246, 95, 30, 180, -180);
}
if (k == true){
/*Keys*/
g.setColor(Color.BLACK);
g.fillOval(376, 350, 15, 15);
g.fillOval(376, 366, 15, 15);
g.fillOval(376, 382, 15, 15);
g.fillOval(360, 500, 15, 15);
g.fillOval(360, 516, 15, 15);
g.fillOval(360, 532, 15, 15);
}
if (h == true){
/*Hole In Bell and Outline of Bell*/
g.setColor(Color.BLACK);
g.fillOval(416, 417, 60, 40);
g.drawArc(410, 431, 70, 37, 186, 150);
}
}
}

import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import javax.swing.JFrame
public class Sax extends JFrame implements KeyListener{
char c;
boolean n, b, m, o, k, h = false;
private Graphics g;

public void paint(Graphics g)
{
super.paint(g);

repaint();
}
@Override
public void keyPressed(KeyEvent e) {
// TODO Auto-generated method stub

}
@Override
public void keyReleased(KeyEvent e) {
// TODO Auto-generated method stub

}
@Override
public void keyTyped(KeyEvent e) {
c = e.getKeyChar();
if (n == true){
/*Neck and Bell*/
g.fillOval(406, 410, 80, 60);
g.fillRoundRect(271, 260, 120, 30, 10, 10);
repaint();
}
if (c == 'n'){
n = true;}
if (c == 'm'){
m = true;}
if (c == 'b'){
b = true;}
if (c == 'o'){
o = true;}
if (c == 'k'){
k = true;}
if (c == 'h'){
h = true;}

if (m == true){
/*Mouthpiece*/
g.setColor(Color.BLACK);
g.fillOval(217, 259, 70, 32);
g.fillRect(238, 260, 50, 30);
repaint();
}
if (b == true){
/*Body of Sax*/
g.setColor(Color.ORANGE);
g.fillArc(350, 560, 120, 120, 180, 180);
g.fillRoundRect(351, 260, 40, 380, 10, 10);
g.fillRoundRect(415, 423, 55, 200, 10, 10);
repaint();
}
if (o == true){
/*Octave Key*/
g.setColor(Color.ORANGE);
g.drawArc(294, 249, 95, 30, 180, -180);
g.drawArc(294, 248, 95, 30, 180, -180);
g.drawArc(294, 247, 95, 30, 180, -180);
g.setColor(Color.BLACK);
g.drawArc(294, 250, 95, 30, 180, -180);
g.drawArc(294, 246, 95, 30, 180, -180);
repaint();
}
if (k == true){
/*Keys*/
g.setColor(Color.BLACK);
g.fillOval(376, 350, 15, 15);
g.fillOval(376, 366, 15, 15);
g.fillOval(376, 382, 15, 15);
g.fillOval(360, 500, 15, 15);
g.fillOval(360, 516, 15, 15);
g.fillOval(360, 532, 15, 15);
repaint();
}
if (h == true){
/*Hole In Bell and Outline of Bell*/
g.setColor(Color.BLACK);
g.fillOval(416, 417, 60, 40);
g.drawArc(410, 431, 70, 37, 186, 150);
repaint();
}
}

}

How are you passing the Graphics object? Why are you not using a JPanel? Creating the Demo Application (Step 1) (The Java™ Tutorials > Creating a GUI With JFC/Swing > Performing Custom Painting) has a good demo.

Welcome saxy
Please do not double post your question. Threads merged.
Please see the Announcements page for the use of code tags and other useful forum tips.
You should not call repaint() from within the paint method

i keep getting something wrong

What do you get wrong?
What does the program do now? What should it do instead? Where do things seem to go unexpected?
Even if someone had the time to compile and run every code sample posted, how are we to guess what it should do if it does not do the right thing as it is?