RGSS1（2、3也同样）中一个并不会触发的奇怪bug

SailCat

RUBY 代码复制

def animation(animation, hit)
      dispose_animation
      @_animation = animation
      return if @_animation == nil
      @_animation_hit = hit
      @_animation_duration = @_animation.frame_max
      animation_name = @_animation.animation_name
      animation_hue = @_animation.animation_hue
      bitmap = RPG::Cache.animation(animation_name, animation_hue)
      if @@_reference_count.include?(bitmap)
        @@_reference_count[bitmap] += 1
      else
        @@_reference_count[bitmap] = 1
      end
      @_animation_sprites = []
      if @_animation.position != 3 or not @@_animations.include?(animation)
        for i in 0..15
          sprite = ::Sprite.new(self.viewport)
          sprite.bitmap = bitmap
          sprite.visible = false
          @_animation_sprites.push(sprite)
        end
        unless @@_animations.include?(animation)
          @@_animations.push(animation)
        end
      end
      update_animation
    end

def animation(animation, hit)


      dispose_animation


      @_animation = animation


      return if @_animation == nil


      @_animation_hit = hit


      @_animation_duration = @_animation.frame_max


      animation_name = @_animation.animation_name


      animation_hue = @_animation.animation_hue


      bitmap = RPG::Cache.animation(animation_name, animation_hue)


      if @@_reference_count.include?(bitmap)


        @@_reference_count[bitmap] += 1


      else


        @@_reference_count[bitmap] = 1


      end


      @_animation_sprites = []


      if @_animation.position != 3 or not @@_animations.include?(animation)


        for i in 0..15


          sprite = ::Sprite.new(self.viewport)


          sprite.bitmap = bitmap


          sprite.visible = false


          @_animation_sprites.push(sprite)


        end


        unless @@_animations.include?(animation)


          @@_animations.push(animation)


        end


      end


      update_animation


    end

请注意这一句：  if @_animation.position != 3 or not @@_animations.include?(animation)

我查了一下什么叫不等于3：也就是动画的“位置”不等于“画面”（即非全屏动画）
这个if后面是两个否定，你改成肯定并unless就好理解了：除非动画是全屏动画，并且缓存里已经有了这个动画（那么跳过），否则就要实际生成动画图块，并将动画加载到缓存里。

看起来，这个判定是为了当你有8个敌人然后你放了个全体大招的时候，不至于要生成8*16＝128个动画元件（因为进行全屏动画所有元件的位置是一样的，这样做完全没有意义），只需要16个元件，然后本体分别闪烁就行了。
但是经过反复实验，放全体大招的时候根本不会触发这个判断，会生成128个动画元件，以毫无意义的方式进行重叠播放（也不能说毫无意义，你要是加法或减法，就能看出来原位叠加了），并且最后咣咣咣释放掉。
这是为啥呢？
战斗时更新精灵的地方在Spriteset_Battle的108-111行：

RUBY 代码复制

# 刷新战斗者的活动块
    for sprite in @enemy_sprites + @actor_sprites
      sprite.update
    end

# 刷新战斗者的活动块


    for sprite in @enemy_sprites + @actor_sprites


      sprite.update


    end

看出问题了吗？它是每个战斗者依次更新的。
而每个战斗者精灵在执行自己的update的时候，第一句都是super（Sprite_Battler第36行），也就是去执行RPG::Sprite的update。这个操作会干吗呢？会将@@_animations（也就是全屏动画缓存）予以清空。
所以，战斗者A预约调用全屏动画1（animation2_id设置为1），清空缓存，执行动画。然后战斗者B预约调用全屏动画1，清空缓存，执行动画（这时缓存被清空了，于是无法通过那个检测了，会照样生成16个重复的动画元件）。

然后我为啥说这个是不会触发的奇怪bug呢？这里的bug之处，在于@@_reference_count（也就是对动画位图的引用）是在检查之前更新+1的，但是若是检查不通过，不会生成16个动画元件精灵，也就不会产生对动画位图的引用，到最后释放的时候这个计数-1就执行不到了，然后动画精灵解放的时候，缓存位图就dispose不掉，然后大家都知道，这个错误叫内存泄露。
但是，按照目前的更新逻辑（我以战斗为例了，但地图上也是一样的），每个战斗者自己update时都会清缓存，到下一个战斗者时，根本就不会出现“检查不通过”的情况，也就不会出现@@_reference_count的计数器错误。——结果就是，除了会卡一些之外，它的执行逻辑是完全自恰的。

也就是说，这个看上去可以防止全屏动画叠加播放的判断，根本就是形同虚设，白给了。

看看大家打算怎么修这个bug？
保持不动（因为实际不会触发）——但是齐时战斗系统就不好说会不会触发了
还是去掉这个根本无用的判断
还是保留这个判断，但是让它真的能起防止重复播放的作用，并且不会出现计数错误

后记：XP的动画其实也真没有到特别卡的地步，我试着做了个天气扩展插件，允许天上下刀子——后来做着做着发现RPG::Weather继承的不是原生Sprite，而是RPG::Sprite，也就是说天气元素（雨滴，雪花，刀子）都是允许带效果的精灵——然后我试了40个天气元素循环播放状态动画并同时下落，在这种极端情况下，帧率还能连续10秒钟维持在34左右。
又后记：VA也是一模一样的问题（不会触发的bug，形同虚设的判定），顺便XP的动画是按1秒20帧播放的，VA……居然是按1秒15帧播放的，真是不好说这是进步还是退步了……

百里_飞柳

关于这个，va里的yea的battle core插件就新增了一个修改
为技能备注栏增加了<one anim>的备注
如果有这个，就不再是所有目标调用use_item（里面包含显示动画和执行伤害），而是在迭代前显示一个动画，然后每个目标只执行伤害

89444640

对，就是这个闹得我当年卡的要死，
我这里用的攻击对象同时执行攻击动作是对象全体 动画设置是对象方动画为画面，等于当时用全动画脚本时候，同一个时间加上敌我待机动作，一旦我执行全屏攻击最多同时在播放8敌人+4己方+8对象方画面的动画，这还不算中毒什么的循环状态，
造成我当年战斗时候一旦战斗进行多了，就会卡死跳出。难道是因为这种全屏占用的内存一直没清除？我动画文件是又多又大，而且当初也没调用4G内存的微软官方不定，全屏攻击同时叠加N个不卡死才怪。
话说，用guoxiaomi写的全屏动画，直接ps做640*320的去播放，一旦目标是对象方，画面，也会叠加8次？

XP不能调用显卡，2D序列帧只要我是非像素的光效，一旦体积上来就完蛋。所以我也只好出迷宫什么的手动清一下内存。

srwjrevenger

居然允许天上下刀子雨?你这个魔鬼（滑稽）

灯笼菜刀王

我前些时候也发现了同款BUG(应该)



如图, 让主角播放循环动画, 本来应该是一直持续播放的, 此时, 给另一事件播放个同bitmap的动画, 然后呢, 等这个动画结束的时候, 循环动画也一起消失了.....

初步判断是@@_reference_count的判定没起作用, bitmap被释放了~  感觉应该也是这个BUG的原因

坐等解决方法(伸手)

ppspssss

,我之前使用这个动画时候也是卡的不行, 基本播放动画从40帧掉到只有30帧, 然后修改了@@_reference_count[bitmap]就變回滿40幀了, 不知有什么副作用,

RPGzh500223

怎么分辨是否重叠啊，我是没看出来……
12人测试，感觉FPS少掉 2-3

1. module RPG
   
2.   class Sprite < ::Sprite
   
3.     def dispose_animation
   
4.       if @_animation_sprites != nil
   
5.         sprite = @_animation_sprites[0]
   
6.         if sprite != nil
   
7.           @@_reference_count[sprite.bitmap] -= 1
   
8.           if @@_reference_count[sprite.bitmap] == 0
   
9.             sprite.bitmap.dispose
   
10.           end
    
11.         end
    
12.         for sprite in @_animation_sprites
    
13.           sprite.dispose
    
14.         end
    
15.         @_animation_sprites = nil
    
16.       end
    
17.       #……………………………………………………………………………………
    
18.       if @_animation != nil
    
19.         @@_animations.delete(@_animation) if @_animation.position == 3
    
20.         @_animation = nil
    
21.       end
    
22.       #……………………………………………………………………………………
    
23.     end
    
24.    
    
25.     def update
    
26.       super
    
27.       if @_whiten_duration > 0
    
28.         @_whiten_duration -= 1
    
29.         self.color.alpha = 128 - (16 - @_whiten_duration) * 10
    
30.       end
    
31.       if @_appear_duration > 0
    
32.         @_appear_duration -= 1
    
33.         self.opacity = (16 - @_appear_duration) * 16
    
34.       end
    
35.       if @_escape_duration > 0
    
36.         @_escape_duration -= 1
    
37.         self.opacity = 256 - (32 - @_escape_duration) * 10
    
38.       end
    
39.       if @_collapse_duration > 0
    
40.         @_collapse_duration -= 1
    
41.         self.opacity = 256 - (48 - @_collapse_duration) * 6
    
42.       end
    
43.       if @_damage_duration > 0
    
44.         @_damage_duration -= 1
    
45.         case @_damage_duration
    
46.         when 38..39
    
47.           @_damage_sprite.y -= 4
    
48.         when 36..37
    
49.           @_damage_sprite.y -= 2
    
50.         when 34..35
    
51.           @_damage_sprite.y += 2
    
52.         when 28..33
    
53.           @_damage_sprite.y += 4
    
54.         end
    
55.         @_damage_sprite.opacity = 256 - (12 - @_damage_duration) * 32
    
56.         if @_damage_duration == 0
    
57.           dispose_damage
    
58.         end
    
59.       end
    
60.       if @_animation != nil and (Graphics.frame_count % 2 == 0)
    
61.         @_animation_duration -= 1
    
62.         update_animation  
    
63.       end
    
64.       if @_loop_animation != nil and (Graphics.frame_count % 2 == 0)
    
65.         update_loop_animation
    
66.         @_loop_animation_index += 1
    
67.         @_loop_animation_index %= @_loop_animation.frame_max
    
68.       end
    
69.       if @_blink
    
70.         @_blink_count = (@_blink_count + 1) % 32
    
71.         if @_blink_count < 16
    
72.           alpha = (16 - @_blink_count) * 6
    
73.         else
    
74.           alpha = (@_blink_count - 16) * 6
    
75.         end
    
76.         self.color.set(255, 255, 255, alpha)
    
77.       end
    
78.       #@@_animations.clear
    
79.       @@_animations.delete_if {|i| i.position != 3}
    
80.     end
    
81.   end
    
82. end

复制代码

orochi2k

于是大家终于发现这个BUG了，那么我也等着伸手好了。（大雾）

enghao_lim

以前改一个技能多动画的时候也是发现同款bug，只是当时不知道是bug，以为全屏动画就是那么设计的，现在仔细看来还是太草率了……

SailCat

https://rpg.blue/thread-484741-1-1.html
该bug问题已经在SEP Core 1.8中被完美修复。见插件bug说明的l)-n)项。

三个bug是交织在一起的，所以先修正了计数错误（从而将重复绘制错误暴露出来）然后通过将@@_animations.clear改为一帧执行一次（植入Spriteset.update）而不是一帧执行多次，来修正重复绘制的错误。