1

надо закинуть возвращаемые значения двух первых функций в 3 и объединить их. понимаю, что решение где-то на поверхности, но никак не могу понять, как это сделать аккуратно и красиво. буду рад любой помощи, заранее спасибо этот код без 3 функции с объединением

result: {1: 3.5, 2: 4.67}

lst_movie: 
[{'movieId': 1, 'year': '2018', 'title': 'Bungo Stray Dogs: Dead Apple (2018)', 'genres': 'Action|Animation', 'rating': None},
 {'movieId': 2, 'year': '1991', 'title': 'Andrew Dice Clay: Dice Rules (1991)', 'genres': 'Comedy', 'rating': None}] 

в result закидываю значения по такой схеме {movieId: rating} и на место rating в lst_movie(который None) нужно поставить rating из result в соответствии с movieId. если в result нет rating к какому-то movieId, то остается None

import csv
import re
from collections import defaultdict as df


def csv_movie(file_obj):
    reader = csv.reader(file_obj)
    next(f_obj)
    lst_movie = []
    for row in reader:
        dic = dict()
        dic['movieId'] = int(row[0])
        dic['year'] = ''.join(re.findall('(?<=\()[0-9]{4}(?=\))', row[1]))
        dic['title'] = row[1]
        dic['genres'] = row[2]
        dic['rating'] = None
        lst_movie.append(dic)
    return lst_movie


def csv_rating(file):
    reader = csv.reader(file)
    lst = []
    next(obj)                #     8 строка    
    for row in reader:
        dic = dict()
        dic[int(row[1])] = float(row[2])
        lst.append(dic)
    d = df(list)
    for i in lst:
        for j in i.keys():
            d[j].append(i[j])
    result = {k: round(sum(v) / len(v), 2) for k, v in d.items()}
    return result


if __name__ == "__main__":
    csv_movies = "movies.csv"
    csv_ratings = "ratings.csv"
    with open(csv_movies, "r") as f_obj:
        csv_movie(f_obj)
    with open(csv_ratings, "r") as obj:
        csv_rating(obj)

UPD

добавил как вы посоветовали

def fill_movie_rating(lst_movie, result):
    for movie in lst_movie:
        movie['rating'] = result.get(movie['movieId'])
    print(lst_movie)


if __name__ == "__main__":
    csv_movies = "movies.csv"
    csv_ratings = "ratings0.csv"
    with open(csv_movies, "r") as f_obj:
        csv_movie(f_obj)
        lst_movie = csv_movie(f_obj)      #   48 строка
    with open(csv_ratings, "r") as obj:
        csv_rating(obj)
        result = csv_rating(obj)

выдало ошибку:

Traceback (most recent call last):
  line 48, in <module>
    lst_movie = csv_movie(f_obj)
  line 8, in csv_movie
    next(f_obj)
StopIteration

если удалить из main, то ничего не выводится

3
  • 1
    Вы лучше покажите как выглядят словари (хотя бы по нескольку элементов из них) и что вы хотите из них получить. Что нам толку от кода генерации словарей, если у нас нет ваших .csv файлов.
    – CrazyElf
    7 июл 2021 в 14:10
  • добавил, спасибо
    – Platon
    7 июл 2021 в 14:22
  • Разобрались? Не надо дважды вызывать одну и ту же функцию подряд в вашем случае, тем более это не работает. Логика такая: открыли файл, прочитали, закинули в функцию прочитанное, return которой передали переменной. И так всегда. Функцию fill_movie_rating так же можете вызывать в main после того как вы получили значения для переменных lst_movie, result
    – Dmitry
    8 июл 2021 в 9:48

2 ответа 2

1

В результате исполнения двух вышеописанных функции вы имеете двe переменных. Одна из них lst_movie (тип list), которая содержит общие данные о фильмах и вторая result (тип dict), в котором хранятся рейтинги фильмов.

Входные данные, которые будут использованы:

lst_movie = [{'movieId': 193587, 'year': '2018', 'title': 'Bungo Stray Dogs: Dead Apple (2018)', 'genres': 'Action|Animation', 'rating': None}, {'movieId': 193609, 'year': '1991', 'title': 'Andrew Dice Clay: Dice Rules (1991)', 'genres': 'Comedy', 'rating': None}]

result = {193587: 3.5, 2: 4.67, 3: 4.33}

что будем делать? Брать значение movieId из result (который есть ключ), искать его в lst_movie и определять рейтинг, который соответствует movieId

>>> for i in result:
...   for k in lst_movie:
...     if i == k["movieId"]:
...       k["rating"] = result[i] 

И в итоге имеем:

>>> print(lst_movie)
[{'movieId': 193587, 'year': '2018', 'title': 'Bungo Stray Dogs: Dead Apple (2018)', 'genres': 'Action|Animation', 'rating': 3.5}, {'movieId': 193609, 'year': '1991', 'title': 'Andrew Dice Clay: Dice Rules (1991)', 'genres': 'Comedy', 'rating': None}]

т.е. рейтинг фильма с 'movieId': 193587 изменился на 3.5

UPD сама функция

def union_dict(result:dict, lst_movie:list) -> list:
    for i in result:
        for k in lst_movie:
           if i == k["movieId"]:
               k["rating"] = result[i]
    return lst_movie

вызов

union_dict(result, lst_movie)

Я бы так же обратил внимание на ответ от @CrazyElf. Он объективно лучше

3
  • а как возможно это решение закинуть в функцию? выполняя это задание, я обнаружил свою слабость в функциях, и основная моя проблема - это переменные lst_movie и result, которые локальные. я прошурстил не один сайт с описаниями работы с функциями, но ничего толкового не нашел. буду крайне признателен, если вы поможете мне
    – Platon
    7 июл 2021 в 15:00
  • 1
    Ваш ответ в целом подробнее, но вот реально имея готовый словарь нет смысла делать цикл перебора его ключей, чтобы найти нужный, когда можно сразу взять значение по ключу.
    – CrazyElf
    7 июл 2021 в 15:12
  • @CrazyElf абсолютно согласен, при большом количестве данных слишком много не нужных итераций, да и при малом тоже.
    – Dmitry
    7 июл 2021 в 15:15
1

Если в result точно есть все movieId из lst_movie, то можно так:

def fill_movie_rating(lst_movie, result):
    for movie in lst_movie:
        movie['rating'] = result[movie['movieId']]

А если там есть не все movieId, то нужно делать через get, чтобы не выпало исключение:

def fill_movie_rating(lst_movie, result):
    for movie in lst_movie:
        movie['rating'] = result.get(movie['movieId'])
9
  • ну да, так естественно лучше. Одна вещь только про вариант номер 2: почему вы указываете вторым аргументом None? get же, все равно, если не найдет вернет его без явного указания. Или я ошибаюсь?
    – Dmitry
    7 июл 2021 в 14:58
  • 1
    @Dmitry Да, вы правы. Просто указал такое же значение, как исходно было в словаре, на самом деле не знаю, что автору вопроса нужно там чтобы было
    – CrazyElf
    7 июл 2021 в 15:01
  • а как возможно это решение закинуть в функцию? выполняя это задание, я обнаружил свою слабость в функциях, и основная моя проблема - это переменные lst_movie и result, которые локальные. я прошурстил не один сайт с описаниями работы с функциями, но ничего толкового не нашел. буду крайне признателен, если вы поможете мне
    – Platon
    7 июл 2021 в 15:06
  • @Platon Достаточно написать заголовок функции. Добавил.
    – CrazyElf
    7 июл 2021 в 15:10
  • @CrazyElf спасибо за ответ, но я немного про другое: переменные result и lst_movie находятся внутри функций и "не видны" остальному коду. как их передать в def fill_movie_rating?
    – Platon
    7 июл 2021 в 15:16

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.