Почему не выполняется SQL запрос повторно в while?

Question

[willdes]

Писал скрипт, который берет значения из бд и присваивает их состоянию gpio rasbperry pi. Но столкнулся с проблемой, что запрос не выполняется повторно с каждым прохождением цикла.. БД меняю руками из workbanch'a, а значения в скрипте не меняются..
Код:

while True:
	print("OUT1")
	cursors = db.cursor()
	w = cursors.execute("SELECT numb FROM gpio WHERE type='out' and value=1")
	cursors.execute("SELECT numb FROM gpio WHERE type='out' and value=1")
	dataa =  cursors.fetchall()
	for d in range(w):
		rowc = dataa[d]
		idlegs = rowc[0]
		GPIO.output(idlegs, 1)
		print("set OUT", idlegs, "= 1")
	print("OUT0")
	time.sleep(1)	
	cursora = db.cursor()
	j = cursora.execute("SELECT numb FROM gpio WHERE type='out' and value=0")
	cursora.execute("SELECT numb FROM gpio WHERE type='out' and value=0")
	datal =  cursora.fetchall()
	print(j)
	print(datal)
	for l in range(j):
		rowj = datal[l]
		idlegj = rowj[0]
		GPIO.output(idlegj, 0)
		print("set OUT", idlegj, "= 0")
	time.sleep(2)
	print("wait")

Вывод в консоли:

wait
OUT1
set OUT 3 = 1
set OUT 5 = 1
set OUT 11 = 1
set OUT 12 = 1
set OUT 13 = 1
set OUT 15 = 1
set OUT 16 = 1
set OUT 18 = 1
set OUT 19 = 1
set OUT 21 = 1
set OUT 22 = 1
set OUT 23 = 1
OUT0
0
()

Answer 1

[CrazyElf]

А если закомментировать всё, что в первой половине цикла, то вторая половина выполняется?
У меня подозрение, что менять вы меняете, но изменения не коммитятся и видны только в том клиенте, где меняете.
Так то у вас непонятно зачем селекты повторяются и циклы странно организованы, наверное, где-то не то скопировано.
Но на логику выполнения цикла это не должно влиять.

Comments

[willdes]

я не нашел другого способа вытащить данные по порядку..
закомментировал, и еще оду строку закомментировал, которая была выше по скрипту

cursor.execute("UPDATE gpio SET value=1 WHERE type='out'")
	db.commit()

сейчас так

wait
OUT0
set OUT 3 = 0
set OUT 15 = 0

после запуска скрипта поменял в БД значение, а в выводе скрипта ничего не поменялось

[willdes]

все.. разобрался.. нужно было всего то добавить
db.commit()

после
data = cursor.fetchall()

[CrazyElf]

Насчёт циклов, мне кажется, там достаточно использовать что-то типа for rowc in dataa, а первые селекты и циклы по range можно выкинуть. Ну то есть первая часть тела цикла будет выглядеть так:

print("OUT1")
  cursors = db.cursor()
  cursors.execute("SELECT numb FROM gpio WHERE type='out' and value=1")
  dataa =  cursors.fetchall()
  for rowc in dataa:
    idlegs = rowc[0]
    GPIO.output(idlegs, 1)
    print("set OUT", idlegs, "= 1")
  print("OUT0")
  time.sleep(1)

Просто небольшая оптимизация, не люблю лишний код :)

Answer 2

[alexhott]

попробуйте после обработки удалять объекты

Answer 3

[willdes]

Теперь такой вывод

OUT1
set OUT 3 = 1
set OUT 5 = 1
set OUT 11 = 1
set OUT 12 = 1
set OUT 13 = 1
set OUT 15 = 1
set OUT 16 = 1
set OUT 18 = 1
set OUT 19 = 1
set OUT 21 = 1
set OUT 22 = 1
set OUT 23 = 1
OUT0
wait

скрипт исправил

while True:
	print("OUT1")
	cursor = db.cursor()
	w = cursor.execute("SELECT numb FROM gpio WHERE type='out' and value=1")
	cursor.execute("SELECT numb FROM gpio WHERE type='out' and value=1")
	data =  cursor.fetchall()
	for d in range(w):
		row = data[d]
		idleg = row[0]
		GPIO.output(idleg, 1)
		print("set OUT", idleg, "= 1")
		del row
		del idleg
	del data
	del cursor
	del w
	print("OUT0")
	cursor = db.cursor()
	w = cursor.execute("SELECT numb FROM gpio WHERE type='out' and value=0")
	cursor.execute("SELECT numb FROM gpio WHERE type='out' and value=0")
	data =  cursor.fetchall()
	for d in range(w):
		row = data[d]
		idleg = row[0]
		GPIO.output(idleg, 0)
		print("set OUT", idleg, "= 0")
		del row
		del idleg
	del data
	del cursor
	del w
	time.sleep(2)
	print("wait")

в БД поставил значения поля value=0 на 3 и 15 numb