homework lec.1

[SolReykan]

No description provided.

[IlyaOrlov]

Хорошо, но есть пара вопросов/замечаний.

[IlyaOrlov]

В этой строке с отступами все в порядке. PyCharm может подсвечивать эту строку, как ошибочную из-за строки 15. Интерпретатор Python требует, чтоб все строки внутри блока выравнивались одинаковыми отступами, однако не указывает конкретный размер этих отступов. Строка 15 и строки 16-19 относятся к одному блоку, но имеют разное выравнивание. Это несоответствие и сбивает PyCharm с толку. Согласно рекомендациям PEP8, размер отступа - 4 пробела, поэтому строки 16-19 можно считать отформатированными корректно, а строку 15 - ошибочной.

[IlyaOrlov]

Только в этой строке или во всем блоке 28-30?

[IlyaOrlov]

Небольшие замечания по стилю:

1. В строках 1, 3, 5, 6, 8 перед скобкой пробелы не нужны PEP8: Avoid extraneous whitespace in the following situations
2. В строках 6 и 13 нужно отделить вызов функции от определения функции двумя строками PEP8: Surround top-level function and class definitions with two blank lines.
   В остальном все ОК.

[IlyaOrlov]

Замечательно! Особенно решение с палиндромом! Но есть небольшие замечания.

[IlyaOrlov]

Можно чуть упростить: if len(w) < 2

[IlyaOrlov]

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

[IlyaOrlov]

Замечательная идея - реализовать функцию через рекурсию! Если вместо 0/1 возвращать True/False, то и проверку можно чуть упростить:

if palindrom(w):
    print("Это палиндром")
else:
    print("Введённое слово не является палиндромом")

Есть, правда пара нюансов:

1. w[1:-1] - это копирование подсписка, что может оказаться достаточно затратной по времени операцией; лучше передавать тот же список (w) и два индекса, которые будут сравниваться (1 и -1, 2 и -2, и так пока первый не превысит len(w)/2).
2. Лучше если lower() будет выполняться внутри функции, т.к. эта операция тоже относится к проверке на палиндром, а чтоб lower() не выполнялся на каждом рекурсивном вызове, можно сделать функцию-обертку:

def palindrome_ci(w):  # ci - case insensitive - регистронезависимый
    def palindrome(w):
        # здесь ваш код для рекурсивной проверки на палиндром
    
    return palinfrome(w.lower())

w = input("Введите слово: ")
if palindrome_ci(w):
    ....

[IlyaOrlov]

Здесь и в стр.45: функции и методы без тела интерпретатор не пропустит. Надо добавить хотя бы pass.

[IlyaOrlov]

Просто так объявлять поля класса бессмысленно. Надо им присвоить какие-то значения.

[IlyaOrlov]

Нужно еще создать экземпляр класса Tank (ниже) и написать примеры вызова его методов.

[IlyaOrlov]

Да, если работать c num, как с числом, то проще не получится. И с этой точки зрения решение корректное. НО мы же запросто можем привести число к строковому типу и работать с ним, как с последовательностью символов (тем более input сразу возвращает строку), тогда и никакие сложные вычисления не потребуются.