<div>Hola Josep,<br></div><div>Bueno, Salvador se me ha adelantado, pero quiero agregar unos matices al código que nos enviaste.<br></div><div class="gmail_quote">2011/7/4 josep el wendo <span dir="ltr"><<a href="mailto:josepbord@hotmail.com" target="_blank">josepbord@hotmail.com</a>></span><br>
<blockquote style="margin: 0px 0px 0px 0.8ex; padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;" class="gmail_quote">




<div><div dir="ltr">
Hola; he cambiado de programa (en vez de dev ahora con codeblocks) y me pasa lo mismo: no separo bien las funciones.<br><br>Se trata de un programa para aprender, pero que se supone, gestionaria una agenda (aunque de momento hay que darle las entradas a mano y cada vez que se ejecuta el programa).<br>

<br>les mando el codigo, pero no hace falta que comprueben que está bien y que no; me sobra con que resuelvan mis dudas:<br></div></div></blockquote><div> </div><div>Si el código es para aprender, DEBO comentarte algunas cosas que he visto en este código fuente:</div>

<div><br>1. Variables globales: no las necesitas y créeme te pueden complicar la vida MUY fácilmente, por lo que se consideran peligrosas. Si quieres comunicarte con las funciones, usa parámetros, que para eso están.</div>

<div> </div><div>2. NO USES 'goto'. No se debería usar para nada (casi nada). En tu caso, se usa el 'goto' a modo de bucle, por lo tanto, USA un bucle; C++ te da hasta tres diferentes tipos de bucles a escoger.</div>
<div> </div><div>En cuanto a la segunda duda, te comento que para la opción del menú, lees un solo carácter, pero no recoges el segundo carácter '\n' generado por pulsar la tecla de [ENTER] o [RETURN]. En este caso, aconsejo que leas una cadena de caracteres, pero que aceptes solamente el primer carácter. Por ejemplo,</div>
<div> </div><div>char resposta[10]="";</div><div>...</div><div>while( resposta[0] != 'x' and resposta[0] != 'X' )</div><div> </div><div> </div><div>Por cierto, veo un error en 'BuscaxNom()' donde comparas cadenas de caracteres con el operador ==. Escribes:<br>
<br>if ( nombusc==Agenda[s].nom )</div><div> </div><div>Esto no funcionará como esperas. Deberías usar la función estándar 'strcmp()', declarada en <cstring>. Esto es,</div><div> </div><div>if ( strcmp(nombusc, Agenda[s].nom) == 0 )</div>
<div> </div><div> </div><div>Espero que esto te oriente.</div><div> </div><div>Steven</div><div> </div></div>