c语言标记耦合的例子,不良代码展示-高耦合度代码的例子

原创文章，如有转载，请注明出处：http://blog.youkuaiyun.com/yihui823/article/details/6833131

前段时间，看到别人的一段代码，其实没什么逻辑，却做的丰富异常。

代码运行极其不稳定，却无从下手去改动。后来下定决心重写了。

昨晚有空，把原来不良代码抽取出来，供大家参考。

package testpj;

import java.util.Timer;

import java.util.TimerTask;

/**

* 高耦合的展示类

* @author yihui823

*/

public class CouplingTester {

/**

* @param args the command line arguments

*/

public static void main(String[] args) {

Connect conn = new Connect();

Listener l = new Listener(conn);

conn.setListener(l);

l.onConnFailed("first connect");

//循环查看Connect的对象

while(true) {

System.out.println(l.getConnect().hashCode());

try {

Thread.sleep(1000);

} catch (InterruptedException ex) {

}

}

}

/**

* 模拟一个连接类

*/

private static class Connect extends Thread {

//监听器

private Listener listener;

/**

* 设置监听器

* @param listener 监听器

*/

public void setListener(Listener listener) {

this.listener = listener;

}

/**

* 线程方法，开始进行连接.

*/

public void run() {

//假设连接失败

try {

//保持连接

Thread.sleep(2000);

listener.onConnFailed("Connecting error!");

System.out.println("连接失败.");

} catch (InterruptedException ex) {

}

//假设连接成功

// System.out.println("连接成功.");

// while (true) {

// try {

// //保持连接

// Thread.sleep(2000);

// System.out.println("保持连接.");

// } catch (InterruptedException ex) {

// }

// }

}

}

/**

* 模拟监听器

*/

private static class Listener {

private Connect conn;

private Timer timer = null;

public Listener(Connect conn) {

this.conn = conn;

}

public Connect getConnect() {

return conn;

}

/**

* 连接失败时调用

* @param msg 失败消息

*/

public void onConnFailed(String msg) {

System.out.println("Connect failued : " + msg);

timer = new Timer();

ConnectTask ct = new ConnectTask(timer, conn);

timer.schedule(ct, 3000, 100);

}

}

/**

* 一个连接任务类

*/

public static class ConnectTask extends TimerTask {

private Connect conn;

private Timer timer = null;

public ConnectTask(Timer timer, Connect conn) {

this.timer = timer;

this.conn = conn;

}

public void run() {

if (conn.isAlive()) {

System.out.println(conn.hashCode() + " is still alive.");

if (timer != null) {

timer.cancel();

timer = null;

}

} else {

System.out.println(conn.hashCode() + " is not alive.");

conn = new Connect();

conn.setListener(new Listener(conn));

conn.start();

}

}

}

}

Connect类里的run方法，原来是做连接用的，这里用简单的代码去模拟了。

我们可以看到，这段代码的逻辑，其实是要做一个重连接。

也就是说，当连接断了之后，自动触发onConnFailed事件，然后再次去做连接。

在这个例子里面，Connect里面保存了Listener，而Listener里面又保存了Connect，

ConnectTask里面也保存了Connect。三个类紧紧的耦合在一起，

一个简单的逻辑有两个线程在运作。其结果，是一旦有一次失败了，

则会多一个Timer在后台持续运行，而这个Timer不会被标记停止。

这里主要的问题，就是三个类的高耦合。而良好的代码，是需要做到：高内聚，低耦合的。也就是说，类与类之间的关系，越松散越好，最好能相互独立。

解耦改动的方法：

1，Connect里面设置Listener。但是，Listener里面，不记录Connect实例。

2，在onConnFailed事件中，只是做启动连接的动作，不要启动线程。再次连接之前，暂停一段时间即可。

具体代码就不贴出来了。重点就是：不要高耦合。