musician 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a24cae27533f98ff5e6efc2d6a29fd5dbb9eaa1c
-  data.tar.gz: dc38bae199112096122f03113813fa926f3d470e
+  metadata.gz: 9aa91808647c10fa546b0ec0fc1994afa5585578
+  data.tar.gz: 78a4be48e645e66c76103c3ba63da2b46fda4301
 SHA512:
-  metadata.gz: 720944e72c8b1b96539b0ed567b4ae496883517f980a027a711a3be5a207a0b59a0470fca8fd3143040609d04f7e3fb73be7517151cdc822906e13116da87d4b
-  data.tar.gz: 18bb880a7c8a7690a58f1eb59102065dad79afe9b5b7aab1c3137486e87078367b488f12357a14ed602943877acd1dbf6cef90b2d34ab437444b05d381bc66a0
+  metadata.gz: b09a4cd13a9b2185057f26ff8c2391ca43be1022714de3ff50686a278e096209141359b9ecefb3f3125f18c676f669db66a2976deafab8d9d8aec47eaa373747
+  data.tar.gz: 246c1ba61b6f4421e3e127f726343b3b886b63d5ebb0c2b5fa585c1277aba8227a5cee1ffb026dc1d5ea1a743d3a3b35489a13c64e1711f9330e70fcf66bcbca

data/lib/files/chord.rb

@@ -2,59 +2,54 @@ class Chord < Note
 
   attr_accessor :name, :raw_name, :addition
 
-  def initialize(name)
-    # ちょっとややこしいし良くないかもしれないけど、
-    # 一応 @name が chord の name になってて、@raw_name が addition (maj7, dim とか)なしの name
-    # こう： @name #=> "G♭m" @raw_name #=> "G♭"
-    # でも、Note.name は普通は maj7 や dim とかはないから、OOP においては Chord.name はおかしいかもしれない
-    # Note.name は Chord.name と違うので、OOPの考え方においては良くないかもしれない。
-    # でも、Note.name と Chord.name はバリ分かりやすいと思うのでそのようにしたい。
-    # とりあえす、Chord.name は Note.name とは違うという風に書いています。
-    @name = name
-    raw_elements = sanitize_method
-    super(raw_elements[0]) # 一応 raw_name で Note のインスタンス変数を作る
-    @raw_name = change_symbol
-    @addition = raw_elements[1]
-    @name = name
-    @name = change_symbol
+  def initialize(initial_name)
+    # Chord.name と Note.name はそもそも違うものなのでOOPにおいては良くないかもしれない
+    # こう: 「Note.name #=> "G"」  「Chord.name #=> "Gm7"」
+    # でも、Note.nameとChord.nameは分かりやすいので、これで行きます。
+    # そのために、Noteのインスタンス変数をadditionなしで作ってから、name, raw_nameとadditionを定義しています。
+
+    raw_elements = sanitize(initial_name)
+    super(raw_elements[:raw_name]) # Noteのインスタンス変数を作る
+
+    @name = change_symbol(initial_name)
+    @raw_name = change_symbol(raw_elements[:raw_name])
+    @addition = raw_elements[:addition]
   end
 
-  # note.rbだけで置くようにしたいけど、raw_nameが。。。
   def position
     if sharp?
-      SHARP_CHORDS.index(raw_name)
+      SHARP_CHORDS.index(@raw_name)
     else
-      FLAT_CHORDS.index(raw_name)
+      FLAT_CHORDS.index(@raw_name)
     end
   end
 
   private
-  def sanitize_method
+  def sanitize(initial_name)
 
-    addition = String.new
-    raw = Array.new
+    raw_elements = {
+      raw_name: String.new,
+      addition: String.new
+    }
 
-    # additionがあるか確認する
+    # intial_nameにはadditionがあるかどうか確認する
     finished_search = false
-    until !addition.empty? || finished_search == true
-      ADDITIONS.each do |add|
-        regexp = Regexp.new(add)
-        raw = Array.new
-        if name.match(regexp)
-          addition = add
-          raw << name.gsub(add, "")
-          raw << add
+    until !(raw_elements[:addition].empty?) || finished_search == true
+      ADDITIONS.each do |addition|
+        regexp = Regexp.new(addition)
+        if initial_name.match(regexp)
+          raw_elements[:raw_name] = initial_name.gsub(addition, "")
+          raw_elements[:addition] = addition
           break
         end
       end
       # もしadditionがなければ、nameをそのまま返して、additionが空の配列のまま返す
-      if addition.empty?
-        raw << name
-        raw << addition
+      if raw_elements[:addition].empty?
+        raw_elements[:raw_name] = initial_name
       end
       finished_search = true
     end
-    raw
+    raw_elements
   end
 
 end

data/lib/files/note.rb

@@ -3,29 +3,30 @@ class Note
   attr_accessor :name
 
   def initialize(name)
-    @name = name
-    @name = change_symbol
+    @name = change_symbol(name)
   end
 
-  def change_symbol
-    @name = @name.tr("#b", "#{Sharp}#{Flat}") # gsub()の変わりに「tr」を使ってる（・ω・）
-  end
-
-  # Noteを初期化する時に#は♯に変換されるから両方が要るかな。。。
+  # 「#」と「b」要らんと思う
   def sharp?
-    !!name.match(/#/) || !!name.match(/♯/) # 真偽を返して欲しかったから次のように書きました
+    !!@name.match(/#/) || !!@name.match(/♯/) # 真偽を返して欲しかった「!!」を入れました
   end
 
   def flat?
-    !!name.match(/b/) || !!name.match(/♭/)
+    !!@name.match(/b/) || !!@name.match(/♭/)
   end
 
   def position
     if sharp?
-      SHARP_CHORDS.index(name)
+      SHARP_CHORDS.index(@name)
     else
-      FLAT_CHORDS.index(name)
+      FLAT_CHORDS.index(@name)
     end
   end
 
+  private
+
+  def change_symbol(note)
+    note = note.tr("#b", "#{Sharp}#{Flat}") # gsub()の変わりに「tr」を使ってる（・ω・）
+  end
+
 end

data/lib/files/song.rb

@@ -2,12 +2,23 @@ class Song
 
   attr_accessor :key, :chords, :title, :tempo
 
-  # keyは文字列で、chordsは文字列の配列
+
   def initialize(key, chords)
-    @key = Chord.new(key)
+    # 文字列かChordのインスタンス変数、両方で定義できるように
+    unless key.instance_of? Chord
+      @key = Chord.new(key)
+    else
+      @key = key
+    end
+
     @chords = chords.map do |chord|
-      Chord.new(chord)
+      unless chord.instance_of? Chord
+        Chord.new(chord)
+      else
+        chord
+      end
     end
+
     # 作る時に定義しなくていいけど、一応自分で定義できる
     @title = ""
     @tempo = ""
@@ -20,8 +31,10 @@ class Song
   end
 
   def key_change(new_key, option=:sharp)
-    # unless new_key.instance_of? Chord then new_key = Chord.new(new_key)
-    new_key = Chord.new(new_key) # もうすでにChordクラスであればこれを飛ばすようにする
+
+    unless new_key.instance_of? Chord
+      new_key = Chord.new(new_key)
+    end
 
     if key.position < new_key.position
       key_up = true

data/lib/musician.rb

@@ -2,7 +2,7 @@ Sharp = "♯"
 Flat = "♭"
 SHARP_CHORDS = ["C", "C♯", "D", "D♯", "E", "F", "F♯", "G", "G♯", "A", "A♯", "B"]
 FLAT_CHORDS = ["C", "D♭", "D", "E♭", "E", "F", "G♭", "G", "A♭", "A", "B♭", "B"]
-ADDITIONS = ["dim7", "dim", "2", "sus4", "sus", "maj7", "maj", "m7", "m", "aug", "7", "6", "9", "11"]
+ADDITIONS = ["add2", "add9", "dim7", "dim", "sus4", "sus", "maj7", "maj", "m7", "m", "aug", "2", "6", "7", "9", "11"]
 
 require 'files/note'
 require 'files/chord'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: musician
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Gabriel Arcangel Zayas