I am currently using Dependency Injection and when I create a controller I always create a constructor and type hint Twig $view and LoggerInterface $logger which create use statements. Do I need to type hint still or am I using dependency injection incorrectly?
You should always TypeHint, this allows you to better test your class structure as you can use Mocks/Partials to test the outputs of your classes.
Should I not use dependency injection for Twig $view and LoggerInterface $logger then?
In my opinion you should use dependency injection and type hints.
You are using dependency injection.