Можно ли визуально сократить кусок кода?

Question

[cyberkilla]

Не суть важно - зачем, но требуется визуально сократить кусок кода, конкретно вот эту часть:



Перепробовала все, что знала и могла, но ничего не вышло. Может, у кого-нибудь найдется пара идей. Заранее спасибо!

from __future__ import division
import mpmath
import sympy
from sympy import *

arr1 = (open("f.txt")).read()
mark = 0
kits = set()
kits.add(arr1)


for i in range(1,9) :
    for i in range(len(kits)) :
        arr1 = list(kits)[i]
        for j in range(1,9) :
            array = arr1
            array = array.replace(str(j), "0")
            kits.add(array)

for i in range(len(kits)) :
    array = list(kits)[i]
    x=symbols('x1:9', positive=True)
    x1,x2,x3,x4,x5,x6,x7,x8=x
    mark=0
   
    for j in range(1,9):
      if array[j-1] == "0" :
        if j == 1:
          x1 = 0
        if j == 2  :
          x2 = 0
        if j == 3:
          x3 = 0
        if j == 4:
          x4 = 0
        if j == 5:
          x5 = 0
        if j == 6 :
          x6 = 0
        if j == 7:
          x7 = 0
        if j == 8 :
          x8 = 0
        mark+=1
        
    if mark >= 3 :
      eq=(3*x1 - 11*x2 + 11*x3 + 4*x4 - 5*x5 + 6*x6 - 2*x7 + 8*x8 - 112, -2*x1 + 0*x2 - 50/11*x3 + 11*x4 + 12*x5 - 6*x6 -1*x7 + 3*x8 - 113,-2*x1 + 4*x2 + 6*x3 + 7*x4 + 12*x5 -11*x6 -1*x7 + 13*x8 - 109,-1*x1 + 3*x2 - 11/4*x3 - 4*x4 + 5*x5 - 1*x6 + 12*x7 - 1*x8 - 132,2*x1 + 5*x2 + 7*x3 -11*x4 +6*x5 +1*x6 + 1*x7 - 2*x8 - 211)
      solver = sympy.solve(eq, x1,x2,x3,x4,x5,x6,x7,x8)
      if str(type(solver)) != "<class 'list'>" :
        print('\n Точка с координатами: '+ (str(solver)[1:-1]), end =', если ')
        xs=(x1,x2,x3,x4,x5,x6,x7,x8)
        for i in range(8):
          if xs[i]==0:
            print('x'+str(i+1)+'=0', end = ', ')
        print('\n ______________________________________________________')

Comments

[Сергей П]

отформатируйте код специальным тегом. Неужели вы не ведите, что он не читаемый у вас?

[javedimka]

Не распаковывай.
x1,x2,x3,x4,x5,x6,x7,x8=x

->
x = list(x)
x[j-1] = 0
x1, x2, x3, x4, x5, x6, x7, x8 = x

[cyberkilla]

Сергей Паньков, извиняюсь! Навела красоту!

[cyberkilla]

javedimka, что-то вроде такого?

[Алмаз]

а как если exec ом

for j in range(1,9):
    if array[j-1] == "0" :
        if j in range(1,9):
            exec('x{}=0'.format(j))
        mark+=1

[Алан Гибизов]

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

Вот это:
check = (0, 1, 2, 3, 4, 5, 6, 7)

мне кажется лучше записать как-то типа:
check = tuple(range(8))

Это у тебя их сейчас 8 шт., а если бы 256?

[Алан Гибизов]

И вообще, зачем эти x1, x2 и т.п., когда можно было бы везде вместо них применять x[0], x[1] ... x[7]? Опять же, сейчас их 8. А ну как будет 64?

[iMaximus]

У меня глаз задергался от этого п.....а
for j in range(1,9):
if array[j-1] == "0" :
if j == 1:
x1 = 0
if j == 2 :
x2 = 0
if j == 3:
x3 = 0
if j == 4:
x4 = 0
if j == 5:
x5 = 0
if j == 6 :
x6 = 0
if j == 7:
x7 = 0
if j == 8 :
x8 = 0

[javedimka]

cyberkilla, Ага, только зачем check непонятно, можно получать нужный индекс через j-1.
Если для красоты, то это не особо красиво

Answer 1

[Андрей Соболев]

Добрый день. Почитайте про eval
https://pythonz.net/references/named/eval/
Это как раз ваш случай.

В целом если у вас возникает подобная ситуация, вы точно что то делаете не так.