Зачем нам нужно проверять, является ли пользователь экземпляром пользовательского интерфейса

Да, если ваш код не предназначен для использования с открытым исходным кодом, в противном случае нет.

Отсутствие проверки экземпляра объекта не гарантирует, что объект, возвращаемый методом getUser(), будет иметь все ожидаемые методы (пример: getUsername()).

Если вы посмотрите на метод getUser() из Controller.php он не обязательно возвращает объект пользователя. Фактически, вы могли бы настроить брандмауэр Symfony2 таким образом, чтобы getUser() возвращал другой объект другого экземпляр.

Признавая, что у нас есть интерфейс UserInterface, который определяет getUsername().

В следующем коде наш объект пользователя не реализует UserInterface.

$user = $this->getUser();
if (!is_object($user)) {
    $user->getUsername();
}

Этот код выдаст ошибку, потому что getUsername() не существует на объекте, вместо этого код должен был быть следующим:

$user = $this->getUser();
if (!is_object($user) || !$user instanceof UserInterface) {
    $user->getUsername();
}

Если объект пользователя не реализует соответствующий интерфейс, то код не выдаст ошибку, потому что он не будет выполнен.

Избегайте проверки объекта, подобного следующий

$user = $this->getUser();
if (!is_object($user) || !$user instanceof User) {
    $user->getRoles();
}

Если кто-то расширит объект пользователя, то оператор if больше не будет выполняться, потому что $user не будет экземпляром User, но скажет ExtendedUser, хотя в нем есть все необходимые методы.

Другое преимущество использования интерфейсов заключается в том, что вы можете реализовать несколько интерфейсов на объекте.

class A implements C {}

class B extends A implements C, D {}

interface C {}

interface D {}

$nA = new A();
$nB = new B();

$nA instanceof A; // true - instance of A
$nA instanceof B; // false - pretty obvious, no relationship with B
$nA instanceof C; // true - A implements C
$nA instanceof D; // false - A does not implement D

$nB instanceof A; // false - B is not an instance of A
$nB instanceof B; // true - instance of B
$nB instanceof C; // true - A implements C, that's the key:
                  //        both A and B implements C but B is not an
                  //        instance of A.
$nB instanceof D; // true - A implements D

TLDR; интерфейсы - отличный способ установить ожидания и избежать серьезных головных болей.

Когда вы прочитаете код, вы сможете быстро определите тип передаваемых объектов. Если кто-то изменит код, он либо покажет значимую ошибку, либо изящно ухудшится (в этом случае пользователю будет отказано в доступе).