|
锁定老帖子 主题：新浪微博API一定是实习生写的
该帖已经被评为良好帖
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2011-11-08
最后修改：2011-11-08
advantech 写道 这段代码3个问题：
1.文件路径的问题，使用src这个不是特别的合理，有n多的办法来指定一个和平台无关的路径，不管是类路径也好，绝对路径也罢。 2.没关闭流。 3.异常处理不应该e.printStackTrace();草草了事。 advantech说的很全面。但我觉得src这个问题最大，SDK必须能正常作为一个三方库引用到别人工程里面，怎么可能像源文件一样拷来拷去。我贴这个CODE不是为了证明我有多牛逼，而是觉得这种错误是我们JAVA程序员最不应该犯的，特别是像新浪这种大公司里面。 |
| 返回顶楼 | |
|
发表时间：2011-11-08
洪帮主 写道 这代码太多问题了。楼主哪来的代码啊？
文件路径是一个问题， 读文件是一个问题， 构造方法没有私有化， 工具方法没有将类final， 异常处理太坑爹了， 方法命名很有问题， prop居然还改里面的属性。  这位说得比较全呢。
|
| 返回顶楼 | |
|
发表时间：2011-11-08
jacksv123 写道 pipilu 写道 jacksv123 写道 是不是没关闭流啊？请楼主说明一下理由啊！
应该从类路径读取文件，而不是一个src/***这样的相对路径。 我菜鸟一个想请教一下，这样做会有什么区别？ 做成相对路径，部署起来肯定要麻烦一些了。 |
| 返回顶楼 | |
|
发表时间：2011-11-08
这个代码写的很好呀，，非常的漂亮，而且简单到极点，最求的就是简单代码。
|
| 返回顶楼 | |
|
发表时间：2011-11-08
路径可以找类编译的路径：
this.getClass().getResource("/").getPath() + "config.properties" |
| 返回顶楼 | |
|
发表时间：2011-11-08
我特别想问一句 这种config类 不应该设计成单例么？
|
| 返回顶楼 | |
|
发表时间：2011-11-08
up up up
this kind of code is just like students wrote. |
| 返回顶楼 | |
|
发表时间：2011-11-08
saturn 写道 这个代码写的很好呀，，非常的漂亮，而且简单到极点，最求的就是简单代码。
其实就是这样的，一个demo而已 |
| 返回顶楼 | |
|
发表时间：2011-11-08
InputStream in = null;
try {
in = AbstractBankRequestHandler.class.getResourceAsStream("xxx.properties");
props.load(in);
} catch (IOException e) {
logger.warn("加载xxx属性文件(AbstractBankRequestHandler所在类路径下xxx.properties)异常："+e.getMessage());
} finally {
if(in!=null){
try {
in.close();
} catch (IOException e) {
}
}
}
俺的习惯性写法~LZ的意思是应该关闭流，记日志，能用JDK提供的类，就用JDK提供的类吧 ?!!! |
| 返回顶楼 | |
|
发表时间：2011-11-08
只是example，而且是eclipse工程，又不是产品发布，不必较真。example只是展示api的使用，自己的程序当然需要好好规划...
|
| 返回顶楼 | |
