Verkko-ohjelmointi apua kaivataan

verkkis

Hei !

Ongelmani on seuraava. Koodaan parhaillaan erästä verkkopeliä C:llä ja pitäisi saada pelaajan antama nickname näkymään jokaisen siirron yhteydessä, eli kuka teki kyseisen siirron. Kysessä client-Server tyyppinen ratkaisu jossa servu hoitaa kaiken tietoliikenteen clientien välillä.
Ohessa koodia siitä mitä tässä ajan takaa...
eli Servu välittää toiselle Clientille vastustaja Clientin siirron

memset(bufout, 0, sizeof(bufout));
strcpy(bufout, "Move from %s: ");
strcat(bufout, &bufin[1]);
strncpy(send, bufout, sizeof(send));
sendto(sock, bufout, strlen(bufout), 0, (struct sockaddr *)&clients[i], len);

7

487

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • Anonyymi

      Voisitko hieman selventää mikä kokonaisuuden osa-alue tuottaa ongelmia. Onko se:
      1. Pelaajien nickien selvittäminen servulla
      2. Tunnistaminen miltä pelaajalta paketti tulee
      3. char* nicknamen yhdistämien lähetettävään puskuriin
      4. Joku muu, mikä?

      Koodista päätellen se vaikuttaisi olevan vaihtoehto 3. Kuitenkin esimerkkikoodisi tuntuu olevan oikeilla jäljillä enkä täysin kykenen ymmärtämään, mikä on ongelma. Tosin itseäni hieman häiritsee tuo %s joka näyttää enemmän stringien stream-tyyppiseltä käsittelyltä kuin heti tämän perässä tuleva strcat. Tosin en tunne C:tä kauhean hyvin, vaan pohjaan vastauksen täysin C :aan.

      • verkkis

        Hei !

        Kyse on nimen omaan tuosta mainitsemastasi 3. vaihtoehdosta. eli char* nicknamen yhdistämien lähetettävään puskuriin. tyyliin:

        move from Pekka: H5
        yms.

        Olen koittanyt tuota kyseista koodia myös %c määrittelyllä, mutta silloin tuo kaikki on
        sekavaa lähetettyä puppua tuon nicknamen kohdilla. Ohjelmassa on määritetty array nickeille jotka
        lisätään ks. taulukkoon ohjelmaan kirjautumisen yhteydessä.


      • Anonyymi
        verkkis kirjoitti:

        Hei !

        Kyse on nimen omaan tuosta mainitsemastasi 3. vaihtoehdosta. eli char* nicknamen yhdistämien lähetettävään puskuriin. tyyliin:

        move from Pekka: H5
        yms.

        Olen koittanyt tuota kyseista koodia myös %c määrittelyllä, mutta silloin tuo kaikki on
        sekavaa lähetettyä puppua tuon nicknamen kohdilla. Ohjelmassa on määritetty array nickeille jotka
        lisätään ks. taulukkoon ohjelmaan kirjautumisen yhteydessä.

        Tutkiskelin tuossa hieman dokumentaatiota ja näyttäisi, että haluamaasi lopputulokseen voidaan päästä kahdella eri tavalla. Esimerkkikoodisi tuntuu tosin olevan jotain näiden välillä.

        Muotoilu voidaan tehdä:
        sprintf(bufout, "Move From %s: %s", nickname, movestring);
        btw. En ole aivan varma, mutta pitäsi kai tarkistaa, että lopputulos mahtuu bufout-taulukkoon, niin ei tule ilkeitä puskuriylivuotoja...

        Tai toisaalta kelpaa kait
        memset(bufout, 0, sizeof(bufout));
        strcpy(bufout, "Move from ");
        strcat(bufout, nickname);
        strcat(bufout, ": ");
        strcat(bufout, movestring);

        Tässäkin kai tulisi tarkistaa koko ennen kopioimista...

        Huomaathan, että nämä ovat vain esimerkkejä, eli ne eivät välttämättä toimi lainkaan (vaikka kyllä pitäisi). En ole koodeja testannut, eikä tuo C:n standardikirjasto ole kauhean hyvin hallussa, mutta periaatteen luulisin olevan kyllä oikein.


      • verkkis
        Anonyymi kirjoitti:

        Tutkiskelin tuossa hieman dokumentaatiota ja näyttäisi, että haluamaasi lopputulokseen voidaan päästä kahdella eri tavalla. Esimerkkikoodisi tuntuu tosin olevan jotain näiden välillä.

        Muotoilu voidaan tehdä:
        sprintf(bufout, "Move From %s: %s", nickname, movestring);
        btw. En ole aivan varma, mutta pitäsi kai tarkistaa, että lopputulos mahtuu bufout-taulukkoon, niin ei tule ilkeitä puskuriylivuotoja...

        Tai toisaalta kelpaa kait
        memset(bufout, 0, sizeof(bufout));
        strcpy(bufout, "Move from ");
        strcat(bufout, nickname);
        strcat(bufout, ": ");
        strcat(bufout, movestring);

        Tässäkin kai tulisi tarkistaa koko ennen kopioimista...

        Huomaathan, että nämä ovat vain esimerkkejä, eli ne eivät välttämättä toimi lainkaan (vaikka kyllä pitäisi). En ole koodeja testannut, eikä tuo C:n standardikirjasto ole kauhean hyvin hallussa, mutta periaatteen luulisin olevan kyllä oikein.

        Kiitos avusta..nyt sain tuon toimimaan jotenkin tuolla jälkimmäisellä rakenteellla

        memset(bufout, 0, sizeof(bufout));
        strcpy(bufout, "Move from ");
        strcat(bufout, nickname);
        strcat(bufout, ": ");
        strcat(bufout, movestring);

        mutta vielä on eräs ongelma. Jokaiseen viestiin tulostuu vain toisen pelaajan nickname. Eli ihan sama kumpi Clienteista lähettää viestin,niin sama ekan clientin nickname tulee viestin lähettäjäksi.

        nicknamet on määritelty seuraavasti ( vain nicknameen liittyvät jutut esitetty):

        char nicks[MAXCLIENTS][NICKLENGHT 1]; /*array for nicks*/

        /* copy nick to the buffer and send it to server*/

        memset(buffer_in, 0, sizeof(buffer_in));
        strncpy(buffer_in, nick, 10);

        /* send nick to the server */
        n_sent = sendto(sk, buffer_in, strlen(buffer_in), 0, (struct sockaddr*) &server, sizeof(server));


      • Anonyymi
        verkkis kirjoitti:

        Kiitos avusta..nyt sain tuon toimimaan jotenkin tuolla jälkimmäisellä rakenteellla

        memset(bufout, 0, sizeof(bufout));
        strcpy(bufout, "Move from ");
        strcat(bufout, nickname);
        strcat(bufout, ": ");
        strcat(bufout, movestring);

        mutta vielä on eräs ongelma. Jokaiseen viestiin tulostuu vain toisen pelaajan nickname. Eli ihan sama kumpi Clienteista lähettää viestin,niin sama ekan clientin nickname tulee viestin lähettäjäksi.

        nicknamet on määritelty seuraavasti ( vain nicknameen liittyvät jutut esitetty):

        char nicks[MAXCLIENTS][NICKLENGHT 1]; /*array for nicks*/

        /* copy nick to the buffer and send it to server*/

        memset(buffer_in, 0, sizeof(buffer_in));
        strncpy(buffer_in, nick, 10);

        /* send nick to the server */
        n_sent = sendto(sk, buffer_in, strlen(buffer_in), 0, (struct sockaddr*) &server, sizeof(server));

        Hmm...

        Ensimmäisenä nyt tulisi mieleen varmistaa, että nick = nicks[i], eikä esimerkiksi unohtunut muotoon nick = nicks; tai nick = nicks[0] kohdassa
        strncpy(buffer_in, nick, 10);
        tai aiemmin...

        Nickithän tulevat kuitenkin onnistuneesti ja ne lisätään taulukkoon oikein? Senkin voisi tarkistaa parilla printf:llä. Ja muutenkin voisi olla hyvä ainakin aluksi printata konsoliin (printf) kaikki mitä servu luulee lähettävänsä, eli tässä tapauksessa tuo nick tai buffer_in.


      • verkkis
        Anonyymi kirjoitti:

        Hmm...

        Ensimmäisenä nyt tulisi mieleen varmistaa, että nick = nicks[i], eikä esimerkiksi unohtunut muotoon nick = nicks; tai nick = nicks[0] kohdassa
        strncpy(buffer_in, nick, 10);
        tai aiemmin...

        Nickithän tulevat kuitenkin onnistuneesti ja ne lisätään taulukkoon oikein? Senkin voisi tarkistaa parilla printf:llä. Ja muutenkin voisi olla hyvä ainakin aluksi printata konsoliin (printf) kaikki mitä servu luulee lähettävänsä, eli tässä tapauksessa tuo nick tai buffer_in.

        Mulla on määrittely clientin startissa:

        else if(port != 0 && address[0] != '\0' && nick[0] != '\0') Joka asettaa tuon nickarrayn nollaksi ??
        {
        startClient(port, address, nick); /*start client is port, address and nick is given*/
        }


      • Anonyymi
        verkkis kirjoitti:

        Mulla on määrittely clientin startissa:

        else if(port != 0 && address[0] != '\0' && nick[0] != '\0') Joka asettaa tuon nickarrayn nollaksi ??
        {
        startClient(port, address, nick); /*start client is port, address and nick is given*/
        }

        Tuossa koodissa ei minun silmiini näy kyllä mitään vikaa. Tuo koodihan ei muuta ainakaan nick-muuttujaa mitenkään, vain vertaa sen ensimmäistä arvoa ( = alkiota? ), että nick ei ole tyhjä.

        Itse tutkisin ensimmäiseksi nyt yhdistyskoodin servupäässä, sillä clientti todennäköisesti toimii oikein, koska toinen clientti tässä operaatiossa onistuu. Voisi siis olettaa, että serverissä on tämä vika, koska joinimisjärjestys tuskin vaikuttaa juurikaan clientin logiikkaan toisin kuin serverillä.

        Sitä paitsi, eikös servu luo täysin tuon "Move from X: YZ" palautteen, eikä tähän ole clientillä osaa eikä arpaa. Veikkailisin siis serveripään ongelmaa...


    Ketjusta on poistettu 0 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Päivän Sanna: Nvidia sijoittaa miljardi dollaria Nokian osakkeisiin

      Nvidia merkitsee osakkeet 6,01 Yhdysvaltain dollarin osakekohtaisella merkintähinnalla tavanomaisten toteutumisehtojen t
      Maailman menoa
      65
      5726
    2. Sanna Marin teki sen, mihin muut eivät pystyneet - sote kerralla maaliin

      Yli 15 vuotta Suomessa vatvottu sote-uudistus meni lopulta läpi Sanna Marinin hallituksen aikana. Edeltävät hallitukset
      Maailman menoa
      11
      3733
    3. Palkansaajilta kupattiin 27,5 mrd euroa työeläkkeisiin

      Jo pelkän himmelin toimintakulut olivat 400 miljoonaa euroa, jolla olisi mukavasti tuottanut myös sote-palveluja hyvinvo
      Maailman menoa
      109
      3617
    4. Suomessa Pohjoismaiden tyhmimmät demarit......aijaijai..

      Lasse Lehtinen sanoo suoraan, että Ruotsissa on fiksummat demarit kuin Suomessa. Ja Tampereella fiksummat demut kuin Hel
      Maailman menoa
      57
      3458
    5. Marinin hallitus hyväksyi soten (105-77) vuonna 2021

      vastaan äänesti Kok, persut, KD, Liike Nyt. Nyt on sitten käynyt niin kuin on käynyt. Pääkirjoitus: Sanna Marin jätti
      Maailman menoa
      56
      3090
    6. Lehtinen: "Oli demareidenkin onni, että valkoiset voittivat sodan 1918"

      Lasse Lehtisen mukaan vasemmalla on radiohiljaisuus hänen uutuuskirjastaan, "Läheltä piti. Kansakunnan hurjat hetket" L
      Maailman menoa
      94
      2695
    7. SDP:n valtuutettu valehtelee koulutuksensa

      SDP:n helsinkiläinen kaupunginvaltuutettu Mahad Ahmed käyttää maisterin titteliä suoritettuaan 60 opintopisteen epäviral
      Maailman menoa
      73
      2548
    8. IL - 100 000 potentiaalista sotilasta pakeni Ukrainasta!

      "Ukrainasta nuorten miesten joukkopako Liki 100 000 asevelvollisuusikäistä miestä on poistunut Ukrainasta parin viime k
      Maailman menoa
      19
      2482
    9. Nepotismi jytkyttää porvaristossa

      Taas uutisoidaan, että useampi perussuomalainen kansanedustaja on palkannut oman perheenjäsenensä eduskunta-avustajaksi
      Maailman menoa
      87
      1985
    10. Persut palkkaavat verorahoilla

      avustajikseen lapsiaan ja naisystäviään. Juuso on palkannut avustajakseen oman poikansa. Hänen koulutustaustastaan ei
      Maailman menoa
      342
      1924
    Aihe