Блог

обновленный код Я новичок в Python и пытаюсь сделать свой код более эффективным. В настоящее время я переношу значения датчиков в базу данных Firebase в реальном времени в разные поля, но код повторяется. Могу ли я в любом случае уменьшить размер скрипта, используя цикл или альтернативный метод.

 def getSound():
    sound_value = grovepi.analogRead(sound_sensor)
    result = firebase.patch('/SoundValues/Monday/'{'Sound1':int(sound_value)})
    print("the sound value is%d" %sound_value)
    time.sleep(1)

def getSound1():
    sound_value = grovepi.analogRead(sound_sensor)
    result = firebase.patch('/SoundValues/Monday/'{'Sound2':int(sound_value)})
    print("the sound value is%d" %sound_value)
    time.sleep(1)
  

Любая помощь будет с благодарностью принята 🙂

Измените функцию так, чтобы она принимала номер звука в качестве параметра:

 def getSound(number):
    sound_value = grovepi.analogRead(sound_sensor)
    key = 'Sound%d' % number # this will be 'Sound1', 'Sound2', etc.
    result = firebase.patch('/SoundValues/Monday/'{key :int(sound_value)})
    print("the sound value is%d" %sound_value)
    time.sleep(1)
  

Затем вы должны вызвать функцию как getSound(1) getSound(2) , и т.д.

Я бы всегда смотрел на то, что отличается между getSound() и getSound1(). Похоже, что функции имеют точно такой же код, за исключением этой строки:

 result = firebase.patch('/SoundValues/Monday/'{'Sound1':int(sound_value)})
  

Таким образом, вы можете написать одну функцию со всем повторяющимся кодом и использовать ‘sound_spec’ в качестве параметра вашей функции. Это будет выглядеть примерно так:

 sound_value = grovepi.analogRead(sound_sensor)

def getSound(sound_spec):
    result = firebase.patch(sound_spec)
    print("the sound value is%d" %sound_value)
    print("firebase.path result %s" % str(result))
    time.sleep(1)
  

Затем вы могли бы иметь список ‘sound_specification’ и использовать цикл for для получения результата для нескольких значений:

 sound_specs = ['/SoundValues/Monday/'{'Sound1':int(sound_value)},'/SoundValues/Monday/'{'Sound2':int(sound_value)}]

for sound_spec in sound_specs:
    getSound(sound_spec)