|
Just ask the Witch to help you port it to the Teensy and she will rewrite it in a couple of days for you!
If you can't laugh at yourself - ask me and I will do it for you.
|
|
|
|
|
The best quote for deciding on Green-field or Maintenance on Brown-field...
"It is easier to give birth than raise the dead." ~unknown
Build the new one!!! 
|
|
|
|
|
It sounds like you should be able to rewrite it a page at a time.
Bonus:
Have both deployed side by side so you can compare them/use them.
Once they are happy with the new version, delete the old one.
|
|
|
|
|
> but it's beginning to show its age
That is a vague problem description.
What is the real problem?
- performance,
- concurrency,
- GUI (look & feel)
- quality of code,
- no people that know the technology?
- ....
|
|
|
|
|
All of the above.
People know the technology, they just don't want to work with it (including me, to be honest).
|
|
|
|
|
Avoid doing a hard rewrite/relaunch, do a gentle "sunsetting" over time
1 Keep the old application but keep track of how many hits it gets
2 Make a new competing application, also tracking its hits
Then launch them side-by-side, giving the user a choice about which one they want to use, maybe even having cross links to make them able to "jump to the other side". Keep a look at the statistics to see if the new one is gaining or losing. I think you get the basic idea.
Suggested reading: https://medium.com/@herbcaudill/lessons-from-6-software-rewrite-stories-635e4c8f7c22[^]
|
|
|
|
|
It's an internal business application.
No competing or statistics there, just the one or the other.
|
|
|
|
|
You haven't given us any idea how big the system is (in terms of lines of code or whatever). Although, arguably, the bigger it is the more it would make sense to rewrite it rather than having to keep some old, creaking codebase working.
I recently went through my entire C++ codebase replacing a lot of old-fashioned manual memory management with 'modern' C++ idioms. It was a daunting prospect initially but I am sooooooo glad I did it. So, I would say, if you can handle the issues that Sander (quite rightly) raises and have sufficient resources then I would go for it. Just my $0.02
Paul Sanders.
If I had more time, I would have written a shorter letter - Blaise Pascal.
Some of my best work is in the undo buffer.
|
|
|
|
|
Well, what do you expect after 20 years of development?
It's not small, it has a few dozen pages with features such as entering sales orders, purchase orders, importing data from various sources, exporting to various sources, product and client management, stock management...
Basically everything a mid-sized organization needs to do business.
Paul Sanders (the other one) wrote: So, I would say, if you can handle the issues that Sander (quite rightly) raises and have sufficient resources then I would go for it.
If I can handle the issues I raised myself? 
|
|
|
|
|
Oh, it was you that started the thread, lol. Sorry, need coffee.
Paul Sanders.
If I had more time, I would have written a shorter letter - Blaise Pascal.
Some of my best work is in the undo buffer.
|
|
|
|
|
Having been involved in a big ground-up rewrite (painful, everything takes longer than everybody thinks or is willing to stomach) I would tend to favour an incremental refactoring approach.
So (if possible) put the new code in the same site as the old, and start replacing features one by one.
I wouldn't envy the person who has to extract that SQL out of WebForms though!
|
|
|
|
|
others likly given similar response, my basic points
WHY, what does client or end user benefit. Developers are not the user
that said - if want to reduce COSTS (time or money) for adding features, that a good why for client. Changing underlying code to fix "mistakes" (things that have newer ways, or low quality patterns)
Platform compatibility. Main reason had to take a silverlight/lightswitch to HTML.
Security issues
maintenance vs feature add
feature add turn around time
and best for client - features that at the time of request could not, or costly to implement can be scoped in this work of redesign. Some of these features might replace existing features thus not need transfering.
|
|
|
|
|
I've awoken my old account just to chime in on this as I've had some recent, terrible experience here that might help. I'll try to keep my rants to a minimum.
Sadly my last position was working in this tech stack, with over 150 customers using various versions of the product.
- Business logic strewn throughout SQL procs, UI and a little bit of mis-informed backend abstractions coupled with embedded and sitewide javascript.
- No source control, at all
 . - Zero, or inadequate out of date documentation.
- Team works on network shares.
- Manual deployment.
- No IDE or any other tooling (except for SQL Management studio).
- Building was done using cmd scripts split into arbitrary "modules" due to the limitations of the command buffer string length.
- Versioning on live system was mandated - rename the old file with a date, and leave it there.
- Sometime in the past they switched from VB to C# which was the dominant language when I started. The mix led to some awful days where I realised I had to touch the VB, and those days were long, unproductive, usually doomed to fail.
Staff turnover (all senior devs) was 100% for new hires, lasting from 4 months to 1.5 yrs (me, the fool).
10 years of constant losses, mostly negative customer relations with some "entrenched" customers buying out the parent company on more than 2 occasions to keep their critical systems from vaporising led to my current attitude to this type of product.. return to the ship and nuke it from orbit.
Seriously though, I would park it in a vm(s), charge a fortune for anything that needs change management. "Try" to fix any security holes that are essential, but first try to mitigate those within your new virtual infrastructure if you can first. Product is not cost effective, and the customer needs to understand that through passing on that aspect of the business. If cost is not an issue, maybe think about selling it on to an outfit that doesn't mind/understand financial loss, the technical debt or the talent attrition it will cause  .
You mentioned updating the html/css and providing a more modern approach to responsivness. The product I worked with had all that already and still had the above issues. My opinion is that it will not improve your developers lives, nor maintenance timescales, but might reduce the front end developers attrition in the long run. It might also increase the appeal of the application to the customer, making it seem "better". Do you really want that?
Moving to C# might seem like a good plan, but the application and its users would not benefit from a straight conversion. I would plan a rewrite, and maybe a targetted plan for specific use cases only in a new application written on a more modern, and versatile framework. Due to the cost of re-writes (coming from a change management line of business in my old days) I would go to the business and start discussions around re-assessing their requirements in leiu of modernisation.
SVN/git.. meh whatever your team prefers.
|
|
|
|
|
Ouch, sounds painful.
Seriously though, how old was that product, no IDE or tooling, how did they write it in the first place!? 
No source control!? Were these even developers? Did they have any sort of training at all? 
The more I read this thread the more convinced I am doing a straight up rewrite is not the way to go.
|
|
|
|
|
Painful was an understatement. Initially I was under pressure to find a job quickly and this paid ok.
Then I tried too hard to fix things, I should have quit an hour in...
Reasons for where they got to imo .. Misled non technical management, a priority on marketing, political division along dev/design lines and an old guard with speed, but no control or experience on their side from the start leading to a protective attitude.
Speed to deliver was the ONLY consideration when I was there. As a result there was no time to fix anything even when experienced devs like me tried to put pressure on. Of course every project was underestimated with the lack of documentation leading to scope creep, overruns and customer disatisfaction from project inception.
Product would have been brand new around 2009, ASP web forms was old even then. Only 1 original dev left, and a contractor that seemed to get work intermittently but never actually work/talk with anyone in the team. The original founder was kept around as a consultant, lol... This guy interviewed me, with the other original dev, and one question they posed was a simple relational data model question. Then when I hit the actual code I found they had no foreign keys defined, only indexes. I raised this in a formal review down the line, and was given a 5/10 grade for my work lol
I'm currently on the bench as a carer, not employed since Covid redundancy so I have too much time to mull over this stuff.
The organisational problems are interesting, and help understand the "how the F..." but from your response I am sure you don't have to worry about this junk. The tech stack is old and knackered and, imo, not really the platform I would have used even back in the 90's.
One note of caution - watch out for use of the "viewstate". Check any page in your browser with form controls (or not sometimes) and look for a massive utf8 encoded field(s). Depending on if this is a public website or not, and your customers level of security paranoia, you might want to look into getting an upgrade to a later version of the .net framework at a minimum (4.8) and ensure you turn on the relevant secure features (machine id and encryption I believe are the keywords there).
|
|
|
|
|
One very good and not-going-to-flip-the-canoe improvement might be Flyway.
The learning curve to some very basic "make sure my database state is what I think it should be" is not at all steep, but there are also some nice to haves if you dig deeper.
You essentially version control your DDL/DML into numbered scripts which it then facilitates applying to a given database by checking to see what was the most recent one applied and applying anything newer. A nifty option (that you have to also create if you want) are undo scripts correlated to the "do" scripts that allow you to rollback the change made by the "do".
This marries into build/release pipelines in easy and good ways.
For an Azure DevOps build pipe, you might first have a commandline step "clean my database and start over so nothing is funky from a previous mistake or someone mucking around because it is the build box".
In both build and release ADO pipes it is another commandline step added to flyway migrate the db. You could deploy and run the flyway scripts and flyway infrastructure required to run them (a bunch of java) from the artifact project folders being deployed if you wanted to.
Typically, we don't do that, opting instead to deploy and run the migrations from a totally different environment. We do this partially because there would be multiple service hosts in a deployment group and you only need migrations to run once. While it's not a huge bunch of more binaries/data to throw out there, on all but one machine it would generally exist for very little reason.
"LINQ queries are a pain in VB!"
I'd forgotten that particular horror, thanks.
|
|
|
|
|
Yeah, I know Flyway, actually already mention it with my client 
jochance wrote: I'd forgotten that particular horror, thanks.
You're welcome
|
|
|
|
|
I think this would depend very heavily on how useful and unique the program is. Back in the day, I was a co-operative education engineer (sort of a paid intern) at a company that had a flight simulator that they ran in this huge sphere of a room. It was a massive thing that cost vast amounts to run. Our boss tasked me and one other co-op to convert - or at least begin to convert - the program to a c version to be run on an x terminal. It was lots of fun with very little risk involved for us or them (we were just interns after all). I don't know if the project was ever completed, but we got a really good start on it. The question is whether it was finished before the plane it was designed to simulate went out of production. So, you see, it all depends on value for work, same as every other project undertaken.
|
|
|
|
|
In short, without getting detailed: Refactor.
1. Isolate and extract business code, and if it contains SQL, change it to stored procedures.
2. Translate (using online tools) the VB to C#.
3. Refactor the C# as necessary to be object oriented.
4. Capture the HTML and CSS for use in Blazor Client.
5. Isolate the UI logic that was JavaScript or VB, convert to C#, and use in Blazor for the UI layer.
6. Convert any C# code necessary from .NET Framework 4.8 to be compatible with .NET 6.
7. Start debugging and get it working with no new features yet.
8. Once what you have is working, then add any new requirements.
|
|
|
|
|
Scope...Scope
Best practices, on web especially, tend to lean towards the concept that your site is going to be the next viral app. A low volume site will generally outlive the technology behind it. Having 5k users today doesn't mean you'll have 50 or 500k next year, or anytime in the future.
So, I'd want to look at traffic, and make some decisions based on that. Really, I'd roll back the entire look to see if .Net is the best choice for what you're looking for. Since I'm about 67% desktop development, I've been tempted to try .Net on the web, but use LAPP/J-various on the web side of things. (And it's way cheaper and easier to maintain) That's just me though.
We're re-writing 25yr old VB6 desktop code and it starts out as fun, but goes downhill from there.  Not knowing how deep the site is, it's tough to make a really good call. Also, how deep are your customer's pockets?
I think this might be a good start: Quote: Maybe we can start by modernizing the generated HTML and CSS to make things more modern, like making the pages responsive.
since in any case it's going to need to be done as part of any upgrade and won't be wasted effort. Your client may be more responsive to a multi-step approach.
Been too many years since I've looked at SQL Server to comment on that part; heck, I don't even know what Quote: SQL queries written directly into the HTML
means, lol.
|
|
|
|
|
MikeCO10 wrote: Been too many years since I've looked at SQL Server to comment on that part; heck, I don't even know what Quote:SQL queries written directly into the HTML means, lol.
Exactly what it says.
Pseudo code (I don't know the exact syntax or commands from the top of my head):
<asp:Table runat="server" Source="MySqlQuery">
<columns>
<tr>
<asp:Cell runat="server" Text="<% #Bind('MyField') %>"><asp:Cell>
</tr>
</columns>
</table>
<DataSource ID="MySqlQuery" SelectQuery="SELECT MyField FROM MyTable" UpdateQuery="...">
<SelectParameters>...</SelectParameters>
<UpdateParameters>...</UpdateParameters>
</DataSource>Visual Studio doesn't indent it (anymore?) though, so most of this is just hundreds of lines of randomly (un)indented HTML with SQL queries strewn about.
|
|
|
|
|
Ahh, I remember those now.
I'm curious if you have an approach you are leaning towards to address your question?
The more I thought about, the more I think it goes to a client choice question. It's one I've presented to clients several times over the years and have gone the rewrite route many times, including two current ones. Also depends on your time/desire to do a rewrite. You can limit it some by using waterfall approach, but that depends on the client. I have one now that goes beyond agile to a "let's roll with it" mentality, resulting in about 60% of code written with about 25% released. Probably wouldn't be the case on a web app. Though I have a few curves on my current one, those are understood to be at the end of the project.
|
|
|
|
|
I think our problem now is that we simply don't have the capacity to rebuild while also keep developing what we have.
Also, there are probably two people who know what does what and where to find it.
One is a user that uses or used everything in the last twenty years, the other is a developer who built most of it or was somehow involved.
It's in their heads though, so it should get out first.
So I'm leaning towards a "let's refactor the hell out of this thing first when we touch something" approach.
|
|
|
|
|
I have a similar situation. Code is in classic ASP.
Best approach is to get the rules as they SHOULD BE from the business owners if you can.
If not, at least document the screen level functionality.
When you feel comfortable, create a road map . Take one piece at a time. I would start by identifying what could be converted to a service prioritizing what is re-usable first. (Authentication/Authorization would be a good start)
For ease of maintenance alone, I think a rewrite is in order. The longer the client waits, the larger the risk of something being deprecated in a future release.
|
|
|
|
|
Our company has an enormous monolith that shares a lot of similarities with this situation. While we are currently road mapping a full migration/rewrite, in the mean time we were able to get all our Sql stuff into source control by generating "create/alter" scripts for every procedure, function, view, and table. So while we still have tons of logic in Sql, it is at least source controlled. This might be a helpful first step, regardless of what happens next, since doing so gives you all the logic for those parts in text files.
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
