there is nothing wrong with your code BUT it’s not the recommended way to do it … it’s better to use get_template_directory_uri as that will always get the assets from the parent theme rather than the current theme e.g. a child theme.
Lets say I build a child theme on your theme, using stylesheet_directory_uri the css from the main theme won’t be loaded, so by all purposes my child theme won’t work as it was intended.