remove old code

vapidbabble · Sep 2, 2015 · c7e60d7 · c7e60d7
1 parent 5351197
commit c7e60d7
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 29 deletions.
diff --git a/README.md b/README.md
@@ -8,3 +8,4 @@ need to:
 
 1. remove convert_to_arabic_old
 2. added error_handling?
+3. move code to lib?
diff --git a/roman.rb b/roman.rb
@@ -29,28 +29,6 @@ def convert_to_roman(arabic)
         roman
     end
 
-    def convert_to_arabic_old(str)
-        arabic=0
-        myind=0
-        edge=false
-        str.each_char do |ele|
-            if edge
-                edge=false
-                myind+=1
-                next
-            end
-            value=@romanlist[ele]
-            substr=str.slice(myind,2)
-            if (substr.length == 2)  && (value < @romanlist[substr[1]])
-                value=@romanlist[substr]
-                edge=true
-            end
-            myind+=1
-            arabic+=value
-        end
-        arabic
-    end
-
     def convert_to_arabic(str)
         arabic=0
         @romanlist.each do |rsymbol,num|

diff --git a/test_roman.rb b/test_roman.rb
@@ -13,13 +13,6 @@ def test_to_roman
     assert_equal "MMMMMMDCCLXXXIX", @convert.convert_to_roman(6789)
   end
 
-  def test_to_arabic_old
-    assert_equal @convert.convert_to_arabic_old("VIII"), 8
-    assert_equal @convert.convert_to_arabic_old("XLIX"), 49
-    assert_equal @convert.convert_to_arabic_old("MMXV"), 2015
-    assert_equal @convert.convert_to_arabic_old("MMMMMMDCCLXXXIX"), 6789
-  end
-
   def test_to_arabic
     assert_equal @convert.convert_to_arabic("VIII"), 8
     assert_equal @convert.convert_to_arabic("XLIX"), 49