【问题标题】:Reusing a single object instead of creating new ones重用单个对象而不是创建新对象
【发布时间】:2017-04-12 14:19:17
【问题描述】:

案例研究
我有一种方法可以扫描文件系统中的某些条目,并在找到时将它们存储为新对象。 这些对象包括几个用于入口位置的字符串和表示入口状态的标志。

由于我实例化的每个新对象占用 > 16 个字节,如果找到 1000 个条目,我将使用 (16 * 1000) 个字节!由于我一次只需要一个实例,因此我想重用一个对象(即一次使用 new)并更新其字段。

这就是我现在处理对象的方式:

for (final Object object : scanFor(PATTERN)) {
  if (aName.equals(object.getName()) {
    ... use the object
  }
}

我想我可以使用与 ResultSet 相同的技术。
有什么建议吗?

附: 我知道 16kb 的内存不算什么,但这只是一个例子。

编辑:我看到有人投票结束我的问题,因为它不清楚。老实说,我真的不知道还能写什么。

编辑,真实代码:
调用者

DspPgmRef dspPgmRef = new DspPgmRef("MYLIBRARY", "*ALL", J400Type.ALL);
dspPgmRef.addFilter(DspPgmRefEntryField.REFERENCED_OBJECT, name);

for (final DspPgmRefEntry entry : dspPgmRef.run()) {
   if (entry.referencedObject.equals(name)) {
      users.add(J400ObjectFactory.get("*LIBL", entry.object));
   }
}

DspPgmRef 实现:

public final class DspPgmRef implements J400Command<List<DspPgmRefEntry>>
{
   /**
    * Rappresenta un valore di ritorno del comando
    */
   public static class DspPgmRefEntry
   {
      public final String library;
      public final String object;
      public final String referencedLibrary;
      public final String referencedObject;
      public final String referencedType;

      public DspPgmRefEntry(
            final String library,
            final String object,
            final String referencedLibrary,
            final String referencedObject,
            final String referencedType) {
         this.library = library;
         this.object = object;
         this.referencedLibrary = referencedLibrary;
         this.referencedObject = referencedObject;
         this.referencedType = referencedType;
      }
   }

   /**
    * Campi dei riferimenti ritornati dal comando
    */
   public enum DspPgmRefEntryField
   {
      LIBRARY("WHLIB"),
      OBJECT("WHPNAM"),
      REFERENCED_LIBRARY("WHLNAM"),
      REFERENCED_OBJECT("WHFNAM"),
      REFERENCED_TYPE("WHOTYP");

      /**
       * Nome reale del campo di database
       */
      public final String value;

      private DspPgmRefEntryField(final String value) {
         this.value = value;
      }
   }

   private final String _library;
   private final String _object;
   private final J400Type _type;
   private final Map<DspPgmRefEntryField, String> _filters;

   {
      _filters = new EnumMap<>(DspPgmRefEntryField.class);
   }

   public DspPgmRef(final String library, final String object, final J400Type type) {
      _library = library;
      _object = object;
      _type = type;
   }

   public DspPgmRef(final J400GenericObject object) {
      this(object.getLibrary(), object.getName(), object.getType());
   }

   @Override
   public List<DspPgmRefEntry> run() {
      final String tempLibrary = J400Util.getRandomAS400String();
      final String tempFile = J400Util.getRandomAS400String();
      final int filtersSize = _filters.size();

      final StringBuilder builder = new StringBuilder(120 + filtersSize * 20);
      builder.append("DSPPGMREF PGM(");
      builder.append(_library);
      builder.append("/");
      builder.append(_object);
      builder.append(") OUTPUT(*OUTFILE) OBJTYPE(");
      builder.append(_type.ASType);
      builder.append(") OUTFILE(");
      builder.append(tempLibrary);
      builder.append("/");
      builder.append(tempFile);
      builder.append(")");

      final J400Connection connection = J400Connection.instance();
      connection.createLibrary(tempLibrary, "Libreria temporanea per DSPDBR");

      if (!connection.executeCommand(builder.toString()).containsKey("CPF3030")) {
         return Collections.EMPTY_LIST;
      }

      builder.delete(0, builder.length());
      builder.append("select distinct WHPNAM, WHLNAM, WHFNAM, WHOTYP from ");
      builder.append(tempLibrary);
      builder.append(".");
      builder.append(tempFile);

      if (filtersSize > 0) {
         builder.append(" where ");

         for (final Entry<DspPgmRefEntryField, String> filter : _filters.entrySet()) {
            builder.append(filter.getKey().value);
            builder.append(" = '");
            builder.append(filter.getValue());
            builder.append("' and ");
         }

         final int stringLenght = builder.length();
         builder.delete(stringLenght - 5, stringLenght);
      }

      PreparedStatement statement = null;
      ResultSet result = null;

      try {
         statement = connection.getConnection().prepareStatement(builder.toString());
         result = statement.executeQuery();

         final List<DspPgmRefEntry> references = new ArrayList<>(128);

         while (result.next()) {
            references.add(new DspPgmRefEntry(
                  _library,
                  result.getString(DspPgmRefEntryField.OBJECT.value).trim(),
                  result.getString(DspPgmRefEntryField.REFERENCED_LIBRARY.value).trim(),
                  result.getString(DspPgmRefEntryField.REFERENCED_OBJECT.value).trim(),
                  result.getString(DspPgmRefEntryField.REFERENCED_TYPE.value).trim()));
         }

         return Collections.unmodifiableList(references);
      } catch (final SQLException e) {
         e.printStackTrace();
      } finally {
         try {
            if (result != null) {
               result.close();
            }

            if (statement != null) {
               statement.close();
            }
         } catch (final SQLException e) {
            e.printStackTrace();
         }

         connection.deleteLibrary(tempLibrary);
      }

      return Collections.EMPTY_LIST;
   }

   public void addFilter(final DspPgmRefEntryField field, final String value) {
      _filters.put(field, value);
   }

   public void removeFilter(final DspPgmRefEntryField field) {
      _filters.remove(field);
   }

   public void clearFilters() {
      _filters.clear();
   }
}

【问题讨论】:

  • 在scanFor中使用lambda函数怎么样?
  • @brummfondel mmmh 我不明白你的意思,你能再解释一下吗?
  • 请发布扫描以获得更好的帮助。
  • 不清楚您想要什么,因为您说您只想更新其字段,这很简单:您编写 setter 方法。这是你想知道的吗?
  • 我从问题标题中删除了 Immutable 这个词,因为根据您的问题它显然是错误的。基本上,据我了解,您是在询问在迭代过程中一遍又一遍地重用同一对象(在每一步更新其字段)与创建新对象实例的效果。

标签: java arrays object optimization memory-management


【解决方案1】:

假设您的建议是从这个 sn-p 中的 scanFor 方法返回 相同的对象实例 对性能的影响和代码异味:

for (final Object object : scanFor(PATTERN)) {
  if (aName.equals(object.getName()) {
    ... use the object
  }
}

根据其余代码的详细信息(例如,equals 检查的频率是否正确以及... use the object 中发生的情况),这可能确实会大大节省分配率。然而,它确实违反了Iterable&lt;T&gt;.iterator().next() 的普遍理解行为——即它每次都返回一个唯一的对象。这已在some length before 讨论。如果您使用此技巧衡量对您而言意义重大的性能提升,它可能仍然值得,但一如既往地衡量。

Java Mission Control 包含在 (Oracle) JDK 中,可免费用于非商业用途,并且可以测量实际对象分配率和位置。您可以使用它来验证临时对象是否是整体分配的重要部分(在某些情况下,内联 + 转义分析可能能够完全消除它们,从而使您假设的问题变得毫无意义)。

【讨论】:

  • 基本上你建议创建一个自定义迭代器,它总是返回相同的对象,只是改变了属性。
  • 是的,我绝对需要学习如何使用任务控制!你是对的!
  • @LppEdd - 明确地说,这就是我认为 you 的建议,我正在列出一些利弊。你说:“因为我一次只需要一个实例,我想重用一个对象(即一次使用 new )并更新它的字段。” - 然后你展示了一个使用 foreach 的循环(需要 Iterable)。您可以将“一个对象,变异”方法与 Iterable 一起使用,或者您可以更改循环结构并避免使用 Iterable,这可能会更清晰一些(因为代码的未来读者可以明确地看到一个对象被一遍又一遍地修改)。
  • 如果您查看上面发布的代码(真实代码),您可能最终会说迭代器是最佳选择,因为我还需要获取结果集。谢谢!
猜你喜欢
  • 2017-04-23
  • 2013-07-06
  • 2017-02-25
  • 1970-01-01
  • 2012-04-24
  • 1970-01-01
  • 2012-07-15
  • 2015-07-26
相关资源
最近更新 更多