[C#]Generatore di Password

[Kleidemos]

Sto cercando di fare un generatore di password in C# ed ecco che ho pensato:

// 
//
//Generatore di password (Console version) 0.1 Alpha
//
//
using System;
using System.Random;

// La classe contenente il generatore
public class PasGen {

	public PasGen() {
		Console.WriteLine("\t\t -- Benvenuto nel generatore di password! -- \n");
	    Console.WriteLine("Inserisci di quanti caratteri vuoi la password:\n");
		int quant= Console.ReadLine();
	}
	// L'array delle vocali
	public string[] vocali= new string[4];
	vocali[0]= "a";
	vocali[1]= "e";
	vocali[2]= "i";
	vocali[3]= "o";
	vocali[4]= "u";
	// L'array delle consonanti
	public string[] consonanti= new string[20];
	consonanti[0]= "b";
	consonanti[1]= "c";
	consonanti[2]= "d";
	consonanti[3]= "f";
	consonanti[4]= "g";
	consonanti[5]= "h";
	consonanti[6]= "l";
	consonanti[7]= "m";
	consonanti[8]= "n";
	consonanti[9]= "p";
	consonanti[10]= "q";
	consonanti[11]= "r";
	consonanti[12]= "s";
	consonanti[13]= "t";
	consonanti[14]= "v";
	consonanti[15]= "z";
	consonanti[16]= "$";
	consonanti[17]= "w";
	consonanti[18]= "x";
	consonanti[19]= "y";
	consonanti[20]= "@";
            // la gunzione che genera la password
 		public void Genera(int quantiCar) {
 			int quanto = quantiCar;
 			int i, y;
 			string pwd;
 			for (i=0;i<quanto;i++) {
 				for (y=0;i<quanto+5;y++) {
					Random rdm1 = newRandom(unchecked((int)DateTime.Now.Ticks));
					Random rdm2 = newRandom(unchecked((int)DateTime.Now.Ticks)); 
					Random rdm3 = newRandom(unchecked((int)DateTime.Now.Ticks));
					Random rdm3 = newRandom(unchecked((int)DateTime.Now.Ticks));
					pwd = consonanti[y]+ vocali[rdm1] +vocali[rdm2]+ vocali[rdm3]+consonanti[y]; 
 				}
 			return pwd;
 			} 			
 		Console.WriteLine("Ecco la password:{0}\n", pwd);	
 		}

	
}

public class Print:PasGen {
	public static void Main(){
	PasGen gen = new PasGen();
	gen.Genera(quant);	
	}
}

Ma mi da:

[soalle]

Ma ci vedo storto io o il codice che c'è nell'immagine è diverso da quello te postato????

quello da te postato c'è vocali[0]="a";
quello nell'immagine (se non ci vedo storto) vocali[[0]]="a";

[Kleidemos]

Originally posted by "soalle"

Ma ci vedo storto io o il codice che c'è nell'immagine è diverso da quello te postato????

quello da te postato c'è vocali[0]="a";
quello nell'immagine (se non ci vedo storto) vocali[[0]]="a";

To, ora lo aggiorno
Che vor di?

[soalle]

Due o tre cose... innanzi tutto quando definisci un array tipo new int32[4] gli indici di questo array vanno da 0 a 3 per un totale di 4 elementi...
Prova ad inizializzarlo così:
String[] vocali= new String{"a","e","i","o","u"};

Ciaociao
Soalle

PS: hai pensato ad usare un array di caratteri??

[Kleidemos]

// 
//
//Generatore di password (Console version) 0.1 Alpha
//
//
using System;

// La classe contenente il generatore
public class PasGen {

	public PasGen() {
		Console.WriteLine("\t\t -- Benvenuto nel generatore di password! -- \n");
	    Console.WriteLine("Inserisci di quanti caratteri vuoi la password:\n");
	    this.Genera();
	}
	// L'array delle vocali
        public String[] vocali= new String[]{"a","e","i","o","u"}; 
	// L'array delle consonanti
    public String[] consonanti= new String[]{"b","e","d","f","g", "h", "l", "m", "n", "p", "q", "r", "s", "t", "v", "z", "w", "x", "$", "@"}; 
     // la gunzione che genera la password
 	public void Genera(int quantiCar) {
		int quanto = quantiCar;
		int i, y;
		string pwd;
			for (i=0;i<quanto;i++) {
				for (y=0;i<quanto+5;y++) {
							pwd = consonanti[y]+ vocali[i] +vocali[i]+ vocali[i]+consonanti[y];
							public string password= pwd;
				}
			}
 	}
 		Console.WriteLine("Ecco la password:{0}\n", password);	
 		}
	
    }

public class Print:PasGen {
	public static void Main(){
	PasGen gen = new PasGen();
    int prova_prima= 10;
	gen.Genera(prova_prima);	
	}
}

Ora dove sbaglio?

[soalle]

se fai la cortesia di postare l'errore.... non ho un compilatore al posto della testa...

[Kleidemos]

// 
//
//Generatore di password (Console version) 0.1 Alpha
//
//
using System;

// La classe contenente il generatore
public class PasGen {
		public PasGen(int quanto) {
		this.Genera(quanto);	
		}
		// L'array delle vocali
		public String[] vocali= new String[]{"a","e","i","o","u"}; 
		// L'array delle consonanti
		public String[] consonanti= new String[]{"b","e","d","f","g", "h", "l", "m", "n", "p", "q", "r", "s", "t", "v", "z", "w", "x", "$", "@"}; 
		 // la gunzione che genera la password
		public void Genera(int quantiCar) {
			int i, y;
			string pwd;
				for (i=0;i<quantiCar;i++) {
					for (y=0;i<quantiCar+5;y++) {
						pwd = consonanti[y] + vocali[i] + vocali[i ]+ vocali[i] + consonanti[y];
						public string password = pwd;
					}
					
				}
				return password;
			    Console.WriteLine("Ecco la password:{0}\n", password);
		}
 	
}

public class Print {
	public static void Main(){
	Console.WriteLine("\t\t -- Benvenuto nel generatore di password! -- \n");
	Console.WriteLine("Inserisci di quanti caratteri vuoi la password:\n");
	PasGen gen = new PasGen();
    int prova_prima= 10;
	gen.Genera(prova_prima);	
	}
}

Errori:
[/quote]

[soalle]

Cerca di strutturare meglio il programma in termini di classi... perché la classe Print deve ereditare da PasGen? Comunque l'errore è probabile che sia nel costruttore di PasGen che richiama Genera senza passargli alcun parametro...

[soalle]

oppss hai cambiato il codice mentre rispondevo....

[soalle]

o levi la chiamata al metodo genera nel costruttore oppure quando crei l'oggetto gli passi effettivamente un numero e poi (poiché è il costruttore che si occpua di chiamare Genera) eviti di chiamare Genera....

[Kleidemos]

quindi cosi

	public PasGen(int quanto) {
		this.Genera(quanto);	
		}

nn va?

[soalle]

nooooooo!!! perché dalla classe print richiami:
PasGen gen=new PasGen()

ma non esiste nessun costruttore di PasGen senza parametri!!! quindi nella chiamata al costruttore effettuata nella classe print gli passi un intero (e dopo eviti di chiamare Genera perché lo chiama già il costruttore):

PasGen gen=new PasGen(10);

Oppure togli la chiamata a Genera dal costruttore di PasGen e gli levi anche il parametro intero che deve ricevere e poi dopo richiami tu la funzione Genera

PasGen gen=new PasGen();
gen.Genera(10);

Peccato che con questo secondo metodo il costruttore perde di significato allora lo levi e poi, poiché non ha senso neppure un metodo di istanza, fai il metodo genera statico...

[Kleidemos]

con questo:

// 
//
//Generatore di password (Console version) 0.1 Alpha
//
//
using System;

// La classe contenente il generatore
public class PasGen {
		public PasGen(int quanto) {
		this.Genera(quanto);	
		}
		// L'array delle vocali
		public String[] vocali= new String[]{"a","e","i","o","u"}; 
		// L'array delle consonanti
		public String[] consonanti= new String[]{"b","e","d","f","g", "h", "l", "m", "n", "p", "q", "r", "s", "t", "v", "z", "w", "x", "$", "@"}; 
		 // la gunzione che genera la password
		public void Genera(int quantiCar) {
			int i, y;
			string pwd;
				for (i=0;i<quantiCar;i++) {
					for (y=0;i<quantiCar+5-1;y++) {
						pwd = consonanti[y] + vocali[i] + vocali[i ]+ vocali[i] + consonanti[y];
						public string password = pwd;
					}
					
				}
				return password;
			    Console.WriteLine("Ecco la password:{0}\n", password);
		}
 	
}

public class Print:PasGen {
	public static void Main(){
	Console.WriteLine("\t\t -- Benvenuto nel generatore di password! -- \n");
	Console.WriteLine("Inserisci di quanti caratteri vuoi la password:\n");
	int prova_prima= 10;
	PasGen gen = new PasGen(prova_prima);
   
	
	}
}

Mi da sempre sti erreri!!!!!!!!
Ma dove azzo sono?

[soalle]

ma lo sai quello cha stai facendo nell'algoritmo???

nel for metti una variabile che è valida solo nel blocco del for e NON deve avere la keywork "public"; togliendo quella rimane un problema: la variabile password oltre che essere inutile è valida come dicevo prima solo nello scope del for quindi quando esci dal for password non esiste più e return password naturalmente dda dei problemi. Ma c'è un altro errore la tua funzione l'hai dichiarata come ua procedura che non restituisce niente ("void Genera....") quindi non ha neppure senso il return. Inoltre perché continui a fare derivare Print da PasGen che è concettualmente sbagliato??? Infine con un parametro errato il tuo programmino genererà un'eccezione che poiché non è stata colta farà scoppiare tutto il tuo computer....
Se chiamo PasGen(100) lui nel for prima o poi inizierà ad andare a prendere delle stringhe nell'array passando degli indici al di fuori della lunghezza dell'array-> ArrayOutOfBoundException o qualcosa del genere...

PS: Ero sotto linux e ho provato a compilare con mono

[Kleidemos]

Prova a compilare questo e dirmi che ti dice:

using System; 
// La classe contenente il generatore 
public class PasGen {
	  public string pwd;
      public PasGen(int quanto) { 
      this.Genera(quanto);    
      } 
      // L'array delle vocali 
      public String[] vocali= new String[]{"a","e","i","o","u"}; 
      // L'array delle consonanti 
      public String[] consonanti= new String[]{"b","e","d","f","g", "h", "l", "m", "n", "p", "q", "r", "s", "t", "v", "z", "w", "x", "$", "@"}; 
       // la gunzione che genera la password 
      public string Genera(int quantiCar) { 
         int i, y;
      	 double min= 1;
      	 int max=29; 
            for (i=0;i<quantiCar*System.Math.Round(min,max);i++) { 
               for (y=0;i<quantiCar*System.Math.Round(min, max);y++) { 
                  pwd = consonanti[y] + vocali[i] + vocali[i ]+ vocali[i] + consonanti[y];  
               }                 
            }        
         return pwd;
              
      }     
} 

public class Print { 
   public static void Main(){ 
   Console.WriteLine("\t\t -- Benvenuto nel generatore di password! -- \n"); 
   Console.WriteLine("Inserisci la percentuale di complessita che vuoi nella password:\n"); 
   int prova_prima= 10; 
   PasGen gen = new PasGen(prova_prima);
   string password;
   password = gen.pwd;	
   System.Console.WriteLine("Ecco la password:{0}\n", password); 
    
   } 
}

[soalle]

Sono forse il tuo paggio...
Il problema è quello che ti ho spiegato prima... ma a quanto pare non ci senti....
vocali è un array lungo 5. ci accedi con un for i cui indici vanno da 0 a quantiCar*(sbrodaglia); quanticar=10 la sbrodaglia=1 quindi tu accedi nell'array con indici fino a 9 -> Exception...
Ti conviene prima guardarti un po' di programmazione di base...

[cn73]

Che maestro paziente...

[Mezzetti0903]

[Kleidemos]

Originally posted by "soalle"

Sono forse il tuo paggio...

Poi in pvt decidiamo la cifra!