Mensajes

Jajaja. Un error lo tiene cualquiera Pero no he llegado a entender tu explicación D: No digo que esté mal explicada, solo digo que no sé como resuelve mi duda!

He entendido por qué se debe retornar el objeto de flujo, pero no entiendo como tu respuesta resuelve mi duda :S Aun así, gracias por tu rápida respuesta

Por cierto, a lo mejor es que no entiendo/sé suficiente C++, pero ¿y esta estructura?

Código (cpp) [Seleccionar] Expandir

istream& >>(istream &in, UnaClase objeto)

Parece la estructura de sobrecarga de un operador, pero no tiene la keyword 'operator'. ¿Significa otra cosa?

Hola, muy buenas. En mi rato libre, quise mirar un poco el standard de C++. Una cosa de las que he mirado es los operadores definidos para los streams definidos en este (lo he mirado tanto en cplusplus como en el propio archivo de mi compilador MinGW.

Un ejemplo de lo que encontré en el archivo "ostream" es el siguiente:

Código (cpp) [Seleccionar] Expandir

__ostream_type& operator<<(long __n);

La duda es: ¿No necesita una clase o estructura en los parámetros del operador para ser correcto?

Bueno. Para intentar no tener que preguntar, he hecho el siguiente ejemplo:

Código (cpp) [Seleccionar] Expandir

#include <cstdio>

struct stream {
    int null;
};

stream& operator<< (int val);

int main() {
    stream ostream = {0};

    ostream << 123 << 5;

    return 0;
}


stream& operator<< (int val) {
    printf("%i", val);
}

Pero en las definiciones del operador he tenido que poner esto, porque si no me daba error:

Código (cpp) [Seleccionar] Expandir

stream& operator<< (stream& ostream, int val);

Esto no esclarece nada :S Gracias por su ayuda!

Creo que el algoritmo sería:

Determinas en que posición debe ir la cadena (según tus criterios o los del ejercicio)
Buscas esa posición entre los nodos
Intercalas un nuevo nodo en esa posición

Qué tienes que ordenar de forma ascendente o descendente... Las cadenas no se pueden ordenar de esta forma (al menos de la forma en la que entendemos ordenar...). Tienes que establecer un criterio para ver que cadenas van primero y cuál va última

Problemas que he detectado en la función mostrar_lista:
1º Utilizas un while sin sentido. Tienes que usar un if si no quieres un ciclo infinito XD
2º Si tu objetivo era la recursividad, se te ha olvidado el elemento más importante: llamar a la función recursivamente (véase en el código de abajo)

Código (cpp) [Seleccionar] Expandir

void mostrar_lista(nodo_t* p) {
    if(p!=NULL) {
        printf(". %s\n",p->filename);
        //p=p->sig; Aquí está el error
        mostrar_lista(p->sig); // Esta es la solución
    }
}

En la función borrar_todas_lista comentes errores similares. Así quedaría:

Código (cpp) [Seleccionar] Expandir

void borrar_todas_lista(nodo_t* p) {
    nodo_t* borrar=NULL;

    if(p!=NULL) {
        borrar = p->sig;
        free(p->filename);
        free(p);
        borrar_todas_lista(borrar);
    }
}

Por cierto, date cuenta de que también tienes que liberar de la memoria la cadena de caracteres que has generado con malloc en la función agregar_nodo.

En la función main, declaras un array de un solo elemento (es decir, no cabe el elemento NULL que indica el final de la cadena). La cadena de caracteres tiene que tener como mínimo dos elementos disponibles. También tienes que pedir una cadena de caracteres con la función scanf (sustituye %c por %s)

He probado el código y va perfectamente. El único problema que veo es que cuando imprimes la cadena de caracteres, lo haces con un %c. %c es para caracteres. Para imprimir una cadena tendrías que usar %s:

Código (cpp) [Seleccionar] Expandir

#include<stdio.h>
#include<string.h>
#include<stdlib.h>

typedef struct Nodo {
    char* filename;
    struct Nodo* sig;
} nodo_t;

void agregar_nodo(nodo_t**p, char* ruta) {
    nodo_t* nuevo;
    nodo_t* aux;

    nuevo = (nodo_t*)malloc(sizeof(nodo_t));
    nuevo->filename = ruta;
    nuevo->sig = NULL;

    if(*p == NULL) {
        *p = nuevo;
    } else {
        aux = *p;

        while(aux->sig != NULL)
            aux = aux->sig;

        aux->sig = nuevo;
    }
}

int main() {
    nodo_t* nodo = NULL;
    nodo_t* aux = NULL;

    agregar_nodo(&nodo, "Hola x1");
    agregar_nodo(&nodo, "Hola x2");
    agregar_nodo(&nodo, "Hola x3");

    printf(nodo->filename);
    aux = nodo->sig;
    while(aux != NULL) {
        printf("\n%s", aux->filename);
        aux = aux->sig;
    }

    return 0;
}

También tienes que tener cuidado con lo siguiente: Cuando le envias una cadena de caracteres a agregar_nodo, le tienes que enviar una diferente. Por ejemplo, el siguiente código no haría lo esperado:

Código (cpp) [Seleccionar] Expandir

int main() {
    char cadena[16] = "Codigo";
    nodo_t* nodo = NULL;
    nodo_t* aux = NULL;

    agregar_nodo(&nodo, cadena);
    cadena[0] = 'c'; // Ahora cadena vale "codigo"
    agregar_nodo(&nodo, cadena);
    cadena[6] = 's';
    cadena[7] = '\0'; // Ahora cadena vale "codigos"
    agregar_nodo(&nodo, cadena);

    printf(nodo->filename);
    aux = nodo->sig;
    while(aux != NULL) {
        printf("\n%s", aux->filename);
        aux = aux->sig;
    }

    return 0;
}

El resultado que uno podría esperar es:

Código [Seleccionar] Expandir

Codigo
codigo
codigos

Pero en realidad el resultado obtenido es:

Código [Seleccionar] Expandir

codigos
codigos
codigos

Lo que pides es dependiente del sistema operativo. Dado que usas Dev-Cpp, asumiré que usas Windows. Para hacer esto en Windows, puedes usar el comando mode, cuya estructura es "mode ancho" donde "ancho" es el número de caracteres de ancho de la consola. Por ejemplo:

Código (cpp) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h> // system

int main() {
        system("mode 124");

        printf("Ahora caben 124 caracteres de ancho!");

        fgetc(stdin);
        return 0;
}

No sé si hay algún comando para GNU/Linux que haga la misma función.

Curioso. Nunca había oído hablar de la notación polaca inversa. Cada día se aprende algo

Una cosilla. No he analizado tu código, pero ¿no sería mejor usar

Código (c) [Seleccionar] Expandir

scanf("%d %d %c", &operando1, &operando2, &operador);
?

Ya te digo que no he analizado tu código

Si, está bien