初学Java,自己写的小程序，请指正。(100分)

初学Java,看到多线程的时候，记起原来delphi的例子里有一个多线程的demo，是用
三个框演示冒泡，选择，希尔排序速度的。我就想自己试着搞一个类似的。看帮助，
翻书，终于弄出来两个框的，比较冒泡和选择排序。
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import com.borland.jbcl.control.*;
import com.borland.jbcl.layout.*;
import javax.swing.border.*;
import java.util.Random;

public class Frame1 extends JFrame
{
BorderLayout borderLayout1 = new BorderLayout();
TitledBorder titledBorder1;
JPanel jPanel1 = new JPanel();
JPanel jPanel2 = new JPanel();
GridLayout gridLayout1 = new GridLayout();
JPanel jPanel3 = new JPanel();
TitledBorder titledBorder2;
TitledBorder titledBorder3;
TitledBorder titledBorder4;
TitledBorder titledBorder5;
JButton jButton1 = new JButton();
boolean flag1,flag2;
//ÅÅÐòÍê³É±êÖ¾
//Construct the frame
public Frame1()
{
enableEvents(AWTEvent.WINDOW_EVENT_MASK);
try
{
jbInit();
}
catch(Exception e)
{
e.printStackTrace();
}
}
//Component initialization
private void jbInit() throws Exception
{
titledBorder1 = new TitledBorder("");
titledBorder2 = new TitledBorder("");
titledBorder3 = new TitledBorder("");
titledBorder4 = new TitledBorder("");
titledBorder5 = new TitledBorder("");
this.getContentPane().setLayout(borderLayout1);
this.setSize(new Dimension(428, 322));
this.setTitle("Frame Title");
jPanel2.setLayout(gridLayout1);
jPanel2.setAlignmentX((float) 0.0);
jPanel2.setAlignmentY((float) 0.0);
jPanel2.setBorder(titledBorder5);
jPanel2.setPreferredSize(new Dimension(200, 250));
jPanel3.setBorder(titledBorder4);
jPanel3.setPreferredSize(new Dimension(200, 250));
jButton1.setText("Bubble");
jButton1.addActionListener(new java.awt.event.ActionListener() {
public void actionPerformed(ActionEvent e) {
jButton1_actionPerformed(e);
}
});
this.getContentPane().add(jPanel1, BorderLayout.CENTER);
jPanel1.add(jPanel3, null);
jPanel1.add(jPanel2, null);
jPanel1.add(jButton1, null);

}
//Overridden so we can exit on System Close
protected void processWindowEvent(WindowEvent e)
{
super.processWindowEvent(e);
if(e.getID() == WindowEvent.WINDOW_CLOSING)
{
System.exit(0);
}
}
class BubbleSortPanel extends Thread
{ int[] line1=new int[81];
public BubbleSortPanel()
{
Random r = new Random();
//ÓÃαËæ»úÊý³õʼ»¯Êý×é
int i;
for(i = 0;i<line1.length;i++)
line1 = r.nextInt(195);
paintBubble();
}
private void BubbleSort()
{
int m,n,temp;
for(m = 0;m<line1.length-1;m++)
for(n = m+1;n<line1.length;n++){
if (line1[m]>line1[n]){
temp = line1[m];
line1[m] = line1[n];
line1[n] = temp;
paintBubble();
try{ sleep(5);} catch (InterruptedException e){}
}
}
}
public void paintBubble()
{ Graphics g = jPanel3.getGraphics();
g.setColor(Color.red);
g.clearRect(3,
2,
jPanel3.getWidth()-6,
jPanel3.getHeight()-6);
int j;
for(j=0;j<line1.length;j++)
g.drawLine(3,(j+1)*3,line1[j],(j+1)*3);
}
public void run()
{
BubbleSort();
flag1 = true;
if (flag1&amp;&amp;flag2) jButton1.setEnabled(true);
}
}
class SelectSortPanel extends Thread
{ int[] line2=new int[81];
public SelectSortPanel()
{
Random r = new Random();
//&amp;Oacute;&amp;Atilde;&amp;Icirc;±&amp;Euml;&amp;aelig;&amp;raquo;ú&amp;Ecirc;&amp;yacute;&amp;sup3;&amp;otilde;&amp;Ecirc;&amp;frac14;&amp;raquo;&amp;macr;&amp;Ecirc;&amp;yacute;×é
int i;
for(i = 0;i<line2.length;i++)
line2 = r.nextInt(195);
paintSelect();
}
private void SelectSort()
{
int m,n,temp,lm;
lm = 0;
for(m = 0;m<line2.length-1;m++)
{ lm=m;
for(n = m+1;n<line2.length;n++)
if (line2[lm]>line2[n])
lm = n;
temp = line2[m];
line2[m] = line2[lm];
line2[lm] = temp;
paintSelect();
try{
sleep(5);
}catch(InterruptedException e){}
}
}
public void paintSelect()
{ Graphics g = jPanel2.getGraphics();
g.setColor(Color.red);
g.clearRect(3,
2,
jPanel2.getWidth()-6,
jPanel2.getHeight()-6);
int j;
for(j=0;j<line2.length;j++)
g.drawLine(3,(j+1)*3,line2[j],(j+1)*3);
}
public void run()
{
SelectSort();
flag2 = true;
if (flag1&amp;&amp;flag2) jButton1.setEnabled(true);
}
}
void jButton1_actionPerformed(ActionEvent e) {
BubbleSortPanel bp = new BubbleSortPanel();
SelectSortPanel sp = new SelectSortPanel();
jButton1.setEnabled(false);
flag1 = false;
flag2 = false;
bp.start();
sp.start();
}
}
但是觉得不太符合oop,比如直接操纵jpanel，于是该了一下，用Runnable接口：
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import com.borland.jbcl.control.*;
import com.borland.jbcl.layout.*;
import javax.swing.border.*;
import java.util.Random;
public class Frame1 extends JFrame
{
private Thread runner;
BorderLayout borderLayout1 = new BorderLayout();
TitledBorder titledBorder1;
JPanel jPanel1 = new JPanel();
SelectSortPanel jPanel2 = new SelectSortPanel();
GridLayout gridLayout1 = new GridLayout();
BubbleSortPanel jPanel3 = new BubbleSortPanel();
TitledBorder titledBorder2;
TitledBorder titledBorder3;
TitledBorder titledBorder4;
TitledBorder titledBorder5;
JButton jButton1 = new JButton();
boolean flag1,flag2;
//&amp;Aring;&amp;Aring;&amp;ETH;ò&amp;Iacute;ê&amp;sup3;&amp;Eacute;±ê&amp;Ouml;&amp;frac34;
//Construct the frame
public Frame1()
{
enableEvents(AWTEvent.WINDOW_EVENT_MASK);
try
{
jbInit();
}
catch(Exception e)
{
e.printStackTrace();
}
}
//Component initialization
private void jbInit() throws Exception
{
titledBorder1 = new TitledBorder("");
titledBorder2 = new TitledBorder("");
titledBorder3 = new TitledBorder("");
titledBorder4 = new TitledBorder("");
titledBorder5 = new TitledBorder("");
this.getContentPane().setLayout(borderLayout1);
this.setSize(new Dimension(428, 322));
this.setTitle("Frame Title");
jPanel2.setLayout(gridLayout1);
jPanel2.setAlignmentX((float) 0.0);
jPanel2.setAlignmentY((float) 0.0);
jPanel2.setBorder(titledBorder5);
jPanel2.setPreferredSize(new Dimension(200, 250));
jPanel3.setBorder(titledBorder4);
jPanel3.setPreferredSize(new Dimension(200, 250));
jButton1.setText("Bubble");
jButton1.addActionListener(new java.awt.event.ActionListener() {
public void actionPerformed(ActionEvent e) {
jButton1_actionPerformed(e);
}
});
this.getContentPane().add(jPanel1, BorderLayout.CENTER);
jPanel1.add(jPanel3, null);
jPanel1.add(jPanel2, null);
jPanel1.add(jButton1, null);
}
//Overridden so we can exit on System Close
protected void processWindowEvent(WindowEvent e)
{
super.processWindowEvent(e);
if(e.getID() == WindowEvent.WINDOW_CLOSING)
{
System.exit(0);
}
}
class SortPanel extends JPanel
{
int[] line=new int[81];
public SortPanel()
{
Random r = new Random();
//&amp;Oacute;&amp;Atilde;&amp;Icirc;±&amp;Euml;&amp;aelig;&amp;raquo;ú&amp;Ecirc;&amp;yacute;&amp;sup3;&amp;otilde;&amp;Ecirc;&amp;frac14;&amp;raquo;&amp;macr;&amp;Ecirc;&amp;yacute;×é
int i;
for(i = 0;i<line.length;i++)
line = r.nextInt(195);
}
public void paintComponent(Graphics g)
{ super.paintComponents(g);
g.setColor(Color.red);
g.clearRect(3,
2,
this.getWidth()-6,
this.getHeight()-6);
int j;
for(j=0;j<line.length;j++)
g.drawLine(3,(j+1)*3,line[j],(j+1)*3);
}
}
class BubbleSortPanel extends SortPanel implements Runnable
{
private void BubbleSort()
{
int m,n,temp;
for(m = 0;m<line.length-1;m++)
for(n = m+1;n<line.length;n++){
if (line[m]>line[n]){
temp = line[m];
line[m] = line[n];
line[n] = temp;
this.paintImmediately(3,
2,
this.getWidth()-6,
this.getHeight()-6);
try{ Thread.sleep(5);}catch(InterruptedException e){}
}
}
}
public void start()
{
runner = new Thread(this);
runner.start();
}
public void run()
{
BubbleSort();
flag1 = true;
if (flag1&amp;&amp;flag2) jButton1.setEnabled(true);
}
}
class SelectSortPanel extends SortPanel implements Runnable
{
private void SelectSort()
{
int m,n,temp,lm;
lm = 0;
for(m = 0;m<line.length-1;m++)
{ lm=m;
for(n = m+1;n<line.length;n++)
if (line[lm]>line[n])
lm = n;
temp = line[m];
line[m] = line[lm];
line[lm] = temp;
this.paintImmediately(3,
2,
getWidth()-6,
getHeight()-6);
try{Thread.sleep(5);} catch (InterruptedException e){}
}
}
public void start()
{
runner = new Thread(this);
runner.start();
}
public void run()
{
SelectSort();
flag2 = true;
if (flag1&amp;&amp;flag2) jButton1.setEnabled(true);
}
}
void jButton1_actionPerformed(ActionEvent e) {
jButton1.setEnabled(false);
flag1 = false;
flag2 = false;
jPanel3.start();
jPanel2.start();
}
}
请各位打下批评指正。另外，看到书上的Java程序的类里的变量声明都在类的末尾，
仅仅是这样的习惯吗？我喜欢放在类开始的。

其实这无关紧要，关键是只要符合企业文化就可以了。
我觉得最好放在前面，便于维护。

swing并不是线程安全的。不如改为awt.

很好!!!

i trytry

多人接受答案了。