为什么不能使用成员地图来存储工具栏?

时间:2016-06-22 17:02:06

标签: javascript qooxdoo

我定义了以下窗口:

qx.Class.define('my.Window', {
   ...

   construct: function(caption, icon) {
       this.base(arguments, caption, icon);
       this.setLayout(new qx.ui.layout.VBox(10));

       this.add(new qx.ui.form.renderer.Single(this.getForm('some')));
       this.add(new qx.ui.form.renderer.Single(this.getForm('other')));

       this.add(this.getToolbar('test'));  // This is shown
       this.add(this.getToolbar('some'));  // This is not shown
       this.add(this.getToolbar('other')); // This is not shown
   },

   members: {
      _forms: {},
      _toolbars: {},

      getForm: function(id) {
          if(this._forms[id]) return this._forms[id];

          if(id = 'some') this._forms[id] = new Form();
          else if(id == 'other') this._forms[id] = new OtherForm();

          return this._forms[id];
      },

      getToolbar: function(id) {
          if(id == 'test') {
             if(this._tb) return this._tb;
             this._tb = new TestToolbar();
             return this._tb;
          }
          else if(id == 'some') this._toolbars[id] = new SomeToolbar();
          else if(id == 'other') this._toolbars[id] = new OtherToolbar();

          return this._toolbars[id];              
      }
  }
});

当我显示窗口时,会显示表单('some'和'other')。还会显示其他项目,例如TestToolbar和List。但是,其他工具栏('test','some'和'other')未显示..

我尝试将_toolbars从对象更改为数组,但我也有相同的行为。

1 个答案:

答案 0 :(得分:3)

在您的示例中,getToolbar()方法将始终创建一个新的工具栏,无论如何;这意味着当构造函数将向窗口添加一个新工具栏时,如果您随后调用getToolbar()并向其添加小部件,则会添加到错误的工具栏(即添加到尚未添加到窗口的工具栏)。

以下是您在游乐场中运行的示例的略微修改版本

qx.Class.define('my.Window', {
   extend: qx.ui.window.Window,

   construct: function(caption, icon) {
       this.base(arguments, caption, icon);
       this.setLayout(new qx.ui.layout.VBox(10));

       this._toolbars = {};

       //commented out because
       //this.add(new qx.ui.form.renderer.Single(this.getForm('some')));
       //this.add(new qx.ui.form.renderer.Single(this.getForm('other')));

       this.add(this.getToolbar('test'));  // This is shown
       this.add(this.getToolbar('some'));  // This is not shown
       this.add(this.getToolbar('other')); // This is not shown

       this.getToolbar('test').add(new qx.ui.toolbar.Button("Test Button"));
       this.getToolbar('some').add(new qx.ui.toolbar.Button("Some Button"));
       this.getToolbar('other').add(new qx.ui.toolbar.Button("Other Button"));
   },

   members: {
      _forms: null,
      _toolbars: null,

      getForm: function(id) {
          if(this._forms[id]) return this._forms[id];

          if(id == 'some') this._forms[id] = new Form();
          else if(id == 'other') this._forms[id] = new OtherForm();

          return this._forms[id];
      },

      getToolbar: function(id) {
        if (!this._toolbars[id]) {
          if(id == 'test') {
             if(this._tb) return this._tb;
             this._tb = new qx.ui.toolbar.ToolBar();
             return this._tb;
          }
          else if(id == 'some') this._toolbars[id] = new qx.ui.toolbar.ToolBar();
          else if(id == 'other') this._toolbars[id] = new qx.ui.toolbar.ToolBar();
        }
        return this._toolbars[id];              
      }
  }
});
var win = new my.Window("First Window");
win.setWidth(300);
win.setHeight(200);
win.setShowMinimize(false);

this.getRoot().add(win, {left:20, top:20});
win.open();