[Ace] Window.ok_handler funktioniert nicht

[MindXperience]

Hey Guys,
ich experimentiere wieder etwas mit den Window Klassen, damit ich diese endlich mal erlerne und auch verstehen ^^
So versuche ich grad eine Liste zu machen, die jeden Status anzeigt und dabei angibt, ob der Actor mit diesem Status betroffen ist oder nicht. Wenn man Enter drückt, so soll der Status on/off triggern. Aber der ok-Handler scheint einfach nicht zu funktionieren?

Code:

class Window_DebugStates < Window_SkillList
  def col_max
    return 1
  end
  def enable?(item)
    return true
  end
  def include?(item)
    return true
  end
  def make_item_list
    @data = $data_states
  end
  def draw_item(index)
    state = @data[index]
    if state
      rect = item_rect(index)
      rect.width -= 4
      draw_item_name(state, rect.x, rect.y, enable?(state))
      infl = inflicted?(state) ? "[ON]" : "[OFF]"
      draw_text(item_rect_for_text(index), infl, 2)
    end
  end
  def inflicted?(state)
    return @actor.state?(state)
  end
end

Code:

def command_edit_states
    @states_window = Window_DebugStates.new(100, 120, 250, 250)
    @states_window.actor = @actor
    @states_window.set_handler(:ok,     method(:on_state_ok))
    @states_window.set_handler(:cancel, method(:on_state_cancel))
    @states_window.activate
    @states_window.refresh
    @states_window.select(1)
  end

Code:

    def on_state_ok
    state_id = @states_window.index
    if actor.state?(state_id)
      @actor.remove_state(state_id)
    else
      @actor.add_state(state_id)
    end
    @states_window.select(@states_window.index)
    @status_window.refresh
    @states_window.activate
  end

[Cornix]

Es wäre ganz nützlich falls du dazu sagen kannst, um welchen Maker es sich handelt.

[MindXperience]

VX Ace
steht aber auch in meiner Signatur

[-KD-]

Das ist alles weitestgehend richtig. Zwei Fehler sind mir aufgefallen:

Code:

def inflicted?(state)
  return @actor.state?(state)
end

Die Methode state? erwartet die ID des States, nicht das State Objekt selbst. Und die RGSS ist an der Stelle leider sehr sparsam mit Fehlermeldungen. Also einfach state.id hinschreiben.

Code:

@status_window.refresh

Das soll vermutlich @state_window.refresh heißen.

Solche blöden Fehler findet man nur durch exzessives Einfügen von print Befehlen.

[MindXperience]

Vielen Dank für den Hinweis.

Es heißt tatsächlich status_window, da ich noch weitere Fenster habe.
Außerdem habe ich schon eindeutige Befehle in "def on_state_ok" eingefügt (zB prints), aber die Methode wird einfach nicht gestartet

[-KD-]

Ich hab deinen Code so in Scene_Map reingepresst und die zwei Stellen korrigiert. Damit funktioniert es.

Code:

class Window_DebugStates < Window_SkillList
  def col_max
    return 1
  end
  def enable?(item)
    return true
  end
  def include?(item)
    return true
  end
  def make_item_list
    @data = $data_states
  end
  def draw_item(index)
    state = @data[index]
    if state
      rect = item_rect(index)
      rect.width -= 4
      draw_item_name(state, rect.x, rect.y, enable?(state))
      infl = inflicted?(state) ? "[ON]" : "[OFF]"
      draw_text(item_rect_for_text(index), infl, 2)
    end
  end
  def inflicted?(state)
    return @actor.state?(state.id)
  end
end

class Scene_Map < Scene_Base
  
  alias start_debugstates start
  def start
    start_debugstates
    @actor = $game_party.members.first
    command_edit_states
  end
  
  def command_edit_states
    @states_window = Window_DebugStates.new(100, 120, 250, 250)
    @states_window.actor = @actor
    @states_window.set_handler(:ok,     method(:on_state_ok))
    @states_window.set_handler(:cancel, method(:on_state_cancel))
    @states_window.activate
    @states_window.refresh
    @states_window.select(1)
    @states_window.actor =  @actor
  end
  
  def on_state_cancel
    
  end
  
  def on_state_ok
    state_id = @states_window.index
    if @actor.state?(state_id)
      @actor.remove_state(state_id)
      @actor.result.removed_states.clear
    else
      @actor.add_state(state_id)
    end
    #@states_window.select(@states_window.index)
    @states_window.refresh
    @states_window.activate
  end
  
end

Eine Zeile hab ich noch eingefügt:

Code:

@actor.result.removed_states.clear

muss nach @actor.remove_state aufgerufen werden, da ein Zustand der beseitigt wurde in der selben "Kampfrunde" nicht noch einmal gesetzt werden darf.

Kannst du mal deinen kompletten Code zeigen?

[MindXperience]

Nun, das komplette Skript umfasst einige mehr Funktionieren, aber schau es dir mal an

Code:

class Window_CharacterCommand < Window_MenuCommand
  def make_command_list
    add_command("Name ändern", :name)
    add_command("Klasse bearbeiten", :class)
    add_command("Level ändern", :level)
    add_command("Skills bearbeiten", :skills)
    add_command("heilen", :heal)
    add_command("Status hinzufügen/entfernen", :stats)
  end
end

class Window_DebugClass < Window_Command
  def make_command_list
    for i in 1...$data_actors.size
      name = $data_classes[i].name
      text = sprintf("%s", name)
      add_command(text, :switch, true, i)
    end
  end
end

class Window_Level < Window_Command
  def make_command_list
    add_command("<            >", :switch)
  end
  def update
    super
    update_tone
    update_open if @opening
    update_close if @closing
    if Input.repeat?(:RIGHT)
      call_handler(:right)
    end
    if Input.repeat?(:LEFT)
      call_handler(:left)
    end
  end
end

class Window_DebugSkill < Window_SkillList
  def col_max
    return 1
  end
  def enable?(item)
    return true
  end
  def include?(item)
    return true
  end
  def make_item_list
    @data = $data_skills
  end
end

class Window_DebugStates < Window_SkillList
  def col_max
    return 1
  end
  def enable?(item)
    return true
  end
  def include?(item)
    return true
  end
  def make_item_list
    @data = $data_states
  end
  def draw_item(index)
    state = @data[index]
    if state
      rect = item_rect(index)
      rect.width -= 4
      draw_item_name(state, rect.x, rect.y, enable?(state))
      infl = inflicted?(state) ? "[ON]" : "[OFF]"
      draw_text(item_rect_for_text(index), infl, 2)
    end
  end
  def inflicted?(state)
    return @actor.state?(state)
  end
end

class Scene_Character < Scene_Status
  
  def start
    super
    @status_window = Window_Status.new(@actor)
    @status_window.set_handler(:pagedown, method(:next_actor))
    @status_window.set_handler(:pageup,   method(:prev_actor))
    @status_window.refresh
    
    @command_window = Window_CharacterCommand.new
    @command_window.x = 350
    @command_window.y = 200
    @command_window.width = 290
    @command_window.set_handler(:name,      method(:command_change_name))
    @command_window.set_handler(:class,     method(:command_change_class))
    @command_window.set_handler(:level,     method(:command_change_level))
    @command_window.set_handler(:skills,    method(:command_edit_skills))
    @command_window.set_handler(:heal,      method(:command_heal))
    @command_window.set_handler(:stats,     method(:command_edit_states))
    @command_window.set_handler(:cancel,    method(:return_scene))
    @command_window.refresh
    
    @class_window = Window_DebugClass.new(100, 100)
    @class_window.hide
    
    @level_window = Window_Level.new(300, 200)
    @level_window.hide
    
    @skill_window = Window_DebugSkill.new(100, 100, 400, 480)
    @skill_window.hide
    
    @states_window = Window_DebugStates.new(100, 300, 250, 250)
    @states_window.hide
    @states_window.set_handler(:ok,     method(:on_state_ok))
    @states_window.set_handler(:cancel, method(:on_state_cancel))
  end
  
  def command_change_name
    SceneManager.call(Scene_Name)
    SceneManager.scene.prepare(@actor, 10)
  end
  
  def command_change_class
    @class_window = Window_DebugClass.new(100, 40)
    @class_window.width = 250
    @class_window.set_handler(:ok,     method(:on_class_ok))
    @class_window.set_handler(:cancel, method(:on_class_cancel))
    @class_window.activate
    @class_window.refresh
  end
  
  def command_change_level
    @level_window = Window_Level.new(100, 100)
    @level_window.set_handler(:ok,     method(:on_level_cancel))
    @level_window.set_handler(:cancel, method(:on_level_cancel))
    @level_window.set_handler(:left,   method(:decrease_level))
    @level_window.set_handler(:right,  method(:increase_level))
    @level_window.activate
    @level_window.refresh
  end
  
  def command_edit_skills
    @skill_window = Window_DebugSkill.new(100, 0, 400, 400)
    @skill_window.actor = @actor
    @skill_window.set_handler(:ok,     method(:on_skill_ok))
    @skill_window.set_handler(:cancel, method(:on_skill_cancel))
    @skill_window.activate
    @skill_window.refresh
    @skill_window.select(1)
  end
  
  def command_change_equip
  end
  
  def command_heal
    @actor.recover_all
    @command_window.refresh
    @command_window.activate
    @status_window.refresh
  end
  
  def command_edit_states
    @states_window = Window_DebugStates.new(100, 120, 250, 250)
    @states_window.actor = @actor
    @states_window.set_handler(:ok,     method(:on_state_ok))
    @states_window.set_handler(:cancel, method(:on_state_cancel))
    @states_window.activate
    @states_window.refresh
    @states_window.select(1)
  end
  
  def on_class_ok
    cl = @class_window.index + 1
    @actor.change_class(cl)
    @class_window.hide
    @command_window.activate
    @status_window.refresh
  end
  
  def on_class_cancel
    @command_window.activate
    @class_window.hide
  end
  
  def on_level_cancel
    @level_window.hide
    @command_window.activate
    @status_window.refresh
  end
  
  def decrease_level
    @actor.level_down
    @status_window.refresh
  end
  
  def increase_level
    @actor.level_up
    @status_window.refresh
  end
  
  def on_skill_ok
    @actor.learn_skill(@skill_window.index)
    @skill_window.refresh
    @skill_window.activate
    @status_window.refresh
  end
  
  def on_skill_cancel
    @skill_window.hide
    @command_window.activate
    @status_window.refresh
  end
  
  def on_state_ok
    state_id = @states_window.index
    if (4 < 2)
      @actor.remove_state(state_id)
    else
      @actor.add_state(state_id)
    end
    #@states_window.select(@states_window.index)
    @states_window.refresh
    #@status_window.refresh
    @states_window.activate
  end
  
  def on_state_cancel
    @states_window.hide
    @command_window.activate
    @status_window.refresh
  end
end

[-KD-]

Okay, Fehler gefunden. Das Problem liegt darin, dass (bezogen auf die OK und CANCEL Handler) immer nur ein Fenster aktiv sein darf (also sozusagen im Fokus liegen kann). Bei dir fängt ein anderes Window den Tastendruck Enter ab, daher kommt der in Window_DebugState nicht mehr an. Klassen die von Window_Selectable erben, sind automatisch erstmal inaktiv. Klassen die von Window_Command erben, sind automatisch aktiv. Du musst also deine Klassen, Window_DebugClass und Window_Level direkt nach der Initialisierung deaktivieren, sonst liegen die im Fokus und fangen die OK und CANCEL Handler ab. Ob ein Fenster aktiv ist, ist komplett unabhängig davon ob ein Fenster angezeigt wird. hide() allein reicht also nicht aus, du musst auch deactivate() aufrufen.

[MindXperience]

Ahhh so läuft das also ^^

Vielen Dank fr diese ausführliche Erklärung