Использование переменной в качестве параметра массива в C++

Question

Я пытаюсь написать код, который будет сравнивать 10 фрагментов последовательности ДНК с ДНК последовательности 10 родственников. Пользователь вводит свое имя, количество родственников, которых они хотят сравнить, и их ДНК. Компьютер выводит процент соответствия. ATTAGACGCA по сравнению с ATAAGACGCA будет соответствовать 90%. Количество родственников является постоянным после того, как пользователь указывает, сколько родственников. Я попытался использовать const, но, похоже, он не хочет использовать этот номер.

/********************************************************************** 
* Get DNA Sequence 
***********************************************************************/ 
void getMyDNA(char myDNA[]) 
{ 
    cout << "Enter your DNA sequence: "; 
    cin >> myDNA; 
} 

/********************************************************************** 
* Get Potential Relatives 
***********************************************************************/ 
int getRelatives() 
{ 
    int relatives = 0; 
    cout << "Enter the number of potential relatives: "; 
    cin >> relatives; 

    return relatives; 
} 

/********************************************************************** 
* Get Potential Relatives Names 
***********************************************************************/ 
void getRelativeName(string relativeNames[], int relatives) 
{ 
    string name; 
    for (int i = 0; i < relatives; i++) 
    { 
     cout << "Please enter the name of relative #" << i + 1 << ": "; 
     cin >> name; 
     relativeNames[i] = name; 
    } 
} 

/********************************************************************** 
* Get Potential Relatives DNA Sequence 
***********************************************************************/ 
void getRelativeDNA(char relativeDNA[][10], string relativeNames[], int relatives) 
{ 
    for (int i = 0; i < relatives; i++) 
    { 
     cout << "Please enter the DNA sequence for " << relativeNames[i] << ": "; 
     cin >> relativeDNA[i]; 
    } 
} 

/********************************************************************** 
* Display Potential Relatives Match 
***********************************************************************/ 
void displayMatch(string relativeNames, char relativeDNA[][10], int relatives, char myDNA[]) 
{ 
    const int family = relatives; 
    int count[family] = 0; 
    for (int r = 0; r < 3; r++) //relative number r 
    { 
     for (int d = 0; d < 10; d++) //dna piece number d 
     { 
     if (relativeDNA[r][d] == myDNA[d]) 
      count[r]++; 
     } 
    } 
} 

/********************************************************************** 
* Main 
***********************************************************************/ 
int main() 
{ 
    char myDNA[10]; 
    string relativeNames[50]; 
    char relativeDNA[50][10]; 

    // My DNA 
    getMyDNA(myDNA); 


    //# of relatives 
    int relatives = getRelatives(); 
    cout << endl; 

    //thier names 
    getRelativeName(relativeNames,relatives); 
    cout << endl; 


    //their dna 
    getRelativeDNA(relativeDNA,relativeNames,relatives); 
    cout << endl; 

    //display 
    displayMatch(relativeNames,relativeDNA,relatives,myDNA); 

    return 0; 
} 

Answer 1

count Если новый массив создать новый массив динамически, как следует ...

int *count = new int[relatives]; 

Я заметил, что вы используете следующие позже ...

count++; 

ли вы пытаетесь увеличить целое число или переместить указатель? Этот код может помочь сделать его более ясным ...

#include <assert.h> 
#include <iostream> 
#include <typeinfo> 

int main(void) { 
    const int x = 500; 
    int* a = new int[x]; 
    size_t i = 0; 
    for(i = 0; i < x;i++) { 
    a[i] = i; 
    } 

    for(i = 0; i < x;i++) { 
    //Print numbers without moving pointer 
    std::cout << a[i] << std::endl; 
    } 
    for(i = 0; i < x;i++) { 
    //Print numbers moving pointer 
    std::cout << a[0] << std::endl; 
    a++; 
    } 
    a = a - x; 
    delete[] a; 
    return 0; 
} 

Answer 2

Даже если параметр передан как const, он не будет работать. Вы можете попробовать с массива, созданного динамически

int *count = new int[relatives]; 

Answer 3

Вместо

int count[relatives] = 0; 

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

std::vector<int> count(relatives); 

Включите <vector> заголовок.