Почему указатель на арифметику указателя терпит неудачу в моем условном утверждении? - PullRequest
Новая
       25

Почему указатель на арифметику указателя терпит неудачу в моем условном утверждении?

0 голосов
/ 09 ноября 2018

sm - это двумерный массив указателей символов, размещаемых динамически. Мне нужно понять, почему мой указатель на арифметику указателей терпит неудачу в условном, если в структуре цикла.

2-й столбец в sm - это строка, которую мне нужно проверить с помощью ключа оценки gk, который представляет собой массив символов / строку. s содержит размер строки, а q - размер столбца для двумерного массива, hm - мой счетчик кучи памяти для функции освобождения, которая не импортируется для моего вопроса. 

double *cals(char **sm, char *gk, int s, int q, unsigned *hm) {
    int c = 0;
    double *savg = malloc(s * sizeof(double));
    assert(savg);
    *hm += 1;

    for (int i = 0; i < s; *(savg + i) = c / q * 100 , c = 0,  ++i) {
        for (int j = 0; j < q; ++j) {
            if (*(*(sm + i * STUDENTATT + 1) + j) == *(gk + j)) 
                ++c;
        }
    }
    return savg;
}

1 Ответ

0 голосов
/ 10 ноября 2018

Не дано много информации о назначении функции cals, поэтому мне пришлось сделать несколько предположений, чтобы написать этот ответ.

Успенская-1 (смысл) : - Вы хотите узнать, сколько символов в двух строках равны (не все символы), а затем найти процентное соотношение этих же символов к общему количеству символов. Если это так, используйте приведенный ниже код. 

double *cals(char **sm, char *gk, int s, int q, unsigned *hm) {
    float c = 0;        // To force float division the c is declared as a float variable
    double *savg = malloc(s * sizeof(double));
    assert(savg);
    *hm += 1;       
    char* sm_i_key = NULL;
    unsigned int strlen_gk = strlen(gk);    
    unsigned int key_length = string_gk;

    for (int i=0; i<s; ++i) { //The calculation is moved inside for loop
            sm_i_key = *(sm+i*q+1); // You can also use sm_i_key = &sm[i*q+1]
            /* Uncomment this section if length of 2 strings are not bound to be equal
            if(strlen(sm_i_key) < strlen_gk){
                    key_length = sm_i_key; 
            }
            else{
                    key_length = strlen_gk
            }
            */
            for (int j = 0; j < key_length; ++j) {
                    if (sm_i_key[j] == gk[j]) 
                        ++c;
            }
            savg [i] = c / strlen_gk * 100; /* Since gk is the grade key it is assumed
            to be equal to the total number.*/
            c = 0;
        }          
    return savg;
}

Успенская-2 : - Вы хотите проверить, соответствуют ли строки, чей начальный адрес хранится во втором столбце каждой строки двумерного массива sm, строке, сохраненной в массиве, указанном gk, и затем вычислить значение (double). Функция cals возвращает только 0,0 или 100,0, так как формула avgs[i]=c / q * 100 будет выдавать только 0, если строки не равны (поскольку целочисленное деление c / q всегда будет приводить к 0, если c меньше q, как здесь), и 100, если строки равны (Тогда зачем использовать double для сохранения значения, если сохранены только 0 и 100).

Если это так, то то, что вы делаете здесь, хорошо, если только массив gk и массив sm[i][2] не имеют разной длины строки (не q). Было бы лучше использовать strncmp для проверки равенства строк, если длина строки двух массивов должна быть разной. Используйте следующий код для этого: - 

double *cals(char **sm, char *gk, int s, int q, unsigned *hm) {
    int c;
    char* sm_i_key = NULL;
    double *savg = malloc(s * sizeof(double));
    assert(savg);
    *hm += 1;       
    for (int i=0; i < s;++i){//The calculation is moved to a static assignment given below
        if(strncmp(sm_i_key, gk, strlen(gk) == 0)
        {
            savg[i] = 100.0;    // Since c/q * 100 => 100.0 if q == c
        }
        else
        {
            savg[i] = 0.0;      /*Since c/q *100 => 0.0 if q < c since integer 
            division will result in 0.*/
        }
    }              
    return savg;
}

Надеюсь, это поможет.

...