Mensajes

[completo]

Si la primera parte de la cédula es un solo carácter lo puedes almacenar utilizando una variable del mismo tipo:

Código (c) [Seleccionar] Expandir

struct ALGO {
   char N;
   int dato;
};

/* ... */

printf("Cedula (X - XXXXXXXX): ");
fgets(d, 30, stdin);
sscanf(d, " %c - %d", &nuevo->N, &nuevo->dato);

Si por alguna razón no es posible y se debe leer como una cadena de un solo carácter puedes indicar el numero máximo de caracteres a leer con:

Código (c) [Seleccionar] Expandir

printf("Cedula (X - XXXXXXXX): ");
fgets(d, 30, stdin);
sscanf(d, "%1s - %d", &nuevo->N, &nuevo->dato);

Si no funciona por favor indícalo y publica si es posible el código fuente completo.

Un saludo

Listo. Muchas gracias, me sirvió usando el "%1s" aunque extrañamente cuando usaba %c y lo mostraba aparecia V=Venezolano, no tengo ni la mas minima idea de donde salio el resto que no fuera V. Aun asi muchas gracias, un abrazo.

Tengo un programa que necesito que me lea clientes, pero quiero que apartir de la cedula de identidad me muestre el si es extranjero o del pais.

Código (cpp) [Seleccionar] Expandir

printf("Cedula (X - XXXXXXXX): ");
fgets(d,30,stdin);
sscanf(d,"%s - %d",&nuevo->N,&nuevo->dato);

el programa toma la primera X, como cadena para saber si es V (Venezolano) o E (Extranjero) utilizaron una comparacion mas adelante. Y el resto que es XXXXXX, lo deberia guardar en una variable entera

había pensado en esto, pero el problema llega cuando no se colocan los espacios entre X, - y XXXXXX.

es decir, cuando el usuario coloca X-XXXXXXX, todo queda guardado en nuevo->N, y nuevo->dato queda dando numeros al alzar.

claro que ya intente colocando

Código (cpp) [Seleccionar] Expandir

sscanf(d,"%s-%d",&nuevo->N,&nuevo->dato);

en vez del anterior que mostré más arriba.

si conocen alguna manera de hacerlo, o un consejo o funcion que podria usar, se los agradeceria bastante

Mira que pensaba poner un listado de los errores en esa función pero, y recalco que no quiero sonar grosero, son demasiados: reservas memoria cuando no es necesario, no revisas si se alcanza el final de la lista, los nombres de las variables son malos ("nuevo" es el puntero para iterar la lista), hay que tener en cuenta variables "globales" que no son necesarias, etc..

Cuando estaba haciendo el procedimiento para borrar un nodo (Que por cierto, ni idea de lo que hacía en ese momento), me fije bastante de un tema que estaba aquí, básicamente copie, pegue y cambie algunas cosas hasta que me funcionara. Esa misma noche duré bastante buscando solución a esos problemas (Luego de haberlo posteado) hasta que di por fin con todos por mi cuenta.

Hola lerg96 a parte de lo que te ha comentado rir3760 te comento que:

• En la función menu, si escoges una opcion vuelves a llamar a menu recursivamente, esto se va acumulando en la pila de llamadas y no es bueno.
• La función main debe tener un tipo de retorno entero, no puede ser void.
• No deberías mezclar C++ con C, en C++ las bibliotecas adecuadas son cstdlib y cstdio, aunque ésta última es útil en casos muy concretos y su trabajo lo hace iostream prácticamente.
• El typedef está mal puesto, tienes puesto:

Código (cpp) [Seleccionar] Expandir

typedef nodo lista;
Cuando lo correcto es:

Código (cpp) [Seleccionar] Expandir

typedef struct nodo lista;

• En cuanto a lo del fflush(stdin), deberías usar fgets + sscanf(en C) o cin directamente en C++.


• No debes usar system("pause") pues no funciona en todas las plataformas, lo puedes sustituir por un getchar(), o un cin.get()
• No deberías usar system("cls"), podrías poner una directiva de preprocesador pero como trabajas sobre Visual Studio no sé como va en dicho compilador.

Lo del menu lo puedes arreglar con un do - while o con while simplemente.

Aunque como dice rir3760 el problema principal está en esa función.

Sobre el resto del código, sé que hay demasiados errores. Esto es debido a los conocimientos que me imparte mi profesora de algoritmos. El tema que me recomendaste "|Lo que no hay que hacer en C/C++. Nivel básico|." Lo había leído antes, a mitad del trimestre, imagínate mi cara al darme cuenta que la mayoría de las cosas que me enseñan no sirven para hacer algo profesional, la última vez que intente usar algo más adecuado casi pierdo la nota total de un examen T_T  


a la final el procedimiento para borrar quedo de esta manera:

Código (cpp) [Seleccionar] Expandir

    if(inicio != NULL)
    {
int dato;
        nodo *aux = NULL, *ant = NULL;
        ant = new nodo;  
        aux = new nodo;
aux = inicio;
        printf("INTRODUZCA EL ID A ELIMINAR: ");
scanf("%d",&dato);
while(dato != aux->dato)
          {
  ant = aux;
  aux = aux->puntero;
  if (aux==NULL) break;
          }
if(aux==NULL)
{
printf("IDENTIFICADOR NO ENCONTRADO\n");
system("pause");

}
else

{
if(aux == inicio)
          {          
  inicio = inicio->puntero;
             free(ant);
             free(aux);
             printf("ELEMENTO 1 ELIMINADO");
}
else if((ant == inicio)&&(aux==fin))
               {      
ant->puntero= NULL;
inicio = ant;
fin = ant;
                  printf("ELEMENTO ULTIMO ELIMINADO");
               }
else if(ant != NULL)
               {      
ant->puntero=aux->puntero;
free(aux);
                  printf("ELEMENTO x ELIMINADO");
               }
   }
}
else  
      printf("NO HAY ELEMENTOS PARA ELIMINAR");

Pero eso fue lo que hice esa noche, que me ayudo bastante al entendimiento de este. Si creen que puede hacerse mejor o de otra manera, lo agradeceria bastante. Ya me encuentro probando con el codigo de rir3760, intentando completar las partes que mencionaste.

Muchas gracias por su atención, realmente mi intención es aprender, y hacer todo de la manera correcta. Un abrazo!!

Buenas, es la primera vez que posteo en estos foros así que no me amonesten si cometo faltas dentro de ellos, o no tan fuerte. Mi duda es la siguiente: Necesito que mi programa elimine un nodo en especifico a partir de un identificador, cuando coloco el identificador el programa si elimina el nodo. Pero ocurre un problema que es cuando el nodo que intento eliminar no es el primero y luego lo intento mostrar.




Tengo entendido que si el nodo esta al final de la lista(? (no estoy seguro si se dice así) se debe apuntar el nodo anterior a NULL, lo único es que no sé como hacerlo

Además de que tengo otro problema que es cuando el identificador que intento eliminar no lo encuentra, el programa



Estuve buscando ejemplos que pueda ver en el foro, pero no encuentro uno que pueda entender al 100%. Si pudieran ayudarme, se los agradeceria. Además de que puedan explicarme como es este proceso de eliminar (Como leer el código en sí)

Código (cpp) [Seleccionar] Expandir

[
#include <stdio.h>
#include <stdlib.h>
struct nodo
{
int dato;
char nombre[10],telefono[7];
struct nodo *puntero;
};
typedef nodo lista;
lista *nuevo,*inicio,*fin;
void crear_nodos()
{
nuevo=new(nodo);
printf("\nNuevo elemento.\n");
printf("Identificador: ");
scanf("%d",&nuevo->dato);
printf("Nombre: ");
fflush(stdin);
fgets(nuevo->nombre,10,stdin);
printf("Telefono: ");
fgets(nuevo->telefono,7,stdin);
nuevo->puntero= NULL;
if (inicio==NULL)
{
printf("Primer elemento\n");
inicio = nuevo;
fin = nuevo;
}
else
{
fin->puntero = nuevo;
/* hacemos que el nuevo sea ahora el último */
fin = nuevo;
}
}
void mostrar_nodos()
{
lista *auxiliar;
auxiliar=inicio;
while(auxiliar!=NULL)
{
printf("###################\n");
printf("Identificador: %d\n",auxiliar->dato);
printf("Nombre: %s\n",auxiliar->nombre);
printf("Telefono: %s\n",auxiliar->telefono);
auxiliar=auxiliar->puntero;
}
printf("\n");
}
void borrarNodo()
{
    system("cls");
nuevo = inicio;
    if(nuevo != NULL)
    {

        nodo *aux = NULL, *ant = NULL;
        ant = new nodo;  
        aux = new nodo;
        printf("INTRODUZCA EL ID A ELIMINAR: ");
scanf("%d",&aux->dato);
while(aux->dato != nuevo->dato)
          {
  ant = nuevo;
  nuevo = nuevo->puntero;
          }
if(nuevo == inicio)
          {          
  inicio = inicio->puntero;
             free(ant);
             free(aux);
             printf("ELEMENTO 1 ELIMINADO");
}else if(aux->dato == nuevo->dato)
               {                  
ant = nuevo;
nuevo = nuevo->puntero;                
                  free(ant);
                  free(aux);
                  printf("ELEMENTO x ELIMINADO");
               }                              
   }else  
      printf("NO HAY ELEMENTOS PARA ELIMINAR");
}
void menu()
{
int op;
system("cls");
printf (" Menu Principal\n");
printf(" 1.- Insertar Nodos \n");
printf(" 2.- Mostrar Nodos\n");
printf(" 3.- Eliminar Nodos \n");
printf(" 4.- Salir \n");
printf(" Seleccione opcion: ");
scanf("%d",&op);
switch (op)
{
case 1: system("cls");
printf("Ingresa los nodos a la lista");
crear_nodos();
system("pause");
menu();
case 2: system("cls");
printf("Muestra los nodos de la lista\n");
mostrar_nodos();
system("pause");
menu();
case 3: system("cls");
printf("Elimina nodos\n");
borrarNodo();
system("pause");
menu();
case 4: exit(0);
default: printf("Opcion no valida");
system("pause");
menu();
}
}
void main()
{
menu();
}

Sé que no debo usar fflush(stdin); pero aún no sé una manera de pedir una variable entera sin usar scanf para que fgets no se salte xD

lo único que se me ocurre es poner

Código (cpp) [Seleccionar] Expandir


ant->puntero= NULL;
fin->puntero = ant;
fin = ant;

pero nada me funciona