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

472

    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. Useita puukotettu Tampereella

      Mikäs homma tämä nyt taas on? "Useaa henkilöä on puukotettu Tampereen keskustassa kauppakeskus Ratinan lähistöllä." ht
      Tampere
      148
      2814
    2. Asiakas iski kaupassa varastelua tehneen kanveesiin.

      https://www.iltalehti.fi/kotimaa/a/33a85463-e4d5-45ed-8014-db51fe8079ec Oikein. Näin sitä pitää. Kyllä kaupoissa valtava
      Maailman menoa
      363
      2017
    3. Leipivaaran päällä on kuoleman hiljaista.

      Suru vai suuri helpotus...
      Puolanka
      40
      1757
    4. Kuka rääkkää eläimiä Puolangalla?

      Poliisi ampui toistakymmentä nälkiintynyttä eläintä Puolangalla Tilalta oli ollut karkuteillä lähes viisikymmentä nälkii
      Puolanka
      35
      1678
    5. Meneeköhän sulla

      oikeasti pinnan alla yhtä huonosti kuin mulla? Tai yhtä huonosti mutta jollain eri tyylillä? Ei olisi pitänyt jättää sua
      Ikävä
      31
      1393
    6. Jos ei tiedä mitä toisesta haluaa

      Älä missään nimessä anna mitään merkkejä kiinnostuksesta. Ole haluamatta mitään. Täytyy ajatella toistakin. Ei kukaan em
      Ikävä
      94
      1213
    7. Määpä tiijän että rakastat

      Minua nimittäin. Samoin hei! Olet mun vastakappaleeni.
      Ikävä
      54
      1173
    8. Muutama kysymys ja huomio hindulaisesta kulttuurista.

      Vedakirjoituksia pidetään historiallisina teksteinä, ei siis "julistuksena" kuten esimerkiksi Raamattua, vaan kuten koul
      Hindulaisuus
      327
      949
    9. Jumala puhui minulle

      Hän kertoi sinusta asioita, joiden takia jaksan, uskon ja luotan. Hän kuvaili sinua minulle ja pakahduin onnesta kuulles
      Ikävä
      110
      938
    10. Annan meille mahdollisuuden

      Olen avoimin mielin ja katson miten asiat etenevät. Mutta tällä kertaa sun on tehtävä eka siirto.Sen jälkeen olen täysil
      Ikävä
      53
      782
    Aihe