Live a simple life

沉默(zhu_xing@live.cn)
随笔 - 48, 文章 - 0, 评论 - 132, 引用 - 0
数据加载中……

【原创】关于JFace的一个bug(可编辑单元格双击,会频繁调用TableViewer的双击处理)

        闲话稍说,直接上bug视频:
        Bug视频
       大致现象:在一个可编辑tableviewer的单元格中,双击,对应的编辑对话框会不时弹出多次。我这边严重的时候,会连续弹出将近20次,那就只能修改了~_~

      Bug分析:
      调试了一把,发现是SWT底层将同一鼠标事件(通过event.time可以确认)连续发送了多次。而JFace中的TableEditorImpl将这事件转发给了tableviewer,tableviewer会进一步调用对应的double click listener。开发人员做的double click listener实现就是弹出一个编辑对话框。问题就出现了,当弹出的对话框关闭之后,马上有弹出了,上面说过,重复事件又被广播了~_~

    
     修改方法:
     时间较紧,就直接在TableEditorImpl这边修改了一把。将org.eclipse.jface插件以源码工程方式导入,将TableEditorImpl.java类做了一定的修改,加了一个双击事件的时间戳判断。

    【TableEditorImpl.java原有代码,Eclipse 3.2版本】
      mouseListener = new MouseAdapter() {
                        public void mouseDown(MouseEvent e) {
                            // time wrap? 
                            // check for expiration of doubleClickTime
                            if (e.time <= doubleClickExpirationTime) {
                                control.removeMouseListener(mouseListener);
                                cancelEditing();
                                 handleDoubleClickEvent();
                            } else if (mouseListener != null) {
                                control.removeMouseListener(mouseListener);
                            }
                        }
                    };

        【修改后代码】
         1、首先在TableEditorImpl中定义了一个双击事件的时间戳,缓存上次处理的双击事件的时间戳
         2、在转发双击事件之前,做时间戳检查,避免同一双击事件被处理了多次

             abstract class TableEditorImpl {
                ...其他代码省略

                //add by zhuxing:上次处理的双击事件的时间戳
                private int lastDoubleclickEventTime;

                private void activateCellEditor() {
                        ..................................
                        mouseListener = new MouseAdapter() {
                        public void mouseDown(MouseEvent e) {
                            // time wrap? 
                            // check for expiration of doubleClickTime
                            if (e.time <= doubleClickExpirationTime) {
                                control.removeMouseListener(mouseListener);
                                cancelEditing();
                                
                                //检查本次要处理的双击事件是否和上次的事件重复
                                if (e.time != lastDoubleclickEventTime) {
                                     lastDoubleclickEventTime = e.time;
                                     handleDoubleClickEvent();
                                }

                            } else if (mouseListener != null) {
                                control.removeMouseListener(mouseListener);
                            }
                        }
                    };
                }
            }

        

         希望能帮到遇到类似问题的同学

本博客中的所有文章、随笔除了标题中含有引用或者转载字样的,其他均为原创。转载请注明出处,谢谢!

posted on 2008-08-01 11:20 zhuxing 阅读(1678) 评论(1)  编辑  收藏

评论

# re: 【原创】关于JFace的一个bug(可编辑单元格双击,会频繁调用TableViewer的双击处理)  回复  更多评论   

应该在bugs.eclipse.org反映一下啊
2008-08-01 11:29 | Always BaNg.

只有注册用户登录后才能发表评论。


网站导航: