spreadsheet打开多个标签页，关闭最后打开的一个，点击之前标签页的单元格报错。

收藏

本帖最后由李大大于 2017-7-19 17:18 编辑

联系方式：

产品版本: gcspread.sheets.all.9.40
浏览器版本: chrome 58.0.3029.81 (64-bit)
问题描述：打开多个创建的标签页，关闭最后打开的一个，点击之前标签页的单元格报错。sheet对象不正确了，sheet.getValue()方法报错。
关键代码:

/**
* 添加cell click监听事件
* @param id
*/
public native void addCellClickListener(JavaScriptObject spread) /*-{
var $ = $wnd.$;
sheet=spread.getActiveSheet();
var app=this;
sheet.bind($.wijmo.wijspread.Events.CellClick, function (sender, args) {
if(args.sheetArea === $.wijmo.wijspread.SheetArea.colHeader){
return;
}
if(args.sheetArea === $.wijmo.wijspread.SheetArea.rowHeader){
return;
}
if(args.sheetArea === $.wijmo.wijspread.SheetArea.corner){
return;
}
console.log("Clicked column index: " + args.col);
console.log("Clicked row index: " + args.row);
console.log("Clicked cell value: " + sheet.getValue(args.row,args.col));
if(args.col== undefined || args.row==undefined){
return;
}
app.@com.pansoft.query.comp.QueryView::processCellClickScript(Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)(args.row,args.col,sheet.getValue(args.row,args.col));
});
}-*/;

复制代码

使用截图：单个页面点击截图 image.png780174560.png

l._getModel	@	gcspread.sheets.all.9.40.20171.0.min.js:22420
l.getValue	@	gcspread.sheets.all.9.40.20171.0.min.js:19831
(anonymous)	@	GWTEnterprise-0.js:601145
s	@	gcspread.sheets.all.9.40.20171.0.min.js:36
(anonymous)	@	gcspread.sheets.all.9.40.20171.0.min.js:89
trigger	@	gcspread.sheets.all.9.40.20171.0.min.js:78
rt.trigger	@	gcspread.sheets.all.9.40.20171.0.min.js:15423
l._trigger	@	gcspread.sheets.all.9.40.20171.0.min.js:25132
l.triggerCellClick	@	gcspread.sheets.all.9.40.20171.0.min.js:25138
rt.doMouseUpImp	@	gcspread.sheets.all.9.40.20171.0.min.js:14868
rt.doMouseUp	@	gcspread.sheets.all.9.40.20171.0.min.js:14846
(anonymous)	@	gcspread.sheets.all.9.40.20171.0.min.js:12924
s	@	gcspread.sheets.all.9.40.20171.0.min.js:36

Clark.Pan · Clark.Pan

李大大发表于 2017-7-12 16:21
非常感谢技术人员的支持。经过测试发现使我们使用上的问题。图中的sheet没用声明，变全局变量了。问题已 ...

不客气，有什么问题欢迎发帖进行询问。

李大大 · 李大大

ClarkPan 发表于 2017-7-11 14:07
不进行destroy打开的会占用内存无法释放，打开多了可能会出现内存被占满的问题

非常感谢技术人员的支持。经过测试发现使我们使用上的问题。图中的sheet没用声明，变全局变量了。问题已解决。

Clark.Pan · Clark.Pan

李大大发表于 2017-7-11 13:45
如果不进行destroy会有什么问题？

不进行destroy打开的会占用内存无法释放，打开多了可能会出现内存被占满的问题

李大大 · 李大大

ClarkPan 发表于 2017-7-11 11:34
看了以上代码，表面上没看出什么问题，因为只有代码片段，而且您这边提供不了重现问题的demo，并且重现问 ...

如果不进行destroy会有什么问题？

Clark.Pan · Clark.Pan

李大大发表于 2017-7-11 09:47
加载数据的时候会走buildView方法。通过initExcel方法创建并返回spread对象。this.setSpread(spreadObj); ...

看了以上代码，表面上没看出什么问题，因为只有代码片段，而且您这边提供不了重现问题的demo，并且重现问题的步骤也很笼统，所以很难定位问题，这样，您试试逐步注释要无关逻辑的代码，当剩下核心相关逻辑时，再进行逐行注释排查问题，这样能得到具体的一些问题信息，然后发给我们，我们也能够尽快重现问题。

李大大 · 李大大

ClarkPan 发表于 2017-7-10 18:05
那您getSpread()的方法中是否确保各自打开的spread互相独立。这里的代码能否提供下，还有就是您各个sprea ...

public native JavaScriptObject initExcel(String id, String strJSon,int count,String frozenColor) /*-{
var $ = $wnd.$;
var ss = "#"+id;
$(ss).wijspread({sheetCount:count}); // create wijspread control
var spread = $(ss).wijspread("spread"); // get instance of wijspread control
//挂起绘制
spread.isPaintSuspended(true);
spread.fromJSON(JSON.parse(strJSon));
spread.tabStripVisible(true);
spread.grayAreaBackColor("white");
var sheet = spread.getActiveSheet();
sheet.setGridlineOptions({
color: "#D9D9D9",
showVerticalGridline:false,
showHorizontalGridline:false
});
sheet.frozenlineColor(frozenColor);
return spread;
}-*/;

复制代码

private void buildView(){
String strJSon="";
if(formatData.length==2){
strJSon=getExcelFormatAndDataJson(formatData);
}else if(formatData.length==3 || formatData.length==4){
String[] newFormatData={formatData[1],formatData[2]};
strJSon=getExcelFormatAndDataJson(newFormatData);
}else if(formatData.length==5 || formatData.length==6){
strJSon=showBNBQData(this.QO,formatData);
}
if(strJSon==null || strJSon.equals("")){
return;
}
this.viewJson=strJSon;
JavaScriptObject spreadObj=null;
String ldtype=this.PO.getString("ldtype", "init");
if(ldtype.equals("init")){
spreadObj=initExcel(this.getViewId(),strJSon,1,frozenLineColor);
}else if(ldtype.equals("refresh")){
spreadObj=initExcelData(this.getViewId(),strJSon,1,frozenLineColor);
}
this.setSpread(spreadObj);
//设置锁定列
int fccount=0;
if(this.frozenColCount!=null && this.frozenColCount.trim().length()>0){
fccount=Integer.parseInt(this.frozenColCount);
}
initExcelLockColumn(this.getSpread(),fccount);
//设置分级
initExcelGroup(this.getSpread(),0);
if(hideRowHeader){
this.hideRowHeader(this.getSpread());
}
if(hideColHeader){
this.hideColHeader(this.getSpread());
}
if(!hideFooter){
this.hideFooter(this.getSpread());
}
String hideScroll=this.PO.getString("hideScroll", "");
if(hideScroll.equals("1")){
hideHorizontalScrollbar(this.getSpread());
hideVerticalScrollbar(this.getSpread());
}
//设置超链接样式
if(linkQueryCols!=null && linkQueryCols.trim().length()>0){
String lkCol[]=linkQueryCols.split(",");
for(int c=0;c<lkCol.length;c++){
int hcolIndex=Integer.parseInt(lkCol[c]);
setHyperlinkStyle(this.getSpread(),0,this.titleNum+this.headerNum,hcolIndex,"#0000FF");
}
}
addCellClickListener(this.getSpread());
processSameContentSpan();//处理相同内容是否合并
setProtectCell(this.getSpread());
setAutoFitRow(this.getSpread());
//激活重新绘制
suspendPaint(this.getSpread(),false);
}

复制代码

加载数据的时候会走buildView方法。通过initExcel方法创建并返回spread对象。this.setSpread(spreadObj);将spread对象放入。addCellClickListener(this.getSpread());添加监听，即本帖第一个方法代码。

Clark.Pan · Clark.Pan

李大大发表于 2017-7-10 17:18
但是我销毁的只是我关闭掉的spread。
按照打开的顺序123关闭，不会出问题，按照321顺序关闭才会出错。

那您getSpread()的方法中是否确保各自打开的spread互相独立。这里的代码能否提供下，还有就是您各个spread初始化的代码也请提供。感觉可能是您的spread对象之间引用混乱造成。

李大大 · 李大大

ClarkPan 发表于 2017-7-10 16:32
应该是这里的问题，因为destroy掉对象后，重新切换会对象时需要将对象的所有事件全部重新绑定，之前您在s ...

但是我销毁的只是我关闭掉的spread。
按照打开的顺序123关闭，不会出问题，按照321顺序关闭才会出错。

Clark.Pan · Clark.Pan

李大大发表于 2017-7-10 16:25
你好，切换是没有问题的，关闭之前的也不会出现问题，只有关闭刚打开的，再点击之前的才会出现问题。因为 ...

应该是这里的问题，因为destroy掉对象后，重新切换会对象时需要将对象的所有事件全部重新绑定，之前您在sheet上绑定了CellClick事件，destroy后重新切换回来是需要重新绑定的（因为destroy spread 后sheet都没了，上面的事件肯定也没有了）。

		自动登录	找回密码
密码			立即注册

[已处理] spreadsheet打开多个标签页，关闭最后打开的一个，点击之前标签页的单元格报错。

14 个回复

浏览过的版块

讲师达人

悬赏达人

微信认证勋章

SpreadJS 认证

SpreadJS 高级认证

元老葡萄