Как реализовать на python передачу «указателя» изображения opencv в конструктор другого класса?

Question

[Nik87]

Здравствуйте. Как реализовать на питоне такую схему?
p.s Сейчас передача параметра (img) в конструктор класса A идет по ссылке, но при вызове метода painter нет изображения, т.к в переменную img класса В перезаписывается другая ссылка

class A:
  def __init__(self, img):
   self.img = img
  def show(self):
   cv2.imshow(self.img)
   cv2.waitKey(33)

class B:
  def __init__(self):
   ...
   self.img = np.array((1,1,1),np.uin8)
   self.painter = A(self.img)

  def getImg(self):
    whille True:
       _, self.img = self.cap.read()
      self.painter.show()

Comments

[Nik87]

нашел решение.. Если никто не предложит лучшего напишу свое

[Nik87]

если вызвать функцию id(self.img) в getImg то для каждого цикла cv2.read() выделяет новую память под self.img. Видимо это особенности питона и его управления памятью

Answer 1

[Andy_U]

После обсуждения исправил ответ:

from typing import Any

import cv2
import numpy


class A:

    @staticmethod
    def show(img: numpy.ndarray) -> None:
        cv2.imshow('image', img)
        cv2.waitKey(3300)


class B:

    def __init__(self, painter: Any):
        self.painter = painter

    def getimg(self) -> None:
        while True:
            self.painter.show(cv2.imread('Capture.JPG'))


if __name__ == '__main__':
    b = B(A())
    b.getimg()

Comments

[Andy_U]

Nik87,

Вы вопрос то читали?

Да, прочитал, ниже 3 варианта формальных ответов на ваш вопрос:

Как реализовать на python передачу «указателя» изображения opencv в конструктор другого класса?

a) вызвать конструктор этого класса, т.е. создать новый экземпляр класса;
б) в Питоне нет указателей, читайте документацию;
в) изучайте design patterns (такой толстый намек, что то, чего вы хотете добиться, делается совсем не так).

Вам стало легче? Далее, ваш код даже после исправления кучи синтаксических ошибок не полон. Его до сих пор нельзя вставить в IDE и запустить.

Зачем писать что попало?

Ваш вопрос выглядит для профессионала так: я тут написал фигню, с кучей синтаксических ошибок (в дискуссии выше они перечислены) а она не работает, как исправить? Ответ - никак не исправить. Надо сделать по другому. Ну и, похоже, вы не поняли ответа и не пока не способны его доработать под неизвестные нам нужды.

Мне не нужно передавать в метод show img!!!

Вам не нужно (и невозможно без пересоздания экземпляра класса) передавать указатель в конструктор.
Вы бы лучше сформулировали проблему словами, без упоминания питона, которого вы не знаете. Разве не вынести из класса B функциональность вывода картинок на экран?

И зачем мне статический метод???

А зачем вам "обычный"? Вот и мне Pycharm на это намекнул. Вот если вы захотите параметризовать класс A (ну, например, задавать задержку между кадрами в конструкторе класса А, а использовать в A.show(), тогда да - надо декоратор будет убрать, новый коммит в репозиторий сделать.

[Andy_U]

Большой, начиная c ассемблера IBM 360/370 :) Нет, вру, "Аналитик" на МИР-1 и МИР-2 раньше. Дальше долго перечислять. И некая репутация у меня на этом сайте есть. В отличие от вас, задавшего на этом сайте один криво сформулированный вопрос, демонстрирующий полное незнание и непонимание Питона (да и вообще ООП). Про PEP8 молчу, это мои глаза вытекают. Не в первый и не в последний раз. Не нравятся местные правила и обычаи, извините, никто под вас подстраиваться не будет. Скажите еще спасибо, что тут местные модераторы не потоптались по вам. Позвать, что-ли? Или сами завтра в гости зайдут?

Ладно, дело не в этом, а в том, что я не увидел от вас ни одного квалифицированного возражения на мои утверждения или ответа на уточняющие вопросы. Одни брызги. Ну, продемонстрируйте ваш профессионализм?

[Andy_U]

Nik87, Опять ни одного квалифицированного возражения. Вы даже не можете объяснить, чего хотите. Ведь явно что-то иное, чем A(img)? Что?

P.S. А процент - это когда на примерно такие вопросы, как ваш, отвечаю.

[Andy_U]

Nik87, Да не работает, как и ожидалось, ваш вариант (чтобы не возиться с картинками, я в get_img просто присваиваю полю imgB каждый раз новый numpy.array):

import numpy as np


class A:

    def __init__(self, img):
        self.imgA = img

    def show(self):
        print('id(self.imgA) = ', id(self.imgA), 'self.imgA =', self.imgA)


class B:

    def __init__(self):
        self.imgB = np.array((1, 1, 1), np.uint8)
        self.painter = A(self.imgB)

    def get_img(self):
        for i in range(10):
            self.imgB = np.array((i, i+1, i+2), np.uint8)
            print('id(self.imgB) = ', id(self.imgB), 'self.imgB =', self.imgB)
            self.painter.show()


if __name__ == '__main__':
    b = B()
    b.get_img()

Смотрите на вывод:

"C:\Program Files\Python38\python.exe" D:/Documents/PycharmProjects/tstcv/main.py
id(self.imgB) =  2253578859664 self.imgB = [0 1 2]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253597315440 self.imgB = [1 2 3]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253578859664 self.imgB = [2 3 4]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253597315440 self.imgB = [3 4 5]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253578859664 self.imgB = [4 5 6]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253597315440 self.imgB = [5 6 7]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253578859664 self.imgB = [6 7 8]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253597315440 self.imgB = [7 8 9]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253578859664 self.imgB = [ 8  9 10]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]
id(self.imgB) =  2253597315440 self.imgB = [ 9 10 11]
id(self.imgA) =  2253578837808 self.imgA = [1 1 1]

Process finished with exit code 0

Ну никак изменение imgB не сказывается на imgA... Потому что после каждого присваивания в imgB он будет ссылаться на новый объект в памяти (id - это типа ссылки на адрес).

P.S. Вы ведь свой код не запускали, судя по использованию np.uin8 - правильно uint8.

[Andy_U]

Nik87, В вашем предыдущем сообщении вы сообщили, что решили проблему и привели код ее "решения" (а зачем бы вы иначе этот код приводили?). Вам было показано, что оно нерабочее, даже после исправления синтаксической ошибки. Блин, программист-теоретик. Инсталлируйте вы наконец себе Питон на компьютер... Хотелось бы еще обратить ваше внимание, что с автором первого ответа мы общий язык нашли. Только он поленился полный рабочий код написать, а я написал. Формально с нуля, а фактически повторил.

P.S. На всякий случай, оскорблениями тут занимаесь вы. Цитирую предыдущее ваше сообщение, чтобы ходы записать.

Nik87:

что ж вы такой тупой то

P.P.S. На всякий случай, то, что вы хотите, с помощью некоего грязного трюка сделать можно, Но только это ни одно code review не пройдет. И, самое главное, быстрее и/или экономнее по памяти тоже не будет.