Procedure-Pdf-Page-Count #1
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Volian/Applications#1
Reference in New Issue
Block a user
Proc PDF page count
Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
@@ -0,0 +30,4 @@using (var fbd = new FolderBrowserDialog()){fbd.Description = "Select the PDF folder";//fbd.ShowHiddenFiles = true;Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
@@ -0,0 +37,4 @@{pdffiles = Directory.GetFiles(fbd.SelectedPath,"*.pdf");System.Windows.Forms.MessageBox.Show("Files found: " + pdffiles.Length.ToString(), "Message");Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
@@ -0,0 +101,4 @@}catch (Exception ex){System.Windows.Forms.MessageBox.Show("An Error has occured. Please try again. Error: " + ex.Message);Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
@@ -0,0 +114,4 @@ExcelApp.Visible = true;}//private void pageCountView_CellContentClick(object sender, DataGridViewCellEventArgs e)Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
@@ -0,0 +121,4 @@private void pageCountView_CellDoubleClick(object sender, DataGridViewCellEventArgs e){DataGridViewCell cell = pageCountView.Rows[e.RowIndex].Cells[e.ColumnIndex];//System.Windows.Forms.MessageBox.Show("Double Click Row Index: " + cell.Value.ToString());Not worried about it since stand alone app - but in general should remove commented out code and use string interpolation instead of concatenation. Just bringing it up as a "get in the habit thing" as working on the Technical Debt Upgrade & if we are not doing these things moving forward, we will essentially be undoing what we are doing with the Technical Debt upgrade.
looks good