[ad_1]
\n<\/p>\n
Pointing out the mistakes\/unnecessary lines in your code:<\/p>\n
The line Thus, your code can be re-written<\/em> as following:<\/p>\n An elegant<\/strong> way to achieve this would be the following:<\/p>\n Here, In your case, since you are not allowed to change the test code, there’s an alternative in which you change the value of the list in-place instead of returning a new list…<\/p>\n Here, you are changing the values of the list index-by-indexsplit = strings.split().upper()<\/code> is erroneous and at the same time unnecessary as you are mistakenly using strings<\/code> (list)<\/em> instead of string<\/code> (string literal)<\/em>. Also, you can use len(string)<\/code> directly instead of splitting the string into a list and finding the length of list.<\/p>\n<\/li>\nresult.append(char * '#')<\/code> is just appending “#”<\/strong> but not the actual string. The line could be rewritten as result.append(string.upper() + char * '#')<\/code><\/p>\n<\/li>\n<\/ul>\ndef squigler(strings):\n result = []\n for string in strings:\n if len(string) < 5:\n char = 5 - len(string)\n result.append(string.upper() + char * '#')\n else:\n result.append(string.upper())\n \n return result\n<\/code><\/pre>\ndef squigler(strings):\n return list(map(lambda x: x.upper().ljust(5, '#'), strings))\n<\/code><\/pre>\nTesting the Output<\/h2>\n
# TEST\nstrings = ['Right', 'SAID', 'jO']\nstrings = squigler(strings) # Over-writes the original `strings` list\n\nprint(strings) # Output: ['RIGHT', 'SAID#', 'JO###']\n<\/code><\/pre>\n\n
map<\/code> function loops over all the elements in the list strings<\/code> and
\napplies lambda<\/code> function that we provided as an argument to map<\/code> to each of
\nthese elements and returns a map<\/code> object which we then convert to list<\/code> and
\nreturn it.<\/p>\nljust<\/code> method of string, pushes the string to the left and appends a given character at the end of the string which would make the string exactly n characters long (in this case n=5)<\/em><\/p>\n<\/blockquote>\nEDIT:<\/h2>\n
def squigler(strings):\n for(index, string) in enumerate(strings):\n if len(string) < 5:\n char = 5 - len(string)\n strings[index] = string.upper() + char * '#'\n else:\n strings[index] = string.upper()\n<\/code><\/pre>\n\n
\ninstead of returning a new list and reassigning the old one.<\/p>\n<\/blockquote><\/div>\n