Title: återkoppling på students sortering i kmom06 analyzer · Issue #56 · dbwebb-se/python · GitHub
Open Graph Title: återkoppling på students sortering i kmom06 analyzer · Issue #56 · dbwebb-se/python
X Title: återkoppling på students sortering i kmom06 analyzer · Issue #56 · dbwebb-se/python
Description: Här kommer återkoppling på funktionen en student har skapat för att sortera datan i dictionarien korrekt. Här är koden och nedanför finns återkopplingen. def lexicographical_sort(dict_to_sort): """ sorts a dictionary lexicographycally ""...
Open Graph Description: Här kommer återkoppling på funktionen en student har skapat för att sortera datan i dictionarien korrekt. Här är koden och nedanför finns återkopplingen. def lexicographical_sort(dict_to_sort): """...
X Description: Här kommer återkoppling på funktionen en student har skapat för att sortera datan i dictionarien korrekt. Här är koden och nedanför finns återkopplingen. def lexicographical_sort(dict_to_sort): &qu...
Opengraph URL: https://github.com/dbwebb-se/python/issues/56
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"återkoppling på students sortering i kmom06 analyzer","articleBody":"Här kommer återkoppling på funktionen en student har skapat för att sortera datan i dictionarien korrekt.\r\n\r\nHär är koden och nedanför finns återkopplingen.\r\n\r\n```python\r\ndef lexicographical_sort(dict_to_sort):\r\n \"\"\"\r\n sorts a dictionary lexicographycally\r\n \"\"\"\r\n unique_values_list = []\r\n # create a list of unique values\r\n for value in dict_to_sort.values():\r\n if value in unique_values_list:\r\n pass\r\n else:\r\n unique_values_list.append(value)\r\n # sort values\r\n unique_values_list.sort(reverse=True)\r\n\r\n sorted_dictionary = {}\r\n # create a list for multiple values, add to new dictionary\r\n for _outer_key, outer_value in enumerate(unique_values_list):\r\n multiple_values_for_key = []\r\n for inner_key, inner_value in dict_to_sort.items():\r\n if outer_value == inner_value:\r\n multiple_values_for_key.append(inner_key)\r\n multiple_values_for_key.sort(reverse=True)\r\n sorted_dictionary[outer_value] = tuple(multiple_values_for_key)\r\n \r\n return sorted_dictionary\r\n```\r\n\r\nDu har gjort en väldigt avancerad sorteringslösning. Det tog lite tid för mig att komma på exakt vad du gjorde i den.\r\n\r\nJag har inte så mycket att säga om hur koden är skriven, jag hittade bara ett par små saker.\r\n\r\nFöljande if-sats hade jag ändrat på.\r\n\r\n```python\r\nif value in unique_values_list:\r\n pass\r\nelse:\r\n unique_values_list.append(value)\r\n```\r\n\r\nDet är onödigt med en tom if-sats, då är det bättre att kolla specifikt efter att saken inte stämmer.\r\n\r\n```python\r\nif value not in unique_values_list:\r\n unique_values_list.append(value)\r\n```\r\n\r\n\r\nJag hade döpt om `_outer_key` till bara `_` i for-loopen så det blir tydligare att den inte används.\r\n\r\n```python\r\n for _, outer_value in enumerate(unique_values_list):\r\n```\r\n\r\n\r\nDäremot hade jag gjort sorteringen på ett annat sätt. Det finns ett lättare sätt att få till sortering på flera värden och det är med hjälp av tupler.\r\n\r\nPrecis som när man sorterar en sträng, när man sorterar tupler så jämförs ett elemnt åt gången. Om första elementet är olika jämförs inget mer. Om de är lika går det vidare och jämför nästa element.\r\nMed den kunskapen kan vi bygga en lista som innehåller tupler. Tuplerna innehåller värdena i den ordningen som vi vill att det ska sorteras på. T.ex. `(value, word)`.\r\n\r\nSå om vi har en dictionary där nycklarna är orden och värdet är antalet förekomster av ordet i texten, då vill vi göra om den till en lista med tupler istället. Där första elementet är värdet och andra värdet nyckeln.\r\n\r\n```python\r\nlist_with_tuples = []\r\nfor word, nr in dict_to_sort.items():\r\n list_with_tuples.append((nr, word))\r\n```\r\n\r\nNär vi sen sorterar listan så kommer den i första hand sortera på antalet förekomster i texten och om de är lika sorteras det på ordet istället.\r\n\r\n```python\r\nlist_with_tuples.sort(reverse=True)\r\nsorted_dict = {}\r\nfor nr, word in list_with_tuples:\r\n sorted_dict[word] = nr\r\n```\r\n\r\nOch där har vi en sorterad dictionary på förekomst och bokstavsordning.\r\n\r\n\r\n\r\n","author":{"url":"https://github.com/AndreasArne","@type":"Person","name":"AndreasArne"},"datePublished":"2021-10-20T14:54:45.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/56/python/issues/56"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:c81f2e36-ec7c-c0ff-131e-9848080b4f78 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | AA88:4D0D3:2065480:2927383:69919928 |
| html-safe-nonce | 02be6122b5e884b176bd1971b6c5e464bbd56534f770c9e88497cd4f5314d5e3 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBQTg4OjREMEQzOjIwNjU0ODA6MjkyNzM4Mzo2OTkxOTkyOCIsInZpc2l0b3JfaWQiOiI4NjAwNzA1MDU3NDg5NTI1MDMyIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | 8aa828dd33cc83b761ed07f11509d609ee765e0d80634352a3e663badcbabd57 |
| hovercard-subject-tag | issue:1031510538 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/dbwebb-se/python/56/issue_layout |
| twitter:image | https://opengraph.githubassets.com/fa6e5e10ac7e8b4ef5f71fec9cb280234ecb52f3820d24d9d19d6aaa75fd2fc5/dbwebb-se/python/issues/56 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/fa6e5e10ac7e8b4ef5f71fec9cb280234ecb52f3820d24d9d19d6aaa75fd2fc5/dbwebb-se/python/issues/56 |
| og:image:alt | Här kommer återkoppling på funktionen en student har skapat för att sortera datan i dictionarien korrekt. Här är koden och nedanför finns återkopplingen. def lexicographical_sort(dict_to_sort): """... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | AndreasArne |
| hostname | github.com |
| expected-hostname | github.com |
| None | 42c603b9d642c4a9065a51770f75e5e27132fef0e858607f5c9cb7e422831a7b |
| turbo-cache-control | no-preview |
| go-import | github.com/dbwebb-se/python git https://github.com/dbwebb-se/python.git |
| octolytics-dimension-user_id | 17492868 |
| octolytics-dimension-user_login | dbwebb-se |
| octolytics-dimension-repository_id | 19115004 |
| octolytics-dimension-repository_nwo | dbwebb-se/python |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 19115004 |
| octolytics-dimension-repository_network_root_nwo | dbwebb-se/python |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 848bc6032dcc93a9a7301dcc3f379a72ba13b96e |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width