Discussion :

[Thomus38]

Bonjour,

Après moultes et moultes recherches sur internet et moultes essais infructueux, je viens demander de l'aide à la communauté !

Je suis sous Linux et je souhaiterais générer des chaines de caractères aléatoire et de taille aléatoire (entre 3 et 12 caractères). Ils représenteront des noms de fichiers.
Je souhaiterais que les noms de fichiers contiennent uniquement des caractères dans :
- [a -> z]
- [A -> Z]

Voici le code actuel :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
	srand(time(0));
 
	i = (rand()%10)+3; //Taille aléatoire
 
	for(j = 0; j < i; j++) {
		name[j] = (rand()%82)+32;
	}

Je pense que je n'utilise pas la table ASCII correctement car il me génére des noms de fichiers avec bcp de caractères spéciaux (exemple : '89+<%&O\WD')

Comment faire pour arriver mes fins ?

Merci d'avance,
Thomas

[koala01]

Salut,

Commence déjà par créer la "liste" des caractères que tu accepte, par exemple sous la forme de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
std::string acceptes="abcdefghijklmnopqrstuvwyz";
acceptes+="ABCDEFGHIJKLMNOPQRSTUVWXYZ";

Ensuite, tu peux commencer à créer ta chaine en choisissant la taille de la chaine, puis les caractères parmi ceux qui sont acceptés sous une forme (bien que d'aucuns puissent crier au scandale quant à l'utilisation de rand) de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
std::string filename="";
srand(time(0));
size_t taille = (rand()%10)+3;
for(size_t i=0;i<taille;i++)
{
    size_t pos = rand()%52;//j'ai 26*2 caractères dans acceptes
    filename+=acceptes[pos];
}

Après, "YAPUKA" éventuellement préciser le chemin d'acces et/ou donner une extension et, pour finir à ouvrir le fichier sous la forme de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
filename ="/tmp"+filename; //pas obligatoire
filename+=".tmp"; //pas obligatoire
std::ofstream ofs(filename.c_str());

[ThePiiX]

Sauf erreur (j'ai pas envie de chercher :p ) les caractères majuscules ne suivent pas les caractères minuscules.

Donc je procèderais comme suit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
if(rand()%2 == 0) // pile = majuscule ou face = minuscule
{
name[j] = (rand()%26)+'A'; // 'A' est plus stable que sa valeur ascii (eh oué pense que tu peux changer de système )
}
else
{
name[j] = (rand()%26)+'a';
}

Ça évite d'utiliser std::string et c'est plus rapide.

[Thomus38]

Merci les amis !

J'ai opté pour la première solution...

Thom

[koala01]

Sauf erreur (j'ai pas envie de chercher :p ) les caractères majuscules ne suivent pas les caractères minuscules.

Donc je procèderais comme suit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
if(rand()%2 == 0) // pile = majuscule ou face = minuscule
{
name[j] = (rand()%26)+'A'; // 'A' est plus stable que sa valeur ascii (eh oué pense que tu peux changer de système )
}
else
{
name[j] = (rand()%26)+'a';
}

Ça évite d'utiliser std::string et c'est plus rapide.

Que, du point de vue de la valeur ASCII, les majuscules précèdent les minuscules, cela n'a en définitive rien à voir...

De plus, tu oblige à effectuer deux tirages pour un seul caractère, et donc, j'ai vraiment des doutes quant au fait que cela puisse être plus rapide que l'utilisation de std::string, étant donné que l'opérateur [] est en "acces aléatoire" (cela ne prend pas plus de temps d'accéder à l'élément indicé [3] qu'à l'élément indicé [150000])

En outre, si un jour tu envisages de permettre d'autres caractères (le -, le _ ou les chiffres par exemple), il te "suffira" de les rajouter à la chaine de caractères permis, et de modifier les valeurs de modulo (ce qui pourrait se faire de manière automatique en utilisant acceptes.size() plutôt que des "chiffres magiques")

Enfin, il n' y a rien à faire, les chaines de caractères "C style" sont clairement déconseillées, ne serait-ce que par les problèmes d'allocation dynamique de la mémoire qu'elles impliquent souvent, par le fait que, si tu oublie d'ajouter le '\0' final, tu peux en arriver à faire afficher l'ensemble du contenu de la mémoire ou que tu peux trop facilement essayer d'aller modifier une valeur en mémoire qui soit en dehors du nombre d'éléments réservés à ta chaine de caractères...

Tout cela pour dire que ma solution est à la fois plus simple, plus efficace et plus évolutive que celle que tu donnes

[ram-0000]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
if(rand()%2 == 0) // pile = majuscule ou face = minuscule
{
name[j] = (rand()%26)+'A'; // 'A' est plus stable que sa valeur ascii (eh oué pense que tu peux changer de système )
}
else
{
name[j] = (rand()%26)+'a';
}

Le jour où tu voudras accepter les chiffres, tu prendras une pièce à 3 face

Si je peux me permettre, utilise la méthode décrite par koala ici

avec juste une petite modif :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
for(size_t i=0;i<taille;i++)
{
    //size_t pos = rand()%52;//j'ai 26*2 caractères dans acceptes
    size_t pos = rand()%acceptes.size();//le nombre de caractères dans acceptes
    filename+=acceptes[pos];
}