SystemShock: Refactoring (C#) - SystemShock

Jump to content

0
Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

Refactoring (C#) fixing up the old project that's gathering up dust Rate Topic: -----

#1 User is offline   fluf 

  • Member
  • Group: Members
  • Age: 22
  • Posts: 73
  • Joined: 21-February 08
  • Gender:Male
  • Location:Garden Route
Reputation: 0
Neutral

Posted 08 February 2010 - 12:50 PM

When I started last year working we were given a project with a deadline of the day before and we started out as 3(new) people and ended up being 2(new) people working on the project with different coding style :/. This being the first "big" project after uni and working with technologies not known well - Exported data from a LIMS system to Pastel - the code was not the best. Especially with the fact that we haven't found our feet by that time yet.

The application has never been used since we created it and it might be used in March somewhere, the Application work but its not very dev friendly. So ive been thinking of refactoring the code a bit. Now the problem is I will have to re-factor the code without having a dev environment to test the modified code on - Pastel expired and we don't have unit tests -

Can you recommend / give any advice to help speed up the process, because I know its going to take some time as I cant make any real changes/improvements to the code besides creating and moving methods. I am not particularly proud of this project and have created waaay better and standardized projects afterwards, so fixing up this project is also important for me.
0

#2 User is offline   Apoc 

  • The "awesome dutch pimp"
  • View gallery
  • Group: Elite Members
  • Age: 23
  • Posts: 4175
  • Joined: 05-February 06
  • Gender:Male
  • Location:The armpit of SA (Port Elizabeth) :p
Reputation: 52
Excellent

Posted 08 February 2010 - 08:56 PM

View Postfluf, on 08 February 2010 - 12:50 PM, said:

When I started last year working we were given a project with a deadline of the day before and we started out as 3(new) people and ended up being 2(new) people working on the project with different coding style :/. This being the first "big" project after uni and working with technologies not known well - Exported data from a LIMS system to Pastel - the code was not the best. Especially with the fact that we haven't found our feet by that time yet.

The application has never been used since we created it and it might be used in March somewhere, the Application work but its not very dev friendly. So ive been thinking of refactoring the code a bit. Now the problem is I will have to re-factor the code without having a dev environment to test the modified code on - Pastel expired and we don't have unit tests -

Can you recommend / give any advice to help speed up the process, because I know its going to take some time as I cant make any real changes/improvements to the code besides creating and moving methods. I am not particularly proud of this project and have created waaay better and standardized projects afterwards, so fixing up this project is also important for me.


Yeah, I have some advice - don't do it. Never change code (even if its just refactoring) and hand it off without testing.
0

#3 User is offline   Pyro 

  • Senior Member
  • View gallery
  • Group: Elite Members
  • Age: 27
  • Posts: 6623
  • Joined: 15-September 05
  • Gender:Male
  • Location:Helderkruin, Johannesburg
Reputation: 68
Excellent

Posted 08 February 2010 - 09:34 PM

If it ain't broke, don't fix it.

Since you haven't touched it for a while, you might make it slightly more legible, but slightly more broken. I'd prefer hard to read code that actually works over something that doesn't.

Some of my first code is still out there, not the prettiest, but it works.
0

#4 User is offline   AViD 

  • Hamster Terrorist
  • Group: Elite Members
  • Age: 23
  • Posts: 2075
  • Joined: 10-July 07
  • Gender:Male
  • Location:Johannesburg
Reputation: 18
Good

Posted 09 February 2010 - 07:58 AM

There you have it fluf, two guys telling you exactly what I told you - leave it. Cross the performance bridge when you get there.
0

#5 User is offline   fluf 

  • Member
  • Group: Members
  • Age: 22
  • Posts: 73
  • Joined: 21-February 08
  • Gender:Male
  • Location:Garden Route
Reputation: 0
Neutral

Posted 09 February 2010 - 09:18 AM

Thanks, I'll take your advice and leave it be until bugs arise.
0

Share this topic:


Page 1 of 1
  • You cannot start a new topic
  • You cannot reply to this topic

1 User(s) are reading this topic
0 members, 1 guests, 0 anonymous users