refactoring

Author: limenet

Form1.Designer.cs

@@ -1,20 +1,7 @@
-/***********************************************************************
-			b-PAC 3.0 Component Sample (TestPrint)
-
-			(C)Copyright Brother Industries, Ltd. 2009
-************************************************************************/
-
 using System;
-using System.Collections.Generic;
-using System.ComponentModel;
-using System.Data;
-using System.Drawing;
-using System.Text;
 using System.Windows.Forms;
-using System.Runtime.InteropServices;
-using bpac;
 
-namespace BarcodePrint
+namespace BpacBarcode
 {
     public partial class BarcodeForm : Form
     {
@@ -30,53 +17,24 @@ private void Form1_Load(object sender, EventArgs e)
 
             if (args.GetLength(0) == 3)
             {
-                doPrint(args[1], args[2]);
+                (new PrinterAPI(args[1], args[2])).print();
                 Close();
             }
             else if (args.GetLength(0) == 4)
             {
-                doPrint(args[1], args[2], args[3]);
+                (new PrinterAPI(args[1], args[2], args[3])).print();
                 Close();
             }
             else if (args.GetLength(0) == 5)
             {
-                doPrint(args[1], args[2], args[3], Int32.Parse(args[4]));
+                (new PrinterAPI(args[1], args[2], args[3], Int32.Parse(args[4]))).print();
                 Close();
             }
         }
 
         private void btnPrint_Click(object sender, EventArgs e)
         {
-            doPrint(txtTitle.Text, txtBarcode.Text, timestamp.Value, (int)cntCopies.Value);
-        }
-
-        private void doPrint(string title, string barcode, DateTime timestamp, int count = 1)
-        {
-            doPrint(title, barcode, timestamp.ToString("dd.MM.yyyy HH:mm"), count);
-        }
-
-        private void doPrint(string title, string barcode, string timestamp = "", int count = 1)
-        {
-            timestamp = timestamp.Equals("") ? System.DateTime.Now.ToString("dd.MM.yyyy HH:mm") : timestamp;
-
-            bpac.DocumentClass doc = new DocumentClass();
-            string templatePath = System.IO.Path.GetDirectoryName(System.Reflection.Assembly.GetExecutingAssembly().Location) + "\\" + doc.Printer.GetMediaId().ToString() + ".lbx";
-            if (doc.Open(templatePath) != false)
-            {
-                doc.GetObject("title").Text = title;
-                doc.GetObject("barcode").Text = barcode;
-                doc.GetObject("timestamp").Text = timestamp;
-
-                bpac.PrintOptionConstants printOptions = PrintOptionConstants.bpoAutoCut | PrintOptionConstants.bpoQuality;
-                doc.StartPrint("", printOptions);
-                doc.PrintOut(count, printOptions);
-                doc.EndPrint();
-                doc.Close();
-            }
-            else
-            {
-                MessageBox.Show("Open() Error: " + doc.ErrorCode + "\nMedia ID" + doc.Printer.GetMediaId().ToString());
-            }
+            (new PrinterAPI(txtTitle.Text, txtBarcode.Text, timestamp.Value.ToString(), (int)cntCopies.Value)).print();
         }
 
         protected override bool ProcessCmdKey(ref Message msg, Keys keyData)
@@ -88,7 +46,7 @@ protected override bool ProcessCmdKey(ref Message msg, Keys keyData)
             }
             else if (keyData == Keys.Enter)
             {
-                doPrint(txtTitle.Text, txtBarcode.Text);
+                (new PrinterAPI(txtTitle.Text, txtBarcode.Text, timestamp.Value.ToString(), (int)cntCopies.Value)).print();
             }
             return base.ProcessCmdKey(ref msg, keyData);
         }

Form1.cs

@@ -0,0 +1,57 @@
+using System;
+using System.Windows.Forms;
+using bpac;
+
+namespace BpacBarcode
+{
+    class PrinterAPI
+    {
+        private string title;
+        private string barcode;
+        private string timestamp = System.DateTime.Now.ToString("dd.MM.yyyy HH:mm");
+        private int count = 1;
+
+        public PrinterAPI(string title, string barcode)
+        {
+            this.title = title;
+            this.barcode = barcode;
+        }
+
+        public PrinterAPI(string title, string barcode, string timestamp)
+        {
+            this.title = title;
+            this.barcode = barcode;
+            this.timestamp = timestamp;
+        }
+
+        public PrinterAPI(string title, string barcode, string timestamp, int count)
+        {
+            this.title = title;
+            this.barcode = barcode;
+            this.timestamp = timestamp;
+            this.count = count;
+        }
+
+        public void print()
+        {
+            bpac.DocumentClass doc = new DocumentClass();
+            string templatePath = System.IO.Path.GetDirectoryName(System.Reflection.Assembly.GetExecutingAssembly().Location) + "\\" + doc.Printer.GetMediaId().ToString() + ".lbx";
+            if (doc.Open(templatePath) != false)
+            {
+                doc.GetObject("title").Text = title;
+                doc.GetObject("barcode").Text = barcode;
+                doc.GetObject("timestamp").Text = timestamp;
+
+                bpac.PrintOptionConstants printOptions = PrintOptionConstants.bpoAutoCut | PrintOptionConstants.bpoQuality;
+                doc.StartPrint("", printOptions);
+                doc.PrintOut(count, printOptions);
+                doc.EndPrint();
+                doc.Close();
+            }
+            else
+            {
+                MessageBox.Show("Open() Error: " + doc.ErrorCode + "\nMedia ID" + doc.Printer.GetMediaId().ToString());
+            }
+        }
+    }
+}

PrinterAPI.cs

@@ -2,7 +2,7 @@
 using System.Collections.Generic;
 using System.Windows.Forms;
 
-namespace BarcodePrint
+namespace BpacBarcode
 {
     static class Program
     {

Program.cs

@@ -89,6 +89,7 @@
     <Reference Include="System.Xml" />
   </ItemGroup>
   <ItemGroup>
+    <Compile Include="PrinterAPI.cs" />
     <Compile Include="Form1.cs">
       <SubType>Form</SubType>
     </Compile>