Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!feeder.erje.net!eu.feeder.erje.net!newsfeed.fsmpi.rwth-aachen.de!newsfeed0.kamp.net!newsfeed.kamp.net!feeder1.cambriumusenet.nl!feed.tweaknews.nl!194.109.133.81.MISMATCH!newsfeed.xs4all.nl!newsfeed2a.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
Date: Mon, 01 Sep 2014 01:23:36 +0100
From: MRAB <python@mrabarnett.plus.com>
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: python-list@python.org
Subject: Re: This could be an interesting error
References: <pm270ap6g01k6a5ip10ij1lgtebsrv00mr@4ax.com> <mailman.13668.1409523047.18130.python-list@python.org> <2jc70at4qcpqmbes72r2f0tsjk8up0j6mn@4ax.com>
In-Reply-To: <2jc70at4qcpqmbes72r2f0tsjk8up0j6mn@4ax.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.13672.1409531019.18130.python-list@python.org>
Lines: 104
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:77377

On 2014-09-01 01:04, Seymore4Head wrote:
> On Sun, 31 Aug 2014 16:10:27 -0600, Michael Torrie <torriem@gmail.com>
> wrote:
>
>>On 08/31/2014 03:02 PM, Seymore4Head wrote:
>>> def pigword(test):
>>>     for x in range(len(test)):
>>>         if test[x] in "AEIOUaeiou":
>>>             stem = test [x:]
>>>             prefix = test [:x]
>>>             pigword = stem + prefix + "ay"
>>>             print ("Stem ",stem)
>>>             print ("Prefix",prefix)
>>>             print (pigword)
>>>             break
>>>     return (pigword)
>>
>>So, what do you think will happen if the word contains no vowels?  Where
>>is pigword defined?
>>
>>> for x in range(len(newex)):
>>>     sentence = sentence + pigword(newex[x])+ " "
>>>     print (sentence)
>>>     wait = input ("          Wait")
>>
>>You don't need to iterate over range(len(blah)).  The standard idiom
>>when you need index as well as the item itself is to iterate over
>>enumerate().  Or if you don't need the index, just iterate directly.
>>You can iterate directly over the list, or the letters in the word,
>>optionally getting an index. It's much cleaner and less error prone.
>>Consider something like:
>>
>>def pigword(word):
>>    for x,letter in enumerate(word):
>>        # x is index (position), letter is the value at that index
>>        if letter in "AEIOUaeiou":
>>            ...
>>
>>for word in list_of_words:
>>    sentence = sentence + pigword(word) + " "
>>    ...
>>
>>That doesn't solve your little logic problem, though I think you can
>>figure that part out easily!
>>
> I am still kind of shooting in the dark.
>
> I wanted to try your example and it doesn't seem to work.
> This is the latest version of changes I have made so the entire
> program looks like this:
>
> newex='Hey buddy get away from my car'
> newex = newex.split()
> sentence=""
>
> print (newex)
> wait = input ("          Wait")
>
> def pigword(test):
>      for x in range(len(test)):
>          if test[x] in "AEIOUYyaeiou":
>              stem = test [x:]
>              prefix = test [:x]
>              pigword = stem + prefix + "ay"
>              print ("Stem ",stem)
>              print ("Prefix",prefix)
>              print (pigword)
>              break
>      return (pigword)
>
> for x in range(len(newex)):
>      sentence = sentence + pigword(newex[x])+ " "
>      print (sentence)
>      wait = input ("          Wait")
>
> Trying to use your example
>>def pigword(word):
>>    for x,letter in enumerate(word):
>>        # x is index (position), letter is the value at that index
>>        if letter in "AEIOUaeiou":
> BTW I added "AEIOUYyaeiou" y as a vowel.
>
> I tried changing:
> for x in range(len(test)):
> to
> for x in enumerate(test):
>
> That causes an error to popup in a different place.  I don't
> understand why.
> Traceback (most recent call last):
>    File "C:\Functions\piglatin.py", line 26, in <module>
>      sentence = sentence + pigword(newex[x])+ " "
>    File "C:\Functions\piglatin.py", line 15, in pigword
>      if test[x] in "AEIOUYyaeiou":
> TypeError: string indices must be integers
>
Try printing out x. That should give you a clue!

> Since you included:
>>for word in list_of_words:
>>    sentence = sentence + pigword(word) + " "
> I take it you anticipated a fault here.  I don't understand why.
>