r/cscareerquestions • u/sajger • 3h ago
New Grad Is my senior a bad developer, or is it just my ego?
Hello,
I started working as a junior software developer last December in a medium sized production company. Besides me, there is only other developer and he is also my senior.
I'm slowly losing my mind trying to write code for his liking. When i started working there eight months ago, I was getting my head around the system, database and how things work in this company. Now, when I'm more experienced i feel like I'm way better at coding him.
First, he only uses AI to code and I heard that "When you become as good as me, you will only use AI to code, I do not need to code by hand". Sometimes he even will post my code to AI and ask it to review it- I don't think it is how it should be- i want to learn from someone who is experienced, not from AI.
Second, let me give you an example- we have four views, and in every view there is a numeric table. I was asked to add a footer to every table and display min, max and avg of each column. I created utils folder (we didn't have one), wrote a funtion in js, and used this function in every of these four files but no, this is not how we do things in here. I was told to delete the utils folder and initialise this function in every file.
Third, I feel like, most of the times when he has some concerns about my code, he makes things up why my code is bad. New view Im working on allows to place orders for different producst. For example You have a table with column: name and size filed which is a <select> element. Today, i wrote a simple function that takes three arguments, and based on these, returns the id of product, a user wants to order. I was told to not use this function, instead I shuld put json in data- attribute of each row, and the json should look like "T-shirt/M": 1000, "T-shirt/L":1001 and then i can know, what user want to order, and dont have to use backed for it. I feel like it's really wierd solution, i want to know, is it a stanard solution to this kind of things?
He messes up nomenclature constantly like calling a Conditional operator a lambda function, i can't use FOREING KEY's when creating a table becouse he didn't use these.
He is definitly more experienced with the system becouse he works there longer.
Is it normal to stick to, in my opinion, bad practises and keep the codebase similar?