方法和分解

Question

经过几年的HTML/CSS编码，我刚刚开始学习Java，所以希望我在这里不要问一个古老或愚蠢的问题，但任何帮助解释这个问题将非常感激。

我目前正在通过斯坦福CS106A在线资料，并且已经达到了第6周的作业2，问题3（http://see.stanford.edu/materials/icspmcs106a/13-assignment-2-simple -java.pdf）。

正如您所见，它需要在屏幕上放置各种对象来创建图形层次结构，如上所述。我的计划是使用中心坐标将所有对象放在屏幕上。不过，我遇到了一个我似乎无法找到答案的问题。本课程介绍的方法分解应该如何让每一个方法来处理一个问题（单一职责原则，我相信），所以我写了我的代码的第一部分是这样：

//Import any libraries 
import acm.program.*; 
import acm.graphics.*; 

    public class GraphicsHierarchy extends GraphicsProgram { 

//Define constants 
static final int BOX_WIDTH = 200; 
static final int BOX_HEIGHT = 75; 



public void run() { 
    placeGRect(); 
} 

//Find centre x & y 
double centre_x = getWidth()/2; //check this 
double centre_y = getHeight() * 0.5;//and this 

//placeGRect method 
public void placeGRect() { 
    for (int count = 0; count < 4; count++) { 
     GRect box = new GRect (BOX_WIDTH, BOX_HEIGHT); 
     add(box); 
     switch (count) { 
     case 0: 
      box.setLocation(centre_x, 75); 
      break; 
     case 1: 
      box.setLocation((centre_x * 0.5), 250); 
      break; 
     case 2: 
      box.setLocation(centre_x, 250); 
      break; 
     case 3: 
      box.setLocation((centre_x * 1.5), 250); 
      break; 
     } 
    } 
} 
} 

然而，这并不因工作产生零值的centre_x & centre_y。我通过将程序更改为ConsoleProgram，并在run（）方法中使用getHeight行（并在屏幕上打印它们的值），发现了这一点，然后生成所需的值但未将它们传递给GRect方法（如此仍然没有工作）。但是，如果我将getWidth/getHeight行列出run（），那么它们不会为相对定位生成任何值。我的问题是，每个方法都应该处理一个任务，并且（尽可能）方法应该从run（）方法中定义出来，然后我怎么才能将getWidth/getHeight值传递给placeGRect（）方法在run（）方法中没有一大块代码。我理解的是不好的做法。

我没有任何代码解决这个问题，我真的需要了解这个原理，所以我可以在将来编写有效的代码。我更喜欢理解鹦鹉式的代码复制。

在此先感谢您的帮助。

Answer 1

在你的具体的例子：

您已经声明centre_x和centre_y实例变量。当你的程序首先创建的GraphicsHierarchy一个实例对象创建的顺序是这样：

1. 类加载器加载的类...静态变量（BOX_WIDTH，BOX_HEIGHT）被分配指定的值;

2. 空间在堆上分配用于GraphicsHierarchy一个实例（足够的空间容纳实例变量 - 一个double为centre_x和centre_y一个double - 包括用于基类的实例变量空间）

3. 实例变量被设置为默认值：centre_x = 0，centre_y = 0

4. 的GraphicsHierarchy默认构造函数被调用（其不执行任何其他比调用基类的构造函数 - GraphicsProgram）。

5. 基类将通过1-4步，当它执行完毕后，返回到GraphicsHiearchy执行任何余下的构造函数语句之前，现在明确的评估实例变量初始化（在默认的构造函数的情况下，是没有的） 。

说了这一切（在这个过程中http://java.dzone.com/articles/java-object-initialization额外的参考），它会出现在你的GraphicsHierarchy类获取到第5步，并尝试赋值给centre_x和centre_y，该子系统getWidth和getHeight依赖没有准备好（即一个窗口或画布尚未创建，所以方法返回0）。但是，当您将内部任务移动到内部并返回值时，这意味着无论调用什么方法，run run首先会经历必要的窗口创建步骤。

Etienne de Martel的建议很好。它会延迟您的中心值的分配，直到需要之前。如果您愿意，您可以创建一个init方法和移动init方法里面的任务，然后调用初始化为运行的第一步

private void init() { 
    centre_x = getWidth/2; 
    centre_y = getHeight * 0.5; 
} 

public void run() { 
    init(); 
    placeGRect(); 
} 

这是几乎同样的事情，马特尔的建议，但如果你后来发现你有其他的初始化代码需要发生，你可以把它放在同一个地方。

至于起草（如果你喜欢或集合），你可能想重命名placeGRect到placeGRects并通过以点为单位的阵列灵活的代码placeGRects(Point[] points)

（可以使用java.awt.Point中或定义自己的Point类）

这样，你的placeGRects方法就简化了。它不再决定渲染多少个盒子（传入的数组）。它也不能确定这些新盒子的位置（再次是Point对象的数组）。它只是遍历数组的大小，创建一个新的框，添加它，并设置位置。

private Point[] boxPoints; 

public void run() { 
    init(); 
    placeGRects(boxPoints); 
} 

public void placeGRects(Point[] points) { 
    for(int i=0;i<points.length;i++) { 
     GRect b = new GRect(BOX_WIDTH,BOX_HEIGHT); 
     add(b); 
     b.setLocation(points[i].x,points[i].y); 
    } 
} 

而且您可以将您的Point数组初始化放入您的新init（）方法中。

private void init() { 
    centre_x = getWidth/2; 
    centre_y = getHeight * 0.5; 
    boxPoints = {new Point(centre_x, 75),new Point(centre_x * 0.5, 250)}; 
} 

它使您的代码在需要时更易于理解和修改。

Answer 2

也许我不明白你的问题，但为什么不把它们作为参数传递？

protected void placeGRect(double centre_x, double centre_y) { 
    // ... 
} 

你可以调用placeGRect像这样：

public void run() { 
    placeGRect(getWidth()/2, getHeight() * 0.5); 
} 

Answer 3

非常好的问题！如何撰写你的方法是一个直觉问题，而不是严格的指导方针。

当然，方法应该专注于做一件事和一件事。首先，使用简短的方法（甚至是单行程）可以提高代码的可理解性。作为一个非常粗略的例子，认为这样的：

if (DateUtils.before(ticket.getExpirationDate(), new Date())) { 
    accept(ticket); 
} 

，然后这个

if (isNotExpired(ticket)) { 
    accept(ticket); 
} 

... 

private boolean isNotExpired(Ticket t) { 
    return DateUtils.before(t.getExpirationDate(), now()); 
} 

private Date now() { 
    return (new Date()); 
} 

怎么注意）引进的一条线方法isNotExpired（）和现在（显著改善你不知道代码的作用。

下面是另一个例子，这次有做与构造对象：

Loan l1 = new Loan(15000, 36, f7.2, 2.5); 
Loan l2 = new Loan(15000, 36, f7.2); 

与

Loan l1 = Loan.newSubsidizedLoan(15000, 36, f7.2, 2.5); 
Loan l2 = Loan.newNormalLoan(15000, 36, f7.2); 

注意，在这个例子中是如何在两种不同的方法包装的构造显著提高了文档的代码（甚至不需要写评论）;

如果您对编码风格的一般主题感兴趣，您应该阅读this书。

干杯

L.

Answer 4

您的代码似乎没有包括的getWidth（）和getHeight（）方法。另外，下面这段代码是完全错误的位置和应放置在一个构造函数：

double centre_x = getWidth()/2; //check this 
double centre_y = getHeight() * 0.5;//and this 

应该成为

private double centre_x; 
private double centre_y; 
GraphicsHierarchy(){ 
    centre_x = GraphicsHierarchy.BOX_WIDTH/2; 
    centre_y = GraphicsHierarchy.BOX_HEIGHT * 0.5; 
} 

该代码将至少编译，但考虑下面介绍的解决方案，这更好。

考虑到您已将BOX_WIDTH和BOX_HEIGHT定义为静态变量，您始终可以找到centre_x和centre_y。因此，你甚至都不需要定义BOX_WIDTH和BOX_HEIGHT

你可以这样定义类：

//Import any libraries 
import acm.program.*; 
import acm.graphics.*; 

public class GraphicsHierarchy extends GraphicsProgram { 
public void run() { 
    placeGRect(); 
} 
//Define constants 
public static final double CENTRE_X= 100.00; 
public static final double CENTRE_Y = 37.50; 
//placeGRect method 
public void placeGRect() { 
    for (int count = 0; count < 4; count++) { 
     GRect box = new GRect (200, 75); 
     add(box); 
     switch (count) { 
     case 0: 
      box.setLocation(GraphicsHierarchy.CENTRE_X, 75); 
      break; 
     case 1: 
      box.setLocation((GraphicsHierarchy.CENTRE_X * 0.5), 250); 
      break; 
     case 2: 
      box.setLocation(GraphicsHierarchy.CENTRE_X, 250); 
      break; 
     case 3: 
      box.setLocation((GraphicsHierarchy.CENTRE_X * 1.5), 250); 
      break; 
     } 
    } 
} 
} 

在我看来，你可以更进一步通过消除所有的计算和替换这样的东西

GraphicsHierarchy.CENTRE_X * 1.5 

与

150 

来吧，有它容易对你的V irtual机器！你的课程使用了整个静态信息，所以不需要太多的计算。但是有一个BOX_WIDTH和BOX_HEIGHT作为常量是完全没有用的，因为它们只在内部使用，只能在一个地方使用。从BOX_WIDTH和BOX_HEIGHT中计算centre_x和centre_y也是无用的，因为它们是最终的，您可以轻松地自行完成计算并减少不必要的变量创建。

另外，你不要在任何地方使用centre_y值，所以你应该抛弃它。

为了进一步添加一些有用的建议，象NetBeans，Eclipse或IntellIJIDEA这样体面的IDE应该有代码完成和语法高亮显示，并且将帮助你成为一个更好的（或更知名的，甚至更好的）程序员。