.Empty функцией должна быть, а не полем, если код не под нагрузкой живет

по синтаксу: неправильно скобка стоит. по логике: else вообще не нужен. Пить надо всегда.

Помимо скобки я бы ещё сказал, что не хватает сущности вроде person, чтобы именно она пила и наливала кофе. А то тут кофе живёт своей жизнью, что, мягко говоря, странно😅

Фигурная скобка после else