Opened 10 years ago

Closed 8 years ago

#241 closed task (fixed)

Use built-in .Net methods

Reported by: ibboard Owned by:
Priority: trivial Milestone:
Component: WarFoundry-API Version: Trunk
Keywords: codingstandards dotnet papercut Cc:
Blocked By: Blocking:

Description

(Via Twitter)

.Net seemingly has quite a collection of extra methods for common operations (e.g. joining file names to file paths). These may be optimised, but will certainly be more thoroughly tested. A list of suggested alternatives is here.

Code blocks 5 (combining paths using System.IO.Path), 6 (getting file extension using System.IO.Path) and 13 (using File.WriteAllText() to easily write a text file) are the most likely to occur in the code, although 1, 9 and 14 may also turn up in places.

Change History (5)

comment:1 Changed 9 years ago by ibboard

Status: newconfirmed

comment:2 Changed 9 years ago by ibboard

Keywords: papercut added
Priority: majortrivial

comment:3 Changed 8 years ago by ibboard

Component: General/UnknownWarFoundry-API

Most important part for this ticket is the API.

comment:4 Changed 8 years ago by ibboard

In [966]:

Re #241: Use built-in .Net methods

comment:5 Changed 8 years ago by ibboard

Resolution: fixed
Status: confirmedclosed

r966 fixed all the obvious ones

Note: See TracTickets for help on using tickets.