2000 HOUR 2D Game Engine! // Code Review

  Рет қаралды 91,053

The Cherno

The Cherno

Күн бұрын

Пікірлер
@TheCherno
@TheCherno Жыл бұрын
Thanks for watching! Should we continue looking at this 2000-hour 2D Game Engine? Also don't forget you can try everything Brilliant has to offer-free-for a full 30 days! Visit brilliant.org/TheCherno - the first 200 of you will get 20% off Brilliant’s annual premium subscription!
@matin.f139
@matin.f139 Жыл бұрын
do itttttt please!
@dracula5752
@dracula5752 Жыл бұрын
part 2 when
@LovepreetSingh-ye9hd
@LovepreetSingh-ye9hd Жыл бұрын
Let's dive in please!
@Dr_OmarYaser
@Dr_OmarYaser Жыл бұрын
very helpful ... continue
@ferdynandkiepski5026
@ferdynandkiepski5026 Жыл бұрын
Yes
@Renschi.
@Renschi. Жыл бұрын
At 2:23 Cherno mentioned he doesn't know the tool Rene. This is because it is literally me. The person who created this Engine is one of my friends and I asked him to add me as a tool because I gave some recommendations about the UI of the Engine :)
@NoThing-ec9km
@NoThing-ec9km Жыл бұрын
Lmao. Loved it😅
@xTriplexS
@xTriplexS Жыл бұрын
Lol
@blackcitadel37
@blackcitadel37 Жыл бұрын
wait, add you as a... tool? lmfao XD
@and_then_I_whispered
@and_then_I_whispered Жыл бұрын
you mean the egnine?
@hasankaraman
@hasankaraman Жыл бұрын
He can now add cherno as well
@nestor1208
@nestor1208 Жыл бұрын
"I won't roast you too much" *proceeds to making a 30-minute video of roasing the README file alone*
@monad_tcp
@monad_tcp Жыл бұрын
After reading the README file, I've seen it all
@y4ni608
@y4ni608 Жыл бұрын
Hello!! Thanks for reviewing my game engine for some reason my previuos commentar got deleted. it looks like im missing a concept about pointers 😅. I did discuss this vector SpriteRepository structure with my teacher (He is a teacher teaching mainly about os's) (Im still in highschool, 2nd grade) and he actually agreed with this structure. It's cool seing different opinions coming together, but i will defenetly value the advice of a professional gameengine dev more. I can't wait to see the part 2! by the way, if you guys got any questions about my game engine feel free to ask here :) 7 month edit: Yo guys, i have finally implemented all the stuff cherno advised, and im happy AF i love the new project structure
@giobbymenta
@giobbymenta Жыл бұрын
He agreed with your structure cause for an highschool student is really good enough lol. Good job
@ocaly
@ocaly 5 ай бұрын
Congrats bro!
@Berzeger
@Berzeger 4 ай бұрын
Hey, that's amazing on several fronts - that you accepted Cherno's critique as a grown man (and even fixed it according to his suggestions) and also that you're so young and yet managed to write an engine!
@bulentgercek
@bulentgercek Жыл бұрын
Cherno is a type of guy can do code review without looking the code. :D
@sneed1208
@sneed1208 Жыл бұрын
He does look into the code that is presented in the README though, so that's not true.
@MahmoudSayed-hg8rb
@MahmoudSayed-hg8rb Жыл бұрын
I haven't finished the video yet, but i think there might be part 2 and maybe more.
@blackcitadel37
@blackcitadel37 Жыл бұрын
senior dev sorcery
@kzone272
@kzone272 Жыл бұрын
This is accurate, except its actually an insult not a compliment.
@bulentgercek
@bulentgercek Жыл бұрын
@@kzone272 This is only a joke. This is an internet meme. It's not an insult or anything. Cherno has an important place in my life.
@jadrop0037
@jadrop0037 Жыл бұрын
Please continue the review on a future video. I’m really curious to see the engine running!
@marctse3556
@marctse3556 Жыл бұрын
Please continue with this engine. This is super informational! I also like the high level design aspect of this review
@felixp535
@felixp535 Жыл бұрын
Yeah I really enjoy these kinds of videos, learning about how game engines work and are usually structured, and especially WHY it's done this way. It feels like this developer was trying to reinvent the wheel, but to be fair, how do you know you're reinventing the wheel if you don't even know how a wheel looks like and how it works? I started my own game engine a while ago, following Yan's amazing Game Engine series (except I was doing it in a language called Jai). I managed to render simple images, but I stopped following the series when it got to the ECS stuff, as I didn't want to have that in my engine. I just wanted to continue working on the rendering, but most of the stuff was being off-screened, so I really felt stuck when trying to actually create the structure for the rendering system. It's soooo complex, there's just sooooo much to think about. In various projects I've tried to explore there's RenderingContexts, RenderingConfigurations, all sorts of Buffers and Passes, rendering on separate threads, making all of this mess API-agnostic... I just gave up. I would love a series of videos explaining how all of that stuff works, explained with simple text and drawings just like in this video.
@y4ni608
@y4ni608 Жыл бұрын
@@felixp535 Hello, yeah i kinda wanted to try something new, but i can see what issues i have here, i really didnt think that much about it
@NathanHedglin
@NathanHedglin Жыл бұрын
​@@felixp535exactly. JAI?! You got access? How do you like it?
@gelis07
@gelis07 Жыл бұрын
12:24 that literally can become the best bell curve programming meme ever.
@Diventurer
@Diventurer Жыл бұрын
That raw pointer confidence chart is way too relatable. I can't believe how much I trusted myself with them before.
@sub-harmonik
@sub-harmonik Жыл бұрын
As someone who isn't a game dev but is peripherally interested, I really liked the conceptual overviews, thanks. This kind of thing is still relevant to other areas of system design.
@LightTheMars
@LightTheMars Жыл бұрын
Instead of UUIDs for referencing entities during runtime you can also use generational indices. Keeps the index simple and small and takes care of validity.
@HairyPixels
@HairyPixels Жыл бұрын
UUIDs are extremely slow to generate too.
@Mungoes
@Mungoes Жыл бұрын
@@HairyPixels how so? UUID v4 is just a random 128-bit number with a couple of the bits set to specific values
@HairyPixels
@HairyPixels Жыл бұрын
@@Mungoes it's how the numbers are generated I guess. Maybe for sprites it's no so bad but if you start making them real time it's measurable.
@jonforhan9196
@jonforhan9196 Жыл бұрын
@@HairyPixelscreate them in batches and utilize the mm registers maybe
@dealloc
@dealloc Жыл бұрын
@@HairyPixels Extremely slow to generate? What are you on an Amiga? Also, you don't need to generate UUIDs every single frame. You generate them only when you create the instances. It doesn't have to be generate UUIDs from a cryptographically secure random number generator either.
@verminology9999
@verminology9999 Жыл бұрын
In my opinion, this was one of your best code reviews, and you didn't even open the project yet. The way you explained everything visually was very cool and extremely informative. I would love to see a part two!
@donsk324
@donsk324 Жыл бұрын
12:40 Can releate haha. At first I used to avoid dynamic memory management, then I started to embrace pointers after I got the hang of them and considered smart pointers 'pointless', that is after attaching address sanitizer and spending approximately an hour trying to fix memory leaks and memory corruption on a small sized project and later realized how important ownership is and how raw pointers laked ownership.
@hullukana214
@hullukana214 Жыл бұрын
Please make another video about this engine. I'm sure a deeper look into the engine would reveal many more memory management related issues for you to get stuck on for half an hour.
@sigxfs
@sigxfs Жыл бұрын
currently making my own 3d engine and this is the exact design problem i was thinking about
@Patrick_pw345
@Patrick_pw345 Жыл бұрын
I rarely comment on videos. I really liked this video and would love to see more of this engine reviewed. The drawing really helped me understand the concepts. My initial impression is, that a bit more spacing and logically connected structuring would really boost my understanding and make it easier to follow. e.g. stacked in a way that matches the data flow top to bottom. like Scene -> Sprite Objects -> Sprites -> How the memory is referenced / handled. I probably missed a layer in this example. Writing that, a summary in such a structure at the end would've probably really helped me recap and form a better understanding of what concepts you just pointed out. You partly or may have done that in the part about Scenes. Long comment, really loved the video. Hope you can take something from my feedback. :-)
@y4ni608
@y4ni608 Жыл бұрын
Yep you are right! I should have done it like this, i actually do have a scene system but its kinda wierd lol. It just loads the contents from for example scene 2 in memory and deletes the content from Scene 1. I should have wrapped that content that gets loaded into a scene object, like you did it in your comment also managing scenes (refreshing/loading/deleting) etc is much easier with this way
@K.R.O1875
@K.R.O1875 Жыл бұрын
You should do a follow up on this. There was still plenty for people to learn from here. But the kid states at the very start of the documentation that it may be outdated. It would be cool to have a look at the actual code and engine and offer some advice on that.
@thygrrr
@thygrrr Жыл бұрын
Many (Generation-X and older) Germans call the "Strg" key the "String" key, and it really grinds my gears. :D
@CatzHoek
@CatzHoek Жыл бұрын
This project really seems to have some meat to it so i love that you can really talk about stuff. Would love to see that disected even more in other videos.
@TopConductor
@TopConductor Жыл бұрын
I like the code review without even looking at code directly xd It was very interesting to hear this UUID idea. I also was writing some small projects, and came up with the idea of storing the integer instead of the ptr myself, but it always felt awkward for me, and kind of unpreffessional, idk. thanks for the great video
@irfanjames6551
@irfanjames6551 Жыл бұрын
31:31 Yes... Yan. This is a beautiful format. Can't wait for future reviews. Also, the PacMan Game you were going to build yourself. Would really appreciate it, if you can manage to take some time.
@yieldjedimaster1040
@yieldjedimaster1040 Жыл бұрын
Yes, it's really helpful, let's continue!
@simiuciacia
@simiuciacia Жыл бұрын
I was expecting something more of a "continue we shall" with that pfp
@turner7777
@turner7777 Жыл бұрын
Would be cool if you did a vid on how to properly design a project
@The1RandomFool
@The1RandomFool Жыл бұрын
I'm more of a beginner in C++, but I won't make pointers for the sake of having a pointer. From what I learned, it could lead to unsafe memory handling. If I have to pass something to a function, I'll pass it by reference instead. If I need to make something on the heap, I'll use a smart pointer instead. I made a command line scientific calculator at one point and used std::function to create a function map instead of using raw function pointers.
@BoardGameMaker4108
@BoardGameMaker4108 Жыл бұрын
Pointers to functions are perfectly safe. Real bugs start happening when you have a pointer to a variable that no longer exists. Like, if you spawn a character in a game, refer to it using a pointer, then destroy the character. Now you have a pointer that no longer refers to the character it once referred to (because you deleted it). If you ever made a game in Unity and you've coded something in C#, this situation will result in the quite common "null-reference" error. This is because you are trying to use a pointer that refers to something that no longer exists. This problem never occurs if you use a pointer that refers to something that always exists (like a function) or the pointer is on the stack and it refers to another variable previously on the stack. Just ensure that the thing your pointer refers to still exists and doesn't move around. This is quite a simple rule to keep in mind, and it's generally only broken when there's something fundamentally wrong with the logic/structure of your program. std::vector is a unique exception because a vector will move its contents around when new elements get added to it, so be careful when using std::vector. Also, & (reference type) is essentially just a pointer, so using pointers as references is perfectly safe and quite common. Just use smart pointers when you are allocating on the heap. std::shared_ptr is a good alternative to normal pointers but it has overhead (performance cost) so use it only when you need it. Edit: the use of pointers in this video is particularly not good because entities in games can be spawned or deleted anytime, which as I said before can lead to these issues.
@thewelder3538
@thewelder3538 5 ай бұрын
But Cherno... The reality is that under the hood, std::vector is very much like std::vector. Whether you push_back the sprite or the pointer, they're still both going to be heap allocated unless you've got your own allocator. The difference being that with operator new(), your sprites aren't going to be contiguous, but your pointers will be and you've got to to destroy them yourself. The vector iterator is still just a pointer.
@hayabusa1x
@hayabusa1x Жыл бұрын
Very helpful explanation on pointers vs identifiers. Would love to see you review the rest of this code base.
@gniludio
@gniludio Жыл бұрын
Why didn't you roast that the readme had code without syntax highlighting?
@y4ni608
@y4ni608 Жыл бұрын
For some reason it doesnt work
@ludologian
@ludologian Жыл бұрын
I'm c# developer, and I consider myself something between beginner & intermediate lvl but generally I can follow up with you.. please stay with this format
@briumphbimbles
@briumphbimbles Жыл бұрын
1:41 how can you build a game engine and not know about relative paths?
@exosupesu
@exosupesu Жыл бұрын
This guy roasted the whole engine without even seeing the source code
@parabalani
@parabalani Жыл бұрын
Which is questionable
@Mitch_Crane
@Mitch_Crane Жыл бұрын
Isn't that what everyone does
@topg3067
@topg3067 Жыл бұрын
He just needs ad revenue and relevancy, and took the easiest part to pick on. He ran out of libraries to use in his own engine lmao
@JavedAlam-ce4mu
@JavedAlam-ce4mu 5 ай бұрын
@@topg3067 you can't say it wasn't educational
@defter6800
@defter6800 Жыл бұрын
You can also introduce some typed ID (for different types of objects) by using syntax sugar of C++ and allocate this ID by increment atomic int variable (or even non atomic). Only downside if too many threads do this frequently with atomic variable, but there is always way to pre-allocate N ID's very fast and unroll some loops or make like ranged limited local ID allocator that use ID ranges from global ect.
@jakubgiesler6150
@jakubgiesler6150 10 ай бұрын
My first line was written around 2009 in Java (i was 11 btw), and when I learned pointers and c++ around 2015 I felt super smart and used them everywhere, and now it's 2024, and I rarely use pointers, so yeah the statement is true.
@aarondcmedia9585
@aarondcmedia9585 Жыл бұрын
I have read more game egnine books than you could shake a shared pointer at, but discussing something real is what solidifies the concepts in my mind. Much appreciated for this content, and please keep going with this engine.
@aryanbhatia4872
@aryanbhatia4872 Жыл бұрын
loved your review of this, please cover more of this.
@natanaelrabello
@natanaelrabello Жыл бұрын
Please keep roasting this engine. I feel like amazing lessons could come out of it. Thanks :)
@cpaulicka
@cpaulicka Жыл бұрын
This was fun but…very little discussion of abstraction at the very end. Also, discuss user of software vs developer of software (which often have different priorities). Instead you kind of jump into details (charmingly so, yes). API design before implementation helps; you could do that with the implied api here and clean it up, and then walk through possible implementation details afterwards. ;-)
@ParticalStudios
@ParticalStudios Жыл бұрын
Cherno, with the video content you create, you are very inspiring for people like me who want to become a game engine developer in the future. Thank you for that, keep that great work up!
@digital_hoboz
@digital_hoboz Жыл бұрын
Most of the times engines have to move objects around in the heap to fill the holes created by freed memory. If you access pointers directly, you either can't move objects or you can't use memory efficiently.
@kevinthesome
@kevinthesome Жыл бұрын
Man i like this style a lot more plz more
@mikael8276
@mikael8276 Жыл бұрын
Whoa, I didn't know that pushing a new element into a vector would invalidate all earlier references.
@fappylp2574
@fappylp2574 Жыл бұрын
Most of the lines of code I have ever written in C++ have turned out to be undefined behaviour... It's astonishing what you can miss without formal training!
@JavedAlam-ce4mu
@JavedAlam-ce4mu 5 ай бұрын
That's only if you're either pushing from the front, or pushing from the back, but the memory size isn't enough, so it has to make a new vector and copy those items to another location. But it's also specifically only when you are referencing data in the vector, e.g.: pointer* = &vectorName[1] or reference& = vectorName[1] Which really isn't that common. It isn't true for vectors of pointers, e.g. vector Pushing elements to these doesn't invalidate their addresses because they point elsewhere.
@spidermanlift4527
@spidermanlift4527 Жыл бұрын
I wanna see more of this engine
@adamrushford
@adamrushford Жыл бұрын
Thank you Yan that was very helpful and appreciated as always can't wait for more!
@mldy1
@mldy1 Жыл бұрын
25:48 probability would be 1 in 2^64, or 1 in ~18 quintillion or 1 in ~18 billion billion so probably more likely a cosmic ray would case the program to crash
@nicksdrumsticks
@nicksdrumsticks Жыл бұрын
7:04 - What the hell? That pronunciation was so accurate
@-lolus-
@-lolus- Жыл бұрын
i'd love to se another part about this engine
@TehHeiks
@TehHeiks Жыл бұрын
That sprite stunlock truly is something
@NathanHedglin
@NathanHedglin Жыл бұрын
Please continue! This is a great breakdown
@rafaelmateodev
@rafaelmateodev Жыл бұрын
I hope they make the changes, this seems imperative.
@simiuciacia
@simiuciacia Жыл бұрын
People should pay for content like this, this is my kind of videos really, thx again
@Andrumen01
@Andrumen01 Жыл бұрын
I agree with your statement about pointers. I hate them...not that I don't understand them, but it's a pain to keep track of them when they are juggled around and it's better to avoid them whenever possible. It's nothing else, really.
@oussamasoudassi
@oussamasoudassi Жыл бұрын
If you know who owns whatever pointer and who is responsible for clean up why hating and avoiding them??
@Andrumen01
@Andrumen01 Жыл бұрын
@@oussamasoudassi Have you ever worked on projects that have dozens (maybe hundreds) of files, each 2000+ lines long and full of pointers going here and there without indicating who owns what? Yes, fixing a memory leak due to badly un-referenced pointers in those projects is like fixing a hole in the Hoover damn with duct tape. I myself would be careful, but when projects grows out of hand, it is unavoidable to have those leaks, and finding them is a nightmare. Yes, C++20 has helped to mitigate that issue, but still pretty bad!
@the_gobbo
@the_gobbo Жыл бұрын
1:59 Boy was he right about this being a ride lol 🤣
@MotionLive
@MotionLive Жыл бұрын
Really helpful video, thank you !
@michaplucinski142
@michaplucinski142 2 ай бұрын
19:40 Wow, it's really exciting to me, because I have once done exactly that mistake And i had some really nasty bugs because of that It took me quite a lot of time to realise what's going on. I managed to do that, because I was at the same time trying some optymalization on vector, so i preallocated some memory for it and then program lasted longer. This way I realised, the problem is that when vector resizes all the references to it's elements become invalid So yea, I'm not the only one who made this mistake.
@kiririn39m8
@kiririn39m8 Жыл бұрын
Hi Cherno, thanks for another code review. By the way, could you maybe consider reviewing game entity design in Unreal engine? (namely Actors and components) And maybe a UI system? And maybe Compare it with Unity or Cryengine or Hazel or something. The reason is that there's this aura of ideal AAA grade around Unreal and everything related to it's design is perfect, but for me it's exactly the opposite, after unity an O3DE. Every other engineer I have spoken to (Unreal devs) don't even get what's wrong (from my personal opinion ofk) with their favorite engine etc. I think that could be a realy insightful and interesting kind of video
@GonziHere
@GonziHere Жыл бұрын
Yeah. Unreal is incredibly overhyped thanks to it's features, to the point, where we cannot have a normal discussions about it's shortcomings.
@anon1963
@anon1963 Жыл бұрын
​@@GonziHereI'd rather use polymorphism as well instead of creating an array for every type of an entity
@markp1634
@markp1634 Жыл бұрын
Please keep going!
@not_herobrine3752
@not_herobrine3752 Жыл бұрын
i wonder if thecherno would review a c styled rust codebase
@trampflips101
@trampflips101 Жыл бұрын
With 64 bit identifiers you have a 0.7% chance of a collision with 500 million uids, and 35% with 4 billion.
@kplays_6000
@kplays_6000 Жыл бұрын
Even in games such as Elite Dangerous or No Man's Sky, they don't come close to having to make use of 500 million UUIDs at once, let alone 4 billion, so this really is a non-issue with any scale of video game. The game won't store all the planets in one data structure - I'd imagine more of a tree where you segment the universe into quadrants, each quadrant stores its system or planets, then each planet stores its own entities. With careful consideration to structure, the UUIDs only have to be unique per level of the tree, rather than globally. That's where you start considering the difference between UUID and GUID
@trampflips101
@trampflips101 Жыл бұрын
@@kplays_6000 All true, he just asked someone to put the collision chance in the comments in the video.
@SquidSnipes
@SquidSnipes Жыл бұрын
Please continue with this
@loiphin
@loiphin Жыл бұрын
You sounded so excited whilst reading your Brilliant sponsor script... hahaha
@126644
@126644 Ай бұрын
Ive been working on this for about 2000 hrs now, actual project..... Hello world
@olegallito5742
@olegallito5742 Жыл бұрын
Minute 9 is when I started asking myself like "where the fuck is the engine" rather than a few classes and a half (that aren't even handy to use). It's pretty much like making a game from scratch, but using some questionable libraries.
@CodeX3253
@CodeX3253 4 ай бұрын
bro gave the entire CODE review without looking at the the Code 😂😂😂❤
@austinbachurski7906
@austinbachurski7906 Жыл бұрын
I'd really like to hear more about this UUID storage works. I don't believe I'm properly grasping it, obviously (I assume) you're not using the id as the index of the vector or whatever container you're using right? Would like to hear more if possible, and please continue the code review if you have time. Thanks.
@anispinner
@anispinner Жыл бұрын
Basically a 30 minute sprite review
@unkgames-abdullahali4048
@unkgames-abdullahali4048 Жыл бұрын
Another video of this engine
@grigoriy_hacker
@grigoriy_hacker 4 ай бұрын
Nice egnine, I love it
@y4ni608
@y4ni608 4 ай бұрын
Thanks :D
@JavedAlam-ce4mu
@JavedAlam-ce4mu 5 ай бұрын
I'm confused, if you used a 64-bit UUID wouldn't the vector size be huge? Or are you saying to use a map instead?
@kelvinpoetra
@kelvinpoetra 3 ай бұрын
Hi Cherno, I want to ask what steps are needed to create a game engine?
@beigebacchus1
@beigebacchus1 Жыл бұрын
Hey I was wondering if you could do an updated coding tutorial for opengl please?
@leshommesdupilly
@leshommesdupilly 6 ай бұрын
I liked spending hundreds of hours coding games, engines, applications.. It was really fun and I was satisfied of the speed. And then, I learned about the cache, cpu optimizations, ram latency, and now, all I can see is wasted performance and spaghetti code 😂
@NeuroScientician
@NeuroScientician Жыл бұрын
What is a good way to track hours spent in VSCode/Game Engine? Something bit more automatic that just stopwatch and a spreadsheet?
@notlekrut
@notlekrut Жыл бұрын
You can use apps/sites that have extensions for IDEs, like Wakatime, there's several others. There's also some apps that track OS windows and window titles to figure out what you're working on and then you can categorize, but I only know specific ones for mac(ie. Timing), but I'm sure there's similar ones for Windows.
@InfiniteCoder01
@InfiniteCoder01 Жыл бұрын
Wakatime
@Griffin12536
@Griffin12536 4 ай бұрын
That was some incredible insight to how a programmer thinks.
@topg3067
@topg3067 Жыл бұрын
Back to the engine.
@anispinner
@anispinner Жыл бұрын
00:18 Ah, looks like a typo for "Egg nine"
@EmilyGraceSeville7cf
@EmilyGraceSeville7cf Жыл бұрын
I love your content, keep going😄
@alastairtheduke
@alastairtheduke 11 ай бұрын
great video
@tbc1880
@tbc1880 9 ай бұрын
As someone in a similar spot as the guy here, (not 2k hrs deep but grinding through my own engine as basically a higher level novice) I want to shoot two questions I have into the void in hopes I may finally get answers. For my scenes I have the main loop run their update function and every now and then a different one for a transition that may send data over to the new scene (like if I move to the main menu from settings I keep the music playing vs from title screen which has it's own music.) The main just gets a pointer to the new scene while the old one clears itself of all objects and such. Main then runs and creates the new scene. While memory usage is still low I wonder if this is okay due to a not exactly memory leak I am seeing. Basically if I move between scene 1 and 2 the memory use will rise but will stop after a bit. Move to scene 3 and the cap rises. Scene 4 and so on. I have no clue what maybe causing this as I am deleting everything and if it were a true leak would it not continue indefinitely? Is it just my computer managing data in that way or an actual issue with my code.
@lucasgaming4793
@lucasgaming4793 Жыл бұрын
i would like to see more
@loctite417
@loctite417 Жыл бұрын
Thank you for extremely helpful video especially since I recently made a sprite rendering system with a c style array of size 2048 storing pointer to sprites lmao
@zombi1034
@zombi1034 Жыл бұрын
more please❤
@weirddan455
@weirddan455 Жыл бұрын
"Code Review" - Never even looks at the code. He just saw the new keyword and went off on a tangent about entity component systems...
@ZeroUm_
@ZeroUm_ Жыл бұрын
Architecture review is code review. It's way more important to catch wrong assumptions about how stuff works than nitpicking variable names or any easily refactorizable code.
@NathanHedglin
@NathanHedglin Жыл бұрын
If you don't get the architecture correct, the code basically doesn't matter
@TheMrTka4
@TheMrTka4 Жыл бұрын
I expected some code in code review :D But that was useful anyway! I would like to see a continuation about this engine. I miss code reviews from you!
@edmunns8825
@edmunns8825 Жыл бұрын
An egnine is three short of a dozen, but just enough for 2 dosen and an egnine
@arimcbrown
@arimcbrown 10 ай бұрын
Here's some actionable advice: ... set your autofocus to a fixed setting.... pls. :'(
@Brolexios
@Brolexios Жыл бұрын
@TheCherno, what do you use to draw on the screen and whiteboard these webpages?
@I-PixALbI4-I
@I-PixALbI4-I Жыл бұрын
Please help Battlestate Games make their game FASTER! Its so unique game!!! )
@BrainStroming1789
@BrainStroming1789 5 ай бұрын
Si j'avais à faire une revue, déjà je commencerais par poser une question majeur. Est-ce de la responsabilité de l'objet Game de savoir comment créer un sprite et comment manager sa persistance ?! non. Un objet SpriteManager capable de créer des sprites transitant ou persisté serait plus optim. Conception SOLID et le code deviendra plus simple et logique. Ensuite sur le "new", pourquoi sur-utiliser la stl ? qu'il était doux le temps ou nous savions faire la part des choses et ne pas utiliser une class inutile quand elle n'est pas nécessaire.
@marcelbarbosa281
@marcelbarbosa281 Жыл бұрын
Yes, i want sir.
@SiltyWheat7
@SiltyWheat7 Жыл бұрын
I'm not sure I interpreted this correctly. How would you use a random GUID or UUID as an index for a vector? Wouldn't these IDs potentially be massive (i.e. 0 to 2^64 - 1) meaning the vector would have a ridiculous number of elements?
@tardistrailers
@tardistrailers Жыл бұрын
You could use a hashmap instead of a vector in that case.
@fappylp2574
@fappylp2574 Жыл бұрын
Lol, just buy more ram...
@fappylp2574
@fappylp2574 Жыл бұрын
@@tardistrailers But how do you get around losing the contiguous memory? Does it make sense to store a map of (UUID -> Index) and keep that updated somehow? The whole point of this contiguous allocation is better performance during iteration (ECS etc.).
@niallrussell7184
@niallrussell7184 Жыл бұрын
I expected this to be about optimizing drawcalls or something.. didn't even get that far! 🤣
@maltaparkhelp
@maltaparkhelp Жыл бұрын
The downside to a vector of objects is that when the vector resizes, the all the objects need to be moved, instead of just the pointers. Depending on the size of the objects, this could make a big difference.
@mjthebest7294
@mjthebest7294 Жыл бұрын
Traversing the vector is usually done muuuuch more frequently than vector resizing, so optimizing that is a priority. If resizing is happening so frequently that is slows you down, you have an architectural design issue
@kiiikoooPT
@kiiikoooPT Жыл бұрын
I have a stupid question maybe, but is just curiosity I guess, the case you talked in the video and some people are talking in the comments about a really low chance of a 128bit random nr coliding or repeating, isn't it a good idea to make the nr and check if it already exists? Just to make it impossible to happen at all. I just say that, because in this case, the sprites will be loaded once only, and they stay in memory or cache or whatever, it will slow down the loading of the sprites cause it has an if in every sprite nr generation, but it will only happen once right? at the beggining of the game launch. then you just ask to print a specific sprite that is already there or to hide it. I don't know if im making myself clear in what I mean. Is kind of my logic, but I never made a game or engine or anything usefull at all, just few projects that I started to try to learn it, but never finished, or needed to load gigabytes of stuff. Only things with small content. Few images to make buildings with colision and my character images, the way I did it if im not mistaken was with class sprites and push pull to the class array of sprites, kind of what you did with the cherno engine in the begining I believe.
@ScibbieGames
@ScibbieGames Жыл бұрын
Structure of arrays enthusiasts unite
@TheViperZed
@TheViperZed Жыл бұрын
wow this was basic, anyway what this is really missing is a decent description on pointer arithmetic and why that is a very inexpensive operation. This is of course assuming that all the sprite objects have an equal, known, size at compile time and no further de-referencing to other data on the heap happens, cause then the original method of just storing pointers is actually more efficient when drawing the sprite. If you have to explain this to someone, this really should really include an in depth discussion of stack and heap allocations and what that actually means, as that is the underlying optimization issue.
@dejoma.
@dejoma. Жыл бұрын
When somebody writes "You can give the pointer an address by calling ..." you know that this person does not understand how memory allocation works in C++. Which is absurd after 2000 hours :D
@y4ni608
@y4ni608 Жыл бұрын
I apolozige for any confusion in the Readme file, to put it simply my english is just very bad. i meant with this line "You can give the Sprite ptr a address by calling the s2d::Sprite::getSpriteByName("name"); " that well since s2d::Sprite* my_ptr is a nullptr on start you are assigning the nullptr a adress i dont really know how to put it in good english for you. So thats what i meant, and yeah i do agree that it sounds wierd when you say "Give the pointer a adress" but i dont know how to say it better, sorry.
@InShadowsLinger
@InShadowsLinger Жыл бұрын
The Sprite object and its knowledge of sprites vector seems like anti pattern to me and against OOP principles.
@maximiliannowak7860
@maximiliannowak7860 Жыл бұрын
Man, all this talking about sprites made me thirsty
@y4ni608
@y4ni608 Жыл бұрын
Hahaha
@hawns3212
@hawns3212 Жыл бұрын
25:45 The odds of two random 64 bit numbers colliding is 2.916x10^-39% AKA 0.0000000000000000000000000000000000000002196% With more decimals after, of course, but roughly that It is a chance of 1 in 340,282,366,920,938,463,463,374,607,431,768,211,456 chance or a 1 in 2^128 chance or a 1 in 340.2 undecillion chance. Very very unlikely. If this ever happens to you, go buy a lottery ticket. You're likely to win the lottery 11,645,529,300,000,000,000,000,000,000 times more likely to win the lottery than get two 64 bit integers being the same Edit: I made a mistake, its not 1 in 2^127, but 2^128
@HiHi-ur3on
@HiHi-ur3on Жыл бұрын
That doesn't make sense. Shouldn't the chance of two random 64 bit numbers be a straight up 1 in 2^64? Since the second number has to be exactly the first number and can be any of 2^64 numbers?
@felixp535
@felixp535 Жыл бұрын
How can it be a 1 in 2^127 when there's only 64 bits? This makes no sense. If we both pick a number between 1 and 10, the chance we pick the same number is 1/10 (the number you pick doesn't matter, the only thing that matters is that I pick the same). Following the same logic, the chance for a computer to pick two identical numbers between 1 and 2^64 is 1 in 2^64.
@hawns3212
@hawns3212 Жыл бұрын
@@felixp535 Lets make it a bit simpler. Imagine you have two dice and you want them both to roll 6s. Each die has a 1 in 6 chance of landing 6, so dice A has 1/6 chance, and dice 2 has 1/6 chance. These are statistically independent events, therefore to calculate the odds of them both happening we multiply them, therefore both dice have a 1/6 chance. This is called taking the intersection of the probabilities, when two events happen together. The odds of having one number from a 64 bit integer is 1 in 2^64. Assuming perfect randomness in the generation we multiply it (aka square it) just like the dice. 2^64 squared is 2^128, therefore we have odds of 1 in 2^128.
@Rizza257
@Rizza257 Жыл бұрын
@@felixp535 this is a common misconception. The probability of person 1 picking does not affect the probability of person 2. This means they are independent events. The probability for independent events is calculated P = P(A) * P(B). So in your example the probability of both picking the same number out of ten is 1/10 * 1/10 = 1/100. This is because you have to consider every combination they can pick the same number.
@felixp535
@felixp535 Жыл бұрын
​@@Rizza257 You made me doubt so I double checked using Excel. The probability of both picking the same number is indeed 1/10 and not 1/100 as you mention. The number chosen by A doesn't matter. The only thing that matters is that B also chooses the same one, which has a 1/10 chance to happen. The events are dependant because B must pick the same number as A. When you think about it, A does not have to pick the same number as B, so P(A) is the probability that A picks any number, which is 1.
@rudolfbouzek6944
@rudolfbouzek6944 Жыл бұрын
eh :D so pro :D im looking at basic c++ videos... it was 6 years ago... this is like level: GOD :D
@tawandagamedevs
@tawandagamedevs Жыл бұрын
Rust developers in shock seeing a reference to an element in an array
@michaelsteffensen6844
@michaelsteffensen6844 Жыл бұрын
Why would that shock rust developers?
Game Engine Architecture 101 // Code Review
16:19
The Cherno
Рет қаралды 58 М.
2D Flying Dog Game in C++ // Code Review
25:30
The Cherno
Рет қаралды 41 М.
Why no RONALDO?! 🤔⚽️
00:28
Celine Dept
Рет қаралды 32 МЛН
When Cucumbers Meet PVC Pipe The Results Are Wild! 🤭
00:44
Crafty Buddy
Рет қаралды 56 МЛН
World’s strongest WOMAN vs regular GIRLS
00:56
A4
Рет қаралды 48 МЛН
I Made the Same Game in 8 Engines
12:34
Emeral
Рет қаралды 4,2 МЛН
SOME UNIQUE C++ CODE! // Pacman Clone Code Review
26:42
The Cherno
Рет қаралды 286 М.
How to Get a Developer Job - Even in This Economy [Full Course]
3:59:46
freeCodeCamp.org
Рет қаралды 3,1 МЛН
Blazingly Fast Greedy Mesher - Voxel Engine Optimizations
23:35
Using AI to help me with C++
24:31
The Cherno
Рет қаралды 111 М.
Decrusting the tokio crate
3:31:48
Jon Gjengset
Рет қаралды 141 М.
I Optimised My Game Engine Up To 12000 FPS
11:58
Vercidium
Рет қаралды 719 М.
Harder Than It Seems? 5 Minute Timer in C++
20:10
The Cherno
Рет қаралды 214 М.
I made it FASTER // Code Review
38:46
The Cherno
Рет қаралды 549 М.
I Spent 100 HOURS Making a Game With NO GAME ENGINE
13:42
Fuelvin
Рет қаралды 348 М.
Why no RONALDO?! 🤔⚽️
00:28
Celine Dept
Рет қаралды 32 МЛН