narou 1.6.4 → 1.7.0

data/README.md

@@ -24,147 +24,48 @@ Narou.rb ― 「小説家になろう」「小説を読もう！」ダウンロ
 更新履歴 - ChangeLog
 --------------------
 
-
-### 1.6.4 : 2014/09/17
+### 1.7.0 : 2014/09/27
 
 #### 追加機能
-- `list` コマンドに `--echo` オプション(短縮：-e)を追加しました。パイプやリダ
-  イレクトを経由してもリストをそのまま表示します。また、ANSIColorコードの削除
-  も同時に行うので、--no-color オプションは必要ありません
+- `list` コマンドの `--filter` オプションに `frozen` 及び `nonfrozen` が追加
+  されました。凍結状態のフィルタリングが可能になります。また、同時に複数の値を
+  受け付けるようになりました
 
-----
+		narou list -f frozen （凍結された小説のみ表示）
+		narou list -f nonfrozen （凍結されていない小説のみ表示）
+		narou list -f "ss nonfrozen" （凍結されていない短編を表示）
 
-### 1.6.3 : 2014/09/11
+#### 仕様変更
+- 英文判定文字に `&:;_-` を追加しました
+	+ 今までおかしかった例：Tuez-les tous, Dieu reconnaitra les siens）
+- 8文字以上の半角アルファベットは全角に変換せずに半角のままになります
 
 #### Bug Fix
-- ハーメルン、Arcadiaにおいて小説が削除されている場合にエラーが出ていたのを修正
-- １話も投稿されていない小説でエラーが出ていたのを修正
+- 行頭が英文で始まる行がオートインデントできていなかったのを修正
 
-----
+#### その他
+- 起動にかかる時間及び表示処理速度を向上させました(起動時間がv1.6に比べて約50%短縮)
 
-### 1.6.2 : 2014/09/08
+----
 
-#### 仕様変更
-- あらすじだけ変更された場合も更新とみなすようになりました
+### 1.6.4 : 2014/09/17
 
-#### Bug Fix
-- `download.wait-steps` でウェイトを入れるタイミングがおかしくなっていたため、
-  503エラーが頻発するようになっていたのを修正
-- 完結しているにもかかわらず完結タグが付与されない場合があったのを修正。また、
-  完結していた小説が連載を再開した場合、完結タグを削除するようになりました
-- 話数が減っているにも関わらずダイジェスト化検知出来ない場合があったのを修正
+#### 追加機能
+- `list` コマンドに `--echo` オプション(短縮：-e)を追加しました。パイプやリダ
+  イレクトを経由してもリストをそのまま表示します。また、ANSIColorコードの削除
+  も同時に行うので、--no-color オプションは必要ありません
 
 ----
 
-### 1.6.1 : 2014/09/05
+### 1.6.3 : 2014/09/11
 
 #### Bug Fix
-- Ruby1.9.3において `tag` コマンドが使用出来ない問題を修正
-- flag データを設定し、その後削除した場合に flag --convert-tag コマンドがクラッシュする問題を修正
+- ハーメルン、Arcadiaにおいて小説が削除されている場合にエラーが出ていたのを修正
+- １話も投稿されていない小説でエラーが出ていたのを修正
 
 ----
 
-### 1.6.0 : 2014/08/28
-
-#### 追加機能
-- **iBooks** での管理に対応しました
-	+ Mac OS X Mavericks以上が対象です
-	+ `narou s device=ibooks` で変更できます
-	+ 初回登録時はiBooksでEPUBファイルを開くので、ウィンドウが大量に開くことに
-	  ご注意ください。次回以降はiBooksの管理フォルダを直接更新してウィンドウが
-	  開くことはありません
-- **JRuby** での実行に対応しました（Windowsでは環境変数JAVA_OPTSに
-  -Dfile.encoding=UTF8を設定して下さい）
-- `diff` コマンドで外部ツールがなくても更新された部分の差分を表示できるように
-  なりました
-	+ `difftool` が設定されていない場合、独自の差分表示処理を行います。
-	  小説用なので一文字単位の修正がわかりやすいように調整してあります
-
-	  ![diff使用時のイメージ](https://raw.github.com/wiki/whiteleaf7/narou/images/diff_cap.png)
-- `tag` コマンドが追加されました
-	+ 小説に自由にタグを付けることができ、絞り込み等で使用出来ます
-		+ タグを追加: `narou tag -a akatsuki 0`
-		+ タグが付いた小説を表示: `narou tag akatsuki`
-		+ コマンドの対象小説をタグで指定: `narou update akatsuki`
-		+ その他詳細な説明は `narou tag -h` を参照して下さい
-- `list` コマンドが拡張されます
-	+ `--site` オプション(短縮: -s)が追加されました。掲載小説サイト名を合わせて
-	  表示します
-	+ `--author` オプション(短縮:-a)が追加されました。作者名を合わせて表示します
-	+ `--tag` オプション(短縮: -t)が追加されました。タグで小説を検索出来ます
-		* `narou l -t tagname` 、 `narou l -t "tag1 tag2"`
-		* `narou l -t` と引数を省略すると全ての小説をタグつきで表示
-	+ `--grep` オプション(短縮: -g)が追加されました。リストの中から指定された
-	  文字列で検索します。最終的に表示される内容から検索するため、作者名やサイト
-	  名等そのままでは表示されない要素から検索したい場合、それぞれを表示するオプ
-	  ションも合わせて指定して下さい
-		* 例1： リストから作者名に“紫炎”が含まれる小説を探したい場合
-			- `narou list --author --grep 紫炎` 或いは `narou l -ag 紫炎`
-		* 例2: 小説家になろう以外の小説を表示したい場合
-			- `narou list --site --grep "-小説家になろう"` 或いは
-			  `narou l -sg "-小説家になろう"` (-をつけることでNOT検索)
-		* 例3: ハーメルンの小説からSAOを含む小説を探す
-			- `narou l -sg "ハーメルン SAO"` (スペース区切りによるAND検索)
-	+ listコマンドの結果をパイプで他のコマンドに接続できるようになりました
-		* 例1: ハーメルに掲載されている小説にhamelnタグを付けたい
-			- `narou l -sg ハーメルン | narou t -a hameln`
-		* 例2: 短編を全て凍結したい
-			- `narou l -f ss | narou freeze --on`
-		* 注意点として、パイプでつないだ場合は remove コマンドは確認メッセージを
-		  出さずに削除します
-- `convert` コマンドに `--ignore-force` オプションが追加されました。
-  `narou s force.設定名=値` として強制上書きされる設定を一時的に無効にします
-- 空改行の削除処理を制御する `enable_pack_blank_line` オプションが追加されました
-	+ デフォルトでは有効です。無効にするには各小説の setting.ini に
-	  `enable_pack_blank_line=false` を追加するか、
-	  `narou s force.enable_pack_blank_line=false` を実行して下さい
-- `freeze` コマンドに強制凍結及び解除する `--on` 及び `--off` オプションが追加
-  されました
-
-#### 仕様変更
-- `update.strong` の挙動を、「当日分は必ずDLする」から「当日分は内容を見て更新
-  されているか確認する」に変更しました。またタイムスタンプを記録するようにし、
-  アップデート時のパフォーマンスを向上させました
-- デバイス設定時に最低限必要な関連項目を設定変更するようになりました
-- 空改行の間引き具合を若干弱くしました
-- `list` コマンドの `--type` オプションを `--kind` (短縮:-k)に変更しました
-- 差分オブジェクトの書式を `20YY.MM.DD@hh;mm;ss` から `20YY.MM.DD@hh.mm.ss` に
-  変更します
-	+ このバージョンから反映。以前のバージョンで作成された差分は旧書式のままです
-- 新着のみの更新の場合差分データを作成しないように変更しました
-- `backup` コマンドで作成するZIPファイルの内部ファイルパス文字エンコーディング
-  をOS毎に変更するようにしました
-	+ Windows = Windows-31J、その他OS = UTF-8
-- `flag` コマンドは非推奨になり、tagコマンドに統一されます。今後完結フラグを立
-  てる場合、 `narou tag -a end 0` として下さい。flagコマンドは1.7.0で廃止される
-	予定です
-	+ flagデータをtagデータに移行する場合は `narou flag --convert-tag` を実行
-	  して下さい
-- 完結の情報を自動で付与するようになりました。完結を検知したら end タグを追加
-  します
-	+ 現在は小説家になろう関連サイトとハーメルンのみ自動検知します
-- 選択肢を選ぶときエンターを押す必要がなくなりました
-- このバージョンからセマンティックバージョニングに移行します
-	+ Major.Minor.Teen という形式で、特別なアップデートでMajor、追加機能を含む
-	  アップデートでMinor、バグ修正のみのアップデートでTeenの番号を増やします
-
-#### Bug Fix
-- 小説が掲載サイトから削除されていない時(500や503)にも自動凍結していたのを修正
-- iBooksで閲覧時章タイトルページのレイアウトが崩れていたのを修正
-- iBooksで閲覧時Arcadiaの掲載アドレスがあるページでエラーがでていたのを修正 #47
-- ハーメルンで作者名が正常に取得できない場合があったのを修正 #48
-- ハーメルンで連載形式の短編が1話目しか取得できていなかったのを修正 #49
-- 自動字下げの判定処理が正常に動作していなかったのを修正
-- `diff` コマンドの差分表示時に更新日が挿入されない場合があったのを修正
-- `diff` コマンドで小説家になろう以外の小説の差分がHTMLタグ付きのまま表示されて
-  いたのを修正
-- 未対応のURLを入力した場合にエラーが出るようになっていたのを修正
-- 小説家になろうの短編のあらすじで改行が無視されていたのを修正
-- 小説家になろうの短編のあらすじで先頭に縦棒（｜）が入ってしまう問題に対応 #46
-- 小説家になろうの挿絵を認識出来ない場合があったのを修正
-- 前書き・後書き内にある挿絵を単ページ化出来なかったのを修正 #8
-- 漢数字変換で対応桁数以上を変換しようとしてクラッシュしていたのを修正
-- その他ヘルプ文の訂正や無駄な処理削減等多数の細かい調整
+過去の更新履歴は[こちらを参照](https://github.com/whiteleaf7/narou/blob/master/ChangeLog.md)
 
 ----
 

data/Rakefile

@@ -8,5 +8,5 @@ begin
     spec.pattern = 'spec/**/*_spec.rb'
     spec.rspec_opts = ['-cr spec_helper']
   end
-rescue LoadError => e
+rescue LoadError
 end

data/lib/color.rb

@@ -30,14 +30,7 @@ TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
 SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 =end
 
-alias :original_warn :warn
-unless $debug
-  def warn(*a); end
-end
-
-require "termcolor"
-
-alias :warn :original_warn
+require "termcolorlight"
 
 if RbConfig::CONFIG["host_os"] =~ /mswin(?!ce)|mingw|bccwin/i
   require_relative "extensions/windows"

data/lib/command/backup.rb

@@ -41,7 +41,7 @@ module Command
       paths = Dir.glob("**/*").keep_if { |path|
         File.file?(path) && path.split("/", 2)[0] != BACKUP_DIR_NAME
       }
-      FileUtils.mkdir(BACKUP_DIR_NAME) unless File.exists?(BACKUP_DIR_NAME)
+      FileUtils.mkdir(BACKUP_DIR_NAME) unless File.exist?(BACKUP_DIR_NAME)
       Zip.unicode_names = true unless Helper.os_windows?
       Zip::File.open(File.join(BACKUP_DIR_NAME, zipfilename), Zip::File::CREATE) do |zip|
         paths.each do |path|

data/lib/command/diff.rb

@@ -127,7 +127,7 @@ module Command
     def clean_diff(id)
       cache_root_dir = Downloader.get_cache_root_dir(id)
       print Database.instance.get_data("id", id)["title"] + " の"
-      unless File.exists?(cache_root_dir)
+      unless File.exist?(cache_root_dir)
         puts "差分はひとつもありません"
         return
       end
@@ -179,7 +179,7 @@ module Command
         cache_root_dir = Downloader.get_cache_root_dir(id)
         if cache_root_dir
           cache_dir = File.join(cache_root_dir, @options["view_diff_version"])
-          cache_dir = nil unless File.exists?(cache_dir)
+          cache_dir = nil unless File.exist?(cache_dir)
         end
       else
         nth = @options["number"]
@@ -202,7 +202,7 @@ module Command
       cache_sections = []
       cache_section_list.each do |path|
         match_latest_path = File.join(novel_dir, File.basename(path))
-        latest_novel_sections << YAML.load_file(match_latest_path) if File.exists?(match_latest_path)
+        latest_novel_sections << YAML.load_file(match_latest_path) if File.exist?(match_latest_path)
         cache_sections << YAML.load_file(path)
       end
 
@@ -240,7 +240,7 @@ module Command
       end
       puts "差分一覧"
       cache_list.each.with_index(1) do |cache_path, i|
-        puts ("<bold><yellow>" + File.basename(cache_path) + "   -#{i}</yellow></bold>").termcolor
+        puts "<bold><yellow>#{File.basename(cache_path)}   -#{i}</yellow></bold>".termcolor
         cache_section_list = Dir.glob(File.join(cache_path, "*.yaml"))
         if cache_section_list.length > 0
           cache_section_list.map { |section_path|
@@ -258,7 +258,7 @@ module Command
       Database.instance.each do |id, data|
         next if Narou.novel_frozen?(id)
         cache_root_dir = Downloader.get_cache_root_dir(id)
-        next unless File.exists?(cache_root_dir)
+        next unless File.exist?(cache_root_dir)
         FileUtils.remove_entry_secure(cache_root_dir)
         puts "#{data["title"]} の差分を削除しました"
       end

data/lib/command/download.rb

@@ -59,7 +59,7 @@ module Command
     end
 
     def print_prompt(targets)
-      print "#{targets.count}> "
+      print "#{targets.size}> "
     end
 
     def interactive_mode
@@ -92,7 +92,7 @@ module Command
       super
       if argv.empty?
         targets = interactive_mode
-        return if targets.count == 0
+        return if targets.size == 0
         argv += targets
       end
       tagname_to_ids(argv)

data/lib/command/freeze.rb

@@ -41,15 +41,7 @@ module Command
     end
 
     def output_freeze_list
-      database = Database.instance
-      puts "凍結中小説一覧"
-      puts " ID |     タイトル"
-      Inventory.load("freeze", :local).each do |id, _|
-        data = database.get_data("id", id)
-        if data
-          puts "#{id.to_s.rjust(3)} | #{data["title"]}"
-        end
-      end
+      List.execute!(%w(--filter frozen))
     end
 
     def execute(argv)

data/lib/command/init.rb

@@ -67,7 +67,7 @@ module Command
       end
       puts "<bold><green>AozoraEpub3の設定を行います</green></bold>".termcolor
       unless @global_setting["aozoraepub3dir"]
-        puts ("<bold><red>" + "!!!WARNING!!!".center(70) + "</red></bold>").termcolor
+        puts "<bold><red>#{"!!!WARNING!!!".center(70)}</red></bold>".termcolor
         puts "AozoraEpub3の構成ファイルを書き換えます。narouコマンド用に別途新規インストールしておくことをオススメします"
       end
       aozora_path = ask_aozoraepub3_path
@@ -102,7 +102,7 @@ module Command
       src = ["AozoraEpub3.ini", "vertical_font.css", "DMincho.ttf"]
       dst = ["AozoraEpub3.ini", "template/OPS/css_custom/vertical_font.css", "template/OPS/fonts/DMincho.ttf"]
       puts "(次のファイルをコピーor上書きしました)"
-      src.count.times do |i|
+      src.size.times do |i|
         src_full_path = File.join(Narou.get_preset_dir, src[i])
         dst_full_path = File.join(aozora_path, dst[i])
         FileUtils.install(src_full_path, dst_full_path)
@@ -131,8 +131,8 @@ module Command
         when input == ""
           break
         end
-        print ("\n<bold><green>入力されたフォルダにAozoraEpub3がありません。" +
-               "もう一度入力して下さい:</green></bold>\n&gt;").termcolor
+        print "\n<bold><green>入力されたフォルダにAozoraEpub3がありません。" \
+              "もう一度入力して下さい:</green></bold>\n&gt;".termcolor
       end
       nil
     end

data/lib/command/list.rb

@@ -11,7 +11,9 @@ module Command
 
     # MEMO: 0 は昔の小説を凍結したままな場合、novel_type が設定されていないので、
     #       nil.to_i → 0 という互換性維持のため
-    NOVEL_TYPE_LABEL = ["連載", "連載", "短編"]
+    NOVEL_TYPE_LABEL = %w(連載 連載 短編)
+
+    FILTERS_TYPE = Hash[*%w(series 連載 ss 短編 frozen 凍結 nonfrozen 非凍結)]
 
     def self.oneline_help
       "現在管理している小説の一覧を表示します"
@@ -19,7 +21,7 @@ module Command
 
     def initialize
       super("[<limit>] [options]")
-      @opt.separator <<-EOS.termcolor
+      @opt.separator <<-EOS
 
   ・現在管理している小説の一覧を表示します
   ・表示されるIDは各コマンドで指定することで小説名等を入力する手間を省けます
@@ -33,6 +35,7 @@ module Command
     narou list 5 -l        # 最近更新のあった5件表示
     narou list 10 -rl      # 古い順に10件表示
     narou list -f ss       # 短編小説だけ表示
+    narou list -f "ss frozen"   # 凍結している短編だけ表示
 
     # 小説家になろうの小説のみを表示
     narou list --site --grep 小説家になろう
@@ -74,8 +77,9 @@ module Command
         @options["author"] = true
       }
       @opt.on("-f", "--filter VAL", String,
-              "表示を絞るためのフィルターの種類(連載:series, 短編:ss)") { |filter|
-        @options["filter"] = filter
+              "表示を絞るためのフィルター。スペース区切りで複数可\n" \
+              "#{' '*25}#{filter_type_help}") { |filter|
+        @options["filters"] = filter.downcase.split
       }
       @opt.on("-g", "--grep VAL", String,
               "指定された文字列でリストを検索する") { |search|
@@ -102,6 +106,33 @@ module Command
       true
     end
 
+    def filter_type_help
+      FILTERS_TYPE.map { |filter, info|
+        "#{filter}(#{info})"
+      }.join(",")
+    end
+
+    def test_filter(filters, novel_type, frozen)
+      apply_sum = filters.inject(0) do |sum, item|
+        apply = case item
+                when "series"
+                  novel_type == 0 && novel_type == 1
+                when "ss"
+                  novel_type == 2
+                when "frozen"
+                  frozen
+                when "nonfrozen"
+                  !frozen
+                else
+                  error "不明なフィルターです(#{item})"
+                  warn "filters = #{filter_type_help}"
+                  exit 1
+                end
+        sum + (apply ? 1 : 0)
+      end
+      apply_sum == filters.count
+    end
+
     def header
       [
          " ID ", " 更新日 ",
@@ -115,24 +146,22 @@ module Command
     def output_list(novels)
       now = Time.now
       today = now.strftime("%y/%m/%d")
-      filter = @options["filter"]
+      filters = @options["filters"] || []
       selected_lines = {}
       novels.each do |novel|
         novel_type = novel["novel_type"].to_i
-        if filter
-          if filter == "series" && novel_type != 0 && novel_type != 1
-            next
-          elsif filter == "ss" && novel_type != 2
-            next
-          end
+        id = novel["id"]
+        frozen = Narou.novel_frozen?(id)
+
+        unless filters.empty?
+          next unless test_filter(filters, novel_type, frozen)
         end
+
         if @options["tags"]
           next unless valid_tags?(novel, @options["tags"])
         end
-        id = novel["id"]
-        frozen = Narou.novel_frozen?(id)
         disp_id = ((frozen ? "*" : "") + id.to_s).rjust(4)
-        disp_id = disp_id.sub("*", "<bold><cyan>*</cyan></bold>").termcolor if frozen
+        disp_id = disp_id.sub("*", "<bold><cyan>*</cyan></bold>") if frozen
         flags = novel["flags"] || {}   # flagコマンドは1.6.0から非推奨
         tags = novel["tags"] || []
         flags["end"] ||= tags.include?("end")
@@ -162,14 +191,14 @@ module Command
                 color = Tag.get_color(tag)
                 "<bold><#{color}>#{tag.escape}</#{color}></bold>"
               }.join(",") : nil,
-        ].compact.join(" | ").termcolor
+        ].compact.join(" | ")
       end
       if @options["grep"]
         @options["grep"].each do |search_word|
-          selected_lines.keep_if { |id, line|
+          selected_lines.keep_if { |_, line|
             if search_word =~ /^-(.+)/
               # NOT検索
-              not line.include?($1)
+              !line.include?($1)
             else
               line.include?(search_word)
             end
@@ -178,12 +207,12 @@ module Command
       end
       if STDOUT.tty?
         puts header
-        puts selected_lines.values
+        puts selected_lines.values.join("\n").termcolor
       else
         if @options["echo"]
           # pipeにそのまま出力するときはansicolorコードが邪魔なので削除
           puts header
-          puts $stdout.strip_color(selected_lines.values.join("\n"))
+          puts TermColorLight.strip_tag(selected_lines.values.join("\n"))
         else
           # pipeに接続するときはIDを渡す
           puts selected_lines.keys.join(" ")
@@ -197,7 +226,7 @@ module Command
       if !argv.empty? && argv.first =~ /^\d+$/
         num = argv.first.to_i
       else
-        num = database_values.count
+        num = database_values.size
       end
       if @options["latest"]
         database_values = Database.instance.sort_by_last_update