-3

メインからlista->arch->のデータにアクセスしたいときに、この特定のコードに問題がありますnombreArchivo。プログラムがクラッシュします。デバッグすると、問題は「セグメンテーション違反」タイプのエラーであることがわかりました。割り当てが不足していると思いますが、どこにあるのかわかりません。

注: 私がやりたいのは、のリスト (listaArchivo) を作成することです。struct _archivoそのため、関数にエラーがある可能性がありますinsertarArchivoListaArchivos()

main() コードは次のとおりです。

#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <cstdlib>
#include <conio.h>

#include "Sistema.h"
#include "Archivo.h"
#include "Directorio.h"
#include "ListaArchivos.h"
#include "Constantes.h"
using namespace std;

int main()
{
    /* VARIABLES */
    Cadena c_DIR=(char *)"DIR";
    Cadena c_CREATEFILE=(char *)"CREATEFILE";
    Cadena c_DELETE=(char *)"DELETE";
    Cadena c_ATTRIB=(char *)"ATTRIB";
    Cadena c_IF=(char *)"IF";
    Cadena c_DF=(char *)"DF";
    Cadena c_TYPE=(char *)"TYPE";
    Cadena c_CREARSISTEMA=(char *)"CREARSISTEMA";
    Cadena c_DESTRUIRSISTEMA=(char *)"DESTRUIRSISTEMA";
    Cadena c_EXIT=(char *)"EXIT";
    Cadena Segmento;
    Cadena comando;
    Cadena parametro1;
    Cadena parametro2;
    bool seguir;
    int contador=0;
    char frase[50];
    Archivo arch;
    ListaArchivos lista;


    do{
    cout <<"> ";

    /* ALGORITMO PARA DIVIDIR LA FRASE EN TOKENS */
    fgets(frase,50,stdin);
    Segmento=strtok(frase," ");
    while(Segmento!=NULL){
             if( contador == 0 ){
                 comando=Segmento; /* GUARDA EL COMANDO */
             }
             if ( contador == 1 ){
                  parametro1=Segmento; /* GUARDA EL PRIMER PARAMETRO */
             }
             if( contador == 2 ){
                 parametro2=Segmento; /* GUARDA EL SEGUNDO PARAMETRO */
             }
             Segmento=strtok(NULL, " ");
             contador++;
    }

    /* SELECCIONA EL COMANDO INDICADO, COMPARANDO LA CADENA COMANDO CON LAS DIFERENTES CADENAS */
    if (strcmp(comando,c_DIR)== 0){
        cout <<"Uso comando DIR"<<endl<<endl;
    }
    else{
        if (strcmp(comando,c_CREATEFILE)== 0){

            lista=crearListaArchivos(); //CREATE LIST OF ARCH.


            cout<<"LISTA CREted, SU DIRECCION ES "<< &lista<<endl;

            arch=crearArchivo(parametro1); //CREATE ARCH
            cout<<"ARCHIVO CREAted, SU DIRECCION ES"<<&arch<<" SU NOMBRE ES: "<<arch->nombreArchivo<<endl;

            insertarArchivoListaArchivos(lista,arch); //INSERTS ARCH IN LIST OF ARCHS

            cout <<" THE ARCHIVO IN lista es named ..."<<lista->arch->nombreArchivo<<endl<<endl;
        }  /*the problem seems to be when I make this call*/ 

...............................
...............................
...............................

アーカイブヘッダーは次のとおりです。

struct _archivo{  //aca declaro la estructura de un archivo
    Cadena nombreArchivo;
    Cadena Atributos;
    Cadena Contenido;
    _archivo *Siguiente;
    _archivo *Anterior;
};
typedef _archivo* Archivo;

/*
 * Crea e inicializa un archivo con nombre 'nombreArchivo'
 */
Archivo crearArchivo(Cadena nombreArchivo){

    //inicializo
    Archivo nuevoArchivo=new _archivo;

    nuevoArchivo->nombreArchivo=nombreArchivo;
    nuevoArchivo->Siguiente=NULL;
    nuevoArchivo->Anterior=NULL;
    return nuevoArchivo; //devuelvo el archivo inizializado
}

これが listaArchivos ヘッダーです。

struct _listaArchivos{ //defino la estructura _listaArchivos, que sera una lista de nodos de tipo Archivo, definidos en el header Archivo.h
    Archivo arch;
};
typedef _listaArchivos* ListaArchivos;

/*
 * Crea, inicializa y retorna una nueva lista de archivos.
 */
ListaArchivos crearListaArchivos(){
     ListaArchivos lista = new _listaArchivos;
     lista->arch=NULL;
     return lista;
}
/*
 * Inserta el archivo 'archivo' en la lista 'lista'.
 */
void insertarArchivoListaArchivos(ListaArchivos &lista, Archivo archivo){
    ListaArchivos cabezal;

    if (lista==NULL){
        lista->arch=archivo;
        lista->arch->Anterior=NULL;
        lista->arch->Siguiente=NULL;
    }
    else{
        cabezal=lista;
        while (!cabezal->arch->Siguiente)
            cabezal->arch=cabezal->arch->Siguiente;
        cabezal->arch->Siguiente=archivo;
        archivo->Siguiente=NULL;
        archivo->Anterior=cabezal->arch->Siguiente;
    }
}

私に何ができる?その間、ポインター/リストについて読み続け、理解しようとします。

4

1 に答える 1

0

あなたの問題につながる可能性があることがわかったいくつかの問題を次に示します。

文字列を使用する場合、文字ゼロ、「0」、および数値 0 には違いがあります。数値のテキスト表現を内部表現に変換する必要があります。

質問に C++ のタグを付けたので、C 関数ではなくstd::stringとメソッドを使用することをお勧めします。StackOverflow で「[c++] parse number」を検索します。 findstrtok

マジック ナンバーの代わりに名前付き識別子を使用します。
たとえば、次のように使用します。

`#define MAXIMUM_ARRAY_SIZE 50`  
`//...`  
`char frase[MAXIMUM_ARRAY_SIZE];`  
`//...`  
`fgets(frase, MAXIMUM_ARRAY_SIZE, stdin);`   

これにより、1 か所で番号を変更するだけで済み、すべてのコードで同じ番号が使用されるようになります。

C++ を使用std::listしているので、独自のリンク リストを作成して時間を無駄にしないでください。はstd::listすでにテストされているため、テストする必要はありません。

C++ を使用してメニュー選択を読み込むには、次を試してください。

int selection;
cin >> selection;
switch (selection)
{
  //...
}
于 2013-11-03T18:10:50.667 に答える