11个答案 按时间排序 按投票排序

0 0

采纳的答案

不用switch的那个方法稍作修改

    public void textIdentify(File f)
    {
        HashMap<String, String> globalMode = new HashMap<String, String>();
        String[] input = readAllFile(f).replaceAll("   ", ";").split(";");
        for (int i = 0; i < input.length; i++)
        {
            String re = input[i].replace(",", " ").trim();
            String[] reSplited =  re.split(" - ");
            
            String key = KEY_MAP.get(reSplited[0]);

            if (key == "Author")
            {
                String authorNames = globalMode.get(key);
                authorNames = (authorNames == null) ? reSplited[1] : (authorNames
                        + "," + reSplited[1]);

                globalMode.put(key, authorNames);
            }
            else
            {
                globalMode.put(key, reSplited[1]);
            }
        }
    }

有点疑问的地方
1. 你的代码里变量booktitle, title, author, year, isbn分别是做什么用处的, 如果仅这个方法里面用, 那么可以想我那样, 直接不用.
2.

String[] input = readAllFile(f).replaceAll("   ", ";").split(";");

这里replace完再split是否有必要, 是否可以直接

String[] input = readAllFile(f).split(""   "");

3.

String re = input[i].replace(",", " ").trim();

2,3两个代码我觉得比较奇怪, 不过你可能有你自己的意图啦.

2008年12月23日 19:56

dch1287
3
0 0 1

• 添加评论

0 0

说几个我的观点，
   1）if/switch在java6下是没有差别的， 在JAVA5才有差别， 而且不小。
   2）另外， startsWith其实性能还是很高的， 以你的情况， 比较一个字符就出结果的。
  楼上puroc的方法给的很好， 我也是这么类似做的。

  除非你对性能有极高的要求， 我不建议你使用这么复杂的做法， 除非你做类似html分析之类事情， 处理tag的要求高。

2008年12月23日 22:30

sdh5724
874
0 0 1

• 添加评论

0 0

低级错误了 呵呵呵呵

2008年12月23日 20:03

dch1287
3
0 0 1

• 添加评论

0 0

if (key == "Author")

改成

if ("author".equals(key)) 

 

2008年12月23日 20:02

dch1287
3
0 0 1

• 添加评论

0 0

不用switch的那个方法稍作修改

后

再做修改

if ("Author".equals(key))

改成

if ("author".equals(key))

2008年12月23日 20:01

dch1287
3
0 0 1

• 添加评论

0 0

还有一点 你的globalMode最后要返回给方法调用者不?

2008年12月23日 19:58

dch1287
3
0 0 1

• 添加评论

0 0

用enum和switch大概是这个样子的

    static enum KeyEnum
    {
        booktitle, title, author, year, isbn
    }

    private final static Map<String, KeyEnum> KEY_MAP;

    static
    {
        KEY_MAP = new HashMap<String, KeyEnum>();
        KEY_MAP.put("Book Title", KeyEnum.booktitle);
        KEY_MAP.put("Chapter Title", KeyEnum.title);
        KEY_MAP.put("Author", KeyEnum.author);
        KEY_MAP.put("Copyright", KeyEnum.year);
        KEY_MAP.put("isbn", KeyEnum.isbn);
    }

    public void textIdentify(File f)
    {
        HashMap<String, String> globalMode = new HashMap<String, String>();
        String[] input = readAllFile(f).replaceAll("   ", ";").split(";");
        for (int i = 0; i < input.length; i++)
        {
            String re = input[i].replace(",", " ").trim();
            String[] reSplited = re.split(" - ");

            KeyEnum key = KEY_MAP.get(reSplited[0]);

            switch (key)
            {
            case booktitle:
                globalMode.put(KeyEnum.booktitle.name(), reSplited[1]);
                break;
            case title:
                globalMode.put(KeyEnum.title.name(), reSplited[1]);
                break;
            case author:
                String authorNames = globalMode.get(key.name());
                authorNames = (authorNames == null) ? 
                        reSplited[1] : (authorNames + "," + reSplited[1]);
                globalMode.put(key.name(), authorNames);
                break;
            case year:
                globalMode.put(KeyEnum.year.name(), reSplited[1]);
                break;
            case isbn:
                globalMode.put(KeyEnum.isbn.name(), reSplited[1]);
                break;
            default:
                // do nothing here.
            }
        }
    }

2008年12月23日 19:47

dch1287
3
0 0 1

• 添加评论

0 0

你的这个情况用 enum 和 switch 不太现实

试了试 大概也就这样了

    private final static Map<String, String> KEY_MAP;
    
    static {
        KEY_MAP = new HashMap<String, String>();
        KEY_MAP.put("Book Title", "booktitle");
        KEY_MAP.put("Chapter Title", "title");
        KEY_MAP.put("Author", "author");
        KEY_MAP.put("Copyright", "year");
        KEY_MAP.put("isbn", "isbn");
    }

    public void textIdentify(File f)
    {
        HashMap<String, String> globalMode = new HashMap<String, String>();
        String[] input = readAllFile(f).replaceAll("   ", ";").split(";");
        for (int i = 0; i < input.length; i++)
        {
            String re = input[i].replace(",", " ").trim();
            String[] reSplited =  re.split(" - ");

            if (reSplited[0] == "Author")
            {
                String authorKey = KEY_MAP.get(reSplited[0]);

                String authorNames = globalMode.get(authorKey);
                authorNames = (authorNames == null) ? reSplited[1] : (authorNames
                        + "," + reSplited[1]);

                globalMode.put(authorKey, authorNames);
            }
            else
            {
                globalMode.put(KEY_MAP.get(reSplited[0]), reSplited[1]);
            }
        }
    }

1. 用switch并不能提高效率
2. 你的关键判断 re.startsWith("XXXXX") 本身无法转程enum 从而使用switch
   如果换成我的 reSplited[0] 然后再弄成enum仍然很麻烦, 我认为没必要那样写

2008年12月23日 19:46

dch1287
3
0 0 1

• 添加评论

0 0

用enum，然后你就可以用switch了。

2008年12月23日 12:37

puroc
134
0 0 1

• 添加评论

0 0

	private Map<String, String> GlobalMode;

	/**
	 * name 和 key 的关系对应
	 */
	private String[][] nameKeys = { { "Book Title", "booktitle" }, { "Chapter Title", "title" }, { "Author", "author" }, { "Copyright", "year" },
			{ "isbn", "isbn" } };

	/**
	 * <p>Description: 从一个项目得到key. 比如:从"Book Title - 对应值" 得到 "Book Title" 对应的 "booktitle"  </p>
	 */
	private String getKey(String item) {
		String name = item.substring(0, item.indexOf("-")).trim();
		for (int i = 0; i < nameKeys.length; i++) {
			String[] nameItem = nameKeys[i];
			if (name.equals(nameItem[0])) {
				return nameItem[1];
			}
		}
		return null;
	}

	/**
	 * <p>Description: 从一个项目得到value. 比如:从"Book Title - 对应值" 得到 "对应值"  </p>
	 */
	private String getValue(String item) {
		return item.substring(item.indexOf("-") + 1).trim();
	}

	private void initParse() {
		GlobalMode = new HashMap<String, String>();
	}

	private void parseConent(String content) {
		String[] input = content.split(";");
		for (String item : input) {
			String value = getValue(item);
			String key = getKey(item);
			if (key.equals("author")) {
				if (GlobalMode.get(key) != null) {
					value = GlobalMode.get(key) + value;
				}
			}
			GlobalMode.put(key, value);
		}
	}

	public static void main(String[] args) {
		A a = new A();
		a.initParse();
		a.parseConent("Book Title - 对应值; Author - 对应值;");
		System.out.println(a);
	}

只会比你复杂, 没有你的简单.

2008年12月23日 10:23

lggege
1787
0 0 0

• 添加评论

0 0

switch（）
中间的条件只能是 int,byte,char,short

所以, 你的不可以用switch

2008年12月23日 09:34

lggege
1787
0 0 0

• 添加评论