7

Примечание: надо сделать это без использования особых операторов (вручную). Вот сам код:

import random

numbers = []
k = 0

for i in range(-10,10):
 i = random.randrange(-10,10)
 numbers.append(i)

print(numbers)  

for k in len(numbers):
 if k % 2 == 0:
  k = 0
0

5 ответов 5

9

Нужно всего лишь немного поправить написанный Вами код, а именно цикл for:

for k in range(len(numbers)):
    if k % 2 == 0:
        numbers[k] = 0

Дело в том, что Вы каждую итерацию обнуляли значение k вместо значения элемента numbers c индексом k.

UPD. Есть ещё такая реализация, мне она нравится больше:

for k in range(0, len(numbers), 2):
    numbers[k] = 0
1
  • numbers[::2] = 0 работает для numpy массивов. Для Питон-списков это похоже на numbers[::2] = [0]* ceil(len(numbers) / 2) (без создания временного списка)
    – jfs
    4 авг 2017 в 14:42
5
for k in len(numbers):
for k in range(len(numbers)):

И ещё пара способов с созданием нового списка: http://ideone.com/0paBth

print([x * (i % 2) for i,x in enumerate(numbers)])
print([x if i % 2 else 0 for i,x in enumerate(numbers)])
4
  • 2
    x * (i % 2) - изящно!
    – Xander
    9 июн 2017 в 10:59
  • 1- списки создавать следует, если вы их используете в дальнейшем (не используйте списковые включения только ради побочного эффекта/side-effect) @Александр x & 1 and x если хочется краткости. Хотя лучше явно x if x & 1 else 0 как в ответе уже показано, чтобы более читаемо было.
    – jfs
    4 авг 2017 в 14:48
  • @jfs, что-то я не понял про побочные эффекты. Создаётся новый список и он выводится. Никаких побочных эффектов там нет.
    – Qwertiy
    4 авг 2017 в 18:30
  • 1
    Очевидно, что список у вас используется (он печатается). Мой комментарий для будущих читателей, чтобы они не использовали [f(x) for x in L] вместо for x in L: f(x)
    – jfs
    4 авг 2017 в 19:30
4

Вот поправил ваш код:

In [29]: def foo():
    ...:     import random
    ...:     numbers = []
    ...:     k = 0
    ...:     for i in range(-10,10):
    ...:         i = random.randrange(-10,10)
    ...:         numbers.append(i)
    ...:     print(numbers)
    ...:     for k in range(len(numbers)):
    ...:         if k % 2 == 0:
    ...:             numbers[k] = 0
    ...:     print(numbers)
    ...:

In [30]: foo()
[3, -4, 0, -7, 2, 2, 2, -5, -2, 8, -2, -5, 8, -1, 2, -8, 9, 3, -4, -9]
[0, -4, 0, -7, 0, 2, 0, -5, 0, 8, 0, -5, 0, -1, 0, -8, 0, 3, 0, -9]
7
  • Какие-то у тебя странные чётные позиции - двух ненулевых значений подряд не должно быть в результате.
    – Qwertiy
    9 июн 2017 в 9:13
  • @Qwertiy не вижу такого 9 июн 2017 в 9:14
  • Начало списка: [-2, 4, 0,
    – Qwertiy
    9 июн 2017 в 9:16
  • @Qwertiy ну так все правильно, у -2 индекс 0 у 4 индекс 1 а дальше четный индекс 9 июн 2017 в 9:16
  • 1
    Вообще-то 0 - это чётное число.
    – Qwertiy
    9 июн 2017 в 9:17
3

Еще один вариант:

In [126]: numbers
Out[126]: [9, 8, 7, 6, 8, 1, 2, 1, -9, -8, -7, -10, -2, 8, -9, -8, 0, 6, 4, 7]

In [127]: numbers[::2] = [0] * len(range(0, len(numbers), 2))

In [128]: numbers
Out[128]: [0, 8, 0, 6, 0, 1, 0, 1, 0, -8, 0, -10, 0, 8, 0, -8, 0, 6, 0, 7]

Если использовать Numpy, то получится коротко и быстро:

In [129]: import numpy as np

In [130]: a = np.array(numbers)

In [131]: a
Out[131]: array([  9,   8,   7,   6,   8,   1,   2,   1,  -9,  -8,  -7, -10,  -2,   8,  -9,  -8,   0,   6,   4,   7])

Решение в стиле Numpy:

In [132]: a[::2] = 0

In [133]: a
Out[133]: array([  0,   8,   0,   6,   0,   1,   0,   1,   0,  -8,   0, -10,   0,   8,   0,  -8,   0,   6,   0,   7])

Timing для массива из 100 миллионов элементов:

In [144]: a = np.random.randint(10**3, size=10**8)

In [145]: a.shape
Out[145]: (100000000,)

In [146]: %timeit a[::2] = 0
10 loops, best of 3: 42.4 ms per loop
7
  • Жизнь меня к этому не готовила :) 9 июн 2017 в 10:32
  • 1
    @Alban, к чему вас жизнь не готовила? :) 9 июн 2017 в 10:33
  • 1
    к варианту с numpy) 9 июн 2017 в 10:34
  • 1
    @Alban, я постоянно использую Numpy из-за его скорости - я добавил "timing" в ответ... 9 июн 2017 в 10:39
  • 1
    @gil9red ipython 9 июн 2017 в 10:42
2

Можно немного изменить цикл:

for k in numbers:
    i = numbers.index(k)
    if i % 2 == 0:
        numbers[i] = 0

Или так:

new = [0 if (c.index(i) % 2 == 0) else i for i in numbers]
4
  • Эм.. Это же проверка чётности элемента, а не чётности позиции?
    – Qwertiy
    9 июн 2017 в 9:11
  • 1
    @Qwertiy да, читать условие полезно :) Ща поправлю 9 июн 2017 в 9:19
  • А теперь асимптотика квадратичная вместо линейной, не?
    – Qwertiy
    9 июн 2017 в 10:35
  • 1
    @Qwertiy Похоже на то, наверное для общего развития можно оставить, как пример получения индекса эл-та в списке, мне кажется, что если говорить о скорости обработки больших итерируемых объектов, то работа со списком по индексу в принципе не лучшая идея 9 июн 2017 в 10:47

Ваш ответ

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

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