7
def addToDb(list):
    if type(list[0]) == str:
        if type(list[1]) == str:
            if type(list[2]) == str:
                if type(list[3]) == int:
                    cursor.execute("""INSERT INTO clients VALUES (?, ?, ?, ?)""", (list[0], list[1], list[2], list[3]))
                    conn.commit()
                    print('Выполнена запись в базу данных')
                else:
                    print('Аргумент number не является Int ')
            else:
                print('Аргумент time не является Str')
        else:
            print('Аргумент ip не является Str')
    else:
        print('Аргумент status не является Str')
7
  • Осваиваю питон, не хотелось бы 'быдлокодить' начинать :)
    – Stan
    17 авг 2017 в 10:38
  • Незачем делать много вложенных ифов. Проверяйте переменные по одной и если они некорректны — пишите в лог причину и прерывайте выполнение (с помощью выброса исключения, return или exit(1).
    – Nick Volynkin
    17 авг 2017 в 10:53
  • 1
    Подумайте также, а здесь ли нужно проверять тип данных. Возможно, ошибку можно заметить раньше, там где вы их получаете. Там с этим можно будет сделать что-то конструктивное, например переспросить ввод из консоли, переотправить запрос и т.п.
    – Nick Volynkin
    17 авг 2017 в 10:55
  • 1
    Другими словами, пересчитывайте сдачу, не отходя от кассы ))
    – Nick Volynkin
    17 авг 2017 в 10:56
  • 1
    По стилю кода: никогда не используйте названия типов (list, str и т.д.) в качестве имен переменных.
    – insolor
    17 авг 2017 в 15:44

2 ответа 2

12
In [101]: lst = ['a','b','c',1]

проверка на "строки" первых трех элементов:

In [104]: all(isinstance(x, str) for x in lst[:3])
Out[104]: True

проверка четвертого элемента на "int":

In [105]: isinstance(lst[3], int)
Out[105]: True

Немного улучшенный ответ:

In [135]: lst = [1,'b',2,'x']

In [136]: chk_type = (
     ...:   ['status', str],
     ...:   ['ip', str],
     ...:   ['time', str],
     ...:   ['number', int]
     ...: )
     ...:

In [137]: for i,x in enumerate(chk_type):
     ...:     if not isinstance(lst[i], x[1]):
     ...:         print('parameter [{}] has wrong type (should be [{}])!'.format(x[0], x[1].__name__))
     ...:
parameter [status] has wrong type (should be [str])!
parameter [time] has wrong type (should be [str])!
parameter [number] has wrong type (should be [int])!
8
  • Круто, но потерялись сообщения о том, где конкретно ошибка.
    – Nick Volynkin
    17 авг 2017 в 10:59
  • @NickVolynkin, да, действительно, спасибо! Сейчас попробую исправить... 17 авг 2017 в 11:00
  • Мне кажется, надо сначала у автора выяснить контекст. Я вот не уверен, что там будет кому читать эти сообщения. А если будет — то что проверку нужно делать не сразу при получении, а только перед записью в базу.
    – Nick Volynkin
    17 авг 2017 в 11:02
  • @NickVolynkin, да по поводу проверки "не отходя от кассы" - я полностью согласен. Проверять лучше сразу при вводе/получении значения... 17 авг 2017 в 11:04
  • Да эта функция будет вызываться сервером при получении данных от клиента. принты для меня чтоб меньше ошибок делать.
    – Stan
    17 авг 2017 в 11:09
6

Явную проверку типов во время исполнения редко в Питоне нужно делать. Как правило это указывает на ошибку в дизайне кода.

Как валидировать входные данные зависит от задачи. По известным спискам значений (whitelisting), схеме. К примеру, cerberus.

Часто можно упростить код до:

def add_to_db(*args):
    with conn:
        conn.execute("INSERT INTO clients VALUES (?, ?, ?, ?)", args)

Если не произойдёт исключения, то происходит commit.

Для проверки типов аргументов, можно использовать аннотации типов, читаемые статическим анализатором типа mypy и/или декоратор, чтобы во время исполнения аргументы валидировать (не тестировано):

from typing import NamedTuple
import enforce  # $ pip install enforce

class Client(NamedTuple):
    status : str
    ip : str
    time : str
    number : int 

@enforce.runtime_validation
def add_client(client : Client) -> None:
    with conn:
        conn.execute("INSERT INTO clients VALUES (?, ?, ?, ?)", client)

Связанный вопрос: How to use Python decorators to check function arguments?

Если вы такой код часто пишите, то попробуйте ORM. Вот пример кода для SQLAlchemy:

my_client = MyClient(status, ip, time, number)
session.add(my_client)
session.commit()

SQLAlchemy по умолчанию опирается на базу данных как на самый эффективный и надёжный источник о корректности данных, но вы можете и свои проверки добавить How can I verify Column data types in the SQLAlchemy ORM?

Ваш ответ

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

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