Как решить задачу с возвращением ряда?

Question

[Дарья]

Описание задания:
get_range()
Данная функция должна для заданного положительного числа аргумента n возвращать список чисел от нуля до n, не включая само число n. Если при вызове было передано отрицательное число или ноль, функция должна возвращать пустой список.

>>> get_range(5)
[0, 1, 2, 3, 4]

Для решения используйте цикл while и метод списка append.

Моё видение решения:

def get_range(n):
    if n >= 0:
        for i range(start: 0, stop: n, step: 1):
            print(i, end=', ')
    elif:
        print("[ ]")

Answer 1

[Ivan Yakushenko]

def get_range(n):
    if n > 0:
        result = []
        counter = 0
        while counter < n:
            result.append(counter)
            counter += 1
        return result
    else:
        return []

Вы использовали for, а нужно while. Про циклы можете тут почитать.
Когда функция должна что-то возвращать, то это означает наличие return, а не print. О том как устроены функции тут.
"[ ]" - это строка, а не пустой список. О типах данных тут.

А вообще все это очень подробно расписано на первой сотне страниц любой книги по Python. Возьмите хотя-бы A Byte of Python, она бесплатная, на русском языке и оооочень простая.

Comments

[Дарья]

Спасибо за подробный ответ. Помогло:)

[h0w4rd]

Ivan Yakushenko, куратор тега и такой код :)
Вот более правильный:

def get_range(n):
    if n < 0:
        return []

    result = []
    counter = 0

    while counter < n:
        result.append(counter)
        counter += 1

    return result

[Ivan Yakushenko]

h0w4rd, и чем он правильный?

[h0w4rd]

Ivan Yakushenko, Если не ошибаюсь, чем площе код тем лучше. А у вас в коде без нужды сразу идет 2-й уровень отступа и ненужный блок else. Мой вариант читается легче.

[Ivan Yakushenko]

h0w4rd, и где вы такое вычитали, особенно про блок else? =)

[h0w4rd]

Ivan Yakushenko, Как минимум это логично) А так, как бы глупо не звучало, на хабре.

[Ivan Yakushenko]

h0w4rd, ну звучит действительно глупо, особенно что это как раз таки не логично звучит. Попробуйте прочитать вслух русскими словами код, звучать это будет примерно так: "если N меньше нуля, то выполняем это действие, иначе...". Практически в любых блоках if присутствие else ну никак не может быть не логичным и тем более не нужным т.к. он для этого и нужен, что-бы выполнять дополнительный код в случае если ожидаемые условия не выполнились.
На счет отступов: в рамках спецификаций прыгайте сколько угодно, это все полнейшая вкусовщина ничем не закрепленная. На каждого человека которому "не нравится" найдется тот, кому "нравится". Более того, в том-же PEP есть куча листингов, где уход на верхний уровень приветствуется (я говорю не о конкретно этом примере).

К вашему коду; в условиях есть фраза "Если при вызове было передано отрицательное число или ноль", соответственно логично ожидать в коде сравнение n > 0 или n <= 0, ни того ни другого у вас нет.

Эти передергивания, если что, можно продолжать совершенно бесконечно и к "правильности" кода это не имеет совершенно никакого отношения, кто-бы что не говорил. Пока код соблюдает хотя-бы базовые принципы программирования (типа DRY, KISS, YAGNI и т.д.), соблюдает спецификацию и выдает ожидаемый результат, то этот код правильный пожалуй во всех случаях.

[Ivan Yakushenko]

h0w4rd, и кстати на счет этого, забыл еще упомянуть: если передать 0, то у вас просто так отработает весь код, хотя он не должен этого делать и это уже не вкусовщина, а реальная проблема.