Программа входа и регистрации

@saharok2000 · Регистрация: 01.07.2014

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

Здравствуйте! Я написал для себя программку: она записывает логин и пароль в файл, а затем сверяет его в следующий раз при логине.
Проблема состоит в том что логин и пароль сохраняются кракозяброй(ввожу по английски), и как результат не может сравнить с тем чем ввожу при логине. Вот код:

C++

#include <iostream>
#include <cstdlib>
#include <string>
#include <fstream>
using namespace std;
 
ifstream* openFile()
{
    ifstream* pFileStream = 0;
    for(;;)
    {
        char fileName[50] = {'l', 'o', 'g', 'i', 'n', 's', '_', 'p' ,'a', 's', 's', 'w', 'o', 'r', 'd', 's', '.', 't', 'x', 't'};
        //îòêðûâàåì ôàéë äëÿ ÷òåíèÿ
        //íå ñîçäàâàòü, åñëè íå ñóùåñòâóåò
        pFileStream = new ifstream(fileName);
        if (pFileStream->good())
        {
            break;
        }
        cerr << "Невозможно открыть файл. " << fileName << endl;
        delete pFileStream;
    }
    return pFileStream;
}
 
int register_()
{
    char login[40];
    char password[40];
    char theLogPass[100];
    cout << "Введите логин " << endl;
    cout << "->";
    cin >> login;
    cout << "Введите пароль " << endl;
    cout << "->"; 
    cin >> password;
    theLogPass[100] = login[40] + password[40];
    // çàïèñü äàííûõ
    ofstream fout("logins_passwords.txt", ios::app);
    fout << theLogPass;
    fout << endl;
    fout.close();
    cout << "Вы успешно зарегистрированы!!!" << endl;
}
 
int loginer()
{
    char the_log[40];
    char the_pass[40];
    char the_log_pass[100];
    cout << "Введите свой логин" << endl;
    cout << "->";
    cin >> the_log;
    cout << "Введите пароль" << endl;
    cout << "->";
    cin >> the_pass;
    the_log_pass[100] = the_log[40] + the_pass[40];
    cout << the_log_pass << endl;
    
    //ïîëó÷åíèå ôàéëîâîãî ïîòîêà
    ifstream* pFileStream = openFile();
    
    //îñòàíîâèòñÿ ïî äîñòèæåíèþ êîíöà ôàéëà
    while (!pFileStream->eof())
    {
        //÷òåíèå çíà÷åíèÿ
        char value[100];
        (*pFileStream) >> value;
        
        // îñòàíîâêà ïðè îøèáêå ÷òåíèÿ
        //(íàïðèìåð, ñ÷èòûâàåòñÿ íå öåëîå ÷èñëî, èëè
        //ñ÷èòàí ñèìâîë íîâîé ñòðîêè ïîñëå êîòîðîãî 
        //íè÷åãî íåò
        if (pFileStream->fail()){
            break;
        }
        //âûâîä ñ÷èòàííîãî çíà÷åíèÿ
        if(value == the_log_pass){
            cout << "Доступ разрешен" << endl;
        }else{
            cout << "Ошибка аунтефикации!!!" << endl;
        }
    }
    delete pFileStream;
}
 
 
 
int main ()
{
    setlocale(LC_ALL, "");
    string v_log = "login";
    string v_reg = "register";
    string otvet;
    
    cout << "Привет!!! \nТебе нужно войти " 
    << "или зарегистрироваться." << endl;
    cout << "Введите login чтобы войти или register чтобы зарегистрироваться." << endl;
    cout << "->";
    cin >> otvet;
    
    if(otvet == v_log){
        loginer();
    }
    if(otvet == v_reg){
        register_();
    }
    system("pause");
    return 0;
}

Пожалуйста скажите в чем проблема.

@zss · 01.07.2014, 20:19

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

the_log_pass[100] = the_log[40] + the_pass[40];

Это что за чудо?
Так можно только со string.
char массивы надо самому копировать

C++

1 2	strcpy(the_log_pass,the_log); strcat(the_log_pass,the_pass)

@Kerry_Jr · 01.07.2014, 20:21

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

char fileName[50] = {'l', 'o', 'g', 'i', 'n', 's', '_', 'p' ,'a', 's', 's', 'w', 'o', 'r', 'd', 's', '.', 't', 'x', 't'};

Оригинально, а главное не лень же было писать

@saharok2000 · 01.07.2014, 20:38 **[ТС]**

zss, спасибо.
Kerry_Jr, просто я пытался вставить страйнг, но компилятор возмущался.

Добавлено через 6 минут
zss,
[Error] 'strcpy' was not declared in this scope

@nmcf · 01.07.2014, 20:44

Подключи <cstring>.

@saharok2000 · 01.07.2014, 20:46 **[ТС]**

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

Подключи <cstring>.

Спасибо, сработало

@Kerry_Jr · 01.07.2014, 21:03

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

вставить страйнг, но компилятор возмущался.

Вы, наверное, стринг имели в виду. Не обязательно стринг. Можно ведь и так

C++

1	char fileName[50] = "logins_passwords.txt";

Это практически то же самое, что сделали Вы, но более компактно.

@saharok2000 · 01.07.2014, 21:14 **[ТС]**

Kerry_Jr, я имел ввиду string;
я писал

C++

1 2	string fileName = "logins_passwords.txt"; pFileStream = new ifstream(fileName);

он возмущается

C++

1	pFileStream = new ifstream("logins_passwords.txt");

так еще больше

@nmcf · 01.07.2014, 21:17

А зачем new? Просто

C++

1	ifstream ifs(fileName);

@recoder · 01.07.2014, 21:17

C++

1	std::ifstream inFile(filename);

или

C++

std::cin.getline(filename, SIZE);
std::ifstream inFile;
inFile.open(filename);
if ( !inFile.is_open() )
{ ... }

@saharok2000 · 01.07.2014, 21:28 **[ТС]**

C++

#include <iostream>
#include <cstdlib>
#include <string>
#include <fstream>
#include <cstring>
using namespace std;
 
ifstream* openFile()
{
    ifstream* pFileStream = 0;
    for(;;)
    {
        char fileName[50] = {'l', 'o', 'g', 'i', 'n', 's', '_', 'p' ,'a', 's', 's', 'w', 'o', 'r', 'd', 's', '.', 't', 'x', 't'};
        //îòêðûâàåì ôàéë äëÿ ÷òåíèÿ
        //íå ñîçäàâàòü, åñëè íå ñóùåñòâóåò
        pFileStream = new ifstream(fileName);
        if (pFileStream->good())
        {
            break;
        }
        cerr << "Íåâîçìîæíî îòêðûòü ôàéë " << fileName << endl;
        delete pFileStream;
    }
    return pFileStream;
}
 
int register_()
{
    char login[40];
    char password[40];
    char theLogPass[100];
    cout << "Ïðèäóìàéòå ëîãèí " << endl;
    cout << "->";
    cin >> login;
    cout << "Ïðèäóìàéòå ïàðîëü " << endl;
    cout << "->"; 
    cin >> password;
    strcpy(theLogPass,login);
    strcat(theLogPass,password);
 
    // çàïèñü äàííûõ
    ofstream fout("logins_passwords.txt", ios::app);
    fout << theLogPass;
    fout << endl;
    fout.close();
    cout << "Âû óñïåøíî çàðåãèñòðèðîâàíû!!!" << endl;
}
 
int loginer()
{
    char the_log[40];
    char the_pass[40];
    char the_log_pass[100];
    cout << "Ââåäèòå ñâîé ëîãèí" << endl;
    cout << "->";
    cin >> the_log;
    cout << "Ââåäèòå ïàðîëü" << endl;
    cout << "->";
    cin >> the_pass;
    strcpy(the_log_pass,the_log);
    strcat(the_log_pass,the_pass);
    cout << the_log_pass << endl;
    
    //ïîëó÷åíèå ôàéëîâîãî ïîòîêà
    ifstream* pFileStream = openFile();
    
    //îñòàíîâèòñÿ ïî äîñòèæåíèþ êîíöà ôàéëà
    while (!pFileStream->eof())
    {
        //÷òåíèå çíà÷åíèÿ
        char value[100];
        (*pFileStream) >> value;
        
        // îñòàíîâêà ïðè îøèáêå ÷òåíèÿ
        //(íàïðèìåð, ñ÷èòûâàåòñÿ íå öåëîå ÷èñëî, èëè
        //ñ÷èòàí ñèìâîë íîâîé ñòðîêè ïîñëå êîòîðîãî 
        //íè÷åãî íåò
        if (pFileStream->fail()){
            break;
        }
        //âûâîä ñ÷èòàííîãî çíà÷åíèÿ
        if(value == the_log_pass){
            cout << "Äîñòóï ðàçðåøåí" << endl;
        }else{
            cout << "Îøèáêà àóíòåôèêàöèè!!!" << endl;
        }
    }
    delete pFileStream;
}
 
 
 
int main ()
{
    setlocale(LC_ALL, "");
    string v_log = "login";
    string v_reg = "register";
    string otvet;
    
    cout << "Ïðèâåò ÷åëîâåê!!! \n×òîáû ïîïàñòü â ìèð êîìïüþòåðà " 
    << "íåîáõîäèìî âîéòè èëè çàðåãèñòðèðîâàòüñÿ." << endl;
    cout << "Ââåäèòå login ÷òîáû âîéòè èëè register äëÿ ðåãèñòðàöèè." << endl;
    cout << "->";
    cin >> otvet;
    
    if(otvet == v_log){
        loginer();
    }
    if(otvet == v_reg){
        register_();
    }
    system("pause");
    return 0;
}

все равно не совпадает вводимый логин с тем что записан

@nmcf · 01.07.2014, 21:43

82-я строка - нельзя так строки сравнивать, надо через strcmp(). С древними строками ничего делать нельзя без функций. Перейди на string, если хочешь, чтобы было как в Паскале.

@saharok2000 · 01.07.2014, 21:52 **[ТС]**

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

82-я строка - нельзя так строки сравнивать, надо через strcmp(). С древними строками ничего делать нельзя без функций. Перейди на string, если хочешь, чтобы было как в Паскале.

Я что-то ничего не понял(и с паскалем я не знаком)

@nmcf · 01.07.2014, 21:54

C++

1	if(strcmp(value, the_log_pass))

@saharok2000 · 02.07.2014, 19:41 **[ТС]**

nmcf, а все понял

Добавлено через 21 час 36 минут
Люди, я похоже совсем тупой...

У меня снова не совпадает в 82 строке

C++

#include <iostream>
#include <cstdlib>
#include <string>
#include <fstream>
#include <cstring>
using namespace std;
 
ifstream* openFile()
{
    ifstream* pFileStream = 0;
    for(;;)
    {
        char fileName[50] = {'l', 'o', 'g', 'i', 'n', 's', '_', 'p' ,'a', 's', 's', 'w', 'o', 'r', 'd', 's', '.', 't', 'x', 't'};
        //îòêðûâàåì ôàéë äëÿ ÷òåíèÿ
        //íå ñîçäàâàòü, åñëè íå ñóùåñòâóåò
        pFileStream = new ifstream(fileName);
        if (pFileStream->good())
        {
            break;
        }
        cerr << "Íåâîçìîæíî îòêðûòü ôàéë " << fileName << endl;
        delete pFileStream;
    }
    return pFileStream;
}
 
int register_()
{
    char login[40];
    char password[40];
    char theLogPass[100];
    cout << "Ïðèäóìàéòå ëîãèí " << endl;
    cout << "->";
    cin >> login;
    cout << "Ïðèäóìàéòå ïàðîëü " << endl;
    cout << "->"; 
    cin >> password;
    strcpy(theLogPass,login);
    strcat(theLogPass,password);
 
    // çàïèñü äàííûõ
    ofstream fout("logins_passwords.txt", ios::app);
    fout << theLogPass;
    fout << endl;
    fout.close();
    cout << "Âû óñïåøíî çàðåãèñòðèðîâàíû!!!" << endl;
}
 
int loginer()
{
    char the_log[40];
    char the_pass[40];
    char the_log_pass[100];
    cout << "Ââåäèòå ñâîé ëîãèí" << endl;
    cout << "->";
    cin >> the_log;
    cout << "Ââåäèòå ïàðîëü" << endl;
    cout << "->";
    cin >> the_pass;
    strcpy(the_log_pass,the_log);
    strcat(the_log_pass,the_pass);
    cout << the_log_pass << endl;
    
    //ïîëó÷åíèå ôàéëîâîãî ïîòîêà
    ifstream* pFileStream = openFile();
    
    //îñòàíîâèòñÿ ïî äîñòèæåíèþ êîíöà ôàéëà
    while (!pFileStream->eof())
    {
        //÷òåíèå çíà÷åíèÿ
        char value[100];
        (*pFileStream) >> value;
        
        // îñòàíîâêà ïðè îøèáêå ÷òåíèÿ
        //(íàïðèìåð, ñ÷èòûâàåòñÿ íå öåëîå ÷èñëî, èëè
        //ñ÷èòàí ñèìâîë íîâîé ñòðîêè ïîñëå êîòîðîãî 
        //íè÷åãî íåò
        if (pFileStream->fail()){
            break;
        }
        //âûâîä ñ÷èòàííîãî çíà÷åíèÿ
        if(strcmp(value, the_log_pass)){
            cout << "Äîñòóï ðàçðåøåí" << endl;
        }else{
            cout << "Îøèáêà àóíòåôèêàöèè!!!" << endl;
            cout << "value        = " << value << endl;
            cout << "the_log_pass = " << the_log_pass << endl;
            
        }
    }
    delete pFileStream;
}
 
 
 
int main ()
{
    setlocale(LC_ALL, "");
    string v_log = "login";
    string v_reg = "register";
    string otvet;
    
    cout << "Ïðèâåò ÷åëîâåê!!! \n×òîáû ïîïàñòü â ìèð êîìïüþòåðà " 
    << "íåîáõîäèìî âîéòè èëè çàðåãèñòðèðîâàòüñÿ." << endl;
    cout << "Ââåäèòå login ÷òîáû âîéòè èëè register äëÿ ðåãèñòðàöèè." << endl;
    cout << "->";
    cin >> otvet;
    
    if(otvet == v_log){
        loginer();
    }
    if(otvet == v_reg){
        register_();
    }
    system("pause");
    return 0;
}

подскажите что не так

@nmcf · 02.07.2014, 20:19

C++

#include "stdafx.h"
#include <iostream>
#include <fstream>
#include <string>
#include <cstring>
 
using namespace std;
 
void register_()
{
    char login[40], password[40];
 
    cout << "Придумайте логин " << endl;
    cout << "->";
    cin.getline(login, 40);
    cout << "Придумайте пароль " << endl;
    cout << "->"; 
    cin.getline(password, 40);
    // запись данных
    ofstream ofs("logins_passwords.txt", ios::app);
    ofs << login << password << endl;
    ofs.close();
    cout << "Вы успешно зарегистрированы!!!" << endl;
}
 
void loginer()
{
    char the_log[40], the_pass[40], the_log_pass[100];
 
    cout << "Введите свой логин" << endl;
    cout << "->";
    cin.getline(the_log, 40);
    cout << "Введите пароль" << endl;
    cout << "->";
    cin.getline(the_pass, 40);
    strcpy(the_log_pass, the_log);
    strcat(the_log_pass, the_pass);
    //получение файлового потока
    ifstream ifs("logins_passwords.txt");
    if(!ifs)
    {
        cerr << "Невозможно открыть файл " << "logins_passwords.txt" << endl;
        return;
    }
    char value[100];
    //остановится по достижению конца файла
    while (!ifs.eof())
    {
        //чтение значения
        ifs.getline(value, 100);
        
        // остановка при ошибке чтения
        //(например, считывается не целое число, или
        //считан символ новой строки после которого 
        //ничего нет
        if (ifs.fail()){
            break;
        }
        //вывод считанного значения
        if(strcmp(value, the_log_pass) == 0)
        {
            cout << "Доступ разрешен" << endl;
            ifs.close();
            return;
        }
    }
    cout << "Ошибка аунтефикации!!!" << endl;
        ifs.close();
}
 
int _tmain(int argc, _TCHAR* argv[])
{
    setlocale(LC_ALL, "");
    string v_log = "login";
    string v_reg = "register";
    string otvet;
    
    cout << "Привет человек!!! \nЧтобы попасть в мир компьютера " 
            << "необходимо войти или зарегистрироваться." << endl;
    cout << "Введите login чтобы войти или register для регистрации." << endl;
    cout << "->";
    getline(cin, otvet);
    
    if(otvet == v_log){
        loginer();
    }
    else
        if(otvet == v_reg){
            register_();
        }
 
    system("pause");
    return 0;
}

@nmcf 7785 / 6553 / 2983 Регистрация: 14.04.2014 Сообщений: 28,617
	01.07.2014, 21:43	12
	82-я строка - нельзя так строки сравнивать, надо через strcmp(). С древними строками ничего делать нельзя без функций. Перейди на string, если хочешь, чтобы было как в Паскале. 0

@saharok2000 1 / 1 / 2 Регистрация: 01.07.2014 Сообщений: 49
	01.07.2014, 21:52 [ТС]	13
	Сообщение от nmcf 82-я строка - нельзя так строки сравнивать, надо через strcmp(). С древними строками ничего делать нельзя без функций. Перейди на string, если хочешь, чтобы было как в Паскале. Я что-то ничего не понял(и с паскалем я не знаком) 0

@Kerry_Jr 3106 / 2591 / 1219 Регистрация: 14.05.2014 Сообщений: 7,236 Записей в блоге: 1
	01.07.2014, 20:21	3
	Сообщение от saharok2000 char fileName[50] = {'l', 'o', 'g', 'i', 'n', 's', '_', 'p' ,'a', 's', 's', 'w', 'o', 'r', 'd', 's', '.', 't', 'x', 't'}; Оригинально, а главное не лень же было писать 0

@saharok2000 1 / 1 / 2 Регистрация: 01.07.2014 Сообщений: 49
	01.07.2014, 20:38 [ТС]	4
	zss, спасибо. Kerry_Jr, просто я пытался вставить страйнг, но компилятор возмущался. Добавлено через 6 минут zss, [Error] 'strcpy' was not declared in this scope 0

@nmcf 7785 / 6553 / 2983 Регистрация: 14.04.2014 Сообщений: 28,617
	01.07.2014, 20:44	5
	Подключи <cstring>. 0

@saharok2000 1 / 1 / 2 Регистрация: 01.07.2014 Сообщений: 49
	01.07.2014, 20:46 [ТС]	6
	Сообщение от nmcf Подключи <cstring>. Спасибо, сработало 0