SoloCodigo

13.    int opcion, art, codigo[0], cantidad[0];

Fijate en esto, estas reservando espacio para dos vectores que van a contener "enteros", y cuanto espacio reservas?   >> \[0], osea, nada. Eso esta en la función "menu()", de manera que si llamas a la funcion "VerArticulos()" desde la función "menu()", estas lanzando dicha función sobre unos vectores que no existen.

60.    int codigo[art];
61.    int cantidad[art];

Aquí estamos en la función "AnhiadirArticulos()". Aquí si que estás reservando espacio para los vectores, por eso al terminar de añadir los articulos se pueden ver, gracias a la llamada que haces a la función (    VerArticulos( art, codigo, &cantidad[0] ); ). Habeis oído alguna vez el dicho de "Lo que pasa en Las Vegas se queda en Las Vegas" ? con las funciones pasa otro tanto de lo mismo. si construyes un vector dentro de una función y dicho vector no esta en el ambito global del programa o no lo devuelve la función, al terminar la función se termina también la visibilidad del vector. "Lo que pasa en las funciones se queda en las funciones".

Necesitas una estrategia para hacer el vector visible a todas las funciones.

50.    /* esto la verdad me recomendaron hacerlo pero aun no me queda claro cual es la funcion */
51.    art = tran = op = cod = 0;
52.
53.    system("cls");

Esto es una asignación multiple que no tiene mucho misterio, basicamente estas inicializando todas esas variables a cero, al mismo tiempo, el operador '=' se evalua de derecha a izquierda, lo cual quiere decir que estas inicializando  'cod = 0' despues 'op' al valor que contenga 'cod', esto es cero, despues 'tran' al valor que contenga 'op' y así sucesivamente, esto se hace para asegurarnos que dichas variables no contengan un valor residual que estubiese en el espacio de memoria que estamos reservando para ellas.
system("cls");  simplemente limpia la pantalla de la consola, de esta manera en vez de ir llenando la consola con opciones unas debajo de otras, se limpia en cada función para que aparezca todo mas limpio, y mas claro.

Un saludo,
Nacho

.

Pero te ha dicho que en C++ y le has contestado en C

Ahí va una solución. Yo te pongo un programa para resolver un numero de 5 digitos exactamente y tu te lo trabajas para numeros que tengan cantidades arbitrarias de numeros.

// separa los digitos de un numero
#include <iostream>
using namespace std;

// la función main comienza la ejecución del programa
int main()
{
int num1;  // número que introduce el usuario

cout << "Digite un numero de 5 digitos: ";
cin >> num1;

cout << "\n" << num1 / 10000 << "   " << num1 % 10000 / 1000 << "   "
     << num1 % 1000 / 100 << "   " << num1 % 100 / 10 << "   "
<< num1 % 10 << endl;

return 0; // indica que el programa terminó satisfactoriamente

} // fin de la función main
La explicación es la siguiente:
si tu numero tiene 5 digitos buscas la unidad seguida de ceros que tenga 5 digitos, en este caso 10000, al dividir por 10000 te separa el primer numero y lo imprimes.
para el siguiente numero haces el "modulo" (%) de 10000 y esto te deja de resto 4 digitos que los divides por la unidad seguida de ceros que tenga 4 digitos, osea 1000,
el tercer paso es similar al segundo haces el "modulo" de 1000 para quedarte con solo 3 digitos y los divides por unidad seguida de ceros con 3 digitos en total, osea 100....
y asi sucesivamente....
¡Espero haberme explicado bien!

Un saludo
Nacho

Bueno, primero de todo decir que lo he hecho así para respetar el enunciado del problema, que especifica que debe ser una estructura "while". El programa funciona y da un resultado correcto.

Efectivamente, como tu bien apuntas el "break" no solo salta la ejecución de "else", sino que sale del bucle directamente. (lo he comprobado con el debugger). Con lo cual, básicamente solo comprueba que la entrada es distinta de -1 y continua, pero cuando es igual a -1 se encarga el break de salir del bucle y no se utiliza la comparación del while. Sigue siendo necesario para los casos en que se introducen valores validos para los cálculos.

Ya me parecía a mi que la doble comparación no era una "lógica de programación" demasiado elegante, pero la verdad es que, respetando el enunciado, no se me ocurría mejor forma de hacerlo,  ya había pensado en el "do... while" pero no quise utilizarlo por lo que comento de respetar el enunciado, ya que es un ejercicio y hay que ceñirse a lo que indica.

en cualquier caso con do while me parece mas elegante y mejor planteado, pero me gustaría saber si hay mas sugerencias utilizando únicamente while

Un saludo,
Nacho

Hola,

me gustaría hacer un pequeño aporte mas.

Me rondaba la cabeza el tema y no sabia por que, pero tenia en mente que la función sabersiya() no producía un hash md5 correcto al menos en windows, así que lo comprobé, las opciones por defecto al abrir archivos son 'lectura' y 'modo texto', y ya sabemos todos que microsoft es muy particular a la hora de interpretar los saltos de línea, utilizando dos caracteres en lugar de uno solo, el famoso 'retorno de carro' y el 'salto de línea', si abrimos dando las opciones de lectura y modo binario 'rb', conseguimos un código mas portable y mas independiente de la plataforma.

al abrir el archivo se hace con opciones por defecto.
archi = open(archivo) # Esto produce un hash incorrecto

debería abrirse con las opciones 'rb'
archi = open(archivo, 'rb') # Esto produce el hash correcto

aunque yo trabajo en una maquina con linux, la comprobación la he hecho en una maquina windows, no se si en linux pasara lo mismo...
aquí dejo la comprobación:

C:\Documents and Settings\n\Programacion\Python>md5deep resultados.txt
679b1f3244e3d285bf134ff0de4770fd  C:\Documents and Settings\n\Programacion\Python\resultados.txt

C:\Documents and Settings\n\Programacion\Python>Python
Python 2.7 (r27:82525, Jul  4 2010, 09:01:59) [MSC v.1500 32 bit (Intel)] on win 32
Type "help", "copyright", "credits" or "license" for more information.

>>> import hashlib

>>> def sabersiya(archivo):
...    archi = open(archivo)
...    cabeceramd5 = hashlib.md5(archi.read()).hexdigest()
...    archi.close()
...    return cabeceramd5
...
>>> sabersiya('resultados.txt')
'88e75f1675fba32f982f3ce11eb29d5c'

>>> def sabersiya2(archivo):
...    archi = open(archivo, 'rb')
...    cabeceramd5 = hashlib.md5(archi.read()).hexdigest()
...    archi.close()
...    return cabeceramd5
...
>>> sabersiya2('resultados.txt')
'679b1f3244e3d285bf134ff0de4770fd'
Para empezar he comprobado el hash con una utilidad conocida 'Md5deep' sobre un archivo
luego he definido la función sabersiya() y he comprobado el resultado, como se puede ver es erróneo, para finalizar he definido la función sabersiya2() con la opción de apertura modificada y la he comprobado sobre el mismo 'sufrido' archivo que ha hecho de conejillo de indias y en esta ocasión se ve que el resultado es correcto.

Un saludo,
Nacho

hola,

se que estoy rescatando un tema antiguo, pero cada vez que busco código para "eliminar archivos duplicados" en python me sale este programa.

que ha sido diseminado por la red y se hace referencia a el en distintos blogs y foros.

pues bien, el código básicamente pretende eliminar archivos duplicados y dejar una sola copia de cada archivo, y básicamente "Falla" en el propósito para el que fue creado. Esto es uno de esos errores lógicos de los que tanto se habla en los libros, uno de esos que no tira advertencias y que hace que el resultado del programa no cumpla con el cometido para el que fue creado.

en primer lugar se define una función que devuelve el hash md5 de cada uno de los archivos en el árbol de directorios, luego se crea un diccionario con los nombres de los archivos como claves y con las sumas md5 como valores, y ahora se pretende comparar los archivos unos con otros y borrar los que tengan el mismo valor pero distinta clave. Y es aquí donde esta el error, estamos comparando las entradas del diccionario con 2 bucles anidados, y estamos borrando archivos en el árbol de directorios, las entradas del diccionario no se borran, así que básicamente un archivo que haya sido borrado mantiene su entrada en el diccionario y tarde o temprano se comparara con otro de los duplicados, dando como resultado el borrado de este también.

en resumen, lo que pasará es que se borrarán "todos" los archivos, incluso los originales