Crapos escribió:(...)
Estas cosas solo se solucionan con práctica, pero cuando te funcionan casi te corres de gusto. Una cosa, te sobra la última coma en la inicialización del array de lista.
Ahora, no sé si lo has hecho, pero yo de ti cogería un boli y un papel, y haría el proceso paso a paso que habría que seguir. Por ejemplo:
- Tomo un número de entrada, el 3451234.
(0)Grosso modo, hay dos formas de contar los dígitos de ese número: o empiezas contando por el dígito más significativo (3) y vas descendiendo de nivel, o bien empiezas por las unidades (4) y subes de nivel. Te voy a poner el procedimiento para empezar a contar por las unidades, en contra de tu código que parece querer contar por el nivel más significativo e ir bajando hacia las unidades. Creo que es un pelín más difícil de comprender (me va lo chungo, sorry
), pero espero que aun así te ayude.
(1) Miro si el número de entrada es divisible entre 10. 3451234 no lo es.
(2) Como el número no es divisible entre 10, le resto una unidad y vuelvo a hacer la comprobación. 3451233 tampoco lo es...así hasta 3451230. Tienes que saber de alguna forma cuántas veces has ido restando una unidad, para luego usar ese valor para sumar "1" al array de lista de esa posición.
Como ahora 3451230 sí es divisible entre 10, lo divido y vuelvo a hacer (1) y (2) con 345123, luego con 34512, 3451, 345, 34 y 3.
EDITO: otra forma (creo, más eficiente) de hacer (2) sería:
(2') Asegurándome de haber definido el número 3451234 como entero, lo divido entre 10 y lo vuelvo a multiplicar por 10. El resultado es 3451230. Resto 3451234 menos 3451230, y sumo "1" al índice de la lista del resultado de la resta.
(3') Hago lo mismo con 345123, 34512, 3451, 345, 34 y 3.
Aun así, ¿Por qué no pones trazas en los puntos que tú quieras, y luego cuando el programa funciona como quieres las borras o las comentas? Es decir, colocas trazas para que se te muestre por pantalla las variables cuyos valores sabes de antemano y comprobar su buen funcionamiento.
division = numero/10;
printf("División vale %d", division); //Traza 1
while((numero/10 != 0) || (j != 10))
{
division = division/10;
printf("División vale %d", division); //Traza 2
lista[j]= division;
printf("Lista[%d] vale %d", j, lista[j]); //Traza 3
j++;
for(j=0;j<10;j++)
{
printf("El contenido del array es %d\n", lista[j]);
}
system("PAUSE"); // no me acuerdo que tenia que usar en vez de system("pause")
}
Y por dios, elimina el system ese!!! Si quieres que el programa se quede esperando después de mostrarte por pantalla la lista, una opción muy chusquera, pero más segura, es poner otro scanf en lugar de la línea del system:
scanf("%d",&numero);
La movida esq solo lo divide una vez y se queda hay. La idea serie que lo hiciera siempre que el numero dividido no fuera 0 y lo guardase en las distintas posiciones del array, par aluego de alguna forma ver cuantas veces se repitio un numero.
La primera división, fuera del bucle, creo que no ha lugar. Además, las condiciones del while no son correctas. Podrías poner
while (division > 0) // suponiendo que el número de entrada es positivo
El uso de "j" que haces no es correcto: en cada iteración del bucle sumas "1" a "j" cuando lo que deberías sumar "1" es a lista[j], siendo "j" el dígito que has de obtener dentro del while. La clave es obtener "j" por los métodos que te puse en (0) (o por cualquier otro que tú veas mejor).
El array lo muestra infintas veces, no se porq.
Has de cerrar el while antes del for.
PD: joer, se me ha ido la indentación en el código.
(Y a los demás aún más tus respuestas ^^). Además, el muchacho lo ha intentado. Otra cosa es como aquel que llegó exigiendonos un script en bash para dentro de una hora xD
