C++ Passwort Generator

KingwondA · 04/09/2015, 19:04

Hey,
ich erstelle mir grad einen Passwort Generator, jedoch möchte der nicht "laufen".
Im Spoiler ist die Fehlermeldung. Ich benutzte VS 2013 Ultimate.

Spoiler

Hier der Source Code:

Code:

#include <iostream>
#include <Windows.h>
#include <string>
#include <cmath>
#include <fstream>

#pragma comment(lib, "winmm.lib")

int main()
{
	static const TCHAR* myTitle = TEXT("Passwort-Generator");
	SetConsoleTitle(myTitle);

	std::string Buchstaben = "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz0123456789";
	std::string Passwort;
	int laenge;
	srand(timeGetTime());

	std::cout << "Wie lang soll das Passwort werden: ";
	std::cin >> laenge;

	for (int i = 0; i < laenge; i++)
	{
		
		Passwort += Buchstaben[rand() % (sizeof(Buchstaben))];

	}
	std::cout << Passwort << "\n";

	std::cin.get();
	return 0;
}

bloodx · 04/09/2015, 19:15

#pragma comment(lib, "winmm.lib");

KingwondA · 04/09/2015, 22:14

Quote:

Originally Posted by bloodx

#pragma comment(lib, "winmm.lib");

Danke, hat mir nicht wirklich geholfen.

Kam danach ein weiterer Fehler den hab ich selber behoben.

Stehe grad ziemlich auf den schlauch, auch wenn die Lösung einfach sein mag ich komm einfach nicht drauf. xD

Ich möchte nun, dass ich auch angeben kann wie viele Passwörter ich erstellt haben möchte. Da muss definitiv eine Schleife her, nur bekomme ich es nicht auf die Reihe die richtig zu implementieren^^
Code:

Code:

#include <iostream>
#include <Windows.h>
#include <string>
#include <cmath>
#include <fstream>

#pragma comment(lib, "winmm.lib")

int main()
{
	static const TCHAR* myTitle = TEXT("Passwort-Generator");
	SetConsoleTitle(myTitle);

	char  Buchstaben[] = "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz0123456789";
	std::string Passwort;
	int laenge;
	int erstelle;
	int x = 0;
	srand(timeGetTime());

	std::cout << "Wie lang soll das Passwort werden: ";
	std::cin >> laenge;

	std::cout << "Wie viele Passwoerter sollen erstellt werden: ";
	std::cin >> erstelle;


		for (int i = 0; i < laenge; i++)
		{

			Passwort += Buchstaben[rand() % (sizeof(Buchstaben))];

		}


	std::cout << Passwort << "\n";

	getchar();
	return 0;
}

HolySell · 04/09/2015, 23:10

Quote:
Originally Posted by KingwondA
Danke, hat mir nicht wirklich geholfen.
Kam danach ein weiterer Fehler den hab ich selber behoben.

Stehe grad ziemlich auf den schlauch, auch wenn die Lösung einfach sein mag ich komm einfach nicht drauf. xD

Ich möchte nun, dass ich auch angeben kann wie viele Passwörter ich erstellt haben möchte. Da muss definitiv eine Schleife her, nur bekomme ich es nicht auf die Reihe die richtig zu implementieren^^
Code:
Code:
#include <iostream>
#include <Windows.h>
#include <string>
#include <cmath>
#include <fstream>

#pragma comment(lib, "winmm.lib")

int main()
{
	static const TCHAR* myTitle = TEXT("Passwort-Generator");
	SetConsoleTitle(myTitle);

	char  Buchstaben[] = "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz0123456789";
	std::string Passwort;
	int laenge;
	int erstelle;
	int x = 0;
	srand(timeGetTime());

	std::cout << "Wie lang soll das Passwort werden: ";
	std::cin >> laenge;

	std::cout << "Wie viele Passwoerter sollen erstellt werden: ";
	std::cin >> erstelle;


		for (int i = 0; i < laenge; i++)
		{

			Passwort += Buchstaben[rand() % (sizeof(Buchstaben))];

		}


	std::cout << Passwort << "\n";

	getchar();
	return 0;
}

Du pflanzt die for-Loop einfach in eine neue Loop.

PHP Code:


			
    for (int j = 0; j < erstelle; j++)
    {
        for (int i = 0; i < laenge; i++)
        {

            Passwort += Buchstaben[rand() % (sizeof(Buchstaben))];

        }
        //Gib das Passwort aus oder schiebs in ein Array oder mach sonst was damit
    }

KingwondA · 04/10/2015, 00:20

Das hatte ich versucht, dass klappt so aber nicht.
Weil das Passwort immer wieder das gleiche ist, nur dass mehr Buchstaben drangehängt werden.
Aus "EE3g7" wird "EE3g7Jd4c7" und immer so weiter, bis die zu erstellende Menge erreicht ist, dabei wird aber die eigentlich Passwort länge überzogen.

xinternet · 04/10/2015, 00:23

setz passwort einfach vor der erstellung auf ""

KingwondA · 04/10/2015, 00:33

Quote:

Originally Posted by xinternet

setz passwort einfach vor der erstellung auf ""

Geht immer noch nicht

HolySell · 04/10/2015, 00:35

PHP Code:


			
#include <iostream>
#include <Windows.h>
#include <string>
#include <cmath>
#include <fstream>

#pragma comment(lib, "winmm.lib")

int main()
{
    static const TCHAR* myTitle = TEXT("Passwort-Generator");
    SetConsoleTitle(myTitle);

    char  Buchstaben[] = "AaBbCcDdEeFfGgHhIiJjKkLlMmNnOoPpQqRrSsTtUuVvWwXxYyZz0123456789";
    std::string Passwort;
    int laenge;
    int erstelle;
    int x = 0;
    srand(timeGetTime());

    std::cout << "Wie lang soll das Passwort werden: ";
    std::cin >> laenge;

    std::cout << "Wie viele Passwoerter sollen erstellt werden: ";
    std::cin >> erstelle;


    for (int j = 0; j < erstelle; j++)
    {
        for (int i = 0; i < laenge; i++)
        {

            Passwort += Buchstaben[rand() % (sizeof(Buchstaben))];

        }
        std::cout << Passwort << "\n";
        Passwort = "";
    }



    getchar();
    return 0;
}

So sollte es nun klappen.

Da du immer nur was neues dran hängst sollte nach jedem neuen Passwort die Variable auf "" gesetzt werden.

Delinquenz · 04/13/2015, 02:11

Wieso ist passwort nicht einfach eine lokale Variable der ersten for-Schleife?

C++11 bietet übrigens mit std::random eine tolle Alternative zu rand().

KingwondA · 04/13/2015, 15:35

Quote:

Originally Posted by Delinquenz

Wieso ist passwort nicht einfach eine lokale Variable der ersten for-Schleife?

C++11 bietet übrigens mit std::random eine tolle Alternative zu rand().

höh?
std::random ist mir neu

-ist es aber auch besser als srand(timeGetTime());-
Ist doch nicht schlimm, dass "Passwort" keine lokale Variable ist oder?

HolySell · 04/13/2015, 15:45

Quote:

Originally Posted by KingwondA

höh?
std::random ist mir neu
-ist es aber auch besser als srand(timeGetTime());-
Ist doch nicht schlimm, dass "Passwort" keine lokale Variable ist oder?

Wenn's ne lokale Variable wäre, müsstest du die Variable nicht nach jedem Durchlauf leeren.

KingwondA · 04/13/2015, 16:29

Quote:

Originally Posted by HolySell

Wenn's ne lokale Variable wäre, müsstest du die Variable nicht nach jedem Durchlauf leeren.

Ist doch wumpe, ob leeren oder lokal halten^^

Delinquenz · 04/13/2015, 20:48

Quote:

Ist doch wumpe, ob leeren oder lokal halten^^

Wozu braucht man schon Scopes, einfach alles global machen!

KingwondA · 04/13/2015, 21:19

Quote:

Originally Posted by Delinquenz

Wozu braucht man schon Scopes, einfach alles global machen!

Ich versteh jetzt nicht was ihr wollt?^^
Das Programm läuft einwandfrei.
Wieso sollte ich dann am Code was ändern bzw. die Variable "Passwort" Global machen.
Nur damit ich die nicht leeren muss?
Wäre für mich kein Grund das zu machen!

.SlaP · 04/20/2015, 17:07

Die Rand function ist schwachsin , benutzt std::random