Генерация случайных ключей

Question

я получил задание генерировать случайные 80 ключи байт и я решил следующий strateges

в моем компьютере sizeof(char)=1 так я создал массив английских букв алфавита

char *p=" "; 
char a[0..26] and in cycle 


for (int i=0;i<=80;i++){ 
    *(p+i)+= a[(rand()+100) % 26]; 
} 

но это не работать он прекращает выполнение, пожалуйста, помогите извините, если мой код глупо, но я не могу думать в это время иначе благодаря код

#include <iostream> 
#include <string.h> 
#include <cstdlib> 
using namespace std; 
int main(){ 

    char *p=" "; 
    char a[]= { 'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'}; 
for (int i=0;i<=80;i++){ 
     *(p+i)+=(a[(rand()+100)%26]); 
    } 

    cout<<p<<endl; 


    return 0; 

} 

Answer 1

Попробуйте это:

#include <iostream> 
#include <string.h> 
#include <cstdlib> 
using namespace std; 
int main(){ 
    // ensure the target has enough memory for the key and a null terminator 
    char p[81]; 

    // this string will do as nicely as the character array 
    char a[] = "abcdefghijklmnopqrstuvwxyz"; 

    // no += here. I assign the random character directly to the target buffer 
    for (int i=0;i<=80;i++) 
     p[i] = a[rand()%26]; 

    // alternately, you can calculate a random English character with: 
    // p[i] = rand()%26 + 'a'; 
    // which removes the need for the a[] buffer at all 

    // don't forget to null-terminate 
    p[80] = '\0' 

    // output results 
    cout<<p<<endl; 
    return 0; 
} 

Answer 2

Перед записью в цикле вам нужно выделить место для полного размера вашего массива символов p.

---

Update Кстати, rand() не очень случайным, если вы не семя его с уникальным значением первого использования srand.

Answer 3

Вы назначаете символы в своей переменной p, но вы не выделили память для назначения этих символов. Вероятно, вы хотите что-то вроде этого:

char p[81]; 

а затем идите оттуда.

Answer 4

Ну, обычно я бы сказал, что вы должны предоставить больше информации, чем «он прекращает выполнение», но несколько вещей, которые выскакивают на меня:

char *p=" "; 
    char a[]= { 'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'}; 
    for (int i=0;i<=80;i++){ 
     *(p+i)+=(a[(rand()+100)%26]); 
    } 

Ваш цикл записи значений в *p из индекса От 0 до 80 (81 общий элемент).

Первая итерация итерация:

*(p+0) = a[...]; 

будет работать, но второй один

*(p+1) = a[...]; 

должны терпеть неудачу, так как нет зарезервированной памяти по адресу * (р + 1). Это может быть отключено на единицу, если вы можете записать в пространство, зарезервированное для нулевого \0, которое добавлено в строковый литерал.

При объявлении * р как

char *p=" "; 

Вы выделяет только 1 байт. Итак, когда ваш цикл записывается в p [1], p [2] ... вы пытаетесь записать в нераспределенную память. Измените свою декларацию примерно на

char pArr[81]; 
char *p=pArr; 

и перейти оттуда.

Answer 5

Я изменил свой код ... попробовать его.

#include <iostream> 
#include <string.h> 
#include <cstdlib> 
using namespace std; 
int main(){ 

    char *p= new char[81]; 
    char a[]= { 'a','b','c','d','e','f','g','h','i','j','k','l','m','n','o','p','q','r','s','t','u','v','w','x','y','z'}; 
for (int i=0;i<=80;i++){ 
     *(p+i)=(a[(rand()+100)%26]); 
    } 

    cout<<p<<endl; 


    return 0; 

} 

И какой выходной сигнал?

Answer 6

вопрос об этом коде:

Почему генерировать массив буквенных символов? Почему бы не получить случайный, модулировать на 26 и добавить смещение в ASCII «a»? Это экономит выделение памяти и, на мой взгляд, более яснее.

Answer 7

Для полноты:

std::vector<char> BuildKey(std::size_t keyLength) 
{ 
    const char elements[] = "abcdefg0123!@_"; 
    const std::size_t numElements = sizeof(elements)/sizeof(char); 
     //division technically superfluous, as sizeof(char) == 1, but in general its needed. 

    std::vector<char> key; 
    key.reserve(keyLength); 
    for(std::size_t i = 0; i < keyLength; ++i) 
     key.push_back(elements[rand()%numElements]); 
    //there are a handful of ways using standard algorithms to achieve this, 
    // but I'm largely unconvinced the added complexity is justified for simple loops 

    return key; 
} 

Есть еще несколько изменений можно сделать. Я, вероятно, буду параметризировать rand, поэтому я могу позже выбрать PRNG и, возможно, предоставить массив elements в качестве ввода. И string может быть более подходящим, чем vector в зависимости от того, как вы планируете использовать ключ. Или просто примите ForwardIterator и верните void. Много вариантов.