Python-сообщество

SharkyEXE · Июнь 10, 2020 19:01:03

Здравствуйте

Пожалуйста, будьте атк любезны, научите, как сделать красиво

Есть код

 if not vti_image:
   elif self.type in (self.IS_MULTICHANNEL, self.AUDIO_STEREO):
      # FIXME. but currently iAudioTrackInfo doesn't provide more information.
      audio = service.audioTracks()
      if audio:
       n = audio.getNumberOfTracks()
       idx = 0
       while idx < n:
        i = audio.getTrackInfo(idx)
        description = i.getDescription()
        if not vti_image:
         if description in ("AC3", "AC-3", "AC3+", "DTS"):
          if self.type == self.IS_MULTICHANNEL:
           return True
          elif self.type == self.AUDIO_STEREO:
           return False
         idx += 1
        if vti_image:
         if 'AC3' in description or 'DTS' in description or 'Dolby Digital' in description:
          return True
         idx += 1
       if self.type == self.IS_MULTICHANNEL:
        return False
       elif self.type == self.AUDIO_STEREO:
        return True
      return False[/b]
if vti_image:
   elif self.type in (self.IS_MULTICHANNEL):
     # FIXME. but currently iAudioTrackInfo doesn't provide more information.
      audio = service.audioTracks()
      if audio:
       n = audio.getNumberOfTracks()
       idx = 0
       while idx < n:
        i = audio.getTrackInfo(idx)
        description = i.getDescription()
        if not vti_image:
         if description in ("AC3", "AC-3", "AC3+", "DTS"):
          if self.type == self.IS_MULTICHANNEL:
           return True
          elif self.type == self.AUDIO_STEREO:
           return False
         idx += 1
        if vti_image:
         if 'AC3' in description or 'DTS' in description or 'Dolby Digital' in description:
          return True
         idx += 1
       if self.type == self.IS_MULTICHANNEL:
        return False
       elif self.type == self.AUDIO_STEREO:
        return True
      return False

В котором 100%но одинаковый код - выделено черным, жирным

Как сделать так, чтобы вместо двух 100%но одинаковых блоков ( ыделено черным, жирным) был один блок (выделено черным, жирным), сверху которого было только в одном различие

if not vti_image:
elif self.type in (self.IS_MULTICHANNEL, self.AUDIO_STEREO):
{блок - выделено черным, жирным}
if vti_image:
elif self.type in (self.IS_MULTICHANNEL):
{блок - выделено черным, жирным}

Спасибо.

Отредактировано FishHook (Июнь 10, 2020 19:11:50)

Прикреплённый файлы:
1.py (1,8 KБ)

FishHook · Июнь 10, 2020 19:12:35

SharkyEXE
Как сделать так, чтобы вместо двух 100%но одинаковых блоков ( ыделено черным, жирным) был один блок

вы знакомы с концепцией функций?

SharkyEXE · Июнь 10, 2020 19:16:06

FishHook

Здравствуйте

Нет, в общем я не супер мега программист, есть готовый код, хочется красивого решения, с мозгами, чем у меня, методом тыка, как смог

doza_and · Июнь 10, 2020 22:03:55

В ваш код нельзя упростить, просто потому что он синтаксически неверен. Нельзя помочь написать код, который неизвестно что делает.
Допустим было:

 if not vti_image:
     actions1()
elif self.type in (self.IS_MULTICHANNEL, self.AUDIO_STEREO):
     common_actions()
if vti_image:
     actions2()
elif self.type in (self.IS_MULTICHANNEL):
     common_actions()
return True # добавил тк логическая проблема - что возвращать если ничего не выполнено

Его можно преобразовать так:

 if vti_image:
     actions2()
else:
     actions1()
if self.type in (self.IS_MULTICHANNEL, self.AUDIO_STEREO):
    common_actions()
return True

Надеюсь не ошибся в хитросплетениях.

SharkyEXE · Июнь 10, 2020 22:15:28

doza_and

Здравствуйте

Пожалуйста, будьте так любезны, на моем коде напишите, как надо

Повторюсь, я не супер программист,м етодом чего-то въехать могу, просто я уверен, что по умному в питоне можно сделать, чтобы два раза один и тот же код не писать, его вынести в секцию, label, goto типа

xam1816 · Июнь 10, 2020 23:14:07

а у вас этот код вообще работает? смотря на ваш код у него табуляция сбита и почему-то в в блоке if сразу elif,как буд-то кто-то так защищался от копирования.

а так примерно вот по вашему выше коду(не стал вникать о чем он)

 def beautiful_code():
	if self.type in (self.IS_MULTICHANNEL, self.AUDIO_STEREO):
	# FIXME. but currently iAudioTrackInfo doesn't provide more information.
	audio = service.audioTracks()
	if audio:
		n = audio.getNumberOfTracks()
		idx = 0
		while idx < n:
			i = audio.getTrackInfo(idx)
			description = i.getDescription()
			if not vti_image:
				if description in ("AC3", "AC-3", "AC3+", "DTS"):
					if self.type == self.IS_MULTICHANNEL:
						return True
					elif self.type == self.AUDIO_STEREO:
						return False
				idx += 1
			if vti_image:
				if 'AC3' in description or 'DTS' in description or 'Dolby Digital' in description:
					return True
				idx += 1
		if self.type == self.IS_MULTICHANNEL:
			return False
		elif self.type == self.AUDIO_STEREO:
			return True
	return False[ / b]
if not vti_image:
	beautiful_code()
if vti_image:
	beautiful_code()

xam1816 · Июнь 10, 2020 23:23:14

xam1816
if not vti_image:
beautiful_code()
if vti_image:
beautiful_code()

вот этот блок вообще не логичен,и не нужен,
типа
если нет картинки:
выполнить код
если есть картинка:
выполнить этот же код

либо этот код писался для показухи,как-будто много работы сделано,много написано,

Отредактировано xam1816 (Июнь 10, 2020 23:32:46)

Vladimirv · Июнь 11, 2020 20:39:10

Lazy метод:
С начала ищите, каким редактором пользуется

SharkyEXE
супер мега программист

будут разночтения.
Потом ставите плагин PEP8 для понравившегося редактора кода и немного читаете что это.
Дальше используете

SharkyEXE
методом тыка

для применения PEP8 к вашему коду и будет вам

SharkyEXE
красиво

Python-сообщество

Уведомления

#1 Июнь 10, 2020 19:01:03

Красивое условие

#2 Июнь 10, 2020 19:12:35

Красивое условие

#3 Июнь 10, 2020 19:16:06

Красивое условие

#4 Июнь 10, 2020 22:03:55

Красивое условие

#5 Июнь 10, 2020 22:15:28

Красивое условие

#6 Июнь 10, 2020 23:14:07

Красивое условие

#7 Июнь 10, 2020 23:23:14

Красивое условие

#8 Июнь 11, 2020 20:39:10

Красивое условие

Board footer