大家都来看看这段代码，看看缺陷在哪里？

类名最好是大写字母作为起始字符，如：ReadNote
这样也能避免你的类名和方法readNote()同名。

解决方案 »

  1.   

    作为外部访问的方法，当执行这些方法出现exception的时候，你的返回对象处理。
    从现有的程序看，需要对返回对象做判断，这种处理方法是否足够，请楼主再考虑一下。
      

  2.   

    to: every one提的非常好!继续呀顶!!!!!!!!!!!!!
      

  3.   

    public String readNote(String url)
    _____________________________________
    这个错误比较严重啊，构造函数不能有返回类型的
      

  4.   

    to:： lins(Anders*小明) 你认为执行这些方法出现exception的时候,该怎么去处理呢 ?
      

  5.   

    严重同意,构着函数不应该有返回值,或者改类名或者改变方法名称,类名称最好大写.例外应该分层,try后名跟多个catch,文件未找到或者io例外等等.
      

  6.   

    如果你希望这个类的对象可以完成多个文件的读写，最好添加一个复位函数初始化所有内部数据成员。如下面这段：
    try {
    if(fis==null)
    fis = new FileInputStream(file);}
    如果第二次给不同的名字来读，则fis仍然会读出上一次的内容。 file = new String(bt);返回的名字最好写成fileContent之类以免误会。
      

  7.   

    to: shuneng() 你说的复位方法该怎么写？？？？？？？？？？？？？？？？