Timeline for Pig latin translating program
Current License: CC BY-SA 3.0
12 events
when toggle format | what | by | license | comment | |
---|---|---|---|---|---|
Nov 1, 2016 at 23:24 | comment | added | Michael Howard | After that I plan to attempt to work it all into TKinter and dress it up nice. I honestly do appriciate the comments from you all, they do help. | |
Nov 1, 2016 at 23:23 | comment | added | Michael Howard | Thank you AChampion. I always accept constructive criticisim but I am willing to defend my actions as well as my reasonings. I know the comments really should have been there but I just had a sudden flash of light in my head and wanted to get it down fast rather then risk forgetting the flash while I wrote comments hehe. Short memory span is a bitch and a half :P and I totally agree that some did not truly need to be global variables but I am still working on the program. I am attempting to add a function to translate pygLatin to Engh then plan on giving the option to trans to pyg or to Eng. | |
Oct 31, 2016 at 18:02 | comment | added | AChampion | Absolutely, take the feedback for what it's worth. What works for you accept what doesn't ignore. And good luck. | |
Oct 31, 2016 at 18:01 | comment | added | Michael Howard | list= sentence.split was done so that the translation block to cycle through each word separately translating them as it went through each of them. The "for i in list" statement was done as such because as I said, I am very very new to programming and did not know how to put it all into one line. I can agree with virtually everything that you said and I thank you for the criticism. I hope this explains quite a bit of why I chose to do things as I did. | |
Oct 31, 2016 at 18:00 | comment | added | Michael Howard | You got me on the "enter a word to be translated" it was suppose to be "enter a sentence to be translated" | |
Oct 31, 2016 at 17:59 | comment | added | Michael Howard | it is simply meant to take one sentence and translate it then ask if you wish to do another sentence. if len(original) > 0 and original.isalpha(): I used len(original) > to make sure there was an actual word there or at least a letter. I used original.isalpha() to make sure it was letters and not numbers. print empty if the info entered was either blank or if it was numbers. | |
Oct 31, 2016 at 17:48 | comment | added | Michael Howard | AChampion, certain lines I did the way that I did for my own clarification (to keep it clear in my head). I agree, I should have used a lont of comments rather then coding in the way I did. | |
Oct 30, 2016 at 14:11 | comment | added | AChampion | Yes, a loop structure for the request was part of my feedback, as it was unclear whether the intent was for 1+ vs. just 1 or 2. So an assumption on my part - which may ultimately be incorrect. | |
Oct 30, 2016 at 5:10 | comment | added | 200_success |
I suggest also eliminating pyg = 'ay' and writing return word[1:] + word[0] + 'ay' .
|
|
Oct 30, 2016 at 5:10 | comment | added | 200_success | Note that the original code would translate one or two sentences. Your solution offers to translate infinitely many sentences. | |
Oct 30, 2016 at 1:18 | history | edited | AChampion | CC BY-SA 3.0 |
added 159 characters in body
|
Oct 30, 2016 at 1:12 | history | answered | AChampion | CC BY-SA 3.0 |