Cześć,
komenda rand()%2 nie działa jak należy, wynikiem jest zawsze 0.
Użyłem wcześniej srand(time(NULL));

Dziwne bo rand()%3 i dalej działa normlanie.

Problem zaczyna się gdy wchodzę do głównej pętli programu
tam gdzie odczytywane są komunikaty okna.
Poza tą pętlą rand()%2 zwraca 0 lub 1..

Zgłupiałem

• Odpowiedz

@mateusz_s, post #1

• Odpowiedz

@Rafael/ARMO, post #2

• Odpowiedz

@mateusz_s, post #3

• Odpowiedz

@mateusz_s, post #3

• Odpowiedz

@kiero, post #4

• Odpowiedz

@asman, post #5

• Odpowiedz

@mateusz_s, post #7

if (losuj() = 0)
{
  //...
}

• Odpowiedz

@asman, post #5

• Odpowiedz

@asman, post #6

• Odpowiedz

@bfgmatik, post #9

• Odpowiedz

@mateusz_s, post #7

• Odpowiedz

@mateusz_s, post #7

• Odpowiedz

@asman, post #8

if (losuj() = 0)
{
  //...
}

Oj tak, takie rzeczy można przeoczyć nawet kilka razy analizując fragment kodu (chociaż tutaj bardziej pasowałby przykład z przyrównaniem do zmiennej, a nie funkcji).

Na szczęście dzisiejsze kompilatory bardzo krzyczą gdy w warunku jest =, a nie ==. Chociaż akurat GCC 10, którego używam pod MorphOS całkowicie to toleruje. Ale mój kod prawie codziennie jest też budowany w nowym Xcode dla macOS, a ten już wytyka takie miejsca i od razu rzucają się w oczy.

Ja ostatnio przestałem używać wykrzynika tylko przyrównuję do 0:

if (funkcja()==0)

zamiast

if (!funkcja())

bo wraz z pogarszającym się wzrokiem coraz częściej taki wykrzyknik zlewał mi się z nawiasem i dwa razy straciłem sporo czasu na odkrycie przyczyny problemu, która była oczywista... gdybym ten wykrzynik widział. W ferworze walki z problemem albo w twórczym szale człowiek często widzi to co chce widzieć, a nie to co powinien. Czasem nawet okulary czy większe fonty nie pomagają. Mózg wie swoje i już!

Ostatnia aktualizacja: 15.08.2024 22:06:40 przez MDW

• Odpowiedz

@MDW, post #14

• Odpowiedz

@asman, post #8

void    SYS__run(void)
{
    // Signal mask for window.
    ULONG signal_mask = ( 1 << SYS__window->UserPort->mp_SigBit );

    char debug_string[32];
    
    UBYTE current_buffer = 0;

     srand(time(NULL));


    // Enter main loop.
    while (G_io__prefs.is_loop)
    {  
        // Enter Window Message Loop - handle messages and events.
	    if(SetSignal(0L, signal_mask) & signal_mask)
	    {
		    struct IntuiMessage	*imsg;

            while( imsg = (struct IntuiMessage *)GetMsg(SYS__window->UserPort) )
            {
                switch (imsg->Class)
                {
                    case IDCMP_MOUSEMOVE:
                    break;

                    case IDCMP_RAWKEY:
                    break;
                }
                ReplyMsg((struct Message *)imsg);
            }       
	    }  

        // If using AGA make c2p conversion to the output buffer and display.
        if (SYS__is_aga) 
        {
            // Rendering the game.
            G_run();

            // Chunky to planar conversion.
            c2p1x1_8_c5_040(G_io__prefs.view_buffer, SYS__buffer[current_buffer]->sb_BitMap->Planes[0]);

            // Swap Buffers with sync.
            if (G_io__prefs.is_sync)
            {
                    if (!SYS__safetochange)
                    {
                        while (!GetMsg(SYS__display_port)) WaitPort(SYS__display_port);
                        SYS__safetochange = TRUE;
                    }


                if (ChangeScreenBuffer(SYS__screen, SYS__buffer[current_buffer])) 
                {
                    SYS__safetochange = FALSE;
                }
            }

            // Switch buffers.
            current_buffer ^= 1;  
        }
        else
        {
            // Puting into screen display the otherone bitmap.     
            SYS__screen->RastPort.BitMap = SYS__buffer[current_buffer^1]->sb_BitMap;
            SYS__screen->ViewPort.RasInfo->BitMap = SYS__buffer[current_buffer^1]->sb_BitMap;

            // Sync with the monitor.
            if (G_io__prefs.is_sync)
            {
                RethinkDisplay();
            }

            // Setting the game output buffer adress into one of the system bitmaps.
            // Now the geme will render directly into one of the system bitmaps.
            G_io__prefs.view_buffer = SYS__buffer[current_buffer]->sb_BitMap->Planes[0];
            
            // Rendering the game.
            G_run();

            // Switch buffers.
            current_buffer ^= 1;  
        }

        // Show some debug info.
        if (G_io__prefs.is_debug)
        { 
      
            sprintf(debug_string, "fps: %d  dt: %.4f  sync: %d", rand()%2, G_io__prefs.delta_time, G_io__prefs.is_sync);
            Move(SYS__window->RPort, 0,20);
            Text(SYS__window->RPort, (CONST_STRPTR)debug_string, strlen(debug_string));
        }
    }
}

int a = ((rand()%20) < 10);

• Odpowiedz

@teh_KaiN, post #15

żeby nie było - przypisanie wartości do wywołania funkcji zawsze skończy się błędem w c/cpp.

Tak tak, to na pewno. Dlatego napisałem, że lepszym przykładem byłoby przyrównanie do zmiennej.

Warto mieć włączone -Werror co by kompilator traktował warningi jako errory.

A to ciekawe, bo u mnie GCC 10 (które przecież bardzo stare nie jest) nie pokazuje tutaj warninga. Muszę popatrzeć.

A co do niewidoczności wykrzyknika - może wypadałoby zainwestować w jakiś lepszy schemat kolorowania składni, co by operatory miały inny kolor niż nawiasy. Albo fonta zwiększyć. ;)

Font lekko zwiększony, okulary też sporo pomogły. Ale rzeczywiście muszę pomyśleć o modyfikacji kolorowania składni, bo tutaj aż prosi się o inny kolor.

Ostatnia aktualizacja: 16.08.2024 13:26:50 przez MDW

• Odpowiedz

@mateusz_s, post #16

• Odpowiedz

@Phibrizzo, post #18

• Odpowiedz

@mateusz_s, post #16

#include <stdio.h>
#include <stdlib.h> 
#include <time.h>

int main(void)
{
	char debug_string[32];

	srand(time(NULL));

	for (int i = 0; i < 20; ++i)
	{
		sprintf(debug_string, "fps: %d  \n", rand() & 1);
		printf("%s", debug_string);
	}

	return 0;
}

• Odpowiedz

@MDW, post #14

if( ! funkcja() )

• Odpowiedz

@MDW, post #17

• Odpowiedz

@teh_KaiN, post #22

• Odpowiedz

@teh_KaiN, post #22

• Odpowiedz

@mateusz_s, post #24

• Odpowiedz

@Rafael/ARMO, post #25

• Odpowiedz

@Rafael/ARMO, post #25

• Odpowiedz

@mateusz_s, post #27

• Odpowiedz