Scode.cs2.54KB
SreleaseAPI
SVercelApi
Mso... minor low hanging fruit, you can and should reused the HttpClient.
Myou can also just use a statement based using.
M
CThere's no validation that the data you get back is in the correct format.
C, then you don't need , just
MI'd also use linq instead of your loops, but that's just cause I love LINQ so much.
CPut constants like those URLs in fields, rather than just hard-coding thjem
MThen later:
Mthere is no need to all the time.
MIf console formatting is super important to you, check out a library like Spectr or whatever.
Motherwise, I mean the two biggest no-nos I saw were not reusing the HttpClient and the excess to list.
Si know, i am just new to the linq so i am trying to figure out how to linq
Mmost of the rest is just outdated syntax and preference.
Mfair enough. Well, there isn't anything especially wrong with your first loop. Some people might prefer that logic.
MBut you don't need to versions later on.
MYou can just save the result of and foreach that.
Mor even move it into the foreach call, and save a line.
Mmy personal rule of thumb is less, more complex code is generally better than more verbose "simpler" code. Because the fewer llines of code I have the fewer I have to try and keep in memory.
Sthe reson of the !contains("delta") was to remove duplicate data. any faster way?
Mmmm... probably not. should be pretty well optimized.
MIt's going to use vector instructions to examine the string quickly.
MI suppose since you know that if its present it's going to be at a certain position you could slice the string with
SpanThough I think that's a questionable improvement. It makes the code less flexible and speed is hardly going to matter here.
MHonestly, you might consider using regex instead. It will be slower, but a single regex pattern is easier to update and understand than a lot of split and indexed based magic. And more adaptable to changes in format.
MDon't worry about speed unless it really matters. While string operations and parsing can be slow your bottleneck here is almost certainly going to be the file download anyways.
MSo optimize first for flexibility and readability, rather than performance, I say.
Mactually regex might not even be slower than a lot of splits, come to think of it. You'd have to benchmark to determine I suppose.
S
Program.cs2.88KB