Como eu faço para melhorar este código?
import random
import string
import requests
from django.core.management.base import BaseCommand, CommandError
from django.core.exceptions import ValidationError
from optparse import make_option
from core.models import Movie
class Command(BaseCommand):
help = 'Faz o crawler numa api de filmes e retorna os dados.\n \
Uso: ./manage.py initdata\n \
ou: ./manage.py initdata -m 20\n \
ou: ./manage.py initdata -m 20 -y 2015'
option_list = BaseCommand.option_list + (
make_option('--movies', '-m',
dest='movies',
default=10,
help='Define a quantidade de filmes a ser inserido.'),
make_option('--year', '-y',
dest='year',
default=None,
help='Define o ano de lançamento do filme.'),
)
def get_html(self, year):
'''
Le os dados na api http://www.omdbapi.com/ de forma aleatoria
e escolhe um filme buscando por 2 letras
'''
# escolhe uma letra aleatoriamente
c = ''.join(random.choice(string.ascii_lowercase)for _ in range(2))
# se não for definido o ano, então escolhe um randomicamente
if year is None and isinstance(year, int):
year = str(random.randint(1950, 2015))
url = 'http://www.omdbapi.com/?t=' + c + \
'*&y=' + str(year) + '&plot=short&r=json'
return requests.get(url).json()
def save(self, title, year, released, director, actors, poster, imdbRating, imdbVotes, imdbID):
''' transforma "imdbVotes" em numero inteiro '''
if imdbVotes == "'N/A'":
imdbVotes = 0
else:
imdbVotes = imdbVotes.replace(',', '')
try:
''' Salva os dados '''
Movie.objects.create(
title=title,
year=year,
released=released,
director=director,
actors=actors,
poster=poster,
imdbRating=imdbRating,
imdbVotes=imdbVotes,
imdbID=imdbID,
)
except ValidationError:
print('O objeto não foi salvo.')
def handle(self, movies, year, **options):
''' se "movies" não for nulo, transforma em inteiro '''
if movies is not None:
movies = int(movies)
''' busca os filmes n vezes, a partir da variavel "movies" '''
for i in range(movies):
h = self.get_html(year)
j = 1 # contador
''' Se a resposta for falsa, então busca outro filme '''
while h['Response'] == 'False' and j < 100:
h = self.get_html(year)
print('Tentanto %d vezes' % j)
j += 1
''' se "year" não for nulo, transforma em inteiro '''
if "–" in h['Year']:
h['Year'] = year
# salva
self.save(h['Title'], h['Year'], h['Released'], h['Director'], h['Actors'],
h['Poster'], h['imdbRating'], h['imdbVotes'], h['imdbID'])
print(i + 1, h['Year'], h['Title'], h['Released'], h['Director'],
h['Actors'], h['Poster'], h['imdbRating'], h['imdbVotes'], h['imdbID'])
# print('Foram salvos %d filmes' % movies)
Por exemplo,
def save(self, title, year, released, director, actors, poster, imdbRating, imdbVotes, imdbID):
Entrar somente com um dicionário de argumentos.
title=title,
year=year,
released=released,
director=director,
actors=actors,
poster=poster,
imdbRating=imdbRating,
imdbVotes=imdbVotes,
imdbID=imdbID,
Será que dá pra transforma em algo menor, um dicionário talvez.
self.save(h['Title'], h['Year'], h['Released'], h['Director'], h['Actors'],
h['Poster'], h['imdbRating'], h['imdbVotes'], h['imdbID'])
Também poderia ser otimizado?
try...except
. O ideal seria informar a razão do erro, claro, mas nesse caso está óbvio que é um erro de validação, e se a intenção do AP for que o programa não pare após encontrar um dado inválido (i.e. que ele ignore o inválido e continue processando os demais) então não há muita saída a não ser usartry...except
. Da mesma forma,bulk_create
seria o ideal num cenário de "tudo ou nada", mas da mesma forma não daria pra usar (acho) caso se quisesse processar os válidos e ignorar os inválidos.try
porque não tratei todos os erros provenientes dos dados da api.