Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Commit e67466c

Browse files
committed
Removed comments after watching a great lecture by Kevlin Henney - Seven Ineffective Coding Habits of Many Programmers
https://www.youtube.com/watch?v=ZsHMHukIlJY 
1 parent ad761f6 commit e67466c

20 files changed

+29
-1044
lines changed

‎CS/Scripts/AssemblyInfo.cs‎

Lines changed: 0 additions & 97 deletions
Original file line numberDiff line numberDiff line change
@@ -6,29 +6,12 @@
66
using System.Deployment.Application;
77
using Microsoft.Win32;
88

9-
// <summary>
10-
// This namespaces if for generic application classes
11-
// </summary>
129
namespace ScriptHelp.Scripts
1310
{
14-
/// <summary>
15-
/// Used to get the assembly information
16-
/// </summary>
17-
/// <remarks>
18-
/// http://danielsaidi.wordpress.com/2009/05/25/c-get-assembly-information/
19-
/// </remarks>
2011
public static class AssemblyInfo
2112
{
22-
/// <summary>
23-
///
24-
/// </summary>
2513
public static string versionFolderNumber;
2614

27-
/// <summary>
28-
/// Title
29-
/// </summary>
30-
/// <remarks>
31-
/// </remarks>
3215
public static string Title
3316
{
3417
get
@@ -47,11 +30,6 @@ public static string Title
4730
}
4831
}
4932

50-
/// <summary>
51-
/// Description
52-
/// </summary>
53-
/// <remarks>
54-
/// </remarks>
5533
public static string Description
5634
{
5735
get
@@ -69,11 +47,6 @@ public static string Description
6947
}
7048
}
7149

72-
/// <summary>
73-
/// Company
74-
/// </summary>
75-
/// <remarks>
76-
/// </remarks>
7750
public static string Company
7851
{
7952
get
@@ -92,11 +65,6 @@ public static string Company
9265
}
9366
}
9467

95-
/// <summary>
96-
/// Product
97-
/// </summary>
98-
/// <remarks>
99-
/// </remarks>
10068
public static string Product
10169
{
10270
get
@@ -114,11 +82,6 @@ public static string Product
11482
}
11583
}
11684

117-
/// <summary>
118-
/// Copyright
119-
/// </summary>
120-
/// <remarks>
121-
/// </remarks>
12285
public static string Copyright
12386
{
12487
get
@@ -136,11 +99,6 @@ public static string Copyright
13699
}
137100
}
138101

139-
/// <summary>
140-
/// Trademark
141-
/// </summary>
142-
/// <remarks>
143-
/// </remarks>
144102
public static string Trademark
145103
{
146104
get
@@ -158,11 +116,6 @@ public static string Trademark
158116
}
159117
}
160118

161-
/// <summary>
162-
/// AssemblyVersion
163-
/// </summary>
164-
/// <remarks>
165-
/// </remarks>
166119
public static string AssemblyVersion
167120
{
168121
get
@@ -172,11 +125,6 @@ public static string AssemblyVersion
172125
}
173126
}
174127

175-
/// <summary>
176-
/// FileVersion
177-
/// </summary>
178-
/// <remarks>
179-
/// </remarks>
180128
public static string FileVersion
181129
{
182130
get
@@ -187,11 +135,6 @@ public static string FileVersion
187135
}
188136
}
189137

190-
/// <summary>
191-
/// Guid
192-
/// </summary>
193-
/// <remarks>
194-
/// </remarks>
195138
public static string Guid
196139
{
197140
get
@@ -209,11 +152,6 @@ public static string Guid
209152
}
210153
}
211154

212-
/// <summary>
213-
/// FileName
214-
/// </summary>
215-
/// <remarks>
216-
/// </remarks>
217155
public static string FileName
218156
{
219157
get
@@ -224,11 +162,6 @@ public static string FileName
224162
}
225163
}
226164

227-
/// <summary>
228-
/// FilePath
229-
/// </summary>
230-
/// <remarks>
231-
/// </remarks>
232165
public static string FilePath
233166
{
234167
get
@@ -239,11 +172,6 @@ public static string FilePath
239172
}
240173
}
241174

242-
/// <summary>
243-
/// Return the current file name (used for error logging)
244-
/// </summary>
245-
/// <returns>A method that returns a string of the current file name </returns>
246-
/// <remarks></remarks>
247175
public static string GetCurrentFileName()
248176
{
249177
try
@@ -256,11 +184,6 @@ public static string GetCurrentFileName()
256184
}
257185
}
258186

259-
/// <summary>
260-
/// Returns the ClickOnce location
261-
/// </summary>
262-
/// <returns>A method that returns a string of the current location </returns>
263-
/// <remarks></remarks>
264187
public static string GetClickOnceLocation()
265188
{
266189
try
@@ -278,11 +201,6 @@ public static string GetClickOnceLocation()
278201

279202
}
280203

281-
/// <summary>
282-
/// Returns the assembly location
283-
/// </summary>
284-
/// <returns>A method that returns a string of the current location </returns>
285-
/// <remarks></remarks>
286204
public static string GetAssemblyLocation()
287205
{
288206
try
@@ -299,9 +217,6 @@ public static string GetAssemblyLocation()
299217

300218
}
301219

302-
/// <summary>
303-
/// Set Assembly Folder Version
304-
/// </summary>
305220
public static void SetAssemblyFolderVersion()
306221
{
307222
try
@@ -325,13 +240,6 @@ public static void SetAssemblyFolderVersion()
325240
}
326241
}
327242

328-
/// <summary>
329-
/// set the icon in add/remove programs
330-
/// </summary>
331-
/// <param name="iconName">The referenced icon name for the application.</param>
332-
/// <remarks>
333-
/// only run if deployed
334-
/// </remarks>
335243
public static void SetAddRemoveProgramsIcon(string iconName)
336244
{
337245
if (System.Deployment.Application.ApplicationDeployment.IsNetworkDeployed
@@ -376,11 +284,6 @@ public static void SetAddRemoveProgramsIcon(string iconName)
376284
}
377285
}
378286

379-
/// <summary>
380-
/// Open a file as read only
381-
/// </summary>
382-
/// <param name="filePath">Represents the file path string </param>
383-
/// <remarks></remarks>
384287
public static void OpenFile(string filePath)
385288
{
386289
try

‎CS/Scripts/Data.cs‎

Lines changed: 1 addition & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -13,40 +13,19 @@ class Data
1313

1414
const string dataFolder = "App_Data";
1515

16-
/// <summary>
17-
/// Relative database connection string
18-
/// </summary>
19-
/// <returns>the data source of the database</returns>
2016
public static string Connection()
2117
{
2218
Data.SetUserPath();
2319
string databaseFile = "Data Source=" + Path.Combine(Properties.Settings.Default.App_PathLocalData, AssemblyInfo.Product + ".sdf");
2420
return databaseFile;
2521
}
2622

27-
/// <summary>
28-
/// List of common table alias
29-
/// </summary>
30-
public static DataTable TableAliasTable = new DataTable();
3123

32-
/// <summary>
33-
/// List of date format strings
34-
/// </summary>
24+
public static DataTable TableAliasTable = new DataTable();
3525
public static DataTable DateFormatTable = new DataTable();
36-
37-
/// <summary>
38-
/// List of time format strings
39-
/// </summary>
4026
public static DataTable TimeFormatTable = new DataTable();
41-
42-
/// <summary>
43-
/// List of graph data
44-
/// </summary>
4527
public static DataTable GraphDataTable = new DataTable();
4628

47-
/// <summary>
48-
/// Creates the datatable for the list of common table alias
49-
/// </summary>
5029
public static void CreateTableAliasTable()
5130
{
5231
try
@@ -78,9 +57,6 @@ public static void CreateTableAliasTable()
7857

7958
}
8059

81-
/// <summary>
82-
/// Creates the datatable for the date format strings
83-
/// </summary>
8460
public static void CreateDateFormatTable()
8561
{
8662
try
@@ -112,9 +88,6 @@ public static void CreateDateFormatTable()
11288

11389
}
11490

115-
/// <summary>
116-
/// Creates the datatable for the date format strings
117-
/// </summary>
11891
public static void CreateTimeFormatTable()
11992
{
12093
try
@@ -146,9 +119,6 @@ public static void CreateTimeFormatTable()
146119

147120
}
148121

149-
/// <summary>
150-
/// Creates the datatable for the graph data
151-
/// </summary>
152122
public static void CreateGraphDataTable()
153123
{
154124
try
@@ -184,9 +154,6 @@ public static void CreateGraphDataTable()
184154

185155
}
186156

187-
/// <summary>
188-
/// Sets the server path of the installed sql server ce database and supporting files
189-
/// </summary>
190157
public static void SetServerPath()
191158
{
192159
try
@@ -206,9 +173,6 @@ public static void SetServerPath()
206173
}
207174
}
208175

209-
/// <summary>
210-
/// Sets the user path of the installed sql server ce database and supporting files
211-
/// </summary>
212176
public static void SetUserPath()
213177
{
214178
string userFilePath = string.Empty;
@@ -219,7 +183,6 @@ public static void SetUserPath()
219183
string versionNumber = AssemblyInfo.versionFolderNumber;
220184
string localAppData = Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData);
221185
userFilePath = Path.Combine(localAppData, AssemblyInfo.Copyright.Replace(" ", "_"), AssemblyInfo.Product, dataFolder);
222-
//if (!Directory.Exists(userFilePath)) Directory.CreateDirectory(userFilePath);
223186
System.IO.Directory.CreateDirectory(userFilePath);
224187
DirectoryInfo info = new DirectoryInfo(userFilePath);
225188
DirectorySecurity security = info.GetAccessControl();
@@ -243,11 +206,6 @@ public static void SetUserPath()
243206

244207
}
245208

246-
/// <summary>
247-
/// Ask the user if they would like to add a missing value to the table
248-
/// </summary>
249-
/// <param name="tbl">The table to check if a value exists</param>
250-
/// <param name="text">The value to insert to the table</param>
251209
public static void InsertRecord(DataTable tbl, string text)
252210
{
253211
string tableName = tbl.TableName.ToString();

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /