Use class methods of `File` over `Kernel.open` and `IO.read`

1 files changed, 27 insertions, 17 deletions

diff --git a/sample/mpart.rb b/sample/mpart.rb
index a88eba0ef6..eeb895d3de 100644
--- a/sample/mpart.rb
+++ b/sample/mpart.rb
@@ -2,11 +2,29 @@
 # split into multi part
 # usage: mpart.rb [-nnn] file..
 
+class MPart < File
+  def self.new(basename, extname, part, parts)
+    super(sprintf("%s.%s%02d", basename, extname, part), "w").
+      begin_mpart(basename, part, parts)
+  end
+
+  def begin_mpart(basename, part, parts)
+    printf("%s part%02d/%02d\n", basename, part, parts)
+    write("BEGIN--cut here--cut here\n")
+    self
+  end
+
+  def close
+    write("END--cut here--cut here\n")
+    super
+  end
+end
+
 lines = 1000
 
 if (ARGV[0] =~ /^-(\d+)$/ )
-  lines = $1.to_i;
-  ARGV.shift;
+  lines = $1.to_i
+  ARGV.shift
 end
 
 basename = ARGV[0]
@@ -14,31 +32,23 @@ extname = "part"
 
 part = 1
 line = 0
+ofp = nil
 
 fline = 0
-for i in ifp = open(basename)
-  fline = fline + 1
-end
-ifp.close
+File.foreach(basename) {fline += 1}
 
 parts = fline / lines + 1
 
-for i in ifp = open(basename)
+File.foreach(basename) do |i|
   if line == 0
-    ofp = open(sprintf("%s.%s%02d", basename, extname, part), "w")
-    printf(ofp, "%s part%02d/%02d\n", basename, part, parts)
-    ofp.write("BEGIN--cut here--cut here\n")
+    ofp = MPart.new(basename, extname, part, parts)
   end
   ofp.write(i)
-  line = line + 1
-  if line >= lines and !ifp.eof?
-    ofp.write("END--cut here--cut here\n")
+  line += 1
+  if line >= lines
     ofp.close
-    part = part + 1
+    part += 1
     line = 0
   end
 end
-ofp.write("END--cut here--cut here\n")
 ofp.close
-
-ifp.close