Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solution cinema_visit #1744

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Roksolana-K
Copy link

No description provided.

app/main.py Outdated
from typing import List, Dict


def cinema_visit(
Copy link

@Maaax256 Maaax256 Mar 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

def cinema_visit(customers: list, hall_number: int,
                 cleaner: str, movie: str) -> None:
    customers_inst = [
        Customer(customer_dict["name"], customer_dict["food"])
        for customer_dict in customers
    ]
    for customer_inst in customers_inst:
        food = customer_inst.food
        CinemaBar.sell_product(customer_inst, food)
    cinema_hall_inst = CinemaHall(hall_number)
    cleaner_inst = Cleaner(cleaner)
    cinema_hall_inst.movie_session(movie, customers_inst, cleaner_inst)
    pass
    ```
    
    too many rows, try to compress your code

Copy link

@Maaax256 Maaax256 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compress the code in main

@Roksolana-K Roksolana-K requested a review from Maaax256 March 3, 2025 18:03
Copy link

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job on implementing the classes and methods as required! 🎉 It looks like everything is in place and functioning correctly according to the task requirements. The Customer and Cleaner classes are well-defined, and their methods are being used appropriately in the CinemaBar and CinemaHall classes. Keep up the good work, and feel free to reach out if you have any questions or need further assistance. 😊

✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants