Функция main получила неправильные значения из подпрограммы

Question

Этот код не работает - проблемы с передачей данных из подпрограммы в основной и выделения памяти.

Вычисления правильны внутри подпрограммы, но значения, принимаемые основным, неверны - переменные в главном имеют случайные значения, например sRates.

#include <stdio.h> 
#include <malloc.h> 
#include "sndfile.h" 

int main(int argc, char *argv[]) 
{ 
    int sRates , sRatem , ret; 
    long nSamples=0, nSamplem; 
    float *datas, *datam; 


    printf("Read Test\n"); 
    if (argc != 3) { 
     fprintf(stderr, "Expecting two wav file as argument\n"); 
     return 1; 
    } 


    ret = readWav(argv[1], nSamples, sRates, &datas); 
    if (ret != 0) { 
    printf("Error\n"); 
    } 
    // Output Info 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
     nSamples, argv[1], sRates, (float)nSamples/sRates); 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
     nSamples, argv[1], sRates, (float)nSamples/sRates); 

// free(datas); 

    return 0; 
} 

int readWav(char *fname, long *numFrames, int *sRate, float **buffer ) 
{ 

    // Open sound file 
    SF_INFO sndInfo; 
    SNDFILE *sndFile = sf_open(fname, SFM_READ, &sndInfo); 
    if (sndFile == NULL) { 
     fprintf(stderr, "Error reading source file '%s': %s\n", fname, sf_strerror(sndFile)); 
     return 1; 
    } 


    printf("1Format of the audio file = %i\n", sndInfo.format); 
    printf("2Number of channels = %i\n", sndInfo.channels); 
    printf("3Sample Rate = %d\n", sndInfo.samplerate); 
    printf("4 Sample count = %ld\n", (long)sndInfo.frames); 
    sRate= sndInfo.samplerate; 

    // Allocate memory 
    buffer = (float *)malloc(sndInfo.frames * sndInfo.channels * sizeof(float)); 
    if (buffer == NULL) { 
     fprintf(stderr, "Could not allocate memory for file\n"); 
     sf_close(sndFile); 
     return 1; 
    } 

     // Load data 
    numFrames = sf_readf_float(sndFile, buffer, sndInfo.frames); 

    // Check correct number of samples loaded 
    if (numFrames != sndInfo.frames) { 
     fprintf(stderr, "Did not read enough frames for source\n"); 
     sf_close(sndFile); 
     free(buffer); 
//  return 1; 
    } 
    else { 
     printf("Successfully read file\n"); 
     numFrames = sndInfo.frames; 
    } 
    // Output Info 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
//  numFrames, fname, sndInfo.samplerate, (float)numFrames/sndInfo.samplerate); 
     numFrames, fname, sRate, (float)numFrames/sndInfo.samplerate); 


    sf_close(sndFile); 
// return(buffer); 
    return(0); 

} 

Answer 1

У вас есть несколько ошибок в коде

1. Вы не декларировать readWav() и вы называете его от main(), это работает по совпадению, а именно потому, что он возвращает int.

2. Вы передаете адрес datas к readWav(), обратите внимание, что &datas имеет тип float ** и readWav() ожидает float *.

   Если у вас были предупреждения о компиляторе, вы бы это заметили.

3. Вы передаете значение nSamples и sRate к readWav() и вы ожидали nSamples и sRate в главном, чтобы инициализировать, вам нужно передать их адреса вместо.

4. Вы проверяете возвращаемое значение readWav(), и все же вы по-прежнему пытаетесь ввести указатель datas.

Это исправленная версия кода

#include <stdio.h> 
#include "sndfile.h" 

int readWav(const char *const fname, long *numFrames, int *sRate, float **buffer); 

int main(int argc, char *argv[]) 
{ 
    int sRates, sRatem, ret; 
    long nSamples = 0, nSamplem; 
    float *datas, *datam; 


    printf("Read Test\n"); 
    if (argc != 3) { 
     fprintf(stderr, "Expecting two wav file as argument\n"); 
     return 1; 
    } 


    ret = readWav(argv[1], &nSamples, &sRates, &datas); 
    if (ret != 0) { 
     printf("Error\n"); 
     return 1; 
    } 
    // Output Info 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
     nSamples, argv[1], sRates, (float)nSamples/sRates); 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
     nSamples, argv[1], sRates, (float)nSamples/sRates); 
    free(datas); 

    return 0; 
} 

int readWav(const char *const fname, long *numFrames, int *sRate, float **buffer) 
{ 
    // Open sound file 
    SF_INFO sndInfo; 

    if ((sRate == NULL) || (numFrames == NULL) || (buffer == NULL)) { 
     fprintf(stderr, "Invalid arguments passed to readWav()\n"); 
     return 1; 
    } 

    SNDFILE *sndFile = sf_open(fname, SFM_READ, &sndInfo); 
    if (sndFile == NULL) { 
     fprintf(stderr, "Error reading source file '%s': %s\n", fname, sf_strerror(sndFile)); 
     return 1; 
    } 

    printf("1Format of the audio file = %i\n", sndInfo.format); 
    printf("2Number of channels = %i\n", sndInfo.channels); 
    printf("3Sample Rate = %d\n", sndInfo.samplerate); 
    printf("4 Sample count = %ld\n", (long)sndInfo.frames); 

    // Allocate memory 
    *buffer = malloc(sndInfo.frames * sndInfo.channels * sizeof(float)); 
    if (*buffer == NULL) { 
     fprintf(stderr, "Could not allocate memory for file\n"); 
     sf_close(sndFile); 

     return 1; 
    } 

    *sRate = sndInfo.samplerate; 
    // Load data 
    *numFrames = sf_readf_float(sndFile, *buffer, sndInfo.frames); 
    // Check correct number of samples loaded 
    if (*numFrames != sndInfo.frames) { 
     fprintf(stderr, "Did not read enough frames for source\n"); 
     sf_close(sndFile); 
     free(*buffer); 
    } 
    else { 
     printf("Successfully read file\n"); 
     *numFrames = sndInfo.frames; 
    } 
    // Output Info 
    printf("Read %ld frames from %s, Sample rate: %d, Length: %fs\n", 
     *numFrames, fname, *sRate, (float)*numFrames/sndInfo.samplerate); 

    sf_close(sndFile); 
    return(0); 

} 

Совет: Вы должны попытаться написать функцию таким образом, что он имеет только одну точку выхода, я люблю использовать goto для что, несмотря на то, что религиозные программисты верят в goto, он делает ваш код более понятным и совместимым.

То, что я имею в виду, вы можете иметь ярлык, где вы возвращаете код ошибки из функции и делать все очистки, что-то вроде этого

int function() 
{ 
    /* set errorCode */ 
    if (firstFailureCondition == 1) 
     goto cleanup; 
    if (secondFailureCondition == 1) 
     goto cleanup; 
       . 
       . 
       . 
    if (nthFailureCondition == 2) 
     goto cleanup; 
cleanup: 
    /* do your cleanup */ 
    return errorCode; 
} 

Answer 2

В C все аргументы передаются по значению, так что если вы хотите по-исх-подобный аргумент вы должны передать указатель. И так как вы хотите вернуть float*, вам необходимо пройти float**.

Фактически вы проходите это, но вы не используете его правильно (пожалуйста, используйте -Wall или эквивалент для вашего компилятора, чтобы включить предупреждения).

Код хотел бы более или менее, как это:

int readWav(const char *fname, long *numFrames, int *sRate, float **buffer) 
{ 
    *buffer = malloc(...); 
    //if you do not feel comfortable writing `*buffer` everywhere: 
    float *data = *buffer; 
    ///.... 
    *numFrames = sf_readf_float(...); 
    ///.... 
    *sRate = sndInfo.samplerate; 
    ///.... 
} 

int main() 
{ 
    long nSamples; 
    int sRates; 
    float *datas; 
    ret = readWav(argv[1], &nSamples, &sRates, &datas); 
    //... 
}