Path: csiph.com!usenet.pasdenom.info!aioe.org!news.stack.nl!newsfeed.xs4all.nl!newsfeed2a.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
Date: Wed, 29 Oct 2014 15:12:14 +0100
From: Florian Schweikert <kelvan@logic.at>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0
MIME-Version: 1.0
To: python-list@python.org
Subject: Re: hotel management system
References: <3f10c652-9d9d-4353-b24f-939d21978998@googlegroups.com>
In-Reply-To: <3f10c652-9d9d-4353-b24f-939d21978998@googlegroups.com>
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.15300.1414592507.18130.python-list@python.org>
Lines: 73
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:79914

On 28/10/14 07:42, ngangsia akumbo 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

Beside posting the code inline, it would be easier if you had split the
"employee" part and the part with the classes.

I'm not sure what you want to accomplish with this code, but I picked a
few parts:

> if age >= 18:
>     pass
> 
> else:
>     return age_lim()

using a function just to print a single line is quite overcomplicated.
return the result and handle it on the caller side.
why do you use if/else and just pass first path?

if age<18:
    do_something

do the same thing.

I have to guess what the class part should do.

> class Bar:

why calling a class Bar if you use it like some drink counter?

>     total_cost = 0
>     count = 0

not sure you want to use class variables here, do you want to count
amount of a single type of beer? or everything

>     def set_name(self, name):
>         self.name = name
>         return name

don't use setter if there is 0 logic in the function
also it would be better to set this things in the constructor otherwise
and a setter should not return the name just set

>     def set_count(self):
>         counts = Bar.count =+ 1

not really readable code style, counts variable is unneccessary here

> # still have to adjust this section
> 
>     def set_total_cost(self):                      #I am having some errors #calling this functions
>         total = set_price() * set_count()
>         print "Total cost of drinks: ", total
>         pass

well, it's quite obvious this function crashes, you try to use not
defined functions. you have to use self.function_name to access the methods.
But worse than this you want to use (useless) setter to get values and
print them inside the method instead of returning the value and handle
it outside.
Also this is not even a setter, it does not set anything beside a local
variable.

Maybe it would be best you start over again and make one step after
another. Build one part of code and ensure it works before starting with
more code. And ask specific questions if you stuck somewhere.


-- Florian