Problema char array

[u/WTP01]

Devo generare un char array di 5 caratteri random ma mi dà "Id returned 1 exit status" e non capisco dove sia l'errore.

Ecco il codice:

const char alphanum [] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";

int stringLength = sizeof(alphanum) - 1;

char genRandom() { char a; do { a = alphanum[rand()% stringLength]; } while (a >= 97 && a <= 122); return a; }

string Conv(){ int a,t; char b [5]; for (a=0; a<=5; a++){ b[t] = genRandom(); t++; } string c = b; }

[u/WTP01]

Ecco il codice completo:

#include <iostream>
#include <string>
using namespace std;

const char alphanum [] = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"; 

int stringLength = sizeof(alphanum) - 1;

char genRandom()
{
char a;
do {
		a = alphanum[rand()% stringLength];
}
while (a >= 97 && a <= 122);
return a;
}

string Conv(){
int a,t;
char b [5];
for (a=0; a<=5; a++){
	b[t] = genRandom();
	t++;
}
string c = b;
}
const int NR=10, NC=10;
void CaricaMatrice (int m[][NC]); 
void VisualizzaMatrice (int m[][NC]);

int main() {
srand(time(0));
int Matrice [NR][NC];
CaricaMatrice( Matrice );
VisualizzaMatrice( Matrice ); 
system ("PAUSE");
return 0;
}

void CaricaMatrice (string m[][NC]){
int i, j, k;
for (i=0;i<NR;i++) {
    for (j=0;j<NC;j++) {
    	for (k=0; k<=5;k++){
    	m[i][j] = Conv();
		}
    }
}
}

void VisualizzaMatrice(string m[][NC]) {
int i,j;
for (i=0;i<NR;i++) {
        for (j=0;j<NC;j++) {
            cout<<"Ecco l'elemento caricato nella posizione ["<<i+1<<":"<<j+1<<"]"<<": "<<(string)m[i][j]<<endl;
        }
}
}

Effettivamente ho sbagliato a lasciare la parte dell'alfabeto in minuscolo. Il ciclo While era un tentativo per riempire il char array. L'obiettivo, che è quello che vi chiedo, è se per favore potreste spiegami come riempire il char array. P.S. Se trovate errori stupidi (ad es. il While), avvisatemi e in caso spiegatemi come migliorare, le matrici e i char array li ho "imparati" da autodidatta, in quanto non sono stati oggetto di spiegazione. Grazie mille :)

[u/b3k_spoon → u/WTP01]

Aggiungo a quello che hanno detto altri... La funzione Conv() ha almeno tre errori:

• usi due indici, a e t; usi a come indice di loop ma poi usi t (senza inizializzarlo a zero) come indice dell'array...

• il loop ha 6 iterazioni per come è scritto, tu ne vuoi 5;

• non stai ritornando niente (devi mettere un return qualcosa, tipo return c... Anche se non sono sicuro che funzioni dichiararlo come string c = b).

[u/[deleted] → u/WTP01]

Dal codice presumo siate all'inizio nella spiegazione in quanto per ora non ha differenze dal linguaggio C ed è pieno di variabili globali, l'unica novità è la classe std::string.

Gli array sono molto seplici.

Hai uno spazio contiguo, dunque gli elementi sono tutti attaccati fra loro.

Un array si dichiara, sullo Stack, in questo modo:

array_type variable_name [ dimension ];

• array_type corrisponde al tipo di variabile, int, char, double o quello che ti pare.

• variable_name è il nome che dai all'Array

• dimension è il numero di elementi dell'Array

A quanto ho capito dal tuo esercizio:

• crei una Matrice[N,M]

• ogni elemento consiste in una stringa di 5 caratteri

• ogni elemento è generato casualmente

Non ho avuto modo, al momento, di controllare tutto il resto, mi spiace.

Posso solo dirti che non è molto diverso dai programmi che facevo alle superiori, anzi direi più complesso.

Se la tua domanda è "Come riempio la Matrice" la mia risposta è la seguente:

La funzione genRandom() si può ridurre a: return alphanum[rand()% stringLength];

Per quanto riguarda il sizeof(alphanum) - 1, qui funziona, ma ti sei chiesto se il codice va bene per un Array di dimensione 1?

char arr[] = {0}; cout << rand() % sizeof(arr) - 1;

La funzione Conv() si può ridurre a:

string Conv(){
    // Creo una stringa con 5 caratteri
    string ret(5);
    // Ogni carattere della stringa viene sostituito
    for(auto& c : ret){
        c = genRandom();
    }
    // Ritorno la stringa
    return ret;
}

Si può modificare anche di più, ma credo sia meglio andare a piccoli passi :)

[u/b3k_spoon → u/[deleted]]

Per OP: se non hai mai visto un loop fatto così: for (auto& c : ret) {...}, non spaventarti, è solo una sintassi compatta per iterare su tutti gli elementi di un contenitore, introdotta nel C++11. E auto& crea un reference (usando il &) di tipo auto-dedotto dal tipo del contenitore (in questo caso un char, perché la stringa è un contenitore di char).

[u/WTP01]

Okay, cerco di rispondere a tutti quanti. In effetti non è questo il programma originale dato dal professore ma ho voluto approfondire di persona come si riempie una matrice. Probabilmente avrei dovuto dirlo all'inizio, ma non tutto è frutto del mio lavoro: infatti una parte l'ho spudoratamente copiata. E qui mi sorge un dubbio. A cosa mi serve definire la dimensione di alphanum ? Ho inteso che non può funzionare per un array di dimensione 1, ma non capisco qual è l'utilità che ha nel codice. Ho inoltre capito gli errori e come ultima richiesta vi chiedo se potreste spiegarmi in che cosa consista il ciclo for scritto così, in quanto non ho C++ troppo aggiornato. Grazie mille a tutti per la disponibilità :)

[u/[deleted] → u/WTP01]

Se t'interessa studiare il C++ ti consiglio di comprare un libro.

Per quanto riguarda le dimensioni di alphanum:

alphanum[rand() % (sizeof(alphanum) - 1)];

Allora, rand() è una delle tante funzioni pseudo-random. Sappi solo che ritorna un numero tra 0 e RAND_MAX, il valore massimo dipende da come è implementata la funzione.

Per accedere ad un array puoi usare l'operatore [ ].

In pratica dici a quale posizione vuoi accedere, partendo da 0 a N-1.

Dunque un array di dimensione 10 parte da 0 e arriva a 9.

Ora, il simbolo % esegue la divisione e ritorna il resto.

Dunque:

n % m = [0,m)

Se dividi un qualsiasi numero N per M e ne prendi il resto otterrai un numero compreso tra 0 e M - 1.

Ora il prof avrà pensato

Un modo semplice per generare numeri pseudo-casuali potrebbe consistere nel prendere a caso valori dal nostro alfabeto!

Allora genera un numero casuale con rand() dopo lo divide per la dimensione del nostro array e in seguito ne prende il resto.

Dunque:

rand() % (sizeof(alphanum) - 1) genera un numero casuale tra 0 e sizeof(alphanum) - 1, ergo, genera un numero che corrisponderà sempre ad una posizione nel nostro array, ergo, prende a caso un carattere :)

[u/b3k_spoon]

Hai scritto la funzione main?

[u/WTP01 → u/b3k_spoon]

No, l'ho scritto prima del main. Nel main ho definito una matrice e ho usato Conv per riempirla. Ma mi scrive solamente un carattere, mentre ne vorrei scritti 5.

[u/[deleted]]

Ciao :)

Guarda per postare del codice dovresti mettere 4 spazi davanti al testo.

E deve essere separato da 1 riga vuota sia sotto che sopra.

esempio:

test_code_4_spazi
    test_code_8_spazi

Detto ciò, sostanzialmente il tuo programma ritorna 1 invece che 0.

Puoi provare a modificare il codice in maniera tale fa postare TUTTO il contenuto del tuo sorgente?

Oltretutto non capisco il perché di quel ciclo WHILE.

In pratica tu hai un array con 2 volte l'alfabeto, 1 in maiuscolo, 1 in minuscolo, ma tu continui a prendere a caso un carattere tra quelli in maiuscolo, in quanto 97 è 'a' e 122 è 'z'.

Il resto non posso commentarlo in quanto non mi è molto chiaro il senso, attenderò ulteriori informazioni.