Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
Why would the credentials be in each of the connection strings, rather than established once or atleast read in, that sounds like a horror system 😶
-
rui7257617yEvery changes should be considered and get across within the team devs dbadmin.. plus your credentials should be assigned to a variable through out your code... so that no matter what you do... it would affect the whole system including your teams and clients side.. it is not a good practice to do changes without consulting your client.. changes should be done in your development excluding the production code..
-
Lol. That probably means it's also in the repository. Is it open source by any chance? 😈
-
Junior devs are junior for a reason, the lack of experience.
From your rant it sounds that he was the only one doing the database. Who did the peer reviews? Junior dev probably only did one or two calls before when he was still in school.
Your rant is just blaming one junior dev, but the whole thing comes across wrong to me. -
In fact environment specific stuff like database connections and credentials for other 3rd party services should be injected into the app at runtime. The ambassador pattern helps with this. You should look up DevOps
-
@Codex404 Even as a junior it should be obvious that it's a bad idea. Juniors still have some basic idea of programming and thus should quickly realise that hard coding the value is completely stupid. Especially since OOP is the "status quo" paradigm being taught, which focuses on reuse.
-
@TheCapeGreek that is true, but blaming it on someone who is programming in the real world for about 3 months is a bit harsh.
(assuming the rest of the world also becomes a medior after that time period) -
@TheCapeGreek then you are three months a junior and after that you have done probably three or two projects for two companies.
-
@jAsE the problem I think isn't to replace them, but rather that theres an unknown amount of files all establishing a connection via hardcoded string.
Related Rants
*Client phones me at 11pm*
Client: It's not working!!
Me:What's the error you're getting?
Client: "Database connection error"
*Phones system/dB admin*
DB Admin: Yeah we had to change the SQL logins, I've sent you the new ones
*Phones junior dev in charge of dB programming*
Junior Dev: Yeah you'll just have to go and change the credentials. They're in all the places where we're using the dB, just before the statement, in the connection strings...
We make over 470 calls to the DB 😑
rant
junior devs
pissed
tired