业务逻辑可不可以不依赖于持久层

业务逻辑能否不依赖于持久层?
我现在的架构是po->manager->service->action
其中manager实现业务逻辑,提供事务,action调用service
service中仅获取session,控制事务,load对象,然后调用manager,所以我可以单独的测试业务而不依赖于持久层

目前有这么个场景:一张地图,分成小块。给我一个坐标和半径,然后判断这个范围内是否有水。
因为地图是上下相通,左右相通,所以我要算出这个范围内所有地块的id,然后load出来,再逐格判断
于是代码就得类似这样:
List<Integer> idList=manager.computeIdList(int x,int y);
Query query = session.createQuery("from Field where id in (:list)");
query.setParameterList("list", idList);
boolean hasWater=manager.fieldHasWater(query.list());

我想把这些东西都封到manage里,于是我定义了一个MyMap,与地块是一对多,内部用map,key为地块的id
这时会多个MyMap表,里面只有一行数据,地块表会多一列,以对应MyMap
于是manager就能写成:
boolean hasWater(int x,int y,MyMap map);
然后在内部用may.getField(int id)来得到地块
地块的数量有几十万,所以我用了lazy="extra"。

这种设计是否合理,是否会产生某些问题?


PS:<map-key type="int" formula="id" />,这种情况下不能用extra,hibernate生成的查询条件会出现null
再PS:以前看过一篇文章,select * from table where id in(1,2)的性能不如select * from table where id=1 union select * from table where id=2
因为in查询不能很好的利用索引,所以我觉得hibernate在缓存命中高的情况下,用循环load对象比用in查询要好一点,这个没测试过,高手有经验请说说
1 楼 wolfsquare 2007-05-15  
1.建议事务控制与业务调用一致,否则会造成业务异常而无法完全回滚。
2.当你在设计业务调用接口时,需要考虑持久化相关的问题,那么可以肯定要不就是该业务不是业务,或者后者完全是多虑。
2 楼 deafwolf 2007-05-15  
wolfsquare 写道
1.建议事务控制与业务调用一致,否则会造成业务异常而无法完全回滚。
2.当你在设计业务调用接口时,需要考虑持久化相关的问题,那么可以肯定要不就是该业务不是业务,或者后者完全是多虑。

多谢回复,不过这两点我不是很明白。
这是不使用MyMap的代码
public class Field {
	private Integer id;

	private int x;
	private int y;
	private boolean hasWater;
}

public class CoreManage {
	public static List<Integer> getIdList(int x, int y, int radius) {
		// ....
	}

	public static boolean hasWater(List<Field> list) {
		for (Field tmp : list) {
			if (tmp.isHasWater()) {
				return true;
			}
		}
		return false;
	}
}

public class CoreService {
	private SessionFactory sessionFactory;

	public boolean hasWater(int x, int y, int radius) {
		Session session = sessionFactory.openSession();
		try {
			session.beginTransaction();
			List idList = CoreManage.getIdList(x, y, radius);
			Query query = session.createQuery("from Field where id in (:list)");
			query.setParameterList("list", idList);
			List<Field> list = (List<Field>) query.list();
			return CoreManage.hasWater(list);
		} catch (Exception e) {
			session.getTransaction().rollback();
			return false;
		} finally {
			session.close();
		}
	}
}


使用MyMap后可以改成如下的,Field不变
public class MyMap {
	private Map<Integer, Field> map = new HashMap<Integer, Field>();

	public Field getField(int id) {
		return map.get(id);
	}
}

public class CoreManage {
	public static List<Integer> getIdList(int x, int y, int radius) {
		// ....
	}

	public static boolean hasWater(int x, int y, int radius, MyMap map) {
		List<Integer> idList = CoreManage.getIdList(x, y, radius);
		for (int r : idList) {
			if (map.getField(r).isHasWater()) {
				return true;
			}
		}
		return false;
	}
}

public class CoreService {
	private SessionFactory sessionFactory;

	public boolean hasWater(int x, int y, int radius) {
		Session session = sessionFactory.openSession();
		try {
			session.beginTransaction();
			MyMap map=(MyMap)session.get(MyMap.class, 1);
			return CoreManage.hasWater(x,y,radius,map);
		} catch (Exception e) {
			session.getTransaction().rollback();
			return false;
		} finally {
			session.close();
		}
	}
}


像上面的代码那样,manage可以封上全部的业务,service只提供事务,这样我可以脱离持久层测试业务,我可以伪造一个MyMap用来测试,而从逻辑上讲,从地图上获取某一格,也比较自然
而按第一段的写法,service里是有业务逻辑的,测试的话,就得搭上数据库才行

把hibernate作为DAO来使用,不使用DAO层,service里不实现业务,只提供事务,这种用法在个别帖子里也出现过,不过特别少,所以我拿不准这种用法有什么缺点,所以还是希望能得到一些指点。