0

Есть код

from PIL import Image

image = Image.open("monro.jpg")
red, green, blue, = image.split()
red = red.save("red.jpg")
green = green.save("green.jpg")
blue = blue.save("blue.jpg")
image = Image.open("red.jpg")
coordinates = (200, 0, image.width, image.height)
coordinates2 = (100, 0, 596, 522)
red1 = image.crop(coordinates) .save("red1.jpg")
red2 = image.crop(coordinates2) .save("red2.jpg")
red1 = Image.open("red1.jpg")
red2 = Image.open("red2.jpg")
red = Image.blend(red1, red2, 0.5) .save("red.jpg")

Как можно сократить код?

4
  • 1
    Зачем? дзэн питона: явное лучше не явного. сократить выйдет на 1-3 строки ухудшив читаемость
    – ganz
    14 апр в 12:19
  • @ganzv Потому что ментор сказал надо все в переменные засунуть. Мне тоже так нравится больше, но раз ментор сказал значит надо.
    – Илья
    14 апр в 12:32
  • нуу тогда поговорить с ментором о модуле this DDD или поместить red,green,blue в словарь, обойти его циклом, убрать image = Image.open("red.jpg") тк объект имеется под именм red,убрать red12 = Image.open("red1.jpg") так как переоткрытие не имеет смысла ибо ред12 уже объет пиллоу. ну и сохранения изображений необходимы или сделаны случайно?
    – ganz
    14 апр в 12:49
  • @ganz изображения по шаблу....они не нужны. На входе только аватар
    – Илья
    14 апр в 13:01

1 ответ 1

0
image = Image.open("monro.jpg")
image,, = image.split() #выкидываем блю и грин тк далее не используются

#радикально: нужна только эта строка
red = Image.blend(Image.crop((200, 0, image.width, image.height)), image.crop((100, 0, 596, 522)) , 0.5) .save("red.jpg")

#red = red.save("red.jpg") 
#green = green.save("green.jpg")
#blue = blue.save("blue.jpg")

#image = Image.open("red.jpg") #объект уже имеется
coordinates = (200, 0, image.width, image.height)
#coordinates2 = (100, 0, 596, 522)#переменная ради переменной
red1 = image.crop(coordinates) #.save("red1.jpg") #зачем сохранять промежуточный результат? #можно поступить как с координатес2
red2 = image.crop((100, 0, 596, 522)) #.save("red2.jpg")
#red1 = Image.open("red1.jpg") лишнее переоткрытие
#red2 = Image.open("red2.jpg")
red = Image.blend(red1, red2, 0.5) .save("red.jpg")


2
  • Спасибо за обратную связь и помощь!
    – Илья
    14 апр в 13:03
  • 'image = Image.open("monro.jpg") image,, = image.split() red = Image.blend(Image.crop((200, 0, image.width, image.height)), image.crop((100, 0, 596, 522)) , 0.5) .save("red.jpg")' Ошибка: AttributeError: 'tuple' object has no attribute 'crop'
    – Илья
    15 апр в 7:08

Ваш ответ

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

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