Problemas con ordenamiento burbuja con punteros en lista simple

[Instru]

Hola.
Sinceramente no entiendo exactamente que intentas hacer ni que es lo que no funciona. Pero lo que si te puedo ayudar es a que tu sólito entiendas mejor tu código.

En primer lugar, tienes demasiados niveles anidados, es decir: while-if-if-if-else-if.....
Esto dificulta muchísimo poder seguir y leer tu propio código. Muchas veces los errores son ocasionados por el abuso de estructuras de control anidadas.

Como resolverlo.

En muchos ifs tienes que el caso verdadero realiza muchas operaciones y el falso sólo da un mensaje de error. Invierte esos ifs y usa returns solitos:

Código C++:

if ((id_dir = opendir(".\\Vendedores\\")) == NULL)
    {
        printf("No se pudo abrir el directorio de vendedores\n");
        return;
    }
 
 
    while ((filedir = readdir(id_dir)) != NULL)
    {
        if (strcmp(filedir->d_name, ".") == 0 && strcmp(filedir->d_name, "..") == 0)
        {
            break;
        }
        sprintf(pathFile, "Vendedores\\%s", filedir->d_name);
        archivoVendedor = fopen(pathFile, "rb");
        if (archivoVendedor == NULL)
        {
            printf("Hubo algun error al leer los vendedores\n");
            break;
        }
 
        //Creo una lista enlazada simple con todos los vendedores
        if ((nuevo_elemento = (Lista *) malloc(sizeof(Lista))) == NULL)
        {
            printf("Huvo un error al solicitar memoria para crear el ranking\n");
            return;
        }
        nuevo_elemento->sig = NULL;
        if (InitDir == NULL)
        {
            Ranking = nuevo_elemento;
            InitDir = Ranking;
        } 
        else
        {
            Ranking->sig = nuevo_elemento;
            Ranking = Ranking->sig;
        }
        if (fread(&nuevo_elemento->Vendedor, sizeof(Ventas), 1, archivoVendedor) == 1)
        {
 
        } 
        else
        {
            printf("Huvo algun error al leer los vendedores\n");
            fclose(archivoVendedor);
            break;
        }
        fclose(archivoVendedor);
 
    }
 

Esto es un ejemplo. Habría que revisar bien cada caso.

Por otro lado. Tienes muchas operaciones en una sola función. Una de las prácticas que mas te salvarán es la siguiente: Tus funciones deben ser cortas (muy cortas) y sobre todo, que lo poco que hacen, lo hagan bien. Es mas fácil hacer pequeñas funciones que cumplan su tarea perfectamente, que una función gigante con mil tareas. Es la base de tener una buena abstracción del problema.

Trata de no tener expresiones demasiado complejas dentro de los condicionales de un while o un if:

Código C++:

if(strcmp(filedir->d_name,".") != 0 && strcmp(filedir->d_name,"..") != 0)
 

es mas legible asi:

Código C++:

int is_current_dir = strcmp(filedir->d_name,".");
int is_previous_dir = strcmp(filedir->d_name,"..");
if( !is_current_dir && !is_previous_dir)
 

Si. es más código, pero de que te sirve código comprimido, cuando no funciona?

SIEMPRE inicializa tus punteros. Y Aunque es mas cuestión de gustos, es mas legible que cada variable este definida en una linea diferente.

Todo esto se llama refactorizar código. Y si quieres se un buen programador, debes dominarlo.
Intenta limpiar tu código y verás que hasta con punteros te sale.

Saludos