Path: csiph.com!usenet.pasdenom.info!news.franciliens.net!fdn.fr!usenet-fr.net!nerim.net!novso.com!newsfeed.xs4all.nl!newsfeed2.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <3f10c652-9d9d-4353-b24f-939d21978998@googlegroups.com>
References: <3f10c652-9d9d-4353-b24f-939d21978998@googlegroups.com>
Date: Tue, 28 Oct 2014 18:09:36 +1100
Subject: Re: hotel management system
From: Chris Angelico <rosuav@gmail.com>
Cc: "python-list@python.org" <python-list@python.org>
Content-Type: text/plain; charset=UTF-8
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.15253.1414480185.18130.python-list@python.org>
Lines: 41
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:79825

On Tue, Oct 28, 2014 at 5:42 PM, ngangsia akumbo <ngangsia@gmail.com> wrote:
> Please can someone look at my code and may be advice and may be help me with some correction. I have been learning python for some time now. This is my first project i wish to write. A hotel management system.
>
>
> http://pastebin.com/LMHmuTiC
>

Sure, we can do code reviews! It's usually helpful to provide your
code in-line, though, rather than linking to pastebin.

You're using Python 2, as evidenced by the print statements. So don't
do this, ever:

> def emp_age():
>     """displaying employees age"""
>
>     age = input("Enter age: ")

The input function is extremely dangerous. Everywhere else, you use
raw_input, which is correct.

> class Bar:

This is an old-style class. It's better to explicitly subclass object:

class Bar(object):

> class Beer(Bar):

Not sure why you're doing this. Firstly, you never use your Beer
class... but secondly, your Beer isn't a special type of Bar, it's a
quite different thing. When you subclass, it's usually helpful to
follow the Liskov Substitution Principle:

https://en.wikipedia.org/wiki/Liskov_substitution_principle

Think carefully about your class hierarchy, because it defines all
sorts of things about your code. Be sure you're happy with it before
you code up too much else.

ChrisA