Можно рекурсивно вызвать self.log_in

фу

Можно рекурсивно вызвать self.log_in

я так делал

это ужас

А смысл рекурсивно вызывать, если нужно изменить данные?

Там инпуты до цикла

А смысл рекурсивно вызывать, если нужно изменить данные?

вот-вот

Стоп, чё там делают инпуты???

Я запутался

Почему при попытке логина мы меняем данные юзера??????

еще раз - что нужно-то?

Стоп, чё там делают инпуты???

чтоб можно было "вводить" логин и пароль

еще раз - что нужно-то?

прервать бесконечный цикл в случае else

а еще не называй переменную logging

class User(): def __init__(self, username, password): self.username = username self.password = password def log_in(self, username_check='john76', password_check='www0z0', login_attempts=0, greeting='', logging=True): while logging: self.username = input('Please enter your username: ') self.password = input('Please enter your password: ') if ((self.username.lower() == username_check) and (self.password == password_check)): logging = False greeting = ('Successful. Logging in. . .' + '-------------------' + "You've made " + str(login_attempts) + " login attempts." + '-------------------' + "Welcome back, " + self.username) print(greeting) break else: login_attempts += 1 print('\nAn error loccured. Please try again . . .'+ '\n') my_user = User('username', 'password') my_user.log_in()

break

Можно рекурсивно вызвать self.log_in

И пробить стек?)

break

это просто конец цикла

а что надо?

Какую роль вообще выполняют self.username и self.password, если мы их меняем при каждом логине?

а мне нужно, чтоб вновь вернулся в строку "введите логин и пароль"

это просто конец цикла

запусти то, что я написал

И пробить стек?)

Ну это сколько раз надо неправильно пароль ввести, чтобы стек пробить

Какую роль вообще выполняют self.username и self.password, если мы их меняем при каждом логине?

вот как раз чтобы запоминать input

А зачем они тогда в __init__()?

sys.setrecursionlimit(1000000), чтобы наверняка

тру посоны такие вещи генераторами делают

У меня есть неуловимое впечатление, что ты неправильно понял суть задания

class Bunch(dict): def __init__(self, *args, **kwargs): self.__dict__ = self super().__init__(*args, **kwargs)

шзнх

шзнх

bunch

self.username и self.password — это параметры юзера

Скажем, если бы юзер был кирпичом, у него были бы параметры "форма" и "цвет"

А зачем они тогда в __init__()?

не, я вначале сделал заданными их, по заданию. А потом решил попробовать то же задание, но уже посложнее

"Квадратный" и "красный"

То же самое с юзером

У него фиксированные юзернейм и пассворд

Они не меняются при каждом логине

Они не меняются при каждом логине

фиксированные вот: username_check='john76', password_check='www0z0'

Эээээээээээм

а те как раз уже то, что вводишь

Простите но это какой то аутизм

Сделай так: создавайте объект юзера (устанавливая у него его имя и пароль при создании) + метод логин - которые проверяет валидность полей

На мой взгляд, эта функция должна выглядеть как-то так: def log_in(self): success = False while not success: username = input() password = input() if username == self.username and password == self.password: # Нормально вошли else: # Говно, выдаём ошибку и пробуем ещё раз

self.* — это ПОЛЯ класса, свойства юзера

Они НЕ должны меняться при входе

А если в пароле есть \n?

На мой взгляд, эта функция должна выглядеть как-то так: def log_in(self): success = False while not success: username = input() password = input() if username == self.username and password == self.password: # Нормально вошли else: # Говно, выдаём ошибку и пробуем ещё раз

Я бы вообще цикл вынес из класса + добавил sleep при неверной попытке

А если в пароле есть \n?

Обычно это запрещено

ну вот только надо чтобы log_in не было методом класса User, а так да

Я бы вообще цикл вынес из класса + добавил sleep при неверной попытке

Да-да-да, но если переделывать именно этот код...

я буквально вчера обновил старый (первый) айпад на ту версию ОС, которую он мне предложил (кажется, последнюю). Так что с версиями все ок. Правда, теперь у меня ничего не работает: книжки почти не открываются, приложение курсеры совсем не открывается, все, что открывается, дико тупит. Но версии последние.

Поэтому сижу на 7.01 и ни-ни...?

ну вот только надо чтобы log_in не было методом класса User, а так да

Ну назови его check_auth или что то в таком духе

тогда вынести оттуда цикл

И пускай bool возвращает успешно или нет

Вообще, в идеале как-то так: def check_auth(self, login, password): return (login == self.login) and (password == self.password)

пойду-ка переделаю, авось что выйдет

То есть совсем в идеале ещё хеши, но это выходит за рамки задачи

@TheStardustCrusader Прочитай ещё раз что-нибудь на тему "что такое классы и с чем их едят"

Пока у тебя не совсем верное представление, похоже

Вообще, в идеале как-то так: def check_auth(self, login, password): return (login == self.login) and (password == self.password)

Не имхо лучше проверку сначала и если ложная то sleep() а потом return false

Пока у тебя не совсем верное представление, похоже

+1

@TheStardustCrusader Прочитай ещё раз что-нибудь на тему "что такое классы и с чем их едят"

да просто я вот как думал: почему бы не иметь возможности вносить свои данные в класс

Не имхо лучше проверку сначала и если ложная то sleep() а потом return false

Всё это не должно быть внутри User

да просто я вот как думал: почему бы не иметь возможности вносить свои данные в класс

Свои данные так свои данные

Всё это не должно быть внутри User

Хм, почему ?

Но это не надо делать в методе log_in()

Для этого стоит сделать что-то вроде edit_data()

ERROR: S client not available

Но это не надо делать в методе log_in()

вот сейчас попробую вынести)

В плане сделать проверку пароля и логина с задержкой.

Хотя ок секьюрити никакой

Хм, почему ?

Эмпирическое правило: ввод-вывод и sleep() надо из класса выносить

Эмпирическое правило: ввод-вывод и sleep() надо из класса выносить

В целом верно.

В идеале, они не должны иметь побочных эффектов, кроме тех, единственной целью которых являются побочные эффекты

Эмпирическое правило: ввод-вывод и sleep() надо из класса выносить

Тоже к этому пришел. В классе, увы, не все так гладко, как я этого хотел...

Ты не в Яве, тебе не нужно фигачить классами на всё происходящее. Класс — хранилище данных плюс методы их обработки, всё. Все остальное выноси.

Ты не в Яве, тебе не нужно фигачить классами на всё происходящее. Класс — хранилище данных плюс методы их обработки, всё. Все остальное выноси.

никогда не работал с явой)

В ней у тебя Hello world без классов не выйдет написать

В идеале, они не должны иметь побочных эффектов, кроме тех, единственной целью которых являются побочные эффекты

Это да, но тут заведомо нужное поведение ведь - задержка для защиты перебором

Это да, но тут заведомо нужное поведение ведь - задержка для защиты перебором

И его можно вынести в вызывающий модуль

И его можно вынести в вызывающий модуль

Вот вроде можно начать про то безопасно ли но тут все не безопасно)

В ней у тебя Hello world без классов не выйдет написать

Борщ борщ равно нью борщфэктори от борщтемплейт точка креэйтборщ

Можно сделать колбэк onLoginError и просить его в конструкторе, по дефолту ничего не днлать

Можно сделать колбэк onLoginError и просить его в конструкторе, по дефолту ничего не днлать

Кстати не плохо

Не знаю, на мой взгляд из класса надо вынести всё, что можно вынести

И кстати если уж следовать - то поля надо с _ начать

Всякими слипами и колбэками должна заниматься вызывающая сторона

И кстати если уж следовать - то поля надо с _ начать

Только приватные

И кстати если уж следовать - то поля надо с _ начать

И писать для них get/set или объявлять как property. Но это не обязательно.

Фу жаба

Только приватные

В идеале, все поля приватные

Не в этом языке

Но мне всегда лень это делать

В любом языке, это универсальный паттерн

Фу жаба

Тут это оправданоб как бы читать и писать в поле пароля без спроса - не дело

Для поддержания корректного состояния класса

Зачем писать проперти там, где они не нужны?

Тут это оправданоб как бы читать и писать в поле пароля без спроса - не дело

А кто будет в него писать, если это запущено со стороны сервера?

Зачем писать проперти там, где они не нужны?

Для поддержания корректного состояния класса

Если программа хочет поменять юзеру пароль, она имеет право это сделать