java - 使用实用程序生成 Java 代码使我的项目更加简洁。好主意？

Question

我正在从事的项目需要我编写大量重复代码。例如，如果我想在我的代码中加载一个名为“logo.png”的图像文件，我会这样写:位图标志图片；

...
// Init
logoImage = load("logo.png")

...
// Usage
logoImage.draw(0, 0);

..
// Cleanup
logoImage.release();

必须编写此代码才能使用每个新图像是一件痛苦的事情，包括必须指定 logoImage 应该加载文件“logo.png”。

由于我正在开发 Java Android 游戏并且图像在内部循环中使用了很多，所以我真的想避免像进行虚函数调用这样的缓慢操作，例如在我可以避免的时候访问数组/ map /对象字段。从 Android API(生成的 R 类)复制一个想法，我想我可以在编译之前运行一个实用程序来为我生成一些重复的代码。例如，项目文件中的实际代码将简化为:

logoImage.draw(0, 0);

使用一些命令行工具(例如 grep、sed)，我可以查找“Image.draw(...”的每个实例，然后自动生成其他所需的代码，即加载/释放文件 .png 的代码并在某处声明“Bitmap logoImage”。此代码可以添加到新类中，或者我可以在我的代码中添加占位符，告诉代码生成器在哪里插入生成的代码。

要显示新图像，我只需将图像文件复制到正确的目录并添加一行代码即可。漂亮而简单。这避免了创建图像数组、定义标记的 int 常量以引用数组以及必须指定要加载的文件名等方法。

这真的是个坏主意吗？这似乎有点 hack，但我看不出更简单的方法，而且它似乎确实彻底清理了我的代码。是否有任何标准工具可以进行这种简单的代码生成(即该工具不需要理解代码的含义)？还有其他人做这样的事情来弥补语言功能吗？

Answer 1

像这样使用代码生成是个坏主意。 (IMO，代码生成应该保留给需要生成大量代码的情况，而这听起来不像那种情况。)

如果您担心当前解决方案中的样板代码，更好的解决方案(比代码生成)是实现图像注册表抽象；例如

public class ImageRegistry {
    private Map<String, Image> map = new HashMap<String, Image>();

    public synchronized Image getImage(String filename) {
        Image image = map.get(filename);
        if (image == null) {
            image = load(filename);
            map.put(filename, image);
        }
        return image;
    }

    public synchronized void shutdown() {
        for (Image image : map.valueSet()) {
            image.release();
        }
        map.clear();  // probably redundant ...
    }
}

将 logoImage.draw(0, 0) 等替换为:

registry.getImage("logo.png").draw(0, 0);

删除所有 load 调用，并将所有 release 调用替换为对 registry.shutdown() 的单个调用。

EDIT 回应以下 OP 的评论:

... but I mention that I'm writing a game for a phone. A HashMap lookup every time I'm drawing a sprite will kill performance.

啊...我记得你在另一个线程。

1. 您(又一次)在没有任何实际绩效衡量依据的情况下对绩效做出假设。具体来说，您假设 HashMap 查找的成本太高。我的直觉是，进行查找所花费的时间只占绘制图像所花费时间的一小部分 (< 10%)。到那时，它已接近用户无法察觉的水平。

2. 如果您的测量结果(或直觉)告诉您 HashMap 查找的成本太高，微不足道的修改可以这样写:

   图片 image = registry.getImage("logo.png");尽管 (...) { ... image.draw(0, 0);

---

For example, Google even go as far as to recommend you don't use iterators in inner loops because these cause the GC to fire when the Iterator objects are deallocated.

这是无关紧要且不准确的。

1. 使用字符串键的 HashMap 查找不会产生垃圾。从来没有。

2. 在内部循环中使用迭代器不会“导致 GC 在迭代器对象被释放时触发”。在 Java 中，您不会释放对象。那就是C/C++的思想。

3. 在内部循环中实例化一个迭代器会new一个对象，但是GC只会在new操作命中时触发内存阈值。这种情况只是偶尔发生。

---

Also, writing "file_that_does_not_exist.png" will not be picked up as a compile time error with your example.

您的原始解决方案或代码生成方法都不会为您提供丢失文件的编译时错误。