bioshogi 0.0.12 → 0.0.14

data/lib/bioshogi/application_memory_record.rb

@@ -3,22 +3,15 @@ module Bioshogi
 
   concern :ApplicationMemoryRecord do
     included do
-      include MemoryRecord
+      unless self < MemoryRecord
+        include MemoryRecord
+      end
 
-      def self.fetch(*)
+      def self.fetch(...)
         super
       rescue KeyError => error
         raise KeyNotFound, error.message
       end
-
-      # fetch できなかったとき default_key が指すレコードを返す
-      def self.safe_fetch(key)
-        v = lookup(key)
-        unless v
-          v = fetch(default_key)
-        end
-        v
-      end
     end
 
     def inspect

data/lib/bioshogi/assertion.rb

@@ -1,18 +1,22 @@
+# frozen-string-literal: true
+
 module Bioshogi
   module Assertion
     extend self
 
-    def assert(actual = nil, message = "assert failed", &block)
+    def assert(actual = nil, message = nil, &block)
       if block_given?
         actual = yield
       end
       unless actual
+        message ||= "assert failed"
         raise MustNotHappen, "#{message}: #{actual.inspect}"
       end
     end
 
-    def assert_equal(expected, actual, message = "assert_equal failed")
+    def assert_equal(expected, actual, message = nil)
       if expected != actual
+        message ||= "assert_equal failed"
         raise MustNotHappen, "#{message}: #{expected.inspect} != #{actual.inspect}"
       end
     end

data/lib/bioshogi/assets/kifu_formats/sample.ki2

@@ -8,11 +8,11 @@
 戦型：ゴキゲン中飛車
 先手の囲い：居飛車穴熊
 後手の囲い：片美濃囲い, 銀冠
-先手の戦型：対振り持久戦, 三間飛車
+先手の戦法：対振り持久戦, 三間飛車
 先手の手筋：垂れ歩
 後手の手筋：垂れ歩
-先手の備考：居飛車, 対振り, 対抗型
-後手の備考：振り飛車, 対抗型
+先手の備考：居飛車, 対振り飛車, 対抗形
+後手の備考：振り飛車, 対抗形
 
 ▲２六歩   △３四歩   ▲２五歩 △３三角   ▲７六歩   △４二銀   ▲４八銀   △５四歩 ▲６八玉 △５五歩
 ▲３六歩   △５二飛   ▲３七銀 △５三銀   ▲４六銀   △４四銀   ▲５八金右 △６二玉 ▲７八玉 △７二玉

data/lib/bioshogi/board/basic.rb

@@ -17,7 +17,7 @@ module Bioshogi
       include ReaderMethods
       include TechniqueMatcherMethods
 
-      prepend Explain::BoardPillerMethods
+      prepend Analysis::BoardPillerMethods
       prepend PieceCountsMethods
     end
   end

data/lib/bioshogi/board_parser/compare_board_parser.rb

@@ -20,11 +20,19 @@ module Bioshogi
             when "*"
               # ▲でどれかがここに含まれる
               soldier = build(place, something, :black)
-              any_exist_soldiers << soldier
+              black_any_exist_soldiers << soldier
             when "?"
               # △側でどれかがここに含まれる
               soldier = build(place, something, :white)
-              any_exist_soldiers << soldier
+              white_any_exist_soldiers << soldier
+            when "~"
+              # ▲でここに含まれない
+              soldier = build(place, something, :black)
+              black_not_exist_soldiers << soldier
+            when "^"
+              # △でここに含まれない
+              soldier = build(place, something, :black)
+              white_not_exist_soldiers << soldier
             else
               soldier = build(place, something, prefix_char)
               soldiers << soldier
@@ -45,8 +53,23 @@ module Bioshogi
       end
 
       # どれかが含まれる用
-      def any_exist_soldiers
-        @any_exist_soldiers ||= SoldierBox.new
+      def black_any_exist_soldiers
+        @black_any_exist_soldiers ||= SoldierBox.new
+      end
+
+      # どれかが含まれる用
+      def white_any_exist_soldiers
+        @white_any_exist_soldiers ||= SoldierBox.new
+      end
+
+      # 含まれない用
+      def black_not_exist_soldiers
+        @black_not_exist_soldiers ||= SoldierBox.new
+      end
+
+      # 含まれない用
+      def white_not_exist_soldiers
+        @white_not_exist_soldiers ||= SoldierBox.new
       end
 
       # 高速に比較するためにメモ化したアクセサシリーズ

data/lib/bioshogi/container/core_methods.rb

@@ -10,8 +10,8 @@ module Bioshogi
 
       def params
         @params ||= {
-          :skill_monitor_enable           => false,
-          :skill_monitor_technique_enable => false,
+          :analyzer_enable           => false,
+          :analyzer_technique_enable => false,
           :candidate_enable               => true,
           :validate_enable                => true,
           :board_class                    => Board,

data/lib/bioshogi/container/execute_methods.rb

@@ -10,7 +10,7 @@ module Bioshogi
         end
 
         attr_writer :kill_count      # 駒を取った回数
-        attr_accessor :critical_turn # 最初の駒が取られる直前の手数           (avg: 21.6328)
+        attr_accessor :critical_turn # 最初の駒が取られる直前の手数           (avg: 22.6328)
         attr_accessor :outbreak_turn # 「歩と角」を除く駒が取られる直前の手数 (avg: 41.8402)
 
         def kill_count

data/lib/bioshogi/container/fast.rb

@@ -9,7 +9,7 @@ module Bioshogi
       include ExecuteMethods
 
       def executor_class
-        PlayerExecutor::WithoutMonitor
+        PlayerExecutor::WithoutAnalyzer
       end
     end
   end

data/lib/bioshogi/container/simple.rb

@@ -10,7 +10,7 @@ module Bioshogi
       include TestMethods
 
       def executor_class
-        PlayerExecutor::WithoutMonitor
+        PlayerExecutor::WithoutAnalyzer
       end
     end
   end

data/lib/bioshogi/converter.rb

@@ -27,6 +27,8 @@ module Bioshogi
       @files.each { |e| process_one(e) }
     end
 
+    private
+
     def process_one(in_file)
       in_file = Pathname(in_file).expand_path
       info = Parser.file_parse(in_file)
@@ -35,7 +37,7 @@ module Bioshogi
       when @options[:overwrite]
         in_file.write(str)
         puts "[overwrite] #{in_file} => #{out_file}"
-      when @options[:output_dir].present?
+      when output_dir
         out_file = output_dir + in_file.basename.sub_ext(extname)
         FileUtils.mkdir_p(out_file.dirname)
         out_file.write(str)
@@ -46,7 +48,9 @@ module Bioshogi
     end
 
     def output_dir
-      Pathname(@options[:output_dir]).expand_path
+      if v = @options[:output_dir]
+        Pathname(v).expand_path
+      end
     end
 
     def extname

data/lib/bioshogi/csa_header_info.rb

@@ -23,6 +23,8 @@ module Bioshogi
       { key: "$START_TIME:", kif_side_key: "開始日時", },
       { key: "$END_TIME:",   kif_side_key: "終了日時", },
       { key: "$OPENING:",    kif_side_key: "戦型",     },
+      { key: "$X_FINAL:",    kif_side_key: "結末",     },
+      { key: "$X_WINNER:",   kif_side_key: "勝者",     },
       {
         key: "$TIME_LIMIT:",
         kif_side_key: "持ち時間",

data/lib/bioshogi/errors.rb

@@ -44,7 +44,7 @@ module Bioshogi
   class FileFormatError < BioshogiError
   end
 
-  # 手合割・囲い・戦型などのあるはずのデータがない
+  # 手合割・囲い・戦法などのあるはずのデータがない
   class KeyNotFound < BioshogiError
   end
 
@@ -75,7 +75,7 @@ module Bioshogi
   end
 
   # .kif ファイルの時間がマイナスになっている
-  class TimeMinusError < SyntaxDefact
+  class ConsumedTimeMinusError < SyntaxDefact
   end
 
   ################################################################################ ルール系 (RuleError) 人間同士で指すと現われる反則。しかし公式の棋譜には残らないグループ

data/lib/bioshogi/extreme_validator.rb

@@ -10,7 +10,7 @@ module Bioshogi
     end
 
     def call
-      # Bioshogi.config[:skill_monitor_enable] = false
+      # Bioshogi.config[:analyzer_enable] = false
 
       error_body.delete rescue nil
       result = Hash.new(0)

data/lib/bioshogi/formatter/animation/demo_builder.rb

@@ -33,7 +33,7 @@ module Bioshogi
           # bin = info.to_animation_mp4(page_duration: 0.0166666666667, end_duration: 0)
           # no_audio_file.write(bin)
 
-          file = Explain::TacticInfo.flat_lookup("トマホーク").sample_kif_file
+          file = Analysis::TacticInfo.flat_lookup("トマホーク").main_reference_file
           info = Parser.parse(file)
           bin = info.to_animation_mp4({
               :color_theme_key => :is_color_theme_real,

data/lib/bioshogi/formatter/core.rb

@@ -76,8 +76,8 @@ module Bioshogi
       def container_new
         container_class.new.tap do |e|
           e.params.update(@parser_options.slice(*[
-                :skill_monitor_enable,
-                :skill_monitor_technique_enable,
+                :analyzer_enable,
+                :analyzer_technique_enable,
                 :candidate_enable,
                 :validate_enable,
                 :validate_double_pawn_skip,
@@ -90,8 +90,8 @@ module Bioshogi
       def container_for_image
         container = Container::Basic.new
         container.params.update({
-            :skill_monitor_enable           => false,
-            :skill_monitor_technique_enable => false,
+            :analyzer_enable           => false,
+            :analyzer_technique_enable => false,
             :candidate_enable               => false,
             :validate_enable                => false,
           })
@@ -163,15 +163,18 @@ module Bioshogi
 
       def container_run_all(container)
         Runner.new(self, container).call
-        if @parser_options[:skill_monitor_enable]
+        if @parser_options[:analyzer_enable]
           SkillEmbed.new(self, container).call
+          StyleEmbed.new(self, container).call
         end
       end
 
+      # FIXME: これは skill_embed のなかだけにあればよくね？
       def skill_set_hash
         @skill_set_hash ||= {}
       end
 
+      # FIXME: メソッドが重複してね？
       def raw_header_part_hash
         pi.header.object.collect { |key, value|
           if value
@@ -234,6 +237,23 @@ module Bioshogi
           [s, *v.lines, s].collect {|e| "#{comment_mark} #{e}" }.join
         end
       end
+
+      # 勝った側を返す
+      # nil の場合もある
+      def win_side_location(container)
+        # 勝者が明示されている場合
+        # SHOGI-EXTEND 側からは常に勝者を入れている
+        # これは切断されたときどちらが切断してどちらが勝ったのか手番では判断できないため
+        if v = @pi.header["勝者"]
+          Location.fetch(v)
+        else
+          # 明示されていないかつ「投了」「時間切れ」「詰み」で終わった場合のみ、手番による勝者を信じる
+          # これは入玉の場合「指した方が負け」になるケースがあるため、入玉などは除外しないといけない
+          if @pi.win_player_collect_p
+            container.win_player.location
+          end
+        end
+      end
     end
   end
 end

data/lib/bioshogi/formatter/last_action_info.rb

@@ -31,20 +31,22 @@ module Bioshogi
     class LastActionInfo
       include ApplicationMemoryRecord
       memory_record [
-        { key: "TORYO",           kakinoki_word: "投了",       reason: nil,              draw: nil,  },
-        { key: "CHUDAN",          kakinoki_word: "中断",       reason: "切断により",     draw: nil,  },
-        { key: "SENNICHITE",      kakinoki_word: "千日手",     reason: "千日手",         draw: true, },
-        { key: "TIME_UP",         kakinoki_word: "切れ負け",   reason: "時間切れにより", draw: nil,  },
-        { key: "ILLEGAL_MOVE",    kakinoki_word: nil,          reason: "反則により",     draw: nil,  },
-        { key: "+ILLEGAL_ACTION", kakinoki_word: nil,          reason: "反則により",     draw: nil,  },
-        { key: "-ILLEGAL_ACTION", kakinoki_word: nil,          reason: "反則により",     draw: nil,  },
-        { key: "JISHOGI",         kakinoki_word: "持将棋",     reason: nil,              draw: nil,  },
-        { key: "KACHI",           kakinoki_word: nil,          reason: nil,              draw: nil,  },
-        { key: "HIKIWAKE",        kakinoki_word: nil,          reason: nil,              draw: nil,  },
-        { key: "MATTA",           kakinoki_word: "中断",       reason: nil,              draw: nil,  },
-        { key: "TSUMI",           kakinoki_word: "詰み",       reason: nil,              draw: nil,  },
-        { key: "FUZUMI",          kakinoki_word: nil,          reason: nil,              draw: nil,  },
-        { key: "ERROR",           kakinoki_word: nil,          reason: "エラーにより",   draw: nil,  },
+        # 手番による勝者判定が可能なら win_player_collect_p を true にする
+        # CHUDAN などはどちらが中断(切断)したのかもしわからない
+        { key: "TORYO",           kakinoki_word: "投了",       reason: nil,              draw: nil,  win_player_collect_p: true, },
+        { key: "CHUDAN",          kakinoki_word: "中断",       reason: "切断により",     draw: nil,  win_player_collect_p: nil,  }, # 切断の場合どちらが切断したのかわからないため win_player_collect_p: true にしてはいけない
+        { key: "SENNICHITE",      kakinoki_word: "千日手",     reason: "千日手",         draw: true, win_player_collect_p: nil,  },
+        { key: "TIME_UP",         kakinoki_word: "切れ負け",   reason: "時間切れにより", draw: nil,  win_player_collect_p: true, },
+        { key: "ILLEGAL_MOVE",    kakinoki_word: nil,          reason: "反則により",     draw: nil,  win_player_collect_p: nil,  },
+        { key: "+ILLEGAL_ACTION", kakinoki_word: nil,          reason: "反則により",     draw: nil,  win_player_collect_p: nil,  },
+        { key: "-ILLEGAL_ACTION", kakinoki_word: nil,          reason: "反則により",     draw: nil,  win_player_collect_p: nil,  },
+        { key: "JISHOGI",         kakinoki_word: "持将棋",     reason: nil,              draw: nil,  win_player_collect_p: nil,  },
+        { key: "KACHI",           kakinoki_word: nil,          reason: nil,              draw: nil,  win_player_collect_p: nil,  }, # この場合、win_player が信用できなくなり、つまりどちらが勝ったのかわからなくなる
+        { key: "HIKIWAKE",        kakinoki_word: nil,          reason: nil,              draw: nil,  win_player_collect_p: nil,  },
+        { key: "MATTA",           kakinoki_word: "中断",       reason: nil,              draw: nil,  win_player_collect_p: nil,  },
+        { key: "TSUMI",           kakinoki_word: "詰み",       reason: nil,              draw: nil,  win_player_collect_p: true, },
+        { key: "FUZUMI",          kakinoki_word: nil,          reason: nil,              draw: nil,  win_player_collect_p: nil,  },
+        { key: "ERROR",           kakinoki_word: nil,          reason: "エラーにより",   draw: nil,  win_player_collect_p: nil,  },
       ]
 
       alias csa_key key
@@ -62,15 +64,17 @@ module Bioshogi
       end
 
       def judgment_message(container)
-        s = []
-        s << "まで#{container.turn_info.turn_offset}手で"
-        s << reason
-        unless draw
-          s << container.opponent_player.call_name
-          s << "の"
-          s << "勝ち"
+        if win_player_collect_p
+          s = []
+          s << "まで#{container.turn_info.turn_offset}手で"
+          s << reason
+          unless draw
+            s << container.opponent_player.call_name # これが信じられるのは win_player_collect_p のときだけ
+            s << "の"
+            s << "勝ち"
+          end
+          s.join
         end
-        s.join
       end
     end
   end