Mensajes

Nota incial: Para el código, usa las etiquetas GeSHi, gracias.

Código (c) [Seleccionar] Expandir

void split(char *s[], char *c1[], char *c2[], char *car){
   if (strstr(s[60], car)){
      char aux=s-strlen(strstr(s[60], car));
      strcat(c1[60], aux);
      strcat(c2[60], (strchr(s[60], (strlen(car)+1))));
   }
}

'strstr' devuelve el puntero a la primera aparición de 'car'... si reutilizases ese valor tendrías medio trabajo hecho...

Una cosa que no entiendo es por qué tienes 's[60]' en el strstr, en el strlen, en... eso está mal.

Código (c) [Seleccionar] Expandir

void split(char *s[], char *c1[], char *c2[], char *car){
   if (strstr(s, car)){
      char aux=s-strlen(strstr(s, car));
      strcat(c1, aux);
      strcat(c2, (strchr(s, (strlen(car)+1))));
   }
}

De primeras no se si así funcionará... no puedo compilar ahora, pero al menos pasas punteros a las funciones en vez de valores que no tienen ningún sentido.

Nota inicial: Para el código, usa las etiquetas GeSHi, gracias.

Código (cpp) [Seleccionar] Expandir


#include <cstdlib>
#include <iostream>
#include <stdio.h>
#include <math.h>

Si estás cargando las librerías cstdlib e iostream, estás programando en C++, no en C. Los includes de la librería estándar de C tienen TODOS el ".h"... en el caso de la librería de C++, NO hay includes con ".h". Te lo comento para que luego no te lleves sustos.

Código (cpp) [Seleccionar] Expandir

using namespace std;

Los espacios de nombre son también un elemento propio de C++... deberías quitarlo y, si te aparecen errores por ello, es porque estás haciendo uso de utilidades propias de C++.

Código (c) [Seleccionar] Expandir


int carga (int x);
int carga (int x) /* Funcion que permite cargar toda la matriz*/
{
}

Las declaraciones de funciones ( en tu caso la primera línea del código que te he puesto ) sólo son necesarias cuando aparecen llamadas a la función antes que su implementación. Un ejemplo:

Código (c) [Seleccionar] Expandir



void func3( void );

void func1( void )
{  puts( "Hola\n" ); }

int main( void )
{
 func1( ); // Ok, su uso es posterior a su implementacion
 func2( ); // Error, esta funcion no esta definida todavia
 func3( ); // Ok, no esta implementada pero si definida
}

void func2( void )
{ puts( "---\n" ); }

void func3( void )
{ puts( "Adios\n" ); }

Más cosas...

Código (c) [Seleccionar] Expandir


int carga (int x) /* Funcion que permite cargar toda la matriz*/
{
 int register k,l;
 for (k=0;k<N;k++)
 {
   for (l=0;l<M;l++)
   {
     printf ("Ingrese un numero ");
     scanf ("%f",x);
   }
 }
}

La llamada a "scanf" esta mal por tres razones:

1. %f se usa para números con decimales (float, double)... x es de tipo entero (int)
2. scanf necesita un puntero a la variable donde debe almacenar los valores que introduzca el usuario... 'x' se está pasando por valor
3. 'carga' está recibiendo 'x' por valor, es decir, todos los cambios que se hagan en 'x' se van a perder una vez el código sale de la función 'carga'.

Ah si... segun la firma actual, 'carga' debería tener un return y devolver un entero... si una función no ha de devolver nada debe ir precedida de 'void'.

La función debería estar más bien así:

Código (c) [Seleccionar] Expandir


void carga (int* x) /* Funcion que permite cargar toda la matriz*/
{
 int register k,l;
 for (k=0;k<N;k++)
 {
   for (l=0;l<M;l++)
   {
     printf ("Ingrese un numero ");
     scanf ("%d",x);
   }
 }
}

Ahora, 'x' es un puntero, por lo que, por un lado, scanf podrá almacenar el valor en la variable apuntada por 'x' sin problemas y, por otro, los cambios en 'x' se reflejarán fuera de la función.

Debido al cambio de la firma de "carga", también hay que cambiar ahora sus llamadas. "carga" ahora necesita un puntero:

Código (c) [Seleccionar] Expandir


int main(int argc, char *argv[])
{
   int num;
   carga (&num);
}

Más cosas...

Código (c) [Seleccionar] Expandir


int arr[N][M];
int cascara;
int i, j;
for (i=0; i<N; i+1)
{
 for (j=0; j<M; j+1)
 {
   cascara = (cascara + arr[j]);
 }
}

'arr' es un arreglo bidimensional, luego para acceder a sus valores necesitas pasarle dos índices... tu estás pasando sólamente uno. Esto tendría que quedar más o menos así:

Código (c) [Seleccionar] Expandir


int arr[N][M];
int cascara;
int i, j;
for (i=0; i<N; i+1)
{
 for (j=0; j<M; j+1)
 {
   cascara = (cascara + arr[i][j]);
 }
}

Además, 'arr' está definda tal que 'arr[N][M]'... no tiene sentido que le pases como primer índice 'j', que está definida en el rango '0..M'; necesitas un índice definido en el rango '0..N' y ese índice es 'i'... no 'j'.

Por otro lado, no has asignado valores a los elementos contenidos en 'arr', así como tampoco has asignado valores a 'cascara' (ni a otras tantas variables)... los resultados finales de las operaciones que tienes puestas ahí van a ser totalmente incoherentes, ya que estás leyendo basura:

Código (c) [Seleccionar] Expandir


int main( )
{
 int numero;
 printf( "%d\n", numero );
}

¿Qué imprime el código anterior? ¿tu lo sabes? yo lo único que se es que la salida del programa puede cambiar en diferentes ejecuciones. Sin embargo:

Código (c) [Seleccionar] Expandir


int main( )
{
 int numero = 10;
 printf( "%d\n", numero );
}

Estoy totalmente seguro que la salida de esta segunda versión será siempre '10'.

Más cosas... podría seguir pero creo que ya es bastante ayuda.

Moraleja: Estudia unos cuantos tutoriales o manuales de C antes de meterte en estos jardines... aprenderá más, será más satisfactorio para tí y aprenderás más ( lo pongo dos veces para darle énfasis... es importante).

Es una chorrada, pero es bueno ir adquiriendo buenas prácticas... sobretodo porque como te de en el futuro por cambiar algo te va a volver a dar problemas. Esto:

Código (c) [Seleccionar] Expandir

        len=strlen("nombre=nombre123correcto&contenido=123456");
        itoa(len,cad,10);
        strcpy(buffer,"POST /index.php HTTP/1.0\r\n");
        strcat(buffer,"Host: mipaginaweb.free33.com\r\n");
        strcat(buffer,"Content-type: application/x-www-form-urlencoded\r\n");
        strcat(buffer,"Content-length: ");
        strcat(buffer,cad);
        strcat(buffer,"\r\n");
        strcat(buffer,"nombre=nombre123correcto&contenido=123456");
        printf("%s\n",buffer);

Se debería parecer más a esto:

Código (c) [Seleccionar] Expandir


        char* contenido = "nombre=nombre123correcto&contenido=123456";
        len = strlen( contenido );
        itoa(len,cad,10);
        strcpy(buffer,"POST /index.php HTTP/1.0\r\n");
        strcat(buffer,"Host: mipaginaweb.free33.com\r\n");
        strcat(buffer,"Content-type: application/x-www-form-urlencoded\r\n");
        strcat(buffer,"Content-length: ");
        strcat(buffer,cad);
        strcat(buffer,"\r\n");
        strcat(buffer, contenido);
        printf("%s\n",buffer);

El motivo es obvio... tener un mensaje duplicado implica tener que actualizarlo en dos o más sitios so pena que el paquete pase a estar mal construido y te falle la comunicación.

Ojala supiera emular un servidor, jaja entonces no estaba aquí supongo xD, ok gracias eferion es cierto ahí sería content, yo creo que es un fallo de cabecera, donde pongo host, pongo el host con el que accedo a la pagina desde navegador, no con el que accedo por ftp con filezilla, no se cual de los 2 poner.

Espero no haber metido la pata con el código, lo he hecho "a mano alzada" y no está probado.

Código (c) [Seleccionar] Expandir


#define MAX_BUFFER 512

// OJO que los puertos inferiores a 1024 estan reservados, lo mismo tienes que cambiar el puerto
#define PUERTO_SERVIDOR "80"

int main( void )
{
    WSADATA wsaData;
   
    // Inicializamos WinSock
    int errorCode = WSAStartup( MAKEWORD( 2, 2 ), &wsaData );
    if ( errorCode != 0 )
   {
        printf("Error en WSAStartup: %d\n", errorCode );
        getch( );
        return 1;
    }

    struct addrinfo configInicial;
    ZeroMemory( &configInicial, sizeof( configInicial) ); // IMPORTANTE limpiar la memoria
    configInicial.ai_family   = AF_INET;
    configInicial.ai_socktype = SOCK_STREAM;
    configInicial.ai_protocol = IPPROTO_TCP;
    configInicial.ai_flags    = AI_PASSIVE;

    // Resolver la dirección IP y el puerto del servidor
    struct addrinfo *configServidor= NULL;
    errorCode = getaddrinfo( NULL, PUERTO_SERVIDOR, &configInicial, &configServidor );
    if ( errorCode != 0 )
    {
        printf("Error en getaddrinfo: %d\n", errorcode );
        WSACleanup( ); // IMPORTANTE reiniciar WinSock
        return 1;
    }

    // Crear socket de escucha
    SOCKET socketServidor= INVALID_SOCKET;
    socketServidor= socket( configServidor->ai_family, configServidor->ai_socktype, configServidor->ai_protocol );
    if ( socketServidor== INVALID_SOCKET )
    {
        // Fijate que podemos obtener el codigo de error al crear el socket
        printf("Error al crear socket de escucha: %ld\n", WSAGetLastError( ) );

        // Hay que limpiar la memoria
        freeaddrinfo( configServidor );

        // Y reiniciar WinSock
        WSACleanup();
        return 1;
    }

    // Se enlaza el socket del servidor para que use la ip y el puerto solicitado
    errorcode = bind( socketServidor, configServidor->ai_addr, (int)configServidor->ai_addrlen );
    if ( errorCode == SOCKET_ERROR)
    {
        printf("Error en bind: %d\n", WSAGetLastError( ) );

        // Liberamos recursos
        freeaddrinfo( configServidor );
        closesocket( socketServidor );
        WSACleanup( );
        return 1;
    }

    // Ya no necesitamos esta configuracion, el socket esta creado
    freeaddrinfo( configServidor );

    // Dejamos al socket esperando conexiones
    errorCode = listen( socketServidor , SOMAXCONN );
    if (errorCode == SOCKET_ERROR)
    {
        printf( "Error listen: %d\n", WSAGetLastError( ) );
        closesocket( socketServidor);
        WSACleanup( );
        return 1;
    }

    // Procesamos las peticiones que nos llegan
    do
    {
        SOCKET socketCliente= INVALID_SOCKET;
        socketCliente= accept( socketServidor, NULL, NULL );
        if ( socketCliente== INVALID_SOCKET)
        {
            printf( "Error en conexion entrante: %d\n", WSAGetLastError( ) );
            closesocket( socketServidor );
            WSACleanup( );
            return 1;
        }

        // Recibimos el mensaje del cliente
        printf( "Conexión entrante:\n" );
        printf( "------------------------------------------\n" );
        int bytesMensaje = 0;
        char recvbuf[ MAX_BUFFER ];
        int recvbuflen = MAX_BUFFER;
        do
        {
            bytesMensaje = recv( socketCliente, recvbuf, recvbuflen, 0 );
            if ( bytesMensaje > 0)
            {
                recvbuf[ bytesMensaje ] = 0;
                printf( "Mensaje recibido:\n" );
                puts( recvbuf );
                printf( "\n" );
                printf("Bytes recibidos: %d\n", bytesMensaje );

                // Tambien podemos enviarle una respuesta al cliente
                // En este caso le devolvemos el mensaje que nos ha enviado
                bytesMensaje = send( ClientSocket, recvbuf, bytesMensaje, 0 );
                if ( bytesMensaje == SOCKET_ERROR)
                {
                    printf("Error al enviar la respuesta: %d\n", WSAGetLastError()  );
                    closesocket( socketCliente );
                    break;
                }
                printf( "Bytes enviados: %d\n", bytesMensaje );
            }
            else if ( bytesMensaje == 0 )
                printf( "Conexion con el cliente cerrada.\n" );
            else
                printf( "Error al establecer conexion con cliente: %d\n", WSAGetLastError( ) );
        } while ( bytesMensaje > 0 );

        closesocket( socketCliente );
    } while ( 1 );

    // En este caso el servidor no para nunca
    // Si se programase una parada habria que realizar las tareas de limpieza
    closesocket( socketServidor );
    WSACleanup( );

    return 0;
}

[Context-Type]

jajaja ya lo siento eferion pensaba que me estabas vacilando no quería ser cruel, pero tus preguntas me parecian absurdas y que estaban en el código todas, sino fallo mio por pasarme de listo xD.

En ese caso sin problemas. Intento que mis preguntas tengan cierto sentido, aunque a veces no lo parezcan. Me puede pasar, como a todo hijo de vecino, que pregunte algo que ya esté contestado en el código... pese a quien le pese soy humano y a veces también me equivoco, aunque no lo hago de mala fe.

La IP es así 84.234.145.130 solo que con numeros cambiados, la obtuve haciendo ping al servidor web, y el socket se crea correctamente,no creo que haya ningún fallo al respecto

Ok, por tu comentario anterior pensé que estabas leyendo la IP dinámicamente haciendo un ping al servidor o algo así... y no me cuadraba el asunto para nada. Cuando consigas corregir esto sería una buena mejora que permitas especificar la dirección del servidor en un archivo de configuración (bien por DNS - lo ideal -, bien por ip). Tener estos datos escritos a pelo en el código solo dan problemas.

printf("%d\n",direccion.sin_family);//muestra 2
printf("%d\n",direccion.sin_port);//muestra 20480
printf("%d\n",direccion.sin_addr.s_addr);//muestra -21023899

Pero no entiendo la IP es la que devuelve el ping no? xD

la función inet_addr hace una conversión para "empaquetar" la ip en formato de red. Esta función tiene en cuenta, entre otras cosas, si tu máquina es BIG ENDIAN o LITTLE ENDIAN a la hora de realizar la conversión... además la IP es de tipo "unsigned", luego si la imprimes con "%d" que se usa para "signed", el valor que te saque no va a servirte de mucho.

PD.: prueba a cambiar Context-Type por Content-Type. Hasta donde llego, Context-Type no es un parámetro válido de cabecera. Lo mismo al tener una cabecera incorrecta el servidor rechaza la conexión... o no sabe como tratarla.

Si esto no soluciona el problema yo realizaría otra prueba para eliminar puntos de fallo:

* Crea un programa que "emule" al servidor web, de tal forma que, simplemente, te saque por pantalla todo lo que reciba por el socket. Después lanza tu cliente contra este programa y mira a ver si imprimes algo. Así al menos puedes verificar que el programa realiza una salida y además puedes comprobar si ésta es la esperada.

Edito: ¿No ves que si? OMG antes de preguntar dedica un minisegundo a mirar el código

Código (c) [Seleccionar] Expandir

direccion.sin_addr.s_addr=inet_addr(ip);

Si, pedazo de comprobación... seguro que esta línea funciona aunque en "ip" esté poniendo "estoesunamierda" en vez de algo tipo "192.168.1.5"... si, no se cómo no he podido ver que, efectivamente, estabas haciendo todas las comprobaciones precisas para asegurar que "ip" tiene una dirección IP válida.

Gracias por tu ayuda eferion, ahora deja a los demás responder, el código ya está posteado no voy a responder preguntas absurdas.

Me parece estupendo... seguro que con tus super conocimientos de C y C++ eres capaz de sacarlo tu solo... a no, espera, que estás aquí pidiendo ayuda.

¿Crees que le estoy quitando a alguien el derecho a responderte? ¿De verdad sabes cómo funciona Internet?

Si alguien te quiere responder lo podrá hacer te pregunte o no absurdeces que, dicho sean de paso, pueden dar lugar a que tu super programa de 30 líneas no funcione.

Ahora permite que un ignorante como yo que, de seguro, ha hecho más programas que tú (y que además funcionan) te explique un pequeño detalle:

Si tu programa tuviese un error de bulto habría aflorado en los primeros 3 o 4 mensajes, la gente que suele contestar en este foro no es manca... Dado que no ha sido así tu problema puede estar en los detalles más absurdos:

* Falta de permisos de escritura
* Estás recuperando mal la IP
* No has inicializado correctamente el socket (ya se ha visto que no es esto)

Detalles así de divertidos y de obvios que hacen que la gente pierda su tiempo en contestar a una persona como tú que recurre aquí porque no tiene ni la más remota idea de cómo solucionar un problema... y encima se queja cuando la intentan ayudar... no se, tu mismo.

Y dicho esto, te tomo la palabra. Tranquilo que no vuelvo a pasarme por este hilo... ni por este ni por ninguno que abras... puede que con esto pierdas algún que otro consejo valioso o puede que no. Yo por mi parte me guardo mi opinión personal al respecto.



EDITO:

Solo una pregunta tonta que vuelvo a repetir porque, insisto, no lo veo en el código

¿Dónde narices está el código que asigna un valor a "ip"?

O mejor, no contestes, seguro que así es más divertido encontrar el fallo a tu programa.

y asumo que has comprobado que el valor almacenado en esa variable es el correcto, cierto?

Cómo estás generando "ip"?

Una pregunta tonta... en qué punto concreto has inicializado el socket. Algo del tipo:

Código (c) [Seleccionar] Expandir

if((s = socket(AF_INET , SOCK_STREAM , 0 )) == INVALID_SOCKET){ }

Si no has inicializado el socket es como intentar escribir en un fichero que no has abierto.

Esta línea o su equivalente debería ir antes del "connect".

Cierto, error mio, no lo ví y eso que estuve mirando el código con "tranquilidad".

Un saludo.

El número que recibes se encuentra en binario y tu has de convertirlo a su representación en base 10.

Para convertir un número de una base a otra tienes que ir procesando el número cifra a cifra. Por tanto, almacenar el número en binario no parece una buena idea (además un número en binario ocupa más que el equivalente en decimal: 100 = 8, por lo que almacenar el binario, en un entero como si fuese un decimal te va a limitar el rango de datos).

Lo suyo es que almacenes el número en una cadena de texto (clase string)... entonces recorres dicha cadena de principio a fin y para cada carácter de la cadena haces lo siguiente:

* Multiplicas el resultado parcial por 2
* Sumas la equivalencia decimal del caracter actual: ( '0' = 0, '1' = 1 )

Al finalizar el proceso tendrás la equivalencia decimal del número binario que has introducido.

PD.: El siguiente código

Código (cpp) [Seleccionar] Expandir

while ((numero / numeroDivisor) == 0)

Solo va a ser verdadero cuando numero < numeroDivisor, es decir, que salvo que metas un "binario" menor que 10 -> 2 en base decimal, el resultado del while va a ser falso... ese es el motivo por el que no te imprime nada.

Estás seguro de que tienes permisos de escritura para la ruta del archivo?