u/Skadarn1
-
58points
-
13points
What can I improve? Currently 1 year into school. (self.dotnet)
-
8points
Exchange for Swedish resident? (self.cardano) [Exchange]
Skadarn1
Skadarn1
Thank you, I am still very unsure if I am going too much up and down, if I should lean more, i read somewhere to lead/aim with the chest, but that feels counter intuitive if I wanna try and maintain a hollow body.. or maybe it just requires more strength.
Skadarn1
Yeah, I was thinking of progressing there, but wanted to make sure that the form is ok before, since it becomes harder the higher my feet goes.
When leaning forward more, do I keep the same motion? "Up and down" or do you adjust and "slide" at an angle instead?
Thanks for the response!
Skadarn1
This is what's wonderful about a social sandbox.. have you ever seen the anime "Sword Art Online"? Just to pull reference from it, some of the people there never even leave the starting areas, they live their entire life there.. Your way of playing the game, by just crafting, is a legit way to play the game. Heck you can make yourself the famous wood chopper and master craftsman that people will give recipes to, pay huge amount of gold to have you craft things, can play as a merchant or really whatever your mind can think of.. don't let others force you into their way of playing, if you enjoy and want to chop trees, then chop your damn trees!
Skadarn1
Hi again, quick question about the required field - I'm doing another part of the project now and I read that using it on non-nullable fields is redudant? (ints, decimal, Guids etc.) but that I should keep using it on navigational properties and nullable ones like string.
What's your take on this? why should I, why should I not use it on these fields? or is what I've read correct? only use required on for example :
public required string TicketCategory { get; set; }
and not on :
public int TicketCount { get; set; }
Thanks again for your insights :)
Skadarn1
Ohh, yeah looks like testing is something I should do better (I barely did any in this project so that needs to be implemented earlier on in the cycle for sure!)
Never heard of that, will add it to my read-up-on-list!
Great input, thank you!
Skadarn1
Logging and error handling, do you mean like more error codes instead of just "successful / failed / could not find user" type of try catches? Or what do you mean? I am thinking of doing a ErrorMessageHandler of some sort. But do you have any recommendations/tips on the error/user feedback? What is good to start with?
Pagination.. not too familiar with the word, gonna have to look that one up, thanks!
Own doubles? I will 100% look into more testing and mocking alternatives, can't harm to know more right? ☺️ Thanks for the input and suggestions!
Skadarn1
Thank you, this was actually my first project where I felt I had some sense of control when writing things haha
Good question, our teacher showed us gRPC and our group said lets run with it since we used REST+Swagger on all other projects this last half year. Our teacher did say after the presentation though that REST would have been the better choice for the microservices we chose to build.
So mostly, to try it out.
Follow up question: why would you, individually, use REST for this project and not gRPC? I heard some performance differences and communication between two gRPC services is slower? But have not dived into it yet. So what is your reasoning? 😀
Skadarn1
Thank you for explaining it a bit more in-detail with code snippets, it really "drives it home".
I will try and incorporate this going forward, especially the required instead of the "Trust me bro, I got this bro" null! haha - thank you u/Atulin!
And yes, I'm probably confusing some things, even a lot of things probably haha
Skadarn1
Thank you for giving me the prompting here as well, that's really useful! Gonna try that out immediately! :)
Skadarn1
Do select on the DB, makes indices more effective and reduces network trafic.
I'm guessing this is more of an answer? that by using select instead of include I'll reduce network trafic and be more effective, especially with larger DBs in the future.
As I answered the other person about the EF conventions vs explicit configuration - is there any real perfomance difference or something to not write it more explicitly? what's your take?
Thanks for chipping in on the conversation :)
Skadarn1
That sounds like a cool feature, gonna take a look at that, thank you! :)
Skadarn1
Is "Required" the "best practice" way of doing it, or a newer way than null! ? I do know it exists but our teacher taught us mostly to use the null! field to be "explicit" - maybe that's not the best way though?
Good point - what would the difference be technically using a GUID instead of a string with Guid.NewGuid().ToString()? I went with the "ToString" version to make it easier in the frontend, not having to do any conversion and that's what we decided on in our group to run with so we all did the same type of things. I'm going to look into the GUID a bit more to see when and where to use it. Good input :)
So try to utilize the EF conventions to clean things up a little bit. - I see the comment below yours that there seems to be a divide on having it more explicit or not. Again, are there any pros / cons to do it one way or the other or is it more of a person preference?
I did the repository first with the REST, not sure if the repo needed to stay or not to I chose to just leave it as is and try and use it in my service. But I do not really get what you are saying, injecting a bare DbContext - I don't need the repository at all then? how would that work?
4.1 & 4,2. I probably don't need all the data, all the time you're right haha - maybe this goes back to planning a bit more to know what I need and when, to not have to load all the tables and all the information at all times. good point, gonna have a look at that and see what I can do better.
- Yeah, I do usually have the braces around but tried to write the "new way" with .NET 9.0 (i think)? but we'll see what I stick to here haha
I appreciate the input and time taken to reviewing and writing back to me here - really, thank you :)
Skadarn1
Hi Thomasz,
Thank you for some input :)
I had no idea, I used GUID to string to make it easier to load it into the frontend part of the application (To save myself the conversion), but I did not know that GUID/UUID could cause such problems - will definitly look into it :) Thank you for taking your time to reply, I appreciate that!
Skadarn1
Hi, thanks for the reply! :)
I do use ChatGPT to give feedback and such, but I also feel that ChatGPTs answers are not always up-to-date with what you use in the real world? I might be prompting it wrong.
The README file on github was 100% AI generated, because I forgot about it completly before turning in the assignment (shame on me).
I will try it though, I have not used the "deep research" function much, gonna give it a go!
Skadarn1
If you do start, hit me up here on reddit and I'll help you out 😊 I'm on the pvp server.
Skadarn1
Don't know if it's against the rules here to "promote" an OT and too lazy to check while at work
check out TibiaScape, it made my passion for Tibia come back.
Really well made server which promotes exploring and actually rewarding you for it. Being able to customize your character to how you want to play.
7.4 based client and active community. The server will not live forever but for the time being really fun to invest into.
Skadarn1
Yeah i know.. looks like a frikkin' broccoli tho!
Skadarn1
Why is there a big ass broccoli in the first scene..?
Skadarn1
It's only on retail.. so no changes!! #nochanges
Skadarn1
The circle is complete.
Skadarn1
They have actually been doing a great job with that lately! :)
As in.. raising my head? Can you explain why, what's the benefit of facing the floor? Thanks for the reply!