`
hgq0011
  • 浏览: 555321 次
  • 性别: Icon_minigender_1
  • 来自: 广州
社区版块
存档分类
最新评论

请点评jdbc通用访问方法,这样可行吗?

    博客分类:
  • Jdbc
阅读更多
    最近在检查其他同事写的代码,看到一同学写的了一个通用JDBC模板(静态工厂模式),似乎没有什么问题,但看起来很是不美观,有点怪异的感觉。我想了很久是不是要在论坛中发布,我觉得这样的代码还是不够OO,为什么自己对应的域对象操作不直接用用对象封装来处理呢?而转用ResultSetMetaData来解析对应的字段的属性,而且每次要把相关的参数保存到MAP、list中,然后又要取出来解析。所以我觉得回影响性能。当然在测试过程中没有发现性能的问题。由于项目中其他同学也在用他这个模板,他们觉得很方便。但对应新人(刚毕业的同学)来说,经常在调试的时候发生了错误,都不知错误发生在那?导致又要其他同学帮忙,乱费太多时间,而且他们还是不知其所以然。请大家帮忙评点一下。谢谢!
//通过SQL反回对应的数据集
//比如:sql select nameSc,age from users
public List getResultDataList(String sql, Connection con) {
		List<Map> dataList = new ArrayList<Map>();
		Map<String, String> map = new HashMap<String, String>();//存放对应字段数据
		PreparedStatement ps = null;
		ResultSet ds = null;
		ResultSetMetaData rsd;
		try {
			ps = con.prepareStatement(sql);
			ds = ps.executeQuery();
			while (ds.next()) {
				map = new HashMap<String, String>();//通过键值对存放一条记录的数据
				rsd = ds.getMetaData();
				for (int i = 1; i <= rsd.getColumnCount(); i++) {
					switch (rsd.getColumnType(i)) {//通过判断数据类型转换数据
					case 12:// varchar(12)
						map.put(rsd.getColumnName(i), ds.getString(i));
						break;
					case 1:// char(1)
						map.put(rsd.getColumnName(i), ds.getString(i));
						break;
					case -7:// bit(-7)
						map.put(rsd.getColumnName(i), Boolean.toString(ds
								.getBoolean(i)));
						break;
					case 4:// int(4)
						map.put(rsd.getColumnName(i), Integer.toString(ds
								.getInt(i)));
						break;
					case 5:// smallint(5)
						map.put(rsd.getColumnName(i), Integer.toString(ds
								.getInt(i)));
						break;	
					case 93:// datetime(93)
						if (ds.getString(i) == null)
							map.put(rsd.getColumnName(i), "");
						else {
							if (ds.getString(i).length() >= 19) {
								map.put(rsd.getColumnName(i), ds.getString(i)
										.substring(0, 19));
							} else {
								map.put(rsd.getColumnName(i), ds.getString(i)
										.substring(0, 10));
							}
						}
						break;
					case 3:// decimal(3)
						map.put(rsd.getColumnName(i), Double.toString(ds
								.getDouble(i)));
						break;
					default:
						map.put(rsd.getColumnName(i), ds.getString(i));
						break;
					}
				}
				dataList.add( map);//存放所有行数据
			}
		} catch (SQLException e) {
			e.printStackTrace();
			throw new RuntimeException(e);
		} finally {
                        [color=red]connection是通过HibernateDaoSupport获取得到的,似乎能够自动回收(反回到连接池),是否真的不用关闭连接呢?[/color]
			/*
			 * try{ if(ds!=null){ds.close();ds=null;}
			 * if(ps!=null){ps.close();ps=null;}
			 * if(con!=null){con.close();con=null;} }catch(SQLException e){}
			 */
		}
		return dataList;
	}



//通过传递SQL,Sql的参数更新数据
//比如 sql insert into users(nameCs,age)valuse(?,?)
//params就是存放?代表的值
public int execUpdateByParams(String sql, Connection con, List params) {
		PreparedStatement ps = null;
		int message = 0;
		try {
			String paramsStr="";
			ps = con.prepareStatement(sql);
			if (params != null) {
				for (int i = 0; i < params.size(); i++) {
					String p = params.get(i)==null?null:"" + params.get(i);//设置对应的参数
					ps.setString(i + 1, p);
				}
			}
			ps.executeUpdate();
		} catch (SQLException e) {
			e.printStackTrace();
			message = -1;
			throw new RuntimeException(e);
		} finally {
			try {
				if (ps != null) {
					[color=red]ps.close();[/color]//怎么在这他有把这个关闭了呢?这会有问题吧?
					ps = null;
				}
				// if(con!=null){con.close();con=null;}
			} catch (SQLException e) {
			}
		}

		return message;
	}
分享到:
评论
35 楼 fangwei 2009-08-12  
好处是封装了ResultSet的操作,直接映射成List<Map>
缺点是上层调用时需要看具体的sql才能知道Map里面是什么东东

ps:我维护过两年的一个系统就是这么干的,完全不需要javabean这种东东,不管在service层还是action层还是jsp直接map.get("xxx"),类少了很多,但是维护起来很不方便,后来在我的建议下增加了javabean,增加了一层对map的包装
34 楼 fjlyxx 2008-12-02  
说实话很不好!我了省事情把整个安全性给破坏了.也许这么说有点过分,但确实是这样的.

提几个意见
第一 注意分析切面,不该设置切面的地方不要这么做
第二 像数据库连接这种东西在小系统下也许不会体现它安全方面的重要性,但是你这么写的后果只会给连接丢失更多的机会.

我个人觉得数据库的切面应该是做在连接获取这块的,获取结果和数据更新操作虽然也可以 但是放置的地方和调用的地方要正确,我不赞同主动去调用这个模块 更赞同的是靠回调去注入连接,回调接口里面爱干嘛就干嘛了.

对于你的解析那块 还有很大的改进余地.不要case了 定义一个解析接口,系统维护一个数据库实际字段类型和RS解析类的映射 动态去调用.
就像

数据库字段A 用JAVA的类 com.xxxx.xxx去解析一样.当然你这个com.xxxx.xxx是你解析接口的一个实现. 这样做的好处是扩展性好而且可以对其他数据库进行支持
个人意见
33 楼 firmgoal 2008-10-01  
看Spring的JdbcTemplate吧,无论连接的获取、关闭,操作的灵活性,ResultSet的Mapping,都是上上之码。
32 楼 coreymylife 2008-09-29  
没认真看,但是认为这段代码极端的丑陋
#    for (int i = 1; i <= rsd.getColumnCount(); i++) { 
#                     switch (rsd.getColumnType(i)) {//通过判断数据类型转换数据 
#                     case 12:// varchar(12) 
#                         map.put(rsd.getColumnName(i), ds.getString(i)); 
#                         break; 
#                     case 1:// char(1) 
#                         map.put(rsd.getColumnName(i), ds.getString(i)); 
#                         break; 
#                     case -7:// bit(-7) 
#                         map.put(rsd.getColumnName(i), Boolean.toString(ds 
#                                 .getBoolean(i))); 
#                         break; 
#                     case 4:// int(4) 
#                         map.put(rsd.getColumnName(i), Integer.toString(ds 
#                                 .getInt(i))); 
#                         break; 
#                     case 5:// smallint(5) 
#                         map.put(rsd.getColumnName(i), Integer.toString(ds 
#                                 .getInt(i))); 
#                         break;   
#                     case 93:// datetime(93) 
#                         if (ds.getString(i) == null) 
#                             map.put(rsd.getColumnName(i), ""); 
#                         else { 
#                             if (ds.getString(i).length() >= 19) { 
#                                 map.put(rsd.getColumnName(i), ds.getString(i) 
#                                         .substring(0, 19)); 
#                             } else { 
#                                 map.put(rsd.getColumnName(i), ds.getString(i) 
#                                         .substring(0, 10)); 
#                             } 
#                         } 
#                         break; 
#                     case 3:// decimal(3) 
#                         map.put(rsd.getColumnName(i), Double.toString(ds 
#                                 .getDouble(i))); 
#                         break; 
#                     default: 
#                         map.put(rsd.getColumnName(i), ds.getString(i)); 
#                         break; 
31 楼 fuwang 2008-09-18  
你们这样把hibernate和jdbc混用了,如何共用连接池、连接何时关闭?事务等方面如何处理?
30 楼 starse7en77 2008-09-13  
编码就是为了解决问题 , 过多的考虑那些不实际的扩展是没意义的 。

诸如楼主这样 , 适用就好 。
29 楼 starse7en77 2008-09-13  
kunanfo 写道
以前我在公司培训的时候,也做过类似的程序,初看起来,很似好用。但是其实有一个很典型的问题,如果数据库有一个BLOB数据类型的怎么办?如果使用了不同的数据库,存在了另外的数据类型怎么办?

当时想了想,姑且不管什么OO,单就扩展性方面,就比较差。
这个问题提出来了,正好也关注下


很简单发现有BLOB就不要用这方法 。
28 楼 vicksong 2008-09-02  
其实我一直是用这种方法,但你这种封装太简单了,用于商业的话太简单。太多的问题没有考滤到。比如在tolist中的,并不是每个数据都是从数据库中直接取出来的,有可以要计算或有可能要再进一步再查询等等情况,保存与更新的情况,没有考滤到的情况就更多了,好在,所有情况我基本都考滤到了,所有我用于商业了,说到性能,其实,软件性能主要体现还是在网络上。
27 楼 hlydlp 2008-09-01  
其实这样做也是一种办法,通用型比较好
26 楼 yefeng 2008-08-29  
直接用apache-dbutils好了,已经做了很好的封装了,性能也不会有问题
25 楼 armorking 2008-08-28  
public List getResultDataList(String sql, Connection con)
方法是传入一个不带参数的完整的sql,
这里存在两个问题
1、对于需要传入参数的情形,只能在调用前手动把参数值拼到sql文当中,这种做法无法防范sql注入攻击
如果万一有一天客户要求你们把这个缺陷给改正的话,估计会有人哭得很惨

2、同样是因为手动把参数值拼到sql文当中,
那么,即便前后的sql文除了参数值完全一样
因为
ps = con.prepareStatement(sql);
这一句里传入的sql文是不同的
所以,db端每次都要重新编译sql文
所以,在批量插入,更新,删除的时候会有性能问题
24 楼 dzmonkey 2008-08-28  
代码写的却是有些不合理的地方。
但是目的本身是值得提倡的,我不赞同那种做一个所谓的“工具”,就一定要面面俱到,考虑了所有情况,我觉得还是要立足于项目或项目组本身,自己够用就好,不需要上岗上线的评论他的美丽与丑陋。
总之,首先要适用,再次要能用,最后在考虑可能的扩展性等等。
23 楼 elvewyn 2008-08-28  
恨死Map了。如果不是自己的程序。都不知道他里面装的是啥类型的。我觉得如果说通用的话,做到ibatis\spring jdbc那种层次就非常不错了。
22 楼 greens.leaf 2008-08-26  
看是不是适合你的应用环境。

这种处理适用于你有很多表,对每个表的操作都不复杂,表之间的关系也不复杂的情况。

关于connetion和statement的资源释放,基本的原则是谁申请谁释放。
connection对象是从函数体外通过参数传入的,那么在函数体内不应该关闭,试想如果你的代码这样调用:
getResultDataList("sql1",conn);
getResultDataList("sql2",conn);
第二次调用函数会抛出空指异常。

反过来,statement对象是在函数体内通过connection.createStatement()创建的,那么应该在函数体内关闭,否则就只能等gc了。statement这种对象比较消耗资源,能避免gc尽量避免。
21 楼 moshalanye 2008-08-26  
引用
由于项目中其他同学也在用他这个模板,他们觉得很方便。但对应新人(刚毕业的同学)来说,经常在调试的时候发生了错误,都不知错误发生在那?导致又要其他同学帮忙,乱费太多时间,而且他们还是不知其所以然。


      楼主:如果你是用的sql Server 数据库   同时还是用微软的那个数据库驱动,好像是有蛮多bug的,最离谱的一个bug 就是不能重复读 resultSet的数据,所以只要debug 取一次数据,运行程序再来一次,就出异常,同时还有长字段不可为空,在有长字段时要按顺序取数据的异常    换成sourceforg 的jdbc驱动就好了
      的确还是感觉springJdbcTemplete 好些,给新人用,又可以学jdbc的使用,又可以很好理解模板和策略,代码也优雅些
20 楼 seemoon 2008-08-26  
做这么个东西跟spring jdbctemplate有什么地方不同?希望能讨论一下
19 楼 土匪一份子 2008-08-26  
楼主测试过性能没问题? 
试下来个50万数据+表中有一个大字段,
应该会有影响吧。
18 楼 jackey3316 2008-08-25  
jacklondon 写道
用 Apache common DBUtils 吧。
它可以直接把 ResultSet 通过反射变成 JavaBean List,比你这里的代码要更高明。
或者用 VelocityWeb, 是在 DBUtils 之上的进一步封装。
顺便说一下,List 的遍历,不管是 LinkedList 还是 ArrayList, 建议都用 Iterator, 不要用 params.get(i).
建议看看 Thinking in Java. 虽然很多人不屑,但是这本书对于 Java Collection 一节,介绍得比其他书都要好。



看完这段代码   我第一想起来的也是dbutil
17 楼 qhfrose 2008-08-25  
感觉太耗内存了。每个Map都字段名和数据对应,如果数据量大,内存浪费还是比较严重的。
16 楼 siriuscor 2008-08-25  
不对sql语句进行分析啊?如果我只取几个字段就出错了.

可以对metadata做一个缓存

相关推荐

    JDBC通用增删改查方法

    本资源主要围绕"JDBC通用增删改查方法"展开,旨在提供一套简单、易用且具有广泛适用性的数据库操作方案。 首先,我们来了解一下JDBC的基本工作流程: 1. 加载驱动:使用`Class.forName()`方法加载数据库驱动,这...

    jdbc通用类

    本篇文章将详细探讨`jdbc通用类`的相关知识点,包括两种连接数据库的方法——基于连接池和属性文件,以及通用方法的设计思路。 首先,我们来看连接数据库的方法: 1. **基于连接池的数据库连接**:连接池是一种...

    jdbc通用数据库连接

    总结来说,"jdbc通用数据库连接"是指通过Java的JDBC API,封装了数据库操作的类或方法,使得在不同项目中可以方便快捷地进行数据库交互。这种封装涵盖了数据库连接的全过程,包括连接建立、SQL执行、结果处理以及...

    JDBC通用封装类

    本资源是对JDBC的封装,方便在项目中使用,其中BaseDao.java是对JDBC操作的封装,使用时让自己的Dao类继承即可,然后调用其中的executeQuery和executeOthe分别执行DQL和DML操作。dbinfo.properties属性文件存储基本...

    自定义jdbc通用类

    在Java编程领域,自定义JDBC(Java Database Connectivity)通用类是提高数据库操作效率和代码复用性的重要手段。通过创建这样一个类,开发者可以封装常见的数据库连接、查询、执行SQL语句等操作,使得数据库访问...

    JDBC通用DAO CRUD(不是Hibernate)

    自己写的一个JDBC通用DAO 有几点需要注意: 1.数据库命明目前只支持没下血杠命名(_) 2.表ID字段名取名有2种方式 (1)ID (2)TableName + ID 例如:user表 则取名为 id 或者 userid 因为没有用xml来映射表结构确定哪一个...

    神州通用数据库JDBC驱动和手册

    本文将深入探讨神通通用数据库的JDBC驱动及其使用方法,同时也会对提供的手册进行解读。 首先,让我们了解JDBC驱动的基本概念。JDBC驱动是Java应用程序连接到数据库的桥梁,它实现了Java API与数据库特定的通信协议...

    JDBC数据库访问技术

    JDBC数据库访问技术

    jdbc访问数据库\jdbc方式直接访问数据库环境的配置

    本文档旨在详细介绍如何通过JDBC方式直接访问数据库环境的配置方法。 #### 二、准备工作 为了确保能够顺利地使用JDBC访问数据库,需要进行以下准备工作: 1. **安装SQL Server 2000并打上sp3补丁**:确保SQL ...

    JDBC访问数据库步骤

    JDBC 访问数据库步骤 JDBC(Java Database Connectivity)是 Java 语言中用来访问数据库的 API,提供了一种标准的方式来连接和操作数据库。下面是 JDBC 访问数据库的步骤: 步骤 1:加载对应数据库驱动 在使用 ...

    JDBC数据访问接口

    JDBC(Java Database Connectivity)是Java编程语言中用于访问数据库的API,它是Java平台的标准,允许Java应用程序与各种数据库进行交互。JDBC提供了一组接口和类,使得开发者能够编写与数据库交互的代码,而无需...

    利用JSP实现基于JDBC_ODBC桥的Web数据库的访问

    本文详细介绍了JSP(Java Server Pages)技术以及JDBC-ODBC桥接技术,并结合Java Bean和JDBC(Java Database Connectivity)技术,展示了如何通过Web浏览器实现对SQL Server 2000数据库的访问。文章首先介绍了JSP的...

    达梦6jdbc通用驱动

    这是十分十分好用的达梦6jdbc通用驱动,欢迎大家下载啊

    通用的jdbc增删改查

    通用的jdbc增删改查,仅仅只是两个通用的方法,利用反射和泛型

    JDBC公共操作方法代码

    这篇名为“JDBC公共操作方法代码”的博客文章可能详细介绍了如何使用JDBC进行常见的数据库操作。下面我们将深入探讨JDBC的关键知识点。 首先,JDBC连接数据库的步骤通常包括以下几个部分: 1. 导入JDBC驱动:在...

    数据库概论实验范文实验八:通过JDBC方式访问数据库

    实验八的主题是“通过JDBC方式访问数据库”,这是数据库概论课程中的一项重要实践内容。JDBC(Java Database Connectivity)是Java语言中用于与各种数据库交互的一种标准接口,由Sun Microsystems开发并定义,现已...

    jdbc通用连接操作类

    在Java编程领域,JDBC(Java Database Connectivity)是Java语言中用来规范客户端程序如何访问数据库的应用程序接口,提供了诸如查询和更新数据库中数据的方法。"jdbc通用连接操作类"是开发者为了简化数据库操作,...

    android通过JDBC直接访问Mysql数据库

    ### Android通过JDBC直接访问MySQL数据库 #### 一、引言 随着移动应用开发的不断发展,Android应用程序常常需要与后端数据库进行交互以提供丰富的功能和服务。传统的Android应用与数据库的交互方式通常采用RESTful ...

    优化JDBC方法功略

    标题“优化JDBC方法攻略”和描述中提到的是关于JDBC(Java Database Connectivity)数据库操作的性能优化,适合初学者和教学使用。本文将详细介绍三个主要的JDBC性能优化技巧,并提供了一些通用的准则,以帮助开发者...

Global site tag (gtag.js) - Google Analytics