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

SailCat · 发表于 2021-3-22 04:04:20

加入我们，或者，欢迎回来。

您需要登录才可以下载或查看，没有账号？注册会员

×

本帖最后由 SailCat 于 2021-3-22 04:53 编辑

[pre lang="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[/pre]

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

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

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

看出问题了吗？它是每个战斗者依次更新的。
而每个战斗者精灵在执行自己的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帧播放的，真是不好说这是进步还是退步了……

百里_飞柳 · 发表于 2021-3-22 09:18:19

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

89444640 · 发表于 2021-3-22 10:06:17

本帖最后由 89444640 于 2021-3-22 10:15 编辑

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

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

srwjrevenger · 发表于 2021-3-22 10:20:27

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

灯笼菜刀王 · 发表于 2021-3-22 10:33:35

本帖最后由灯笼菜刀王于 2021-3-22 10:38 编辑

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

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

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

坐等解决方法(伸手)

ppspssss · 发表于 2021-3-22 12:25:24

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

RPGzh500223 · 发表于 2021-3-22 12:30:30

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

module RPG
class Sprite < ::Sprite
def dispose_animation
if @_animation_sprites != nil
sprite = @_animation_sprites[0]
if sprite != nil
@@_reference_count[sprite.bitmap] -= 1
if @@_reference_count[sprite.bitmap] == 0
sprite.bitmap.dispose
end
end
for sprite in @_animation_sprites
sprite.dispose
end
@_animation_sprites = nil
end
#……………………………………………………………………………………
if @_animation != nil
@@_animations.delete(@_animation) if @_animation.position == 3
@_animation = nil
end
#……………………………………………………………………………………
end
def update
super
if @_whiten_duration > 0
@_whiten_duration -= 1
self.color.alpha = 128 - (16 - @_whiten_duration) * 10
end
if @_appear_duration > 0
@_appear_duration -= 1
self.opacity = (16 - @_appear_duration) * 16
end
if @_escape_duration > 0
@_escape_duration -= 1
self.opacity = 256 - (32 - @_escape_duration) * 10
end
if @_collapse_duration > 0
@_collapse_duration -= 1
self.opacity = 256 - (48 - @_collapse_duration) * 6
end
if @_damage_duration > 0
@_damage_duration -= 1
case @_damage_duration
when 38..39
@_damage_sprite.y -= 4
when 36..37
@_damage_sprite.y -= 2
when 34..35
@_damage_sprite.y += 2
when 28..33
@_damage_sprite.y += 4
end
@_damage_sprite.opacity = 256 - (12 - @_damage_duration) * 32
if @_damage_duration == 0
dispose_damage
end
end
if @_animation != nil and (Graphics.frame_count % 2 == 0)
@_animation_duration -= 1
update_animation
end
if @_loop_animation != nil and (Graphics.frame_count % 2 == 0)
update_loop_animation
@_loop_animation_index += 1
@_loop_animation_index %= @_loop_animation.frame_max
end
if @_blink
@_blink_count = (@_blink_count + 1) % 32
if @_blink_count < 16
alpha = (16 - @_blink_count) * 6
else
alpha = (@_blink_count - 16) * 6
end
self.color.set(255, 255, 255, alpha)
end
#@@_animations.clear
@@_animations.delete_if {|i| i.position != 3}
end
end
end

复制代码

orochi2k · 发表于 2021-3-22 16:22:32

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

enghao_lim · 发表于 2021-3-22 23:11:32

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

SailCat · 发表于 2021-3-28 10:02:04

本帖最后由 SailCat 于 2021-3-28 10:04 编辑

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

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

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

加入我们，或者，欢迎回来。

评分

点评

浏览过的版块

开拓者

极短24评委

极短23参与

极短22参与

极短21评委

老司机慢点开

短篇十吟唱者组别冠军

剧作品鉴家

贵宾