narou 1.7.2 → 2.0.0

data/lib/command/flag.rb

@@ -77,11 +77,11 @@ module Command
       attribute = (argv.shift || "").downcase
       unless ATTRIBUTES.include?(attribute)
         error "有効なフラグを指定して下さい\n指定可能なフラグ：#{ATTRIBUTES.join(', ')}"
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       if argv.length < 1
         error "対象小説を指定して下さい"
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       database = Database.instance
       argv.each do |target|

data/lib/command/list.rb

@@ -126,7 +126,7 @@ module Command
                 else
                   error "不明なフィルターです(#{item})"
                   warn "filters = #{filter_type_help}"
-                  exit 1
+                  exit Narou::EXIT_ERROR_CODE
                 end
         sum + (apply ? 1 : 0)
       end

data/lib/command/mail.rb

@@ -50,7 +50,7 @@ module Command
         return
       rescue Mailer::SettingUncompleteError => e
         error e.message
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       if argv.empty?
         send_all = true
@@ -97,12 +97,12 @@ module Command
           database[id]["last_mail_date"] = Time.now
         else
           error "#{mailer.error_message}"
-          exit 1   # next しても次も失敗する可能性が高いのでここで終了
+          exit Narou::EXIT_ERROR_CODE   # next しても次も失敗する可能性が高いのでここで終了
         end
       end
     rescue Interrupt
       puts "メール送信を中断しました"
-      exit 1
+      exit Narou::EXIT_ERROR_CODE
     ensure
       database.save_database if database
     end

data/lib/command/remove.rb

@@ -5,6 +5,7 @@
 
 require_relative "../narou"
 require_relative "../helper"
+require_relative "../input"
 
 module Command
   class Remove < CommandBase
@@ -77,7 +78,7 @@ module Command
           next
         end
         unless @options["yes"]
-          next unless Helper.confirm("#{title} を#{(@options["with-file"] ? "“完全に”" : "")}削除しますか")
+          next unless Narou::Input.confirm("#{title} を#{(@options["with-file"] ? "“完全に”" : "")}削除しますか")
         end
         Downloader.remove_novel(target, @options["with-file"])
         puts "<bold><green>#{TermColorLight.escape(title)} を削除しました</green></bold>".termcolor

data/lib/command/send.rb

@@ -57,15 +57,15 @@ module Command
         error "デバイス名が指定されていないか、間違っています。\n" +
               "narou setting device=デバイス名 で指定出来ます。\n" +
               "指定出来るデバイス名：" + Device::DEVICES.keys.join(", ")
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       unless device.physical_support?
         error "#{device.display_name} への直接送信は対応していません"
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       unless device.connecting?
         error "#{device.display_name} が接続されていません"
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       end
       send_all = false
       titles = {}
@@ -98,6 +98,7 @@ module Command
         print "#{device.name}へ送信しています"
         exit_copy = false
         copy_to_path = nil
+        Thread.abort_on_exception = true
         Thread.new do
           copy_to_path = device.copy_to_documents(ebook_path)
           exit_copy = true
@@ -111,12 +112,12 @@ module Command
           puts copy_to_path + " へコピーしました"
         else
           error "#{device.name}が見つからなかったためコピー出来ませんでした"
-          exit 1   # next しても次も失敗すると分かりきっているためここで終了する
+          exit Narou::EXIT_ERROR_CODE   # next しても次も失敗すると分かりきっているためここで終了する
         end
       end
-    rescue Interrupt
+    rescue Device::SendFailure, Interrupt
       puts "送信を中断しました"
-      exit 1
+      exit Narou::EXIT_ERROR_CODE
     end
   end
 end

data/lib/command/setting.rb

@@ -5,34 +5,23 @@
 
 require_relative "../inventory"
 require_relative "../novelsetting"
+require_relative "../eventable"
 
 module Command
   class Setting < CommandBase
+    include Narou::Eventable
+
     def self.oneline_help
       "各コマンドの設定を変更します"
     end
 
-    class InvalidVariableType < StandardError
-      def initialize(type)
-        super("値が #{Setting.variable_type_to_description(type).rstrip} ではありません")
-      end
-    end
-
-    class UnknownVariableType < StandardError
-      def initialize(type)
-        super("unknwon variable type (:#{type})")
-      end
-    end
-
-    class InvalidVariableName < StandardError; end
-
     def initialize
       super("[<name>=<value> ...] [options]")
       @opt.separator <<-EOS
 
   ・各コマンドの設定の変更が出来ます。
-  ・Global な設定はユーザープロファイルに保存され、すべての narou コマンドで使われます。
-  ・下の一覧は一部です。すべてを確認するには -a オプションを付けて確認して下さい。
+  ・Global な設定はユーザープロファイルに保存され、すべての narou コマンドで使われます
+  ・下の一覧は一部です。すべてを確認するには -a オプションを付けて確認して下さい
 
   Local Variable List:
         <name>           <value>              説明
@@ -64,25 +53,6 @@ module Command
       }
     end
 
-    def self.variable_type_to_description(type)
-      case type
-      when :boolean
-        "true/false  "
-      when :integer
-        "整数        "
-      when :float
-        "小数点数    "
-      when :string
-        "文字列      "
-      when :directory
-        "フォルダパス"
-      when :file
-        "ファイルパス"
-      else
-        raise UnknownVariableType, type
-      end
-    end
-
     def get_scope_of_variable_name(name)
       [:local, :global].each do |scope|
         if SETTING_VARIABLES[scope].include?(name)
@@ -99,50 +69,12 @@ module Command
     def casting_variable(name, value)
       scope = get_scope_of_variable_name(name)
       unless scope
-        raise InvalidVariableName, "#{name} は不明な名前です"
+        raise Helper::InvalidVariableName, "#{name} は不明な名前です"
       end
-      casted_value = string_cast_to_type(value, SETTING_VARIABLES[scope][name][0])
+      casted_value = Helper.string_cast_to_type(value, SETTING_VARIABLES[scope][name][0])
       [scope, casted_value]
     end
 
-    def string_cast_to_type(value, type)
-      result = nil
-      case type
-      when :boolean
-        case value
-        when /true/i
-          result = true
-        when /false/i
-          result = false
-        else
-          raise InvalidVariableType, type
-        end
-      when :integer
-        if value =~ /^[+-]?\d+$/
-          result = value.to_i
-        else
-          raise InvalidVariableType, type
-        end
-      when :float
-        if value =~ /^[+-]?\d+\.?\d*$/
-          result = value.to_f
-        else
-          raise InvalidVariableType, type
-        end
-      when :directory, :file
-        if File.method("#{type}?").call(value)
-          result = File.expand_path(value)
-        else
-          raise InvalidVariableType, type
-        end
-      when :string
-        result = value
-      else
-        raise UnknownVariableType, type
-      end
-      result
-    end
-
     def output_setting_list
       settings = {
         local: Inventory.load("local_setting", :local),
@@ -159,6 +91,12 @@ module Command
       end
     end
 
+    def output_error(msg, name = nil)
+      @error_count += 1
+      error msg
+      trigger(:error, msg, name)
+    end
+
     def execute(argv)
       super
       if argv.empty?
@@ -170,20 +108,21 @@ module Command
         global: Inventory.load("global_setting", :global)
       }
       device = Narou.get_device
+      @error_count = 0
       self.extend(device.get_hook_module) if device
       argv.each do |arg|
         name, value = arg.split("=", 2).map(&:strip)
         if name == ""
-          error "書式が間違っています。変数名=値 のように書いて下さい"
+          output_error("書式が間違っています。変数名=値 のように書いて下さい")
           next
         end
         scope = get_scope_of_variable_name(name)
         unless scope
-          error "#{name} という変数は存在しません"
+          output_error("#{name} という変数は存在しません", name)
           next
         end
         if value.nil?
-          error "書式が間違っています。#{name}=値 のように書いて下さい"
+          output_error("書式が間違っています。#{name}=値 のように書いて下さい", name)
           next
         end
         if value == ""
@@ -192,14 +131,15 @@ module Command
         end
         begin
           scope, casted_value = casting_variable(name, value)
-        rescue InvalidVariableName, InvalidVariableType => e
-          error e.message
+        rescue Helper::InvalidVariableName, Helper::InvalidVariableType => e
+          output_error(e.message, name)
           next
         end
         hook_call(:modify_settings, settings[scope], name, casted_value)
       end
       settings[:local].save
       settings[:global].save
+      exit @error_count if @error_count > 0
     end
 
     def modify_settings(scoped_settings, name, value)
@@ -232,15 +172,14 @@ module Command
         puts message.string
       end
     rescue Device::UnknownDevice => e
-      error e.message
-      puts "設定できるのは #{Device::DEVICES.keys.join(", ")} です"
+      output_error("#{e.message}\n設定できるのは #{Device::DEVICES.keys.join(", ")} です", "device")
     end
 
     def get_variable_list_strings(scope)
       result = ""
       SETTING_VARIABLES[scope].each do |name, info|
         if @options["all"] || info[2] != INVISIBLE
-          type_description = self.class.variable_type_to_description(info[0])
+          type_description = Helper.variable_type_to_description(info[0])
           result << "    <bold><green>#{name.ljust(18)}</green></bold> #{type_description} #{info[1]}\n".termcolor
         end
       end
@@ -255,10 +194,9 @@ module Command
       puts get_variable_list_strings(:global).gsub(/^ {4}/, "")
     end
 
-    TYPE_OF_VALUE = {
-      TrueClass => :boolean, FalseClass => :boolean, Fixnum => :integer,
-      Float => :float, String => :string
-    }
+    def self.get_setting_variables
+      SETTING_VARIABLES
+    end
 
     INVISIBLE = true
 
@@ -269,16 +207,18 @@ module Command
         "convert.no-mobi" => [:boolean, "MOBI変換を無効にするか"],
         "convert.no-strip" => [:boolean, "MOBIのstripを無効にするか\n" +
                                          " " * 6 + "※注意：KDP用のMOBIはstripしないでください"],
-        "convert.no-zip" => [:boolean, "i文庫用のzipファイルを作らない"],
-        "convert.no-open" => [:boolean, "変換終了時に保存フォルダを開かない"],
+        "convert.no-zip" => [:boolean, "i文庫用のzipファイル作成を無効にするか"],
+        "convert.no-open" => [:boolean, "変換時に保存フォルダを開かないようにするか"],
         "convert.copy_to" => [:directory, "変換したらこのフォルダにコピーする\n" +
                                           " " * 6 + "※注意：存在しないフォルダだとエラーになる"],
         "convert.inspect" => [:boolean, "常に変換時に調査結果を表示するか"],
         "download.interval" => [:float, "各話DL時に指定した秒数待機する。デフォルト0"],
         "download.wait-steps" => [:integer, "この値で指定した話数ごとにウェイトを入れる\n" +
                                        " " * 6 + "※注意：11以上を設定してもなろうの場合は10話ごとにウェイトが入ります"],
+        "download.use-subdirectory" => [:boolean, "小説を一定数ごとにサブフォルダへ分けて保存するか\n" +
+                                                  " " * 6 + "※注意：小説を大量に同一フォルダに保存するとパフォーマンスが劣化する回避策"],
         "send.without-freeze" => [:boolean, "`全話'送信時に凍結された小説は対象外に"],
-        "update.strong" => [:boolean, "更新漏れが無いように改稿日の分は必ずDLする"],
+        "update.strong" => [:boolean, "更新漏れが無い様に改稿日の分は必ずDLするか"],
         "device" => [:string, "変換、送信対象の端末(sendの--help参照)"],
         "multiple-delimiter" => [:string, "--multiple指定時の区切り文字"],
       },
@@ -286,21 +226,25 @@ module Command
         "aozoraepub3dir" => [:directory, "AozoraEpub3のあるフォルダを指定", INVISIBLE],
         "difftool" => [:string, "Diffで使うツールのパスを指定する"],
         "difftool.arg" => [:string, "difftoolで使う引数を設定(オプション)"],
-        "no-color" => [:boolean, "カラー表示を無効にする"],
-        "over18" => [:boolean, "", INVISIBLE],
+        "no-color" => [:boolean, "カラー表示を無効にするか"],
+        "server-port" => [:integer, "WEBサーバ起動時のポート"],
+        "server-bind" => [:string, "WEBサーバのホスト制限(未設定時:起動PCのIP)", INVISIBLE],
+        "over18" => [:boolean, "18歳以上かどうか", INVISIBLE],
       }
     }
 
     NovelSetting::DEFAULT_SETTINGS.each do |default|
       SETTING_VARIABLES[:local]["force." + default[:name]] = [
-        #TYPE_OF_VALUE[default[:value].class], "\n      " + default[:help], INVISIBLE
-        TYPE_OF_VALUE[default[:value].class], "", INVISIBLE
+        Helper.type_of_value(default[:value]), "\n      " + default[:help], INVISIBLE
       ]
     end
 
     Dir.glob(File.expand_path(File.join(File.dirname(__FILE__), "*.rb"))) do |path|
       cmd_name = File.basename(path, ".rb")
-      SETTING_VARIABLES[:local]["default_args." + cmd_name] = [:string, "", INVISIBLE]
+      SETTING_VARIABLES[:local]["default_args." + cmd_name] =
+        [:string, "#{cmd_name} コマンドのデフォルトオプション", INVISIBLE]
     end
+
+    SETTING_VARIABLES.freeze
   end
 end

data/lib/command/tag.rb

@@ -69,6 +69,28 @@ module Command
       }
     end
 
+    #
+    # タグ一覧取得
+    #
+    # ids: 特定の小説のタグだけ取得するためのIDの配列を指定
+    # @return: { "タグの名前" => タグが付けられた小説の数, ... }
+    #
+    def self.get_tag_list(ids = nil)
+      database = Database.instance
+      tag_list = Hash.new(0)
+      database.each do |_, data|
+        if ids.kind_of?(Array)
+          next unless ids.include?(data["id"])
+        end
+        tags = data["tags"] || []
+        tags.each do |tag|
+          tag_list[tag] += 1
+        end
+      end
+      tag_list.default = nil
+      tag_list
+    end
+
     def get_color_list
       COLORS.map { |color|
         "<bold><#{color}>#{color}</#{color}></bold>"
@@ -84,7 +106,7 @@ module Command
           return
         end
         error "対象の小説を指定して下さい"
-        exit 1
+        exit Narou::EXIT_ERROR_CODE
       else
         if @options["mode"] == :list
           search_novel_by_tag(argv)
@@ -96,19 +118,9 @@ module Command
 
     def display_taglist
       database = Database.instance
-      tags_list = {}
-      database.each do |_, data|
-        tags = data["tags"] || []
-        tags.each do |tag|
-          if tags_list[tag]
-            tags_list[tag] += 1
-          else
-            tags_list[tag] = 1
-          end
-        end
-      end
+      tag_list = Tag.get_tag_list
       puts "タグ一覧"
-      puts tags_list.map { |tag, count|
+      puts tag_list.map { |tag, count|
         color = Tag.get_color(tag)
         "<bold><#{color}>#{TermColorLight.escape(tag)}(#{count})</#{color}></bold>"
       }.join(" ").termcolor

data/lib/command/update.rb

@@ -42,6 +42,7 @@ module Command
 
     def execute(argv)
       super
+      mistook_count = 0
       update_target_list = argv.dup
       no_open = false
       if update_target_list.empty?
@@ -66,6 +67,7 @@ module Command
         Helper.print_horizontal_rule if i > 0
         if display_message
           puts display_message
+          mistook_count += 1
           next
         end
         result = Downloader.start(target)
@@ -79,15 +81,18 @@ module Command
           end
         when :failed
           puts "ID:#{data["id"]}　#{data["title"]} の更新は失敗しました"
+          mistook_count += 1
         when :canceled
           puts "ID:#{data["id"]}　#{data["title"]} の更新はキャンセルされました"
+          mistook_count += 1
         when :none
           puts "#{data["title"]} に更新はありません"
         end
       end
+      exit mistook_count if mistook_count > 0
     rescue Interrupt
       puts "アップデートを中断しました"
-      exit 1
+      exit Narou::EXIT_ERROR_CODE
     end
   end
 end