Symfony Relations - symfony

Question

In my app I have 3 entities; User, Booking and Room.
Booking entity:
namespace App\Entity;
/**
* #ORM\Table(name="booking")
* #ORM\Entity(repositoryClass="App\Repository\BookingRepository")
*/
class Booking
{
/**
* #ORM\Column(type="boolean")
* #Assert\NotBlank()
*/
private $isActive;
/**
* #ORM\ManyToOne(targetEntity="App\Entity\Room", inversedBy="bookings")
*/
private $room;
/**
* #ORM\ManyToOne(targetEntity="App\Entity\User", inversedBy="bookings")
*/
private $user;
Room entity:
/**
* #ORM\Table(name="room")
* #ORM\Entity(repositoryClass="App\Repository\RoomRepository")
*/
class Room
{
/**
* #ORM\OneToMany(targetEntity="App\Entity\Booking", mappedBy="room")
* #Expose
*/
private $bookings;
User entity:
/**
* #ORM\Table(name="app_user")
* #ORM\Entity(repositoryClass="App\Repository\UserRepository")
* #UniqueEntity(fields="email", message="This email address is already in use")
*/
class User implements AdvancedUserInterface
{
/**
* #ORM\Column(type="string", length=255, unique=true)
* #Assert\NotBlank()
* #Assert\Email()
*/
private $email;
/**
* #ORM\OneToMany(targetEntity="App\Entity\Booking", mappedBy="user")
* #Expose
*/
private $bookings;
Given a user's email I can get a room id like this:
$user = $this->getEntityManager()
->getRepository(User::class)
->findOneBy([
'email' => 'johndoe#domain.com'
]);
$booking = $this->getEntityManager()
->getRepository(Booking::class)
->findOneBy([
'user' => $user,
'isActive' => true,
]);
$roomId = $booking->getRoom()->getId();
However this seems like a long way to do it. Is it possible to optimise this and query for a room without having to make 2 databases calls?

Answer 1

Yes you can get it directly from the $user variable. I don't see any getters or setters listed here but I'm assuming you have created them. If so then you can do the following:
$bookings = $user->getBookings();
Bookings is an array so you will need to select which booking you want to get the room for. Let's just select the first:
$roomId = $bookings->first()->getRoom()->getId()
To make it cleaner you can add a getCurrentBooking method or something similar to your User class that will return the exact booking you want.
Then you would end up with something like this:
$roomId = $user->getCurrentBooking->getRoom()->getId()

Answer 2

You could use a single join query to get the room for a spcific user
$this->getEntityManager()
->createQueryBuilder()
->select('r')
->from(Room::class, 'r')
->join('r.bookings', 'b')
->join('b.user', 'u')
->where('u.email = :email')
->andWhere('b.isActive = :isActive')
->setParameter('isActive', true)
->setParameter('email', 'johndoe#domain.com')
->getQuery()
->getResult();