Path: csiph.com!usenet.pasdenom.info!news.albasani.net!newsfeed.freenet.ag!news2.euro.net!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <2912c674-e30b-4339-9344-1f460cb96b23@googlegroups.com>
References: <37f23623-8bf5-421a-ab6a-34ff622c69f1@googlegroups.com> <mailman.4027.1364762359.2939.python-list@python.org> <2912c674-e30b-4339-9344-1f460cb96b23@googlegroups.com>
Date: Mon, 1 Apr 2013 08:21:00 +1100
Subject: Re: Help with python code!
From: Chris Angelico <rosuav@gmail.com>
To: python-list@python.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.4029.1364764868.2939.python-list@python.org>
Lines: 66
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:42411

On Mon, Apr 1, 2013 at 8:06 AM, jojo <gerrymcgovern@gmail.com> wrote:
> On Sunday, March 31, 2013 4:39:11 PM UTC-4, Chris Angelico wrote:
>> On Mon, Apr 1, 2013 at 7:10 AM, jojo wrote:
>>
>> > Im used to C# so the syntax looks bizarre to me! Any help would be gre=
at.
>>
>>
>>
>> The first thing you'll need to understand about Python syntax is that
>>
>> indentation is important. By posting this code flush-left, you've
>>
>> actually destroyed its block structure. Could you post it again, with
>>
>> indentation, please? We'd then be in a much better position to help.
>>
>>
>>
>> Chris Angelico
>
>
> Hi Chris, thanks for your reply. See code below...

Ah, you appear to be posting from Google Groups. You may want to check
this page out, as a lot of people rather dislike GG posts.

http://wiki.python.org/moin/GoogleGroupsPython

The best method is simply to avoid Google Groups altogether.

Anyway, some code comments. (Though the biggest comment to make about
the code is its utter lack of comments. Not a good idea in any
language, for anything more than the most trivial script.)

> current_time =3D time.time() + 60*60+24*30

This line doesn't, quite frankly, make a lot of sense; time.time()
returns the current time already, but then an offset of one hour and
twelve minutes is added.

>    if m:
>       sue =3D time.mktime(
>         (int(m.group(7)), int(months[m.group(2)]), int(m.group(3)),
>           int(m.group(4)), int(m.group(5)), int(m.group(6)),
>           int(days[m.group(1)]), 0, 0)
>         )
>         expire_time =3D (sue =AD current_time)/60/60/24

Here's a likely problem. There's supposed to be an operator - probably
a plus sign - between sue and current_time.

>        else:
>         m =3D q.search(line)
>         if m:
>         cert_name =3D m.group(1)

And this last line needs indentation.

The very easiest way to debug Python code is to run it. If it runs,
great! See what output it made and whether it's correct or not. If it
doesn't, Python will give you an exception traceback that points you
to the failing line. Get familiar with them, as you'll be seeing them
a lot :)

Chris Angelico