SoloCodigo

He mirado el codigo por encima (luego lo miro mejor) y me gustaria darte un par de consejos.

El programa parece un ejercicio (o examen) de la facultad, cierto?  
Siempre que programes intenta intercarlar sentencias y/o procedimientos que te ayuden a DEPURAR. Eso te sirve para solucionar de forma rapida problemas como el que comentas.
Create un procedimiento que te muestre todas las entradas del vector y lo llamas cuando te haga falta para ver si insertas/actualizas/ordenas ... bien en el vector.
Por supuesto cuando finalices el programa puedes borrar dicho procedimiento.

Otra cosa que debes tener en cuenta a la hora de programar es la eleccion de las estructuras adecuadas y bucles adecuados para recorrerlas (esto lo valoran muuucho en la facultad).

Si solo buscas un elemento del vector (un empleado con un dni concreto) no uses FOR. FOR recorre todo el vector. Imagina que el empleado es el primer elemento del vector y tu vector tiene 200 empleados mas. Con FOR los miras todos.
Usa un While con la condicion adecuada de modo que al encotrar el elemento buscado, te salgas de la busqueda.

Tambien, hay cosas en los que en vez de usar IF...ELSEIF...ELSEIF podrias usar un Switch y te quedaria mas ordenado  

En general, el codigo esta bien (salvo ese par de detalles). Esta tarde lo mirare mas afondo (ahora estoy en el curro).
De todas formas, programa el metodo ese que te he comentado al principio para que te ayude a depurar y encuentres el error.

Un saludo

En realidad esa costumbre es perjudicial a la larga, lo ideal es usar un depurador en tiempo de ejecucion. No creo ser capaz de cuantificar el beneficio que se obtiene con esta practica.

Hablo de programas de este tipo que son con estructuras sencillas y no son muy extensos.

Por supuesto que es mejor usar un depurador en tiempo de ejecucion, pero no creo que sea malo el programarte tu mismo pequeños procedimientos que aceleren la busqueda de posibles errores.

Hablando sobre proyectos de este tipo a mi en cualquier caso me parece una perdida de tiempo escribir codigo para depurar, es una solucion solo valida ante la ausencia de un depurador o el desconocimiento sobre como usarlo.

Otra cosa es tener codigo para hacer un log (ya sea a archivo, a consola o al propio depurador), eso puede ser util en determinadas circustancias como cuando no es posible reproducir un problema que tiene un cliente.

A mi me gustaría añadir dos cosas:

La primera: Usar un FOR no tiene por qué ser malo en tantoen cuanto uses una variable booleana que controle la salida cuando encuentres o hagas todo lo que tengas que hacer:


Yo uso más bien eso en vez de WHILE. Sin embargo, cada cual elige lo que le resulta más fácil.

La segunda: SObre depuradores no estoy muy puesto y siempre recurro al printf("llega\n") o cout << "llega\n" ; que es un poco feo...Eternal Idol, puedes facilitarme algún tutorial? Y, en que programa lo depuras? Gracias de antemano.

La segunda: SObre depuradores no estoy muy puesto y siempre recurro al printf("llega\n") o cout << "llega\n" ; que es un poco feo...Eternal Idol, puedes facilitarme algún tutorial? Y, en que programa lo depuras? Gracias de antemano.

Pues eso depende de que IDE/Compilador Utilices.

Si Utilizas el Visual Studio (Express o Porfessional), ya viene con depurador integrado, y es muy facil de usar, solo tienes que cambiar las opciones de compilacion para que te genere informacion de depuracion, y listo, utilizas las teclas F5 para comenzar la depuracion y F11/F10 para depuracion paso a paso por instruciones/procedimientos.

El viejo Turbo C tambien incluia un depurador facil de usar.

GCC/Mingw usa su propio depurador GDB, pero es mas dificil de usar, (al menos yo nunca he podido depurar bien con el )

Saludos...

Por cierto, una de las razones (por no decir la principal) por la que uso el WinDbg es que no solo es capaz de depurar en modo Usuario sino que tambien es capaz de hacerlo en modo Kernel. Tambien es perfecto para descargar los simbolos de los binarios de Microsoft (PDBs) y tiene varias extensiones que facilitan increiblemente la depuracion y lo hacen el depurador mas poderoso que yo conozco.

No digo que no sea perfecto, simplemente en cualquier lado te enseñan a usar While en vez de For para recorrer una estructura de elementos cuando buscas uno en concreto.
Por supuesto que hay mil maneras de recorrer la estructura y todas son correctas, pero como te he dicho antes, a la hora de enseñar (hablo de la facultad) te indican que uses While.

Si, muy bien, tambien enseñan con Turbo C y Pascal, eso es irrelevante.

Encuanto al tema de la depuracion...para C yo suelo usar el gdb en linux y nada me impide el "ayudarme" a solucionar de forma rapida mis errores escribiendo sencillos trozos de codigo en el programa. Lo he usado siempre y para mi es mas comodo.

Nada nos impide perdir el tiempo, eso es cierto.

Pretetender enseñar todo de golpe es un error. Se empieza enseñando lo facil (aunque sea lo menos correcto) y posteriormente se continua con lo mas dificil.
El enseñar Pascal y Turbo C (sinceramente dudo que hoy dia enseñen pero bueno) es por motivos didacticos.

Un error es enseñar cosas inutiles que no siguen el standard, no se quien habra tenido la idea de enseñar "todo de golpe" pero seguro que no fui yo.

He visto resolver el problema de las N
reinas en un FOR y no por eso creo que sea correcto usar el FOR para eso.

¿Quien te respondio a vos lo del for? Como dicen por ahi si te pones el saco sera que te queda.

Y en cuanto a lo de perder el tiempo, cada uno es libre de hacer con su tiempo lo que quiera ¿cierto o no?

Creo que esa fue mi afirmacion ...

Has desviado el tema de este hilo y no creo que ni tan siquiera hayas mirado el codigo del muchacho, verdad? .Y si lo has mirado tiene mas delito el hecho de que hallas "gastado tu tiempo" en poner la puntilla a mis mensajes que en ayudar al muchacho.

Si quiero comentar una mala indicacion lo hago, la mejor ayuda que se le podia dar era indicarlre como resolver la situacion por si mismo. Se nota que no coincidimos en las tecnicas de enseñanza, sera que la facultad es muy elevada para algunos de los que si ayudamos en este foro.

La verdad es que para ser moderador no entiendo tu actitud.

Sinceramente me trae sin cuidado, sino te gusta ya sabes lo que podes hacer ademas.

La finalidad del foro es ayudar a la gente que lo necesita, no?? En ningun momento he dicho nada que no sea cierto y en todo momento he respetado la opinion del resto. Es mas, te he dado la razon en todo. En ningun momento he pretendido discutir con nadie y siento si esa ha sido la sensacion que he dado.

Si y seguramente vos sos el que mas ayuda aca ... ayudar no significa resolver problemas ajenos. Para mi si dijiste algo que no es cierto, eso no es una buena costumbre, en ingles lo denominan algo asi como "la depuracion del hombre pobre" y no es por casualidad. Y no hace falta que me des la razon, tampoco me interesa pero eso no quita que deje asentado mi parecer en el hilo. En cuanto a lo del for ya te dije que no te respondi a vos, alguien te respondio y yo le respondi a esa persona, toca estar mas atento para la proxima.

Tampoco me parece correcto que digas algo y luego cierres el hilo.
¿No aceptas criticas constructivas?

Para leer esta clase de cosas es preferible cerrar al hilo antes de tener que responderte. ¿Criticas constructivas? Por favor, no hiciste ninguna critica, simplemente farfullaste conceptos ridiculos escudandote en algo irrelevante como que lo enseñan en la facultad y entonces preferi cerrarlo para no discutir pero parece que tenias tantas ganas de hacerlo que abriste otro hilo ...

Recuerda que nadie tiene la verdad absoluta de ningun tema (sea temas de informatica o no).

Gracias, siempre necesitamos que gente con tanta inteligencia como vos venga a darnos lecciones de este tipo, especialmente con problemas complejos como el del hilo y soluciones magnificas como usar printf. Sera que el que no acepta las criticas es otro, sera que lo del for lo dijo otra persona ... quien sabe.

Pd.- Una vez leas el mensaje, puedes borrarlo tranquilamente. Es mas, te agradecira que lo hicieras dado que ensucia el subforo.

Dejarlo aca es mejor, ya podes continuar con la discucion que tanto querias.

OK. Lo que vos digais. No voy a seguir discutiendo.

Mucho mejor, no ganamos nada haciendolo.

Lo siento si te ha dado la sensacion de que iba de listo o algo. Para nada.
Siento el follon. En ningun momento pretendi liarla.

Ya es agua pasada.

De todas formas, creo hay mejoras formas de dirigirse a la gente.

Puede ser, nadie es perfecto.

Estube revisando tu código... y tu error está al definir la estructura TEmpleado porque definís dni[9] y almacenás un número de 9 dígitos, osea... te falta el caracter de fin de linea, entonces al correr tu programa al principio guardás la dirección correctamente, y al guardar el DNI el caracter de fin de linea se escribe en la variable direccion[80], por eso funciona bien hasta que actualizás donde almacenás algo en dicha variable borrando el carater de fin de linea.

Con respecto al tema del FOR o el WHILE, es indiferente su uso... en ambos casos se pueden definir sentencias de fin (condiciones boleanas, break, continue) el tema es saber utilizar estas sentencias, y con respecto al tema de la depuración, personalmente estoy a favor de los mensajitos antes que un programa de debug, no por un tema de desconocimiento, sinó por practicidad, es más... para mi no representa una pérdida de tiempo colocar pequeñas sentencias de printf o messagebox cada cuanto para ver el comportamiento del programa, sobre todo cuando aparecen fallas muy descolocadas o por errores en el compilador... aunque aveces una mezcla de ambos es muy productivo... como debuger suelo utilizar el win32 dassembler, que es liviano y bastante facil de usar...

Saludos

Naa... es obvio que no se soluciona con eso...   , pero se puede ver hasta donde va corriendo bien el ejecutable...  

Además... no sé que tan complicados sean tus proyectos... pero hasta hora en los que he trabajado... recompilar no toma más de 30 segundos... es más... el error que le acabo de encontrar al código posteado lo hice así en 2 minutos... debugueando no sé si será tan rápido...

Con vos lo que pasa es que no te podes bajar de tu nivel de programador "experto", pero para el que posteó el código originalmente (que se nota a leguas que está comenzando a programar) le es más práctico lo de los mensajesitos que lo del debug...

Sin ir más lejos... (quizás sea un error comun... je) pero muchos de los códigos que andan dando vuelta en la red utilzan este método para debuguear (incluso hasta el código fuente de Win 2000)

Saludos

Lo que yo dije fue esto: En realidad esa costumbre es perjudicial a la larga, lo ideal es usar un depurador en tiempo de ejecucion. No creo ser capaz de cuantificar el beneficio que se obtiene con esta practica.

Primero que aprenda a programar... después a debugear... sinó pasa lo que mensionó "Karkayu", que esperan que se aprenda todo de golpe...

Ese argumento es muy facil de rebatir: millones de moscas comen mierda. Por otro lado que yo recuerde en la filtracion de Windows NT 4.0 y 2000 no habia una sola linea que correspondiera al Kernel.

Si para vos eso es comer mierda... no digo más nada... ... además... yo no dije nada de Kernel... o si?... jo

Saludos

PD: tratá de no ser tan soberbio... let it be...