论坛首页 Java企业应用论坛

新浪微博API一定是实习生写的

浏览 48870 次
该帖已经被评为良好帖
作者 正文
   发表时间:2011-11-07   最后修改:2011-11-08
新浪API一定是实习生写的,居然直接这样读取文件:
props.load(new FileInputStream(“src/config.properties”));

package weibo4j.util;

import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.util.Properties;

public class WeiboConfig {
	private static String filePath = "src/config.properties";
	public WeiboConfig(){}
	private static Properties props = new Properties(); 
	static{
		try {
			props.load(new FileInputStream(filePath));
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
	public static String getValue(String key){
		return props.getProperty(key);
	}

    public static void updateProperties(String key,String value) {    
            props.setProperty(key, value); 
    } 
}


advantech 写道
这段代码3个问题:
1.文件路径的问题,使用src这个不是特别的合理,有n多的办法来指定一个和平台无关的路径,不管是类路径也好,绝对路径也罢。
2.没关闭流。
3.异常处理不应该e.printStackTrace();草草了事。

advantech说的很全面。但我觉得src这个问题最大,SDK必须能正常作为一个三方库引用到别人工程里面,怎么可能像源文件一样拷来拷去。有人说你牛逼你怎么不去新浪去写,这种逻辑就像看球的不能骂踢球的?何况我也算业余选手吧。我贴这个CODE不是为了证明我有多牛逼,而是觉得这种错误是我们JAVA程序员最不应该犯的,特别是像新浪这种大公司里面。

有朋友问哪里的代码,贴个地址:http://open.weibo.com/wiki/SDK#Java_SDK
   发表时间:2011-11-07  
国内的it企业,大牛基本是不码代码的,不是实习生还是谁呢……
所以不要以为大企业的代码就一定牛逼,其实,未必比得上你呢
0 请登录后投票
   发表时间:2011-11-07  
写得还可以啊,没啥问题。
0 请登录后投票
   发表时间:2011-11-07  
有什么问题,实在看不出来?
0 请登录后投票
   发表时间:2011-11-07  
src?          
0 请登录后投票
   发表时间:2011-11-07  
新浪做JAVA有多久哦?
0 请登录后投票
   发表时间:2011-11-07  
你的意思是读文件不关流???
0 请登录后投票
   发表时间:2011-11-07  
新浪微博不是用php写的么?
0 请登录后投票
   发表时间:2011-11-07  
san_yun 写道
新浪API一定是实习生写的,居然直接这样读取文件:
props.load(new FileInputStream(“src/config.properties”));

package weibo4j.util;

import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.util.Properties;

public class WeiboConfig {
	private static String filePath = "src/config.properties";
	public WeiboConfig(){}
	private static Properties props = new Properties(); 
	static{
		try {
			props.load(new FileInputStream(filePath));
		} catch (FileNotFoundException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
	public static String getValue(String key){
		return props.getProperty(key);
	}

    public static void updateProperties(String key,String value) {    
            props.setProperty(key, value); 
    } 
}


楼主的意思 应该是不用new FileInputStream(filePath)这样就可以读取文件的,其实java有现成的方法获取class下面的文件,把它转换成流
0 请登录后投票
   发表时间:2011-11-07  
异常 就这样了  e.printStackTrace(); 
0 请登录后投票
论坛首页 Java企业应用版

跳转论坛:
Global site tag (gtag.js) - Google Analytics