Mensajes

Si no eliminases los espacios, oraciones como "dabale arroz a la zorra el abad" no se cosiderarian palindromos, siendo que si se consideran como tal... o tampoco seria un palindromo el famoso "a man a plan a canal panama", que incluso hay programadores que han trabajado con este ultimo para extenderlo todo lo posible.

ya entendi... Saludos

[%p]

en vez de declarar un puntero como int *a intenta declararlo como int* a. parece una tonteria, y para el compilador es lo mismo, pero de la segunda manera se ve claramente que a es una variable de tipo puntero a int. en cambio de la primera se puede pensar que *a es un puntero a int.

no se entiende lo que queres decir. En una declaracion de puntero cualquiera de las dos formas se entiende que es un puntero

si haces un printf("%d", a), el resultado sera un numero largo que indica la direccion de memoria a la que apunta

No, para saber la direccion a la que apunta se usa %p


Saludos

Exacto yo me referia a esto:

Código (c) [Seleccionar] Expandir

for (i=0; i<strlen(buffer)/2 && cond==1; i++) {
if (buffer[i] != buffer[strlen(buffer)-i-1]) {
cond=0;
}
}
Igual me parece innecesario eliminar los espacios


Saludos

Hola!

Código (c) [Seleccionar] Expandir

if(*comp1 == cod)
{
ns = cantidad - *diferencia1;
¿por que usas el "*" delante de las variables?
En:

Código (c) [Seleccionar] Expandir

void DESCONTAR_STOCK(FILE *stock1, int diferencia1, int comp1)
las variables diferencia1 y comp1 se paso por valor


Pone como hiciste la llamada. Saludos

Hola! no es mas facil tener dos indices, uno al inicio y otro al final del arreglo, para ir comparando letra por letra?? algo asi:

Código (c) [Seleccionar] Expandir

for(i=0,j=strlen(texto)-1; i!=j; i++,j-- )
  if(texto[i]==texto[j])
    "es palindromo"

Saludos

[y para reemplazar a los ifs]

Hola! el codigo esta bastante mejor, mas ordenado pero. Te marco algunas cosas:

1º Revisa el uso del switch-case. Te recomendaron que uses switch para que el codigo se vea mas lindo y para reemplazar a los ifs

2º El problema en la lectura es el orden de como lo haces. En la funcion ALTAS_ARTICULOS() guardas con fwrite en este orden: codigo - art - cantidad - precio. Y en CONSULTA_ARTICULOS(): codigo - precio - cantidad - art... El orden tiene que coincidir tanto para leer como para guardar

3º no queda muy lindo que pongas un return en los case. Aparte no tiene ninguna utilidad ahi

4º El while(op!=0) {} lo podes optimizar con un do-while, asi no tenes que repetir dos veces el menu principal


Saludos

[varios errores conceptuales]

Hola! no te quiero arruinar el regalo pero tenes varios errores conceptuales. Lo mas raro es que digas "recien estoy aprendiendo vectores", cuando ya estas queriendo manejar archivos... Te marco lo errores que vi:

1º Tenes que revisar como se usa la funcion fwrite(). No podes utilizar sizeof(int) en todos los casos, estas guardando mal.

2º Lo mismo fread

3º En el main cuando elegis la opcion: "1 - Alta de un articulo" ó "3 - Clientes" cerras los archivos(con fclose). El problema es: ¿que pasa si el usuario vuelve a elegir esas opciones? los archivos van a estar cerrados por lo que vas a tener un error cuando quieras volver a guardar algo. Tenes que abrir los archivos antes de elegir esas opciones

4º Con este ciclo: while(nombre != NULL) estas generando un bucle infinito. No tiene sentido hacer esa comparacion ¿que es lo que querias hacer?

5º En la funcion STOCK() el principal error esta en el uso de fread. Ya que no sabes manejar estructuras, lo que tenes que hacer para cada ciclo de lectura es:

Código (c) [Seleccionar] Expandir

fread(&x,sizeof(int),1,stock1);
fread(descripcion,sizeof(descripcion),1,stock1);
fread(&cantidad,sizeof(int),1,stock1);
fread(&precio,sizeof(float),1,stock1);

Hay muchas cosas que se pueden mejorar aun. Mi consejo es que antes de seguir con esto practiques un poco mas con manejo de archivos, y con estructuras


Saludos

ah ok, no tenia idea. Respondo a lo que veo ... asi q ya sabes m4rkk0s  reserva memoria

[nombrearchivo]

Tienes que copiar nombrefuncion en nombrearchivo con strcpy(). No se puede hacer mediante un '='.

en este caso no podes utilizar strcpy sino se reserva memoria dinamica, ya que nombrearchivo es un puntero. La asignacion es correcta


m4rkk0s no entendi que es lo que queres hacer, si podes dar un mejor ejemplo. Otra cosa esta combinacion "w+a" no es valida ¿¿te compila??


Saludos

Código (c) [Seleccionar] Expandir

#ifdef __unix__
            sleep(2500);

hola! me parece que en sistemas *nix la unidad de sleep esta dado en segundos. Si queremos hacer una pausa de 5seg, seria

Código (c) [Seleccionar] Expandir

sleep(5);


Saludos