Рекурсия (вызов только 3 раза)

@AlexLag · Регистрация: 03.08.2014

Author24 — интернет-сервис помощи студентам

Привет! Есть программка, проблема заключается в том чтобы сделать вызов рекурсии ТОЛЬКО 3 раза, т.е. как при пароле, есть только 3 попытки. Сейчас она вызывается постоянно, если введено неверное значение. Как сделать вызов только 3 раза...

C++

#include <iostream>
#include <string>
#include <windows.h>
 
using namespace std;
 
bool password_is_valid(string password)
{
    string valid_pass = "qwerty123";
    if (valid_pass == password)
        return true;
    else
        return false;
}
 
void get_pass()
{
    string user_pass;
    cout << "Введите пароль: ";
    getline(cin, user_pass);
    if (!password_is_valid(user_pass)) {
        cout << "Неверный пароль!" << endl;
        get_pass();
    }
    else {
        cout << "Доступ разрешен." << endl;
    }
}
 
int main()
{   
    setlocale(0, "rus");
    get_pass();
    system("pause");
    return 0;
}

Tulosba · 04.03.2015, 15:12

Добавить переменную счетчика попыток вестимо.
Например так:

C++

void get_pass()
{
    static int tryCnt = 0;
    if( ++tryCnt >= 3 )
    {
        cout << "Попытки кончились." << endl;
        return;
    }
    string user_pass;
    cout << "Введите пароль: ";
    getline(cin, user_pass);
    if (!password_is_valid(user_pass)) {
        cout << "Неверный пароль!" << endl;
        get_pass();
    }
    else {
        cout << "Доступ разрешен." << endl;
        tryCnt = 0;
    }
}

@_Ivana · 04.03.2015, 15:17

Маша - это Маша, но 3 раза - это 3 раза! (С)

Заводите интовый параметр в вашу гет пасс, каждый вложенный вызов осуществляете с увеличением его на единицу, в начале функции проверяете его значение, первый вызов в мэйне делаете с нулевым или единичным значением - все.

C++

void get_pass(int tryes)
{
    string user_pass;
    if (tryes>3) cout << "Сколько можно ошибаться! Не дам доступ :)";
    else {
        cout << "Введите пароль: ";
        getline(cin, user_pass);
        if (!password_is_valid(user_pass)) {
            cout << "Неверный пароль!" << endl;
            get_pass(tryes + 1);
        }
        else {
            cout << "Доступ разрешен." << endl;
        }
    }
}
 
int main()
{   
    setlocale(0, "rus");
    get_pass(1);
    system("pause");
    return 0;
}

Добавлено через 2 минуты
Tulosba, не видел ваш вариант когда писал свой. Статик... Имхо, явный аргумент по значению тут элегантнее смотрится.

@AlexLag · 04.03.2015, 15:18 **[ТС]**

Спасибо!)

Tulosba · 04.03.2015, 15:24

Сообщение от _Ivana

Статик... Имхо, явный аргумент по значению тут элегантнее смотрится.

Статические переменные в функциях это вообще не оч.хорошо. При условии использования ООП они должны переехать в поля класса. Но т.к. тут никакого ООП по сути нет, я предложил самое простое (как мне показалось) решение. Инкапсуляция переменной внутри метода (static), имхо в данном случае проще, чем передача аргумента, т.к. вся логика остается внутри метода (параметр при вызове не нужен). Также не меняется сигнатура функции get_pass().

@_Ivana · 04.03.2015, 15:30

Tulosba, все понимаю, кроме одного

А сейчас нет под рукой компилятора чтобы проверить. Если бы вы просто сделали честную глобальную переменную и зануляли ее ДО вызова гет_пасс в мэйне, а внутри гет_пасс только инкрементировали и проверяли ее значение, но не зануляли, тогда было бы все железно и понятно. А сейчас у вас она статик, то есть по сути глобальная, но видна только из гет_пасс, а вы кажый раз ее там зануляете - с какого она должна быть больше единицы при очередном вызове? Или объясните где я ошибаюсь, или скажите что я прав

Tulosba · 04.03.2015, 15:33

Сообщение от _Ivana

кажый раз ее там зануляете

Не каждый, а только при удачном вводе пароля.

Сообщение от _Ivana

А сейчас нет под рукой компилятора чтобы проверить.

ideone.com // и куча других в помощь

@_Ivana · 04.03.2015, 15:41

Хм... Оказывается (внезапно), что при последующих рекурсивных вызовах статическая переменная не переопределяется если она уже была определена и ей было назначено значение при определении в первом вызове.

C++

#include <iostream>
using namespace std;
 
void get_pass()
{
    static int tryCnt = 0;
    cout << "Попытка № " << tryCnt << endl;
    if( ++tryCnt >= 3 )
    {
        cout << "Попытки кончились." << endl;
        return;
    }
    get_pass();
}
 
int main() {
    get_pass();
    get_pass();
    return 0;
}

http://ideone.com/vhQDJz

Tulosba · 04.03.2015, 15:47

Сообщение от _Ivana

Оказывается (внезапно)

Это же базовые вещи

Статическая переменная внутри функции инициализируется только 1 раз. Иначе она была бы обычной локальной.

@_Ivana · 04.03.2015, 15:50

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

ЗЫ а про базовые вещи... Вот когда писал на одном языке, знал его повадки, приоритеты операций, передачу параметров по значению/ссылке и т.п. А когда попробовал другие - уже начинаю иногда путаться где у чего какая область видимости и т.п. Так что звиняйте барин за проколы в базовых вещах

Tulosba · 04.03.2015, 16:04

Не по теме:

Сообщение от _Ivana

звиняйте барин за проколы в базовых вещах

Со всеми случается время от времени. :handshake:

@AlexLag · 04.03.2015, 16:35 **[ТС]**

Вопрос, а как этот код правильно перевести для СИшки?

C++

bool password_is_valid(string password)
{
    string valid_pass = "qwerty123";
    if (valid_pass == password)
        return true;
    else
        return false;
}
 
void get_pass()
{
    static int tryCnt = 0;
    if (++tryCnt > 3)
    {
        cout << "\n >  Доступ запрещён!\n" << endl;
        return;
    }
    string user_pass;
    cout << "Введите пароль: ";
    getline(cin, user_pass);
    if (!password_is_valid(user_pass)) {
        cout << "Неверный пароль!" << endl;
        get_pass();
    }
    else {
        cout << "\n >  Доступ разрешен!\n" << endl;
        tryCnt = 0;
        system("cls");
    }
}

По большей части интересует getline и string.

Tulosba · 04.03.2015, 16:53

Сообщение от AlexLag

getline и string.

scanf() и char*

@AlexLag · 04.03.2015, 17:01 **[ТС]**

Сообщение от Tulosba

scanf() и char*

А эту строчку? getline(cin, user_pass);
scanf(user_pass); ??

C

#include <string.h>
#include <stdio.h>
#include <windows.h>
 
bool password_is_valid(char* password)
{
    char* valid_pass = "qwerty123";
    if (valid_pass == password)
        return true;
    else
        return false;
}
 
void get_pass()
{
    static int tryCnt = 0;
    if (++tryCnt > 3)
    {
        printf("\n >  Доступ запрещён!\n");
        return;
    }
    char* user_pass;
    printf("Введите пароль: ");
    scanf(user_pass);
    if (!password_is_valid(user_pass)) {
        printf("Неверный пароль!");
        get_pass();
    }
    else {
        printf("\n >  Доступ разрешен!\n");
        tryCnt = 0;
        system("cls");
    }
}
 
int main()
{
    setlocale(0, "rus");
    printf("У вас есть 3 попытки! Помните об этом.");
    get_pass();
    printf("Continue...");
    system("pause");
    return 0;
}

Ошибки:

Кликните здесь для просмотра всего текста

Ошибка 1 error C2061: синтаксическая ошибка: идентификатор "password_is_valid"
Ошибка 2 error C2059: синтаксическая ошибка: ;
Ошибка 3 error C2059: синтаксическая ошибка: тип

Tulosba · 04.03.2015, 17:05

AlexLag, не всё так просто как хотелось бы

На самом деле для чтения строки (чтоб с пробелами) в сях стоит использовать скорее gets, а не scanf.
Давно на сях не писал, уже подзабыл.

@AlexLag · 04.03.2015, 17:09 **[ТС]**

Сообщение от Tulosba

AlexLag, не всё так просто как хотелось бы

На самом деле для чтения строки (чтоб с пробелами) в сях стоит использовать скорее gets, а не scanf.
Давно на сях не писал, уже подзабыл.

Ок, буду пробовать...Правда проблема с булом...

Кликните здесь для просмотра всего текста

Ошибка 1 error C2061: синтаксическая ошибка: идентификатор "password_is_valid"
Ошибка 2 error C2059: синтаксическая ошибка: ;
Ошибка 3 error C2059: синтаксическая ошибка: тип
6 IntelliSense: идентификатор "bool" не определен
7 IntelliSense: идентификатор "true" не определен
8 IntelliSense: идентификатор "false" не определен

Tulosba · 04.03.2015, 17:13

Сообщение от AlexLag

Правда проблема с булом...

C

1	#include <stdbool.h>

@_Ivana 4817 / 2278 / 287 Регистрация: 01.03.2013 Сообщений: 5,947 Записей в блоге: 28
	04.03.2015, 15:50	10
	Хотя действительно, что это я... При тех же циклических (а не рекурсивных) вызовах функции статические переменные именно так себя и ведут, просто меня запутала рекурсивная вложенность контекстов, а статики (как глобальные по сути) ничем в данном случае не отличаются по своему поведению. Но в любом случае спасибо за науку ЗЫ а про базовые вещи... Вот когда писал на одном языке, знал его повадки, приоритеты операций, передачу параметров по значению/ссылке и т.п. А когда попробовал другие - уже начинаю иногда путаться где у чего какая область видимости и т.п. Так что звиняйте барин за проколы в базовых вещах 0

@AlexLag 0 / 0 / 1 Регистрация: 03.08.2014 Сообщений: 49
	04.03.2015, 15:18 [ТС]	4
	Спасибо!) 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.03.2015, 15:24	5
	Сообщение от _Ivana Статик... Имхо, явный аргумент по значению тут элегантнее смотрится. Статические переменные в функциях это вообще не оч.хорошо. При условии использования ООП они должны переехать в поля класса. Но т.к. тут никакого ООП по сути нет, я предложил самое простое (как мне показалось) решение. Инкапсуляция переменной внутри метода (static), имхо в данном случае проще, чем передача аргумента, т.к. вся логика остается внутри метода (параметр при вызове не нужен). Также не меняется сигнатура функции get_pass(). 1

@_Ivana 4817 / 2278 / 287 Регистрация: 01.03.2013 Сообщений: 5,947 Записей в блоге: 28
	04.03.2015, 15:30	6
	Tulosba, все понимаю, кроме одного А сейчас нет под рукой компилятора чтобы проверить. Если бы вы просто сделали честную глобальную переменную и зануляли ее ДО вызова гет_пасс в мэйне, а внутри гет_пасс только инкрементировали и проверяли ее значение, но не зануляли, тогда было бы все железно и понятно. А сейчас у вас она статик, то есть по сути глобальная, но видна только из гет_пасс, а вы кажый раз ее там зануляете - с какого она должна быть больше единицы при очередном вызове? Или объясните где я ошибаюсь, или скажите что я прав 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.03.2015, 15:33	7
	Сообщение от _Ivana кажый раз ее там зануляете Не каждый, а только при удачном вводе пароля. Сообщение от _Ivana А сейчас нет под рукой компилятора чтобы проверить. ideone.com // и куча других в помощь 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.03.2015, 15:47	9
	Сообщение от _Ivana Оказывается (внезапно) Это же базовые вещи Статическая переменная внутри функции инициализируется только 1 раз. Иначе она была бы обычной локальной. 0

Tulosba
	04.03.2015, 16:04 #11
	Не по теме: Сообщение от _Ivana звиняйте барин за проколы в базовых вещах Со всеми случается время от времени. :handshake: 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.03.2015, 16:53	13
	Сообщение от AlexLag getline и string. scanf() и char* 1

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.03.2015, 17:05	15
	AlexLag, не всё так просто как хотелось бы На самом деле для чтения строки (чтоб с пробелами) в сях стоит использовать скорее gets, а не scanf. Давно на сях не писал, уже подзабыл. 0

@AlexLag 0 / 0 / 1 Регистрация: 03.08.2014 Сообщений: 49
	04.03.2015, 17:09 [ТС]	16
	Сообщение от Tulosba AlexLag, не всё так просто как хотелось бы На самом деле для чтения строки (чтоб с пробелами) в сях стоит использовать скорее gets, а не scanf. Давно на сях не писал, уже подзабыл. Ок, буду пробовать...Правда проблема с булом... Кликните здесь для просмотра всего текста Ошибка 1 error C2061: синтаксическая ошибка: идентификатор "password_is_valid" Ошибка 2 error C2059: синтаксическая ошибка: ; Ошибка 3 error C2059: синтаксическая ошибка: тип 6 IntelliSense: идентификатор "bool" не определен 7 IntelliSense: идентификатор "true" не определен 8 IntelliSense: идентификатор "false" не определен 0