C/C++ Programming Help Thread

[Gonrai]

Well heres my project, but it links to the graphics.h lib....
http://savefile.com/files/160310

[psphacker12.]

Code:

#include "main.h"

LOCAL char *readInput(FILE *fp) {
    char *retp  = NULL;
    int   curp  = 0, ch;
    int   cursz = EXP_INP_SZ_;

    retp = my_malloc(sizeof *retp * cursz);

    while ( (ch = fgetc(fp)) != EOF ) {
        if ( curp >= cursz ) {
            retp   = my_realloc(retp, sizeof *retp * (cursz + EXP_INP_SZ_));
            cursz += EXP_INP_SZ_;
        }
        retp[curp++] = ch;
    }

    if ( !curp ) {
        MY_FREE(retp);
        return NULL;
    }

    if ( retp[curp-1] == '\n' ) {
        retp = my_realloc(retp, sizeof *retp * curp);
        retp[curp-1] = 0;
    }
    else {
        retp       = my_realloc(retp, sizeof *retp * (curp + 1));
        retp[curp] = 0;
    }
    return retp;
}

LOCAL void printCipherMap(CipherText *ctptr) {
    int i, j;
    CipherChar *ccptr = ctptr->ciphermap;
    int  *ciphermap  = ctptr->plainmap;
    char *plainchars = ctptr->plainchars;

    for ( i = 0; i < ALPHA_SZ_; ++i ) {
        int current = ciphermap[i];

        if ( !ccptr[i].isPresent )
            continue;
        printf("%c => ", 'a' + i);
        if ( plainchars[i] == '?' ) {
            current = ciphermap[i];
            for ( j = 0; j < ALPHA_SZ_; ++j ) {
                if ( current & aleph_codes[j] ) {
                    printf("%c", 'a' + j);
                    current &= ~aleph_codes[j];
                    if ( current )
                        printf(", ");
                }
            }
        }
        else
            printf("%c", plainchars[i]);
        printf("\n");
    }
}

int main(int argc, char *argv[]) {
    char *txt;
    CipherText gctxt;
    clock_t start, end;
    double duration;
    int unsolved;

    if ( argc <= 1 )
        txt = readInput(stdin);
    else {
        FILE *fp = fopen(argv[1], "r");
        if ( !fp ) {
            perror("fopen failed");
            fprintf(stderr, "Input file couldn't be opened\n");
            exit(EXIT_FAILURE);
        }
        txt = readInput(fp);
    }
    if ( !txt ) {
        fprintf(stderr, "No input given!!\n");
        exit(EXIT_FAILURE);
    }

    start = clock();
    unsolved = solveQuip(txt, &gctxt);
    end = clock();

    if ( !unsolved ) {
        printf("Original Text:\n");
        printf("%s\n\n", gctxt.ociphertxt);
        printf("Deciphered Text:\n");
        printf("%s\n\n", gctxt.plaintxt);
        printf("Cipher to Plain text mapping:\n");
        printf("-----------------------------\n");
        printCipherMap(&gctxt);
        duration = (double) (end - start) / CLOCKS_PER_SEC;
        printf("Solving time: %2.1f seconds\n", duration);
    }
    else
        exit(EXIT_FAILURE);

    return 0;
}

Errors:

[Lukeson]

First of all, thanks for your answers!

As for the list of artist, I think the best delimiter would be a newline-character (\n) since it will never appear in an artistname; It would look like this:

536			// First line, number of artists
A perfect circle	// Name of the artist (#1)
3			// Albumcount
Emotive			// Name of the album (#1)
Mer de noms		// Name of the album (#2)
The thiteenth step	// Name of the album (#3)
Aphex Twin		// Name of the artist (#2)
2			// albumcount
Come to daddy		// Name of the album (#1)
Selected ambient works	// Name of the album (#2)
[...]			// Name of the artist (#3)

I thought by sending along the lenght of the artist- album-names I could allocate exactly allocate the amount of memeory I needed, not more not less. No artists- or album-name would have been cut. But as I heard allocating memory using malloc slows down the PSP, so I'll go with your suggestions.

There is only one question left: How do I read a varialble until the next newline as quickly as possible??

[Insert_Witty_Name]

psp12: Learn how to code before trying to copy/paste someone elses code into your own.

Problem fixed.

[psphacker12.]

psp12: Learn how to code before trying to copy/paste someone elses code into your own.

Problem fixed.

??
im trying to port cryptoquip
-= Double Post =-
okay heres my problem..
I have solveQuip declared as a function like this

int solveQuip(char *txt, CipherText *ctptr) {
    CipherWord **cipherwordptrs;
    int i;
    int newmap[ALPHA_SZ_];
    int ncipherwords;
    Bool nosolution = FALSE;
    char *ciphertxt = ctptr->ociphertxt;
    char plainmap[ALPHA_SZ_];

    processInput(txt, ctptr);
    if ( ctptr->ncipherwords <= 0 )
        return 1;

    cipherwordptrs = my_malloc(sizeof *cipherwordptrs * ctptr->ncipherwords);
    for ( i = 0; i < ctptr->ncipherwords; ++i ) {
        cipherwordptrs[i] = ctptr->cipherwords[i];
    }

    for ( i = 0; i < ctptr->ncipherwords-1; ++i ) {
        int j;
        int ikey = cipherwordptrs[i]->len;

        for ( j = i+1; j < ctptr->ncipherwords; ++j ) {
            int jkey = cipherwordptrs[j]->len;
            if ( ikey < jkey ) {
                CipherWord *temp = cipherwordptrs[i];
                cipherwordptrs[i] = cipherwordptrs[j];
                cipherwordptrs[j] = temp;
                ikey = jkey;
            }
        }
    }

    for ( i = 0; i < ctptr->ncipherwords; ++i ) {
        CipherWord *wptr = cipherwordptrs[i];
        getMatchingWordsFromDict(wptr, dicts + wptr->len - 1, ctptr->ciphermap);
    }

    ncipherwords = ctptr->ncipherwords;
    while ( 1 ) {
        int j;

        nosolution = solveWords(cipherwordptrs, ncipherwords, newmap);
        if ( !nosolution )
            break;

        for ( i = 0; i < ncipherwords; ++i ) {
            if ( cipherwordptrs[i]->isIgnored )
                continue;
            cipherwordptrs[i]->isIgnored = TRUE;
            fprintf(stderr, "Couldn't find a solution, Ignoring %s and trying others\n", cipherwordptrs[i]->word);
            break;
        }
        if ( i == ncipherwords )
            break;
        for ( j = 0; j < ALPHA_SZ_; ++j ) {
            CipherChar *ccptr = ctptr->ciphermap + j;
            if ( ! ccptr->isPresent )
                continue;
            ccptr->plainmap = ALEPH_ALL;
        }
        for ( ++i; i < ncipherwords; ++i ) {
            if ( cipherwordptrs[i]->isIgnored )
                continue;
            cipherwordptrs[i]->curmatch = 0;
            if ( cipherwordptrs[i]->possiblewords )
                MY_FREE(cipherwordptrs[i]->possiblewords);
            getMatchingWordsFromDict(cipherwordptrs[i], dicts + cipherwordptrs[i]->len - 1, ctptr->ciphermap);
        }
    }

    if ( nosolution ) {
        fprintf(stderr, "Couldn't find a solution!\n");
        return 1;
    }
    else {
        convertMapToChars(ctptr->ciphermap, newmap, plainmap);

        memcpy(ctptr->plainmap, newmap, sizeof *newmap * ALPHA_SZ_);
        memcpy(ctptr->plainchars, plainmap, sizeof *plainmap * ALPHA_SZ_);

        ctptr->plaintxt = my_malloc(sizeof *ctptr->plaintxt * strlen(ctptr->ociphertxt));
        strcpy(ctptr->plaintxt, ctptr->ociphertxt);
        ciphertxt = ctptr->plaintxt;
        for ( ; *ciphertxt; ciphertxt++ ) {
            char cipherchar = *ciphertxt;
            int  isupper;

            if ( !isalpha((int) cipherchar) )
                continue;
            isupper    = isupper((int) cipherchar);
            cipherchar = tolower((int) cipherchar);
            *ciphertxt = isupper ? toupper((int)plainmap[CHAR_TO_INDEX(cipherchar)]) : plainmap[CHAR_TO_INDEX(cipherchar)];
        }
    }
    return 0;
}

but when i try to use it like this..

unsolved = solveQuip(txt, &gctxt);

it gives me these errors

[email protected] /$
$ make
psp-gcc -I. -I/usr/psp/sdk/include -O2 -G0 -Wall   -c -o main.o main.c
main.c : In function 'main':
main.c(186) : error: called object 'solveQuip' is not a function
make: *** [main.o] Error 1

[Gonrai]

Again, heres my project:
http://savefile.com/files/160310
Please someone help, whats wrong with it? The graphics.h connects to it fine..I dont know whats wrong.

[Raphael]

But as I heard allocating memory using malloc slows down the PSP, so I'll go with your suggestions.

That's plain utter bull. malloc shouldn't be used inside inner loops of realtime-apps, yes, but even there it is possible to some extent. Whoever told you that, don't listen to him.

There is only one question left: How do I read a varialble until the next newline as quickly as possible??

Uh... read char by char into a buffer until one char turns up as newline. Stop. Repeat. Pretty simple :P

[SG57]

ugh...

1. Remove the ] bracket at the start of your main.c...
2. You dont need that extern C stuff since you're already saying it's in C by having main file main.c...

That should fix it.

[Gonrai]

All right its actually working now :P
But is there a way to make it so when I press Up or Down he doesnt move like 1 inch?? Like when i press it he doesnt keep moving he just stops.
How do i make it so when i hold it down he keeps moving, and when i release he stops?

[SG57]

Gonrai - You specifically said to make him only move once (in the code). Remove the UP and DOWN if statements (the button input) OUT of the if statement that says if oldpad ~=pad or w/e.

[SodR]

You need to add something like this mate, at the end of the makefile:

noelf: kxploit
	rm -f myfile.elf

Then when you come to compile just put 'make noelf'.

Note that the rm line is indented with a tab, this is required. You can add more lines after that if you wish to delete other files etc, these need a tab indent also.

Thanks for the code, I appreciate it. The code you gave me worked like a charm. But if only want a "regular" eboot I'll first have to type "make" then "make noex" (I have deleted the kxploit line). Is there a way to short it down so I'll only have to type one line in cygwin?

Psp12.: Try changing 'int' to 'void', that works sometimes with errors like these. At least I had an error that said that it was not a function and it took me a day to figure out what was worng. But I still don't know why it didn't work when I was using int :)

[SG57]

SodR - Im not sure what your question really is, but to spit out a NON kxploit eboot, just type 'make' rather than 'make kxploit'. Once again, Im not 100% what you and Insomniac had discussed and am a little too lazy to read back :o

[SodR]

SodR - Im not sure what your question really is, but to spit out a NON kxploit eboot, just type 'make' rather than 'make kxploit'. Once again, Im not 100% what you and Insomniac had discussed and am a little too lazy to read back :o

Lol, no that is not what I'm asking :) I just wondered if you could type make clean without first having to type make to generate the eboot. That is so I wont have to type two lines into cygwin ;)

[Lukeson]

Uh... read char by char into a buffer until one char turns up as newline. Stop. Repeat. Pretty simple :P

Sounds pretty slow...
Wouldn't it even be faster to write the string to a file and use fgets? That's pretty 'dirty' and I guess fgets does nothing else than doing the char-by-char-thing, but maybe not?

[Moonchild]

Sounds pretty slow...
Wouldn't it even be faster to write the string to a file and use fgets? That's pretty 'dirty' and I guess fgets does nothing else than doing the char-by-char-thing, but maybe not?

That would probably be the slowest way to do it. Just do something like

for(int i=0;i<strlen(str);i++)
{
  if(str[i]=='\n') //newline
}

It should be pretty fast.

[Raphael]

Sounds pretty slow...
Wouldn't it even be faster to write the string to a file and use fgets? That's pretty 'dirty' and I guess fgets does nothing else than doing the char-by-char-thing, but maybe not?

Writing to a file just to read again byte for byte? How should that ever be faster? And yes, fgets also only reads byte for byte, but from a file instead of from memory which is hundreds times slower, plus the additional writing which is even more slower.

for(int i=0;i<strlen(str);i++)
{
if(str[i]=='\n') //newline
}

That's somewhat how I imagined and it's the fastest way possible by any means. Do it like that.

[hallo007]

how do you do this:

let the program while in a loop return to the beginning of that loop
for exemple

    for(leerlingen=1; leerlingen<=21;) {
    
    cout<<"geef de score van leerling "<<leerlingen;
    cin>>punten[leerlingen];
    cout<<"\n";
    leerlingen +1;
   /************************************************
  *********************return now to the beginning*******/
    if (leerlingen=21) {
                      break;
                      }
                      }
}

you see , so it stay asking it ,untiln the variabel "leerlingen = 21" ofcoures;)

[SodR]

how do you do this:

let the program while in a loop return to the beginning of that loop
for exemple

you see , so it stay asking it ,untiln the variabel "leerlingen = 21" ofcoures;)

I don't really understand what you mean but here is an example of a while loop;

int x = 0;

// Endless loop if we don't break it using 'break;'
while(1) {

x++;

// If the integer x = 21 we'll break the loop
if(x==21) {
break;
}

}

// This code will only run when we have broke out of the loop

I hope it'll help

[hallo007]

no i dont help , lol
i understand what a loop is
but i wanne repeat the loop
so if the user put a value for number 1 , it ask a value for number 2 , i have that , the only thing i need to do is that it's start again from the beginning
for exempla

what's the value of number 1? 4
/*now it repeats and x is one more so it prints number 2 and save the value to arrey 2*/
what's the value of number 2? 7

understand?? srry , i know my english is bad

[Moonchild]

Why don't you do it like this

int a = 0;
while(1)
{
  cin>>var;
  a++;
  if(a==21)
    break;
  /*else
  {
    //Not needed at all, just showing you the continue statement
    continue;
  }*/
}

Or in your case

//Reset to avoid any memory problems
leerlingen = 0;
while(1)
{
  cout<<"geef de score van leerling "<<leerlingen;
  cin>>punten[leerlingen];
  cout<<"\n";
  leerlingen++;
  if (leerlingen==21) 
     break;
}

[FreePlay]

You could also put the loop inside another loop. Then, in the inside loop, make it check which run through the outside loop it is. Example:

for(x=0;x<5;x++)
{
   for(y=0;y<5;y++)
   {
      switch(x)
      {
      case 0:
         // something to do the first time through
         break;
      case 1:
         // something to do the second time through
         break;
      case 2:
         // something to do the third time through
         break;
      case 3:
         // something to do the fourth time through
         break;
      case 4:
         // something to do the fifth time through
         break;
      }
   }
}

Or...

for(x=0;x<5;x++)
{
   printf("What's the value of number %i?\n", x);
   //get input here... however you want to do it
}

[yaustar]

    for(leerlingen=1; leerlingen<=21;) {
    
    cout<<"geef de score van leerling "<<leerlingen;
    cin>>punten[leerlingen];
    cout<<"\n";
    leerlingen +1; // This line doesn't do anything to 'leerlingen'
   /************************************************
  *********************return now to the beginning*******/
    if (leerlingen=21) {
                      break;
                      }
                      }
}

Should be:

for( leerlingen = 0; leerlingen < 21; ++leerlingen ) {
    
    cout << "geef de score van leerling " << leerlingen;
    cin >> punten[leerlingen];
    cout << endl;
}

[FreePlay]

Heh. Looks like we all missed that he wasn't incrementing the value, except for you. Also:

for( leerlingen = 0; leerlingen < 21; ++leerlingen ) {
    
    cout << "geef de score van leerling " << leerlingen;
    cin >> punten[leerlingen];
    cout << endl;
}

will start storing values in the second array element.

for( leerlingen = 0; leerlingen < 21; ++leerlingen ) {
    
    cout << "geef de score van leerling " << leerlingen;
    cin >> punten[leerlingen-1];
    cout << endl;
}

will start from the first :)

[hallo007]

so when i put in the first value , the program do first the second loop (and repeats it 100times ) and then it crashes

[SodR]

Now I got a question for you guys:

I use this posted by Insomniac :) at psp-programming to generate a random number:

int getRandomNum(int lo, int hi)
{
SceKernelUtilsMt19937Context ctx;
u32 rand_val = sceKernelUtilsMt19937UInt(&ctx);
rand_val = lo + rand_val % hi;
return (int)rand_val;
}

Then I got an array that looks like this:

char color[5];
// Yellow
color[1] = 0;
// Green
color[2] = 29;
// Black
color[3] = 58;
// Red
color[4] = 87;
// Blue
color[5] = 116;

The numbers are offsets if anyone wonders.

Then I generate a random number between 1 and 5 like this:

	int RandomNum;
	RandomNum = getRandomNum(1,5);

Then I finaly want to use the random number and blit the image:

blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 222);
blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 247);

This compiles fine but when I try it on the psp it'll always generate the number 1 = the color/offset is always 0/yellow. I know I can do this fix it "the lazy way" using alot of if statements but I got [f]Alot[/f] of different images I'm bliting and it would take forever to write if statements to them all.

Any thoughts?

[yaustar]

for( leerlingen = 0; leerlingen < 21; ++leerlingen ) {
    
    cout << "geef de score van leerling " << leerlingen;
    cin >> punten[leerlingen-1];
    cout << endl;
}

will start from the first :)

Woah. That would start from -1 not 0. On the first loop, leerlingen = 0, it doesnt increment until the end of the first loop.

[Waterbottle]

You need to seed the randomizer.

Not sure how you do it with that way.

but you could do:

srand(time(NULL));
RandomNum = rand() % (5+1);

[SodR]

You need to seed the randomizer.

Not sure how you do it with that way.

but you could do:

srand(time(NULL));
RandomNum = rand() % (5+1);

Thanks It worked fine. I guess there is no "magic" way of making the integer change every time you use it? I.e.

// First a number is generated here:
blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 222);
// A new number is automaticly generated using the same integer
blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 247);

That would save me alot of time/space in the source.

[hallo007]

thnx , i m nearly there now i want to parts the punten[leerlingen]


but how do you adds them up if you dont know how many there gonna be????????

[Insert_Witty_Name]

Thanks It worked fine. I guess there is no "magic" way of making the integer change every time you use it?

If you read the full post mate it shows how to seed the Mersenne Twister etc.

http://www.psp-programming.com/dev-f...pic.php?t=1076

I found it a lot more random than using the rand() stuff.
-= Double Post =-

// First a number is generated here:
blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 222);
// A new number is automaticly generated using the same integer
blitAlphaImageToScreen(color[RandomNum],0,29,25, hex_image, 132, 247);

I think you could do something like this...:

blitAlphaImageToScreen(color[GetRandomNum(0, 10)],0,29,25, hex_image, 132, 222);