0

我在c中做一个函数,它接收一个文件名,例如“文件名.网络号”,例如:matriz.0 这表示矩阵与一个代码相关联,例如数字1,0是网络数字。这意味着我必须生成一个带有 1 和 0 的数字代码并返回一个 int。该函数编码良好,如果我在 c 中的另一个程序中单独使用它,它可以工作,但随后将其添加到主程序中,我注意到 char * 字符串操作会产生错误。因为如果我评论函数中的 3 行:

filename = strtok (string, ".");
red = strtok (NULL, ".");
codigored = atoi (red);

分段错误消失。我试图分配内存并初始化所有变量,但错误仍然存​​在,所以任何人都可以帮助找到错误?非常感谢您。

以下是c中函数的代码:

int generasubPalabra(char* nombre) 
{
   char *nombrearchivo=NULL; 
   int codigoarchivo=0;  
   char *red=NULL;                
   int codigored=0;                 

   int subpalabra=1;
   nombrearchivo = (char*) malloc (sizeof(char)*50);  //50 chars

   red = (char*) malloc (sizeof(char)*4); 

   char cadena[strlen(nombre)+1];

   strcpy(cadena, nombre);          



   //ACA ESTA EL ERROR: alguna de las 3 funciones, o las 3, genera una violacion de segmento
   //romper cadena en nombre - nro de red
   //Primera llamada => Primer token (primer elemento hasta el punto, es el nombre del archivo)  

   nombrearchivo = strtok( cadena, "." );       
   printf( "nombre del archivo %s\n", nombrearchivo );

 //Segunda llamada => Segundo token (segundo elemento, desde el punto, es el nro de red)
   red = strtok( NULL, "." );                   
   printf( "numero de red %s\n", red );

   codigored=atoi(red);                 //obtengo el entero correspodiente a la red (cast)
   printf( "codigo de red %d\n", codigored );

   //con la siguiente secuencia se traduce el string contenido en nombrearchivo, al codigo correspondiente para la palabra


   if (strcmp(nombrearchivo,"matriz_incidencia")==0) {
       codigoarchivo=1;
   } 
   else if (strcmp(nombrearchivo,"matriz_brazos_inhibidores")==0) {
        codigoarchivo=2;
   }
   else if (strcmp(nombrearchivo,"matriz_prioridades_red")==0) {
        codigoarchivo=3;
   }
   else if (strcmp(nombrearchivo,"matriz_relacion_disparos_distribuidos")==0) {
        codigoarchivo=4;
   }
   else if (strcmp(nombrearchivo,"matriz_prioridades_disparos_distribuidos")==0) {
        codigoarchivo=5;
   }
   else if (strcmp(nombrearchivo,"vector_marcado_inicial")==0) {
        codigoarchivo=6;
   }
   else if (strcmp(nombrearchivo,"vector_cotas_plazas")==0) {
        codigoarchivo=7;
   }
   else if (strcmp(nombrearchivo,"vector_transiciones_automaticas")==0) {
        codigoarchivo=8;
   }
   else if (strcmp(nombrearchivo,"vector_transiciones_noinformadas")==0) {
        codigoarchivo=9;
   }
   else if (strcmp(nombrearchivo,"vector_mascara_interrupciones")==0) {
        codigoarchivo=10;
   }
   else if (strcmp(nombrearchivo,"vector_cola_entrada")==0) {
        codigoarchivo=11;
   }
   else if (strcmp(nombrearchivo,"vector_consulta_disparo_especifico")==0) {
        codigoarchivo=12;
   }
   else if (strcmp(nombrearchivo,"vector_consulta_plaza")==0) {
        codigoarchivo=13;
   }
   else {
        codigoarchivo=31;  //se retorna el ultimo valor posible (5bits =11111)
        printf("No existe el archivo, verifique ubicacion y permisos \n");
   }

   printf( "codigo de archivo %d\n", codigoarchivo );   

   //se concatena en los 5 bits mas significativos la red, y en los siguientes 5 bits mas significativos el codigo de archivo

   subpalabra = 0x000;                      

   subpalabra+= 0x080*codigored;            //red

   subpalabra+= 0x004*codigoarchivo;        //codigo de archivo

   printf("la subpalabra generada es: %x\n", subpalabra);

return subpalabra;

}

这是完整的代码: http ://www.mediafire.com/download/q782v5cmrcdgl2b/matriz.c

运行,使用命令

“./matriz path_where_are_the_matrices”,例如:./matriz /home/eduardo/hpn/matrices

在该路线上,放置一个带有名称的数组的文件:matriz_brazos_inhibidores.1 文件矩阵示例如下 http://www.mediafire.com/download/ur3ucbf8cdmfpmf/matriz_brazos_inhibidores.1

4

2 回答 2

1

正如泰勒弗洛雷斯指出的那样,您在那里有一些有趣的内存分配,但正如您提到的那样,您正在搞砸,我会假设这就是原因。

确保 cadena 的最后一个索引已初始化。如果不是(在名词字符串中没有空终止字符),则抛出空字符,否则 strtok 将尝试访问内存中的 void。

此外,您将需要使用 malloc(正如 Taylor 所提到的)在运行时动态分配和释放内存,不这样做也可能而且经常会导致问题。

于 2013-06-10T00:27:25.527 回答
1

您的代码的主要问题是您绝对没有错误检查。阅读您正在使用的功能,以便您可以判断它们是否成功返回。此外,您对内存的使用完全倒退。malloc 主要用于不提前知道内存大小的情况。你应该像这样使用它:

char nombrearchivo[50]; // instead of malloc (sizeof (char) * 50); 
char * cadena = malloc (strlen(nombre) + 1);

您缺乏错误检查从这里开始:red = strtok(NULL, ".").Check 以确保调用成功或不通过这样做:

if (nombrearchivo){
    red = strtok (NULL, ".");
    codigored = atoi (red);
}

另一个可能的原因是atoi。您是否正在检查它是否red包含一个以空字符结尾的数字字符串?

您是否正在检查以确保它nombre不为空?添加这个:

if (!nombre) return -1; /* Or just check that you aren't sending a null
                         * pointer in the calling function
                         */

我用一些标准做法重写了你的函数,我没有得到任何错误:

int generasubPalabra (char* nombre){

    if (!nombre) return -1;

    char    * nombrearchivo = 0,
            * red = 0,
            * cadena = malloc (strlen (nombre)); /* strlen could lead to
                                                  * a seg fault if it
                                                  * isn't null-terminated.
                                                  */

    int codigoarchivo  = 0,
            codigored  = 0,
            subpalabra = 1;


    if (!cadena) return -1;

    strcpy(cadena, nombre); /* A seg fault wouldn't happen here without 
                             * first happening above, at strlen
                             */

    nombrearchivo = strtok (cadena, ".");

    if (!nombrearchivo) return -1;

    printf ("nombre del archivo %s\n", nombrearchivo);

    red = strtok (0, ".");

    if (!red) return -1;

    printf ("numero de red %s\n", red);

    codigored = atoi (red);
    printf ("codigo de red %d\n", codigored);

         if (!strcmp(nombrearchivo,"matriz_incidencia"))            codigoarchivo = 1;
    else if (!strcmp(nombrearchivo,"matriz_brazos_inhibidores"))    codigoarchivo = 2;
    else if (!strcmp(nombrearchivo,"matriz_prioridades_red"))       codigoarchivo = 3;
    else if (!strcmp(nombrearchivo,"matriz_relacion_disparos_distribuidos"))        codigoarchivo = 4;
    else if (!strcmp(nombrearchivo,"matriz_prioridades_disparos_distribuidos"))     codigoarchivo = 5;
    else if (!strcmp(nombrearchivo,"vector_marcado_inicial"))               codigoarchivo = 6;
    else if (!strcmp(nombrearchivo,"vector_cotas_plazas"))                  codigoarchivo = 7;
    else if (!strcmp(nombrearchivo,"vector_transiciones_automaticas"))      codigoarchivo = 8;
    else if (!strcmp(nombrearchivo,"vector_transiciones_noinformadas"))     codigoarchivo = 9;
    else if (!strcmp(nombrearchivo,"vector_mascara_interrupciones"))        codigoarchivo = 10;
    else if (!strcmp(nombrearchivo,"vector_cola_entrada"))                  codigoarchivo = 11;
    else if (!strcmp(nombrearchivo,"vector_consulta_disparo_especifico"))   codigoarchivo = 12;
    else if (!strcmp(nombrearchivo,"vector_consulta_plaza"))                codigoarchivo = 13;
    else {
            codigoarchivo = 31;  //se retorna el ultimo valor posible (5bits =11111)
            printf ("No existe el archivo, verifique ubicacion y permisos \n");
    }
    printf ("codigo de archivo %d\n", codigoarchivo);

    subpalabra  = 0x000;
    subpalabra += 0x080*codigored;            //red
    subpalabra += 0x004*codigoarchivo;        //codigo de archivo

    printf ("la subpalabra generada es: %x\n", subpalabra);

    return subpalabra;
}

int main (){

    int ret = generasubPalabra ("matriz_brazos_inhibidores.1");

    if (ret < 0)
            printf ("Error occured\n");
    else    printf ("Result = %d\n", ret);

    return 0;
}

更新:

在获得程序的整个源代码后,我发现了一些导致分段错误的东西。为了测试程序,我这样做了:

  • 制作了一个包含一个文件的测试目录
  • 标记文件 testname.1
  • 添加了一些随机单词

以下是我为获得干净运行所做的一些更改:

  • 我使用了我编辑的功能,而不是已经存在的功能(修复了一个错误)
  • 我通过替换将您的一个for循环更改为for ( j = 0; j < espacios; j++)<=

在那个循环的最后,我添加了这个:

     /* other code went here */
     printf("el valor a ser escrito long int: %s\n", msg);
  }
  // strtok right here was causing another seg fault
  pch = strtok (NULL, "\n "); //esto es como un reset de pch
  if (!pch){ // your solution to this problem might be different
     fclose (pFile); // just as long as it's taken care of before
     return 0; // the next iteration of the loop
  }

这可能无法完全解决您程序的问题。我无法添加最有意义的解决方案,因为我不太了解您的所有代码是如何组合在一起的(我不太了解西班牙语)。但是,如果您只是在代码中添加更多错误检查,您将能够解决问题,或者至少可以解决问题的原因。我看到您确实检查了某些功能中的错误,但这还不够。你必须表现得好像一切都不会按计划进行。问问自己这个:

  • 如果目录中的文件名没有.符号怎么办?
  • 如果有一个没有扩展名的文件会发生什么?喜欢filename.
  • 如果文件的扩展名有字母怎么办?IEfilename.423d2
  • 如果文件中没有任何新行怎么办?
  • 如果文件中没有任何内容怎么办?
  • 如果输入的目录不存在怎么办?还是,打错了?

要点是:您必须假设您的程序正在以完全不可预测的输入运行。

于 2013-06-10T00:19:06.960 回答