0

我想用 Java 绘制hline矩形vline。我在绘制它时遇到了一些问题，我不确切知道，但我认为它是在hline1和vline1方法上。

没有错误，只是参数内的算法有问题。

这是代码..

 package hw1;
 import java.awt.*;
 import java.awt.geom.*;
 import java.awt.event.*;

 public class main extends Frame {

Graphics2D g2d;
main()
{
    addWindowListener(new hw1.main.MyFinishWindow());
}

public class MyFinishWindow extends WindowAdapter
{
    public void windowClosing(WindowEvent e)
    {
        System.exit(0);
    }
}

public void paint (Graphics g)
{
    g2d=(Graphics2D)g;

    hline(0,40,250,40);
    vline(250,40 , 250 , 80);
    hline1(250,80,0,80);
    vline1(0,80 , 0 , 40);

}

public void hline(int x1,int y1 , int x2 , int y2)
{
    for(int x=x1 ; x<=x2 ; x++)
        putpixel(x,y1,Color.blue);
}

public void vline(int x1 ,int y1 , int x2 , int y2 )
{
    for(int y=y1;y<=y2;y++)
        putpixel(x1,y,Color.blue);
}

 public void hline1(int x1,int y1 , int x2 , int y2)
{
    for(int x=x1 ; x<=x2 ; x++)
        putpixel(x,x1,Color.blue);
}

public void vline1(int x1 ,int y1 , int x2 , int y2 )
{
    for(int y=y1;y<=y2;y++)
        putpixel(x1,y,Color.blue);
}

public void putpixel(int x , int y , Color c)

{
    g2d.setColor(c);
    g2d.drawLine(x, y, x, y);
}

 public void putpixel(int x , int y , Color c , int rad)
 {
     g2d.setColor(c);
     if(rad>4) rad=4;
     if(rad<=0) rad=1;
     g2d.drawOval(x-rad/2, y-rad/2, rad, rad);
 }


public static void main(String[] args) {
    // TODO code application logic here

    main f=new main();
    f.setTitle("Computer Graphics:Java 2D prpgram");
    Dimension screenSize=
            Toolkit.getDefaultToolkit().getScreenSize();
    int width=(int) screenSize.getWidth();
    int height=(int) screenSize.getHeight();
    f.setSize(width, height);
    f.setVisible(true);
}

}
4

3 回答 3

2

主要问题是你总是增加循环变量 - 没有考虑哪个边界值大于另一个......

   vline1(0,80 , 0 , 40);

这将从 80 开始 y，并将其递增直到达到 40...

这至少做了它应该做的

public void vline(int x1 ,int y1 , int x2 , int y2 )
{
    //use min and max
    for(int y=Math.min(y1, y2);y<=Math.max(y1,y2);y++) { //always use brackets!!!
        putpixel(x1,y,Color.blue);
    }
}

另外，一定要为 hline() 实现这个...

其他问题

  • 垂直线不需要两个函数，水平线不需要两个函数。方法的目的是重用代码 - 如果您出于某种我们不知道的原因想要保留此结构，请将 1 个函数用于水平线，将 1 个函数用于垂直线。

  • 方法中不应有任何未使用的参数。

    hline(y, x1, x2)
vline(x, y1, y2)

    完全足够了。

  • 更不用说，将其提升到一个新的水平：为各种行提供一种方法会很棒：drawLine()正如@ling.s指出的那样，那就是 。并且可以正确处理各种行...

故事的寓意：了解你的 API，不要重新发明轮子（除非这是为了学校，任务是重新发明它......）

于 2013-10-01T10:07:47.750 回答
1

尝试drawLine() 

public void paint (Graphics g)
{
g2d=(Graphics2D)g;

g.drawLine(0,40,250,40);
g.drawLine(250,40 , 250 , 80);
g.drawLine(250,80,0,80);
g.drawLine(0,80 , 0 , 40);

}
于 2013-10-01T10:07:02.953 回答
0

作为旁注，您可以使用 java.awt.Polygon.

  Polygon p;

   p = new Polygon();
   p.addPoint(10, 10);
   p.addPoint(100,10);       
   p.addPoint(100,100);
   p.addPoint(10,100);

 .....

  public void paintComponent(Graphics g){
    super.paintComponent(g);
    setBackground(Color.white);
    g.fillPolygon(p);   
}
于 2013-10-01T10:10:18.150 回答