@@ -16,19 +16,13 @@ class Controller(object):
16
16
def __init__ (self , storage : Storage ):
17
17
self ._storage = storage
18
18
19
- def _find_available_table_id (
20
- self , request : ReservationCreateRequest
21
- ) -> typing .Optional [int ]:
19
+ def _find_available_table_id (self , request : ReservationCreateRequest ) -> typing .Optional [int ]:
22
20
table_ids = set (self ._storage .list_table_ids (cnt = request .cnt ))
23
- reserved_table_ids = set (
24
- self ._storage .find_reserved_table_ids (cnt = request .cnt , dt = request .dt )
25
- )
21
+ reserved_table_ids = set (self ._storage .find_reserved_table_ids (cnt = request .cnt , dt = request .dt ))
26
22
for table_id in table_ids .difference (reserved_table_ids ):
27
23
return table_id
28
24
29
- def maybe_create_reservation (
30
- self , request : ReservationCreateRequest
31
- ) -> ReservationCreateResponse :
25
+ def maybe_create_reservation (self , request : ReservationCreateRequest ) -> ReservationCreateResponse :
32
26
table_id = self ._find_available_table_id (request )
33
27
if table_id is None :
34
28
logging .warning ("reservation failed" )
@@ -47,16 +41,11 @@ def maybe_create_reservation(
47
41
logging .warning (f"failed to reserve a table due to { repr (e )} " )
48
42
return ReservationCreateResponse (success = False )
49
43
50
- def maybe_cancel_reservation (
51
- self , request : ReservationCancelRequest
52
- ) -> ReservationCancelResponse :
44
+ def maybe_cancel_reservation (self , request : ReservationCancelRequest ) -> ReservationCancelResponse :
53
45
try :
54
46
self ._storage .delete_reservation (phone = request .phone , dt = request .dt )
55
47
logging .warning (f"reservation { request .phone } { request .dt } cancelled" )
56
48
return ReservationCancelResponse (success = True )
57
49
except Exception as e :
58
- logging .warning (
59
- f"failed to cancel reservation for { request .phone } "
60
- f"{ request .dt } due to { repr (e )} "
61
- )
50
+ logging .warning (f"failed to cancel reservation for { request .phone } " f"{ request .dt } due to { repr (e )} " )
62
51
return ReservationCancelResponse (success = False )
0 commit comments