try_3

egorcoders · egorcoders · commit a0b93bbaae80 · 2022-03-20T18:30:23.000+05:00
diff --git a/homework.py b/homework.py
@@ -1,3 +1,4 @@
+import json
 import logging
 import os
 import time
@@ -12,26 +13,35 @@
 
 class ServiceError(Exception):
     """Ошибка отсутствия доступа по заданному эндпойнту."""
-
+    pass
 
 class NetworkError(Exception):
     """Ошибка отсутствия сети."""
+    pass
 
 
 class EndpointError(Exception):
     """Ошибка, если эндпойнт не корректен."""
+    pass
 
 
 class MessageSendingError(Exception):
     """Ошибка отправки сообщения."""
+    pass
 
 
 class GlobalsError(Exception):
     """Ошибка, если есть пустые глобальные переменные."""
+    pass
 
 
 class DataTypeError(Exception):
     """Ошибка, если тип данных не dict."""
+    pass
+
+class ResponseFormatError(Exception):
+    """Ошибка, если формат response не json."""
+    pass
 
 
 CONNECTION_ERROR = '{error}, {url}, {headers}, {params}'
@@ -43,9 +53,10 @@ class DataTypeError(Exception):
 STATUS_IS_CHANGED = '{verdict}, {homework}'
 STATUS_IS_NOT_CHANGED = 'Статус не изменился, нет записей.'
 FAILURE_TO_SEND_MESSAGE = '{error}, {message}'
-GLOBAL_VARIABLE_IS_MISSING = 'Отсутствует глобальная переменная {}'
-GLOBAL_VARIABLE_IS_EMPTY = 'Пустая глобальная переменная {}'
+GLOBAL_VARIABLE_IS_MISSING = 'Отсутствует глобальная переменная'
+GLOBAL_VARIABLE_IS_EMPTY = 'Пустая глобальная переменная'
 MESSAGE_IS_SENT = 'Сообщение {} отправлено'
+FORMAT_NOT_JSON = 'Формат не json {}'
 
 PRACTICUM_TOKEN = os.getenv('PRACTICUM_TOKEN')
 TELEGRAM_TOKEN = os.getenv('TELEGRAM_TOKEN')
@@ -55,7 +66,6 @@ class DataTypeError(Exception):
 ENDPOINT = 'https://practicum.yandex.ru/api/user_api/homework_statuses/'
 HEADERS = {'Authorization': f'OAuth {PRACTICUM_TOKEN}'}
 
-
 HOMEWORK_STATUSES = {
     'approved': 'Работа проверена: ревьюеру всё понравилось. Ура!',
     'reviewing': 'Работа взята на проверку ревьюером.',
@@ -72,19 +82,17 @@ def send_message(bot, message):
             error=error,
             message=message,
         ))
-    else:
-        logging.info(f'Message \'{message}\' is sent')
+    logging.info(f'Message \'{message}\' is sent')
 
 
 def get_api_answer(current_timestamp):
     """Делает запрос к единственному эндпоинту API-сервиса."""
     timestamp = current_timestamp or int(time.time())
     params = {'from_date': timestamp}
-    headers = {'Authorization': f'OAuth {PRACTICUM_TOKEN}'}
-    all_params = dict(url=ENDPOINT, headers=headers, params=params)
+    all_params = dict(url=ENDPOINT, headers=HEADERS, params=params)
     try:
         response = requests.get(**all_params)
-    except telegram.error.TelegramError as error:
+    except requests.exceptions.RequestException as error:
         raise telegram.TelegramError(CONNECTION_ERROR.format(
             error=error,
             **all_params,
@@ -95,27 +103,28 @@ def get_api_answer(current_timestamp):
             response_status=response_status,
             **all_params,
         ))
-    return response.json()
+    try:
+        return response.json()
+    except Exception as error:
+        raise ResponseFormatError(FORMAT_NOT_JSON.format(error))
 
 
 def check_response(response):
     """
     Возвращает домашку, если есть.
     Проверяет валидность её статуса.
     """
-    try:
-        return response['homeworks'][0]
-    except 'code' in response:
+    if 'code' in response:
         raise ServiceError(SERVICE_REJECTION.format(
             code=response.get('code'),
         ))
+    return response['homeworks'][0]
 
 
 def parse_status(homework):
     """Возвращает текст сообщения от ревьюера."""
-    data_type = type(homework)
-    if data_type != dict:
-        raise DataTypeError(WRONG_DATA_TYPE.format(data_type))
+    if not isinstance(homework, dict):
+        raise DataTypeError(WRONG_DATA_TYPE.format(type(homework)))
     homework_name = homework.get('homework_name')
     homework_status = homework.get('status')
 
@@ -129,17 +138,12 @@ def parse_status(homework):
 
 def check_tokens():
     """Проверяет доступность переменных окружения."""
-    for key in (
-        'PRACTICUM_TOKEN',
-        'TELEGRAM_TOKEN',
-        'TELEGRAM_CHAT_ID',
-        'ENDPOINT'
-    ):
-        if globals()[key] is None:
-            logging.error(GLOBAL_VARIABLE_IS_MISSING.format(key))
+    for key in (PRACTICUM_TOKEN, TELEGRAM_TOKEN, TELEGRAM_CHAT_ID, ENDPOINT):
+        if key is None:
+            logging.error(GLOBAL_VARIABLE_IS_MISSING)
             return False
-        elif not globals()[key]:
-            logging.error(GLOBAL_VARIABLE_IS_EMPTY.format(key))
+        if not key:
+            logging.error(GLOBAL_VARIABLE_IS_EMPTY)
             return False
     return True
 
@@ -148,8 +152,6 @@ def main():
     """Основная логика работы бота."""
     if not check_tokens():
         raise GlobalsError('Ошибка глобальной переменной. Смотрите логи.')
-    check_tokens()
-
     bot = telegram.Bot(token=TELEGRAM_TOKEN)
     current_timestamp = int(time.time())
 
@@ -159,20 +161,19 @@ def main():
             homework = check_response(response)
             message = parse_status(homework)
             send_message(bot, message)
-            print(response)
+            logging.info(homework)
             current_timestamp = response.get('current_date')
-            time.sleep(RETRY_TIME)
 
         except IndexError:
             message = 'Статус работы не изменился'
             send_message(bot, message)
-            time.sleep(RETRY_TIME)
+            logging.info('Статус работы не изменился')
         except Exception as error:
             message = f'Сбой в работе программы: {error}'
             send_message(bot, message)
+        finally:
             time.sleep(RETRY_TIME)
-        else:
-            logging.info(MESSAGE_IS_SENT.format(message))
+        logging.info(MESSAGE_IS_SENT.format(message))
 
 
 if __name__ == '__main__':
