C/C++ Programming Help Thread

Zitat:

---

Zitat von Mirzab14

Correct.

Ill try taking out credits.

Edit: Yep, the problem is with credits. I took it out and app works perfectly now. What would I have to do to get credits to work without those errors?

---

Many things, but I'd go by changing the credits file to this:

Code:

---

#include <pspkernel.h>
#include <pspdisplay.h>
#include <pspctrl.h>
#include <stdio.h> //ADDITION FOR SPRINTF
#include "graphics.h"


#define RGB(r, g, b) ((r)|((g)<<8)|((b)<<16))

void Credits()
{
 SceCtrlData pad, lastpad;
 sceCtrlReadBufferPositive(&lastpad, 1);
 
 Image* CreditImage[1];
 CreditImage[0] = loadImage("./Images/Credits/Credits0.png");


 
 while(1) {
      sceCtrlReadBufferPositive(&pad, 1);
 
      if (pad.Buttons & PSP_CTRL_CROSS)
      {
                      break;
      }
     
    blitAlphaImageToScreen(0, 0 , 480, 272, CreditImage[0], 0, 0);

    sceDisplayWaitVblankStart();
    flipScreen();
    }
}

---

and you can also remove this include from the menu.c

As far as I'm aware, this include has been phased out from the SDK

I dont know how I can thank you enough. Game runs now, and credits display too, and it compiles without errors!

Once I get enough cash, I will donate to you!*

*=Forum cash, not real money

Thanks once again!

Zitat:

---

Zitat von Mirzab14

I dont know how I can thank you enough. Game runs now, and credits display too, and it compiles without errors!

Once I get enough cash, I will donate to you!*

*=Forum cash, not real money

Thanks once again!

---

Ohh man.. I thought you'd gimme a million dollars :)

You're welcome mate... my pleasure to help ya!

Just for reference, the correct way to include stat.h for PSP is:

Code:

---

#include <sys/stat.h>

---

Zitat:

---

Zitat von Insert_Witty_Name

Just for reference, the correct way to include stat.h for PSP is:

Code:

---

#include <sys/stat.h>

---

---

Yes, I found some refferences using it that way, but for the code he came up with, I saw no reason to still include it.

Actually, I didn't quite understand why use that. :)

hey alright i have another problem, ive been going good on this piece of code, but then i ran into a problem. i run this code using 200 as price, 30 as amount, and 50 as price, the result should equal 50, but it comes out as 200, wut went wrong..

int price= 0;
double result;
int amount;
int max;

result= pow(10, (-amount/max))* price ;

cout<<" The Price we found for your commodity is "<< result <<"."<<endl;


i have no idea what went wrong, i use a calculator, and i get 50, can anyone help. thanks

Zitat:

---

Zitat von Blackbelttcon

hey alright i have another problem, ive been going good on this piece of code, but then i ran into a problem. i run this code using 200 as price, 30 as amount, and 50 as price, the result should equal 50, but it comes out as 200, wut went wrong..

int price= 0;
double result;
int amount;
int max;

result= pow(10, (-amount/max))* price ;

cout<<" The Price we found for your commodity is "<< result <<"."<<endl;


i have no idea what went wrong, i use a calculator, and i get 50, can anyone help. thanks

---

Are you compiling it for the P.S.P or the P.C??
Gavin/ITDemo

pc sorry i didnt specify

What is max?

Bear in mind the datatypes you are using, dividing two ints together will give you a int. Therefore 30/20 = 1 not 1.5. 1/2 = 0 not 0.5. Your compiler should have given a ton of warnings of mis-matched types.

Zitat:

---

Zitat von Blackbelttcon

hey alright i have another problem, ive been going good on this piece of code, but then i ran into a problem. i run this code using 200 as price, 30 as amount, and 50 as price, the result should equal 50, but it comes out as 200, wut went wrong..

int price= 0;
double result;
int amount;
int max;

result= pow(10, (-amount/max))* price ;

cout<<" The Price we found for your commodity is "<< result <<"."<<endl;


i have no idea what went wrong, i use a calculator, and i get 50, can anyone help. thanks

---

For one you didn't give us a 'max', two you gave us two prices.

If you give us the real values of each variable we can see whether it is your equation or numbers creating the problem ;)
-= Double Post =-

Zitat:

---

Zitat von yaustar

What is max?

Bear in mind the datatypes you are using, dividing two ints together will give you a int. Therefore 30/20 = 1 not 1.5. 1/2 = 0 not 0.5. Your compiler should have given a ton of warnings of mis-matched types.

---

Actually on that note, using Dev-C++ with MinGW I didn't get any warnings (strange...), however I recieved 200 (when max = 50). When changing the data types to float:

Code:

---

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

using namespace std;

int main() {
    float result,
          price = 200.0f,
          amount = 30.0f,
          max = 50.0f;
   
    result = pow(10,(-amount/max)) * price;
   
    cout << "The Price we found for your commodity is " << result << "." << endl;
   
    system("PAUSE");
   
    return 0;
}

---

I recieve 50.2377. :neutral:

DevC++ has warnings off or the lowest settings as default. There should be a -Wall (max warning level) option in the compiler settings.

Well that answers that ;) Seems strange to have errors pop out at you very noticeably while warnings not so much a little flag or something on default settings :-\

Im in need of some help. Im trying to get this to work. I've blitted an image to the screen, but now I want it to block the charector from walking past it, which means the player will have to jump to get over/past the obstacle.

Spoiler for game.c code:

Code:

---

// REAL

#include <pspkernel.h>
#include <pspdisplay.h>
#include <pspctrl.h>
#include <psprtc.h>
#include "graphics.h"

#define RGB(r, g, b) ((r)|((g)<<8)|((b)<<16))

void Game()
{   
 SceCtrlData pad;

 int LinkJump = 0;

 int movespeed = 2;
 float maxspeed = 5;
 float xspeed, yspeed, friction;
 xspeed = yspeed = 0.0f;
 friction = .98f;

 int LinkHealth = 100;
 int LinkEnergy = 100;
 int Enemy = 100;

 int Counter = 0;

 int LinkPosY = 145;
 int LinkPosX = -37;
 int LinkFrameCounter = 44;

 int WidthPos = 50;
 Color White = RGB(255, 255, 255);

 int LinkSwing = 0;
 
 int d;
 int LinkLevel = 1;

 int onGround = 1;
 
 
 #define printf pspDebugScreenPrintf
 #define RGB(r, g, b) ((r)|((g)<<8)|((b)<<16))
 

 Image* Level1;
 Level1 = loadImage("./Images/Level1/1-1.png");
 
 char text[10];

 Image* Status[3];
 Status[0] = loadImage("./Images/Status/health.png");
 Status[1] = loadImage("./Images/Status/energy.png");
 Status[2] = loadImage("./Images/Status/infobox.png");
 Status[3] = loadImage("./Images/Enemy/enem0.png");

 Image* Link[119];
 
 Link[0] = loadImage("./Images/Link/LinkWalking1Backwards.png");
 Link[1] = Link[0];
 Link[2] = Link[0];
 Link[3] = Link[0];

 
 Link[4] = loadImage("./Images/Link/LinkWalking2Backwards.png");
 Link[5] = Link[4];
 Link[6] = Link[4];
 Link[7] = Link[4];

 
 Link[8] = loadImage("./Images/Link/LinkWalking3Backwards.png");
 Link[9] = Link[8];
 Link[10] = Link[8];
 Link[11] = Link[8];

 
 Link[12] = loadImage("./Images/Link/LinkWalking4Backwards.png");
 Link[13] = Link[12];
 Link[14] = Link[12];
 Link[15] = Link[12];
 
 
 Link[16] = loadImage("./Images/Link/LinkWalking5Backwards.png");
 Link[17] = Link[16];
 Link[18] = Link[16];
 Link[19] = Link[16];

 
 Link[20] = loadImage("./Images/Link/LinkWalking6Backwards.png");
 Link[21] = Link[20];
 Link[22] = Link[20];
 Link[23] = Link[20];

 
 Link[24] = loadImage("./Images/Link/LinkWalking5Backwards.png");
 Link[25] = Link[24];
 Link[26] = Link[24];
 Link[27] = Link[24];

 
 Link[28] = loadImage("./Images/Link/LinkWalking4Backwards.png");
 Link[29] = Link[28];
 Link[30] = Link[28];
 Link[31] = Link[28];

 
 Link[32] = loadImage("./Images/Link/LinkWalking3Backwards.png");
 Link[33] = Link[32];
 Link[34] = Link[32];
 Link[35] = Link[32];
 
 Link[36] = loadImage("./Images/Link/LinkWalking2Backwards.png");
 Link[37] = Link[36];
 Link[38] = Link[36];
 Link[39] = Link[36];
 
 Link[40] = loadImage("./Images/Link/LinkWalking1Backwards.png");
 Link[41] = Link[40];
 Link[42] = Link[40];
 Link[43] = Link[40];

 
 Link[44] = loadImage("./Images/Link/LinkWalking1.png");
 Link[45] = Link[44];
 Link[46] = Link[44];
 Link[47] = Link[44];

 Link[48] = loadImage("./Images/Link/LinkWalking2.png");
 Link[49] = Link[48];
 Link[50] = Link[48];
 Link[51] = Link[48];
 
 Link[52] = loadImage("./Images/Link/LinkWalking3.png");
 Link[53] = Link[52];
 Link[54] = Link[52];
 Link[55] = Link[52];


 Link[56] = loadImage("./Images/Link/LinkWalking4.png");
 Link[57] = Link[56];
 Link[58] = Link[56];
 Link[59] = Link[56];

 
 Link[60] = loadImage("./Images/Link/LinkWalking5.png");
 Link[61] = Link[60];
 Link[62] = Link[60];
 Link[63] = Link[60];

 
 Link[64] = loadImage("./Images/Link/LinkWalking6.png");
 Link[65] = Link[64];
 Link[66] = Link[64];
 Link[67] = Link[64];

 
 Link[68] = loadImage("./Images/Link/LinkWalking7.png");
 Link[69] = Link[68];
 Link[70] = Link[68];
 Link[71] = Link[68];

 
 Link[72] = Link[64];
 Link[73] = Link[72];
 Link[74] = Link[72];
 Link[75] = Link[72];
 
 
 Link[76] = Link[56];
 Link[77] = Link[76];
 Link[78] = Link[76];
 Link[79] = Link[76];
 
 
 Link[80] = Link[52];
 Link[81] = Link[80];
 Link[82] = Link[80];
 Link[83] = Link[80];
 
 Link[84] = Link[48];
 Link[85] = Link[84];
 Link[86] = Link[84];
 Link[87] = Link[84];
 
 Link[88] = loadImage("./Images/Link/LinkAttack-Side1.png");
 Link[89] = Link[88];
 Link[90] = Link[88];
 Link[91] = Link[88];
 
 Link[92] = loadImage("./Images/Link/LinkAttack-Side2.png");
 Link[93] = Link[92];
 Link[94] = Link[92];
 Link[95] = Link[92];
 
 Link[96] = loadImage("./Images/Link/LinkAttack-Side3.png");
 Link[97] = Link[96];
 Link[98] = Link[96];
 Link[99] = Link[96];
 Link[100] = Link[96];
 Link[101] = Link[96];
 Link[102] = Link[96];
 
 
 Link[103] = loadImage("./Images/Link/LinkAttack-Side4.png");
 Link[104] = Link[100];
 Link[105] = Link[100];
 Link[106] = Link[100];
 
 Link[107] = Link[96];
 Link[108] = Link[96];
 Link[109] = Link[96];
 Link[110] = Link[96];
 
 Link[111] = Link[92];
 Link[112] = Link[92];
 Link[113] = Link[92];
 Link[114] = Link[92];
 
 Link[115] = Link[88];
 Link[116] = Link[88];
 Link[117] = Link[88];
 Link[118] = Link[88];

 
 while(1)
 {
 sceCtrlReadBufferPositive(&pad, 1);

// 40 is normal
 if (pad.Buttons & PSP_CTRL_LEFT)
      {
      LinkPosX -= 2;
      if (LinkFrameCounter >= 44)
      {
                          LinkFrameCounter = 43;
                          LinkFrameCounter--;
      }
      else
      {
      LinkFrameCounter--;
      }
      if (LinkFrameCounter < 4)
      {
                          LinkFrameCounter = 39;
      }
      }



 if (pad.Buttons & PSP_CTRL_RIGHT)
      {
      LinkPosX += 2;
      if (LinkFrameCounter <= 43)
      {
                          LinkFrameCounter = 48;
                          LinkFrameCounter++;
      }
      else
      {
      LinkFrameCounter++;
      }   
      if (LinkFrameCounter > 87)
      {
                          LinkFrameCounter = 48;
      }
      }
     
 // NOTE! For LinkSwing:
 // 0 is off
 // 1 is on
 
 // It cannot go past 2.
     
 /*
 How LinkSwing Works:
    When LinkSwing is on 1, LinkFrameCounter goes up.
    Once he is done with his sword animation, LinkFrameCounter resets,
    and LinkSwing turns off. (0).
 */   
 if (pad.Buttons & PSP_CTRL_CROSS)
      {             
                LinkSwing += 1;
                if (LinkSwing >= 2)
                {
                              LinkSwing = 1;
                } 
      }     
     
 if (LinkSwing == 1)
    {
              if (LinkFrameCounter <= 87)
              {
                                    LinkFrameCounter = 88;
              }
              LinkFrameCounter++;
              if (LinkFrameCounter > 118)
              {
                                    LinkFrameCounter = 88;
                                    LinkSwing = 0;
              }
    }             
   
     
              if (pad.Buttons & PSP_CTRL_CIRCLE)
              {               
                                yspeed += (float)movespeed;                             
              }

              if(yspeed > maxspeed){yspeed = maxspeed;}
              if(yspeed < -maxspeed){yspeed = -maxspeed;}
              yspeed -= friction;

LinkPosY -= yspeed;

             
              if (LinkPosY < -50)
              {
                            LinkPosY = -50;
              }
             
              if (LinkPosY > 145)
              {
                            LinkPosY = 145;
              }
 
 if (LinkPosX < -37)
    {
      LinkPosX = -37;
    }
 
 if (LinkPosX > 420)
    {
    LinkPosX = 420;
    }
 
 blitAlphaImageToScreen(0, 0, 480, 272, Level1, 0, 0);
 
 /*blitAlphaImageToScreen(0 , 0, 200, 100, Status[2], 0, 0);
 
 blitAlphaImageToScreen(0 , 0, LinkHealth, 30, Status[0], 0, 21);
 blitAlphaImageToScreen(0 , 0, LinkEnergy, 30, Status[1], 0, 43);
 */
 blitAlphaImageToScreen(0,0,Enemy, 200, Status[3], 0,210);
 /*
 sprintf(text, "%d", LinkLevel);
 printTextScreen(30, 75, text, RGB(255, 255, 255));
 */
 blitAlphaImageToScreen(0, 0, 100, 100, Link[LinkFrameCounter], LinkPosX, LinkPosY);
 sceDisplayWaitVblankStart();
 flipScreen();
 
 }
 freeImage(Link[0]);
 freeImage(Link[4]);
 freeImage(Link[8]);
 freeImage(Link[12]);
 freeImage(Link[16]);
 freeImage(Link[20]);
 freeImage(Link[24]);
 freeImage(Link[28]);
 freeImage(Link[32]);
 freeImage(Link[36]);
 freeImage(Link[40]);
 freeImage(Link[44]);
 freeImage(Link[48]);
 freeImage(Link[52]);
 freeImage(Link[56]);
 freeImage(Link[60]);
 freeImage(Link[64]);
 freeImage(Link[68]);
 freeImage(Link[72]);
 freeImage(Link[76]);
 freeImage(Link[80]);   
 freeImage(Link[84]);
 }

---

Enemy, which is Status[3] loads the "enemy" picture, which is a little obstacle. I hope I made sense, thanks in advance.

Image* Status[3];

You only declare 3 images, yet you have four.

Zitat:

---

Zitat von Judas

Image* Status[3];

You only declare 3 images, yet you have four.

---

That the same as
Image* Status0
Image* Status1
Image* Status2
Image* Status3


Which is declaring 4 images.

Zitat:

---

Zitat von TMNT

That the same as
Image* Status0
Image* Status1
Image* Status2
Image* Status3
Which is declaring 4 images.

---

No, it is allocating 3 image pointers hence the '3'. So using your example, you can access, 0, 1, and 2.

Zitat:

---

Zitat von SG57

For one you didn't give us a 'max', two you gave us two prices.

If you give us the real values of each variable we can see whether it is your equation or numbers creating the problem ;)
-= Double Post =-
Actually on that note, using Dev-C++ with MinGW I didn't get any warnings (strange...), however I recieved 200 (when max = 50). When changing the data types to float:

Code:

---

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

using namespace std;

int main() {
    float result,
          price = 200.0f,
          amount = 30.0f,
          max = 50.0f;
   
    result = pow(10,(-amount/max)) * price;
   
    cout << "The Price we found for your commodity is " << result << "." << endl;
   
    system("PAUSE");
   
    return 0;
}

---

I recieve 50.2377. :neutral:

---

hey thanks that was the problem, sorry i messed up, i ment 50 and the max, not the price.


yaustar, i set me warnings now to the highest, thanks
-= Double Post =-
hey sorry i got one problem solved, and another comes up.

im checking a percent for cloaking ( its a game ) and im using 8 as size, 12 as skill, and 4 for area, the answer is supost to be .73 for success and .23 for failure. im getting some outragous number instead. i tryed the float, but its still not working.

Code:

---


float size;
float skill;
float area;
float b;
float c;
float fail;
float success;


b=(size*area) ;
c=(skill*10) ;
fail=b/c;
success= 1-fail ;


 cout<<" Your chance for failure is "<< fail <<"."<<endl;
 cout<<" Your chance for success is "<<  success <<"."<<endl;

---

in between variables and the math, it asks for the input for each of thoughs, it would be to long to put into one post, so i just left it out, there was nothing important, just cout<<"this"<<endl; at thats all.

thanks

great were complaining about his code....

Spoiler for what he was asking for:

but now I want it to block the charector from walking past it, which means the player will have to jump to get over/past the obstacle.

rather than arguing about his code and he hasn't said theirs a prob compiling so lets forget any need for complaint, go ahead and continue arguing if you have to, i'll answer his question:

@Mirzab14

alright so basically you want to have the character stop moving when he collides with an object

heres some peusedo code on how to do it:

Code:

---

int Collision(int x,int y,int xsize, int ysize){
if((LinkPosX>=x && LinkPosX<=x+xsize) || (LinkPosx+linkSizeX>=x && LinkPosx+LinkSizeX<=x+xsize)){
if((LinkPosY>=y && LinkPosY<=y+ysize) || (LinkPosy+linkSizeY>=y && LinkPosy+LinkSizey<=y+ysize)){
return 1
}
}
return 0;
}

---

very simple basically supplying the x, y, and size of the object well show if collision, i'll let you change the code around how you need it=-) hope i helped

also your code is EXTREMLY redudant

Zitat:

---

Zitat von slicer4ever

great were complaining about his code....

Spoiler for what he was asking for:

but now I want it to block the charector from walking past it, which means the player will have to jump to get over/past the obstacle.

rather than arguing about his code and he hasn't said theirs a prob compiling so lets forget any need for complaint, go ahead and continue arguing if you have to, i'll answer his question:

---

It will compile but the behaviour is undefined the code is trying to write beyond the range of the array and he/she should correct it. I really am surprised it didn't crash the application in the first place.

very agreed, i never would think of writing a game in suck a way however if it's not crashing at the very least we could answer his original question rather than ignoring it

Done: http://www.gamedev.net/community/for...opic_id=441670

hey yaustar any answer to mine

Zitat:

---

Zitat von Blackbelttcon

hey yaustar any answer to mine

---

Code:

---

#include <iostream>
int main()
{
    using namespace std;
    float size = 8.f;
    float skill = 12.f;
    float area = 4.f;
    float b = ( size * area );
    float c = ( skill * 10.f );
    float fail = b / c;
    float success = 1.f - fail;

    cout << " Your chance for failure is " << fail << "." << endl;
    cout << " Your chance for success is " << success << "." << endl;
}

---

Gives:

Code:

---

Your chance for failure is 0.266667.
 Your chance for success is 0.733333.

---

Works fine.

really im getting 6.34523342e34 and then that negative again,

the problem is with the floats, the people have to input the numbers, so i cant use:

Code:

---

   
float size = 8.f;
    float skill = 12.f;
    float area = 4.f;

---

is there a way to make the size.f or something like that.
-= Double Post =-
hey i got it. i set all the floats to size = 1.0f
then it worked just fine.

thanks for the guidence
-= Double Post =-
hey what should i use if i wanted to store a bunch of words like a person inputs a message, and it is saved as a variable. im using string, but its only picking up the first word.

Zitat:

---

Zitat von Blackbelttcon

really im getting 6.34523342e34 and then that negative again,

the problem is with the floats, the people have to input the numbers, so i cant use:

Code:

---

   
float size = 8.f;
    float skill = 12.f;
    float area = 4.f;

---

is there a way to make the size.f or something like that.
-= Double Post =-
hey i got it. i set all the floats to size = 1.0f
then it worked just fine.

thanks for the guidence

---

That doesn't like the right 'fix'. Post the whole code, I have a feeling there is something fundamentally wrong with it.
-= Double Post =-

Zitat:

---

hey what should i use if i wanted to store a bunch of words like a person inputs a message, and it is saved as a variable. im using string, but its only picking up the first word.

---

Google getline function.
http://www.cplusplus.com/reference/string/getline.html

here it is,

Code:

---

#include <stdio.h>
#include <windows.h>
#include <iostream.h>
#include <stdio.h>
#include <stdlib.h>
#include <math.h>
#include <string>
#include <fstream>
#include <fstream.h>
#include <cstdlib>
#include <cstdio>
#include <iostream>

using namespace std;


int main()
{

int choice;


cout<<"Welcome to the Pardus EasyNotes."<<endl;
Sleep (500);

start:

cout<<""<<endl;
cout<<"What would you like to check."<<endl;
Sleep (500);
cout<<"1. Notes"<<endl;
Sleep (500);
cout<<"2. Commodity Prices"<<endl;
Sleep (500);
cout<<"3. Cloaking Chance"<<endl;
Sleep (500);
cout<<"4. Movement Costs"<<endl;
Sleep (1000);
cout<<""<<endl;
cout<<"Please enter the number below."<<endl;
cin>>choice;

 if (choice==1)
 {                          //starts the notes

      using namespace std;
 string  name;
 string message= "";


 cout<<" You have chosen to save some notes for further use."<<endl;
 Sleep (500);

 cout<<" Please wait while I access your computer system..."<<endl;

 Sleep (2000);

 cout<<" Access Granted"<<endl;
 Sleep (500);
 cout<<" Please Enter the Name of the System Your Notes are about."<<endl;
 cin>> name;
 Sleep (500);
 cout<<" Thank You, you can now enter your message below."<<endl;
 getline (cin,message);
  Sleep (1000);

 cout<<" Here is your message:"<<endl;
 Sleep (500);
 cout<<"        "<<name<<""<<endl;
 Sleep (500);
 cout<< message <<endl;

 system("PAUSE");











 }                          //ends the notes






if (choice==2)
{          // starts the commodity pricing


 float price= 0;
 float result;
 float amount;
 float max;
string y= "y";
string leave;

cout<<" You have chosen to check the commodity prices."<<endl;
Sleep (500);
cout<<" Please wait while we connect to all the starsystems in the Pardus Universe..."<<endl;
Sleep (2000);

cout<<" Now Connected"<<endl;
Sleep (500);


cout<<" Please enter the Price of the commodity at 0"<<endl;
cin>> price;
 Sleep (500);


cout<<" Please enter the amount on the planet or starbase."<<endl;
cin>> amount;
Sleep (500);

cout<<" And last, please enter the maximum amount allowed."<<endl;
cin>> max;

Sleep (500);

cout<<" Please wait while i access all the StarSystem's Databases..."<<endl;
Sleep (2000);


result= (pow(10, (-amount/max)))* price;

cout<<" The Price we found for your commodity is "<< result <<"."<<endl;
Sleep (500);

cout<<" Would you like to check another?"<<endl;
Sleep (250);

cout<<" y or n" <<endl;
cin>>leave;

if (leave==y)
{  goto start;

}

else
{
cout<<"Please Press enter to exit."<<endl;
getchar();
}


}              //ends the commodity pricing


if (choice==3)
{
                            //starts cloaking chance
    float size = 1.0f;
    float skill = 1.0f;
    float area = 1.0f;
  int a;
  string y= "y";
  string leave;


cout<< " You have chosen to check your ships cloaking chance."<<endl;
Sleep (500);

cout<<" Now accessing Earth's Mainframe for all present ships..."<<endl;

Sleep (1000);

cout<<" Access Granted"<<endl;

Sleep (500);


cout<<" Please enter the size of your ship."<<endl;
cin>>size;

Sleep (500);
cout<<" Please enter your Skill at Cloaking."<<endl;
cin>>skill;


areas:
Sleep (500);

cout<<" Please enter the type of area you are in:"<<endl;
cout<<"                                        1. Energy"<<endl;
cout<<"                                        2. Viral Cloud"<<endl;
cout<<"                                        3. Nebula"<<endl;
cout<<"                                        4. Astroids"<<endl;
cout<<"                                        5. Space"<<endl;
cin>>a;

if (a==1)
{    float area= 1.0f;}

else if (a==2)
{  float area= 2.0f; }

else if (a==3)
{  float area= 4.0f;  }

else if (a==4)
{      float area= 8.0f;}

else if (a==5)
{      float area= 12.0f;}

else

{
cout<<" Please pick a valid answer."<<endl;
goto areas;
}


cout<<" Please wait while we check the systems for cloaking chances."<<endl;
Sleep (1000);


using namespace std;

    float fail =( size * area )/( skill * 10.f ) ;
    float success = 1.f - fail;



 cout<<" Your chance for failure is "<< fail <<"."<<endl;
 cout<<" Your chance for success is "<<  success <<"."<<endl;

cout<<" Would you like to check another?"<<endl;
Sleep (250);

cout<<" y or n" <<endl;
cin>>leave;

if (leave==y)
{  goto start;

}

else
{
cout<<"Please Press enter to exit."<<endl;
getchar();
}


}                              //ends cloaking chance




  return 0;
}

---

-= Double Post =-
hey im having some trouble with that getline stuff.
its in the code up above, but when ever i run that section, it always just skips right over it. so it will be like " your message it:

"
and thats it.

Why have you included the same header several times and also both the C and C++ version of the same header?

Sleep() is not cross platform and neither are system(). Remove them.

Please, PLEASE, indent your code to one of the standards. It is a nightmare to read on forums. I have to keep parsing it via one of my text editors to make any sense of it.

And why are you using goto? There is never a situation where a goto is needed in C or C++. It promotes spaghetti code which makes it harder to maintain and the compiler has a much tougher time trying to optimise the code.

http://imgs.xkcd.com/comics/goto.png

The 'using' keyword obeys scope. Once you put it in file scope, you now longer need to keep redeclaring it.

Use named constants instead of magic numbers. It makes the code much easier to maintain and read.

In this code, you declare a new variable that only exists in the scope of if statements when you meant to change an existing variable outside the scope.

Code:

---

if (a==1)
{    float area= 1.0f;}

else if (a==2)
{  float area= 2.0f; }

else if (a==3)
{  float area= 4.0f;  }

else if (a==4)
{      float area= 8.0f;}

else if (a==5)
{      float area= 12.0f;}

---

The code below is how I believe it should be written with room for further structure improvement. If there is anything you don't understand, please ask.

Code:

---

#include <cmath>
#include <string>
#include <fstream>
#include <cstdlib>
#include <cstdio>
#include <iostream>

using namespace std;

namespace MainOptions
{
        enum EMainOptions
        {
                NOTES = 1,
                COMMODITY_PRICES,
                CLOAKING_PRICES,
                MOVEMENT_COSTS,
                EXIT
        };
}

namespace AreaOptions
{
        enum EAreaOptions
        {
                ENERGY = 1,
                VIRAL_CLOUD,
                NEBULA,
                ASTEROIDS,
                SPACE
        };
}

int main()
{
        int choice = 0;

        do
        {
                cout<<"\nWelcome to the Pardus EasyNotes."<<endl;
                cout<<"\nWhat would you like to check."<<endl;
                cout<<"1. Notes"<<endl;
                cout<<"2. Commodity Prices"<<endl;
                cout<<"3. Cloaking Chance"<<endl;
                cout<<"4. Movement Costs"<<endl;
                cout<<"5. Exit"<<endl;
                cout<<"\nPlease enter the number below."<<endl;
                cin>>choice;
       
                switch(choice)
                {
                        case MainOptions::NOTES:
                        {
                                string name;
                                string message;

                                cout<<" You have chosen to save some notes for further use."<<endl;
                                cout<<" Please wait while I access your computer system..."<<endl;
                                cout<<" Access Granted"<<endl;
                                cout<<" Please Enter the Name of the System Your Notes are about."<<endl;
                                cout<<" Thank You, you can now enter your message below."<<endl;
                                // Clear the whitespace/end line from the previous cin in the buffer.
                                cin.ignore();
                                getline (cin,message);
                                cout<<" Here is your message:"<<endl;
                                cout<<"        "<<name<<""<<endl;
                                cout<< message <<endl;
                        } break;
                       
                        case MainOptions::COMMODITY_PRICES:
                        {
                                float price = 0.0f;
                                float result = 0.0f;
                                float amount = 0.0f;
                                float max = 0.0f;

                                cout<<" You have chosen to check the commodity prices."<<endl;
                                cout<<" Please wait while we connect to all the starsystems in the Pardus Universe..."<<endl;
                                cout<<" Now Connected"<<endl;
                                cout<<" Please enter the Price of the commodity at 0"<<endl;
                                cin>> price;
                                cout<<" Please enter the amount on the planet or starbase."<<endl;
                                cin>> amount;
                                cout<<" And last, please enter the maximum amount allowed."<<endl;
                                cin>> max;
                                cout<<" Please wait while i access all the StarSystem's Databases..."<<endl;
                                result = (pow(10.f, (-amount/max)))* price;
                                cout<<" The Price we found for your commodity is "<< result <<"."<<endl;
                        } break;
                       
                        case MainOptions::CLOAKING_PRICES:
                        {
                                float size = 0.0f;
                                float skill = 0.0f;
                                float area = 0.0f;

                                cout<<" You have chosen to check your ships cloaking chance."<<endl;
                                cout<<" Now accessing Earth's Mainframe for all present ships..."<<endl;
                                cout<<" Access Granted"<<endl;
                                cout<<" Please enter the size of your ship."<<endl;
                                cin>>size;
                                cout<<" Please enter your Skill at Cloaking."<<endl;
                                cin>>skill;
                                cout<<" Please enter the type of area you are in:"<<endl;
                                cout<<"                                        1. Energy"<<endl;
                                cout<<"                                        2. Viral Cloud"<<endl;
                                cout<<"                                        3. Nebula"<<endl;
                                cout<<"                                        4. Astroids"<<endl;
                                cout<<"                                        5. Space"<<endl;
                               
                                int areaType = 0;
                                do
                                {
                                        cin>>areaType;

                                        switch( areaType )
                                        {
                                                case AreaOptions::ENERGY:
                                                {
                                                        area = 1.0f;
                                                } break;
                                               
                                                case AreaOptions::VIRAL_CLOUD:
                                                {
                                                        area = 2.0f;
                                                } break;
                                               
                                                case AreaOptions::NEBULA:
                                                {
                                                        area = 4.0f;
                                                } break;
                                               
                                                case AreaOptions::ASTEROIDS:
                                                {
                                                        area = 8.0f;
                                                } break;
                                               
                                                case AreaOptions::SPACE:
                                                {
                                                        area = 12.0f;
                                                } break;
                                               
                                                default:
                                                {
                                                        cout << "Invalid Area" << endl;
                                                } break;
                                        }
                                } while( areaType < AreaOptions::ENERGY || areaType > AreaOptions::SPACE );

                                cout<<" Please wait while we check the systems for cloaking chances."<<endl;

                                float fail =( size * area )/( skill * 10.f ) ;
                                float success = 1.f - fail;

                                cout<<" Your chance for failure is "<< fail <<"."<<endl;
                                cout<<" Your chance for success is "<<  success <<"."<<endl;
                        } break;
                       
                        case MainOptions::MOVEMENT_COSTS:
                        {
                                cout << "Not implemented yet" << endl;
                        } break;
                       
                        case MainOptions::EXIT:
                        {
                                cout << "Goodbye" << endl;
                        } break;
                       
                        default:
                        {
                                cout << "Invalid option" << endl;
                        } break;
                }
        } while( choice != MainOptions::EXIT );
}

---

That was some very ugly code... AHHHHHHHHHH! :down:

hey sorry about the messy code, ive been coding for about a couple months now. I just have a quick question about the case main option, is that the only thing it can declare, or can it declare other stuff to.

Zitat:

---

Zitat von Blackbelttcon

I just have a quick question about the case main option, is that the only thing it can declare, or can it declare other stuff to.

---

What do you mean?

can i make it case shipoptions, and have that be the options of the ship.

i also have a quick question about saving and opening files.


i am using

Code:

---


        {
            ifstream OpenFile("message.conn");

            if( !OpenFile )
            {
                cerr << "Error opening file" << endl;
                return -1;
            }
                      // open a message if there is one
            string loadmessage= "";

            while ( OpenFile >> loadmessage)
            {
                message += loadmessage;
            }

        }

---

to load the message, and

Code:

---

          {
 ofstream SaveFile("message.conn");
        SaveFile <<message <<endl;
          SaveFile.close();

    }

---

to save the message, the problem is that when it comes out in

cout<<message<<endl;

there are no space, what is the cause of that.

If I did this instead:

Code:

---

namespace MainOptions
{
        enum EMainOptions
        {
                NOTES = 1,
                COMMODITY_PRICES = 2,
                CLOAKING_PRICES = 3,
                MOVEMENT_COSTS = 4,
                EXIT = 5
        };
}

---

Does it make it clearer what you can do and how to do it? If not, read up on enums and swtich statements. Or do you mean you just want to change the name like this?

Code:

---

namespace ShipOptions
{
        enum EShipOptions
        {
                NOTES = 1,
                COMMODITY_PRICES,
                CLOAKING_PRICES,
                MOVEMENT_COSTS,
                EXIT
        };
}

---

About the file opening, when you are reading the file here:

Code:

---

while ( OpenFile >> loadmessage)
{
    message += loadmessage;
}

---

Any whitespace is a delimiter so loadmessage only contains all text from the current position in the file to the nearest whitespace.

E.g If the file contains:

Code:

---

the dog sat on the mat

---

The code above will read it as (each line represents an iteration of the loop):

Code:

---

the
dog
sat
on
the
mat

---

Which when put together with +=:

Code:

---

thedogsatonthemat

---

Again, do some reading around filestreams.

yaustar - "I could restructure the whole program flow... or use one goto statement instead... Ah hell with good practice, how bad could it be?" :ROFL: I think the raptor guy stands for obsolete right?

I never learned goto statements in C/++, rather when writing batch files for bypassing some restrictions at school.

blackbelttcon - Magic numbers...

http://vcard.homer-simpson.de/images/homer/pic18.jpg

When I got smacked about using magic numbers (I believe it was in all capitals with 8 exclamation marks...) I didn't see why until after ^^. You'd be amazed how much better it is to either define them as constants in a global, common header file (not sure the best means of doing this, but including common.h in any source file needing the values has worked fine for me, not too sure if thats the best way to do it) or have a configuration file (this allows quick changes to oyur program without the need of recompilation) - this could save you the trouble of asking what to type in your program - just have them edit the config file and run.

And by the way, don't take yaustar's smack-on-the-hand-with-a-yard-stick nun-like ways the wrong way, we all have scars and stories to tell thanks to yaustar, so don't feel to bad ;)

alright, thanks, both u guys

Zitat:

---

Zitat von SG57

yaustar - "I could restructure the whole program flow... or use one goto statement instead... Ah hell with good practice, how bad could it be?" :ROFL: I think the raptor guy stands for obsolete right?

---

Nah, the author just has a thing for raptors. Never was sure why :/
http://imgs.xkcd.com/comics/substitute.png

Can anyone tell me whats wrong with this code? I made it last night, but for some reason I always get 0x80002001 when trying to load it:

Code:

---

//#include <kubridge.h>
#include <pspkernel.h>
#include <pspdebug.h>
#include <pspvshbridge.h>
#include <stdio.h>
#include <pspmodulemgr.h>
#include <pspsdk.h>

PSP_MODULE_INFO("LoadVSH", 0x1000, 1, 1);
PSP_MAIN_THREAD_ATTR(THREAD_ATTR_USER);
PSP_HEAP_SIZE_KB(20480);

char module[][36] =
{
    "flash0:/kd/sysmem.prx",
    "flash0:/kd/loadcore.prx",
    "flash0:/kd/exceptionman.prx",
    "flash0:/kd/interruptman.prx",
    "flash0:/kd/threadman.prx",
    "flash0:/kd/dmacman.prx",
    "flash0:/kd/systimer.prx",
    "flash0:/kd/iofilemgr.prx",
    "flash0:/kd/stdio.prx",
    "flash0:/kd/memlmd.prx",
    "flash0:/kd/modulemgr.prx",
    "flash0:/kd/systemctrl.prx",
    "flash0:/kd/init.prx",
    "flash0:/kd/loadexec.prx",
    "flash0:/kd/sysreg.prx",
    "flash0:/kd/gpio.prx",
    "flash0:/kd/pwm.prx",
    "flash0:/kd/i2c.prx",
    "flash0:/kd/dmacplus.prx",
    "flash0:/kd/lcdc.prx",
    "flash0:/kd/emc_sm.prx",
    "flash0:/kd/emc_ddr.prx",
    "flash0:/kd/ge.prx",
    "flash0:/kd/idstorage.prx",
    "flash0:/kd/syscon.prx",
    "flash0:/kd/rtc.prx",
    "flash0:/kd/lfatfs.prx",
    "flash0:/kd/clockgen.prx",
    "flash0:/kd/codec.prx",
    "flash0:/kd/audio.prx",
    "flash0:/kd/display.prx",
    "flash0:/kd/ctrl.prx",
    "flash0:/kd/led.prx",
    "flash0:/kd/hpremote.prx",
    "flash0:/kd/power.prx",
    "flash0:/kd/openpsid.prx",
    "flash0:/kd/usb.prx",
    "flash0:/kd/mediaman.prx",
    "flash0:/kd/ata.prx",
    "flash0:/kd/umdman.prx",
    "flash0:/kd/umd9660.prx",
    "flash0:/kd/isofs.prx",
    "flash0:/kd/mscm.prx",
    "flash0:/kd/msstor.prx",
    "flash0:/kd/fatmsmod.prx",
    "flash0:/kd/wlan.prx",
    "flash0:/kd/wlanfirm_magpie.prx",
    "flash0:/kd/registry.prx",
    "flash0:/kd/mgr.prx",
    "flash0:/kd/msaudio.prx",
    "flash0:/kd/mgvideo.prx",
    "flash0:/kd/chkreg.prx",
    "flash0:/kd/mesg_led.prx",
    "flash0:/kd/semawm.prx",
    "flash0:/kd/npdrm.prx",
    "flash0:/kd/mediasync.prx",
    "flash0:/kd/me_wrapper.prx",
    "flash0:/kd/vaudio.prx",
    "flash0:/kd/utility.prx",
    "flash0:/kd/impose.prx",
    "flash0:/kd/avcodec.prx",
    "flash0:/kd/vshbridge.prx",
    "flash0:/kd/vshctrl.prx",
    "flash0:/kd/chnnlsv.prx",
    "flash0:/kd/usersystemlib.prx",
    "flash0:/vsh/module/heaparea1.prx",
    "flash0:/kd/libatrac3plus.prx",
    "flash0:/vsh/module/paf.prx",
    "flash0:/vsh/module/common_gui.prx",
    "flash0:/vsh/module/common_util.prx"
};

int main()
{
  pspDebugScreenInit();
  pspDebugScreenPrintf("Debug output initiated\n");
  int loop, status, vshmain;
 
  for(loop = 0; loop < 69; loop++)
  {
            status = pspSdkLoadStartModule(module[loop], PSP_MEMORY_PARTITION_USER);
            if(status < 0) status = pspSdkLoadStartModule(module[loop], PSP_MEMORY_PARTITION_KERNEL);
            if(status < 0)
            {
                      pspDebugScreenPrintf("0x%08x Failed to load module : %s\n", status, module[loop]);
                      printf("0x%08x Failed to load module %s\n", status, module[loop]);
            }
            if(status >=0) pspDebugScreenPrintf("0x%08x Loaded module        : %s\n", status, module[loop]);
  }
 
  vshmain = vshKernelLoadModuleVSH("flash0:/vsh/module/project4.prx", 0, NULL);
  if(vshmain < 0)
  {
              pspDebugScreenPrintf("0x%08x Failed to load module : vshmain.prx\n", status);
  }
  vshmain = sceKernelStartModule(vshmain, 0, 0, NULL, NULL);
  if(vshmain < 0)
  {
              pspDebugScreenPrintf("0x%08x Failed to start module: vshmain.prx\n", status);
  }

  return 0;
}

---

Thanks for any help.
-Aura

Just a quick thing i finally compiled my code but then the pspDebugkbInit didn't work so is there any way to use the built in psp text input thing.
Also @ Aura now i'm not sure if i am right but in you code did you mean to have the comment thing (//) in-front of the #include <kubridge.h>. If you did then i'm not sure if you didn't then maybe remove the // in-front of it.
Thanks,
Gavin/ITDemo :)

Zitat:

---

Zitat von Auraomega

Can anyone tell me whats wrong with this code? I made it last night, but for some reason I always get 0x80002001 when trying to load it:

Thanks for any help.
-Aura

---

You cannot reload modules that are already loaded.

Yeah, but surely that wouldn't stop the loading module from loading in the first place? The reason I made the output is so I knew which modules would fail to load to remove them later.

-Aura

Code:

---

layer* recursiveClean(layer *layer)

---

Name conflicts. You have a variable named exactly the same as the datatype (take a closer look at the function arguments).

Code:

---

layer* recursiveClean(layer *aLayer)
{
        if(layer)
        {
                layer *temp = recursiveClean(aLayer->next);
                if(temp)
                {
                        aLayer->next = temp->next;
                        temp->destructor();
                        free(temp);               
                }

                if(aLayer->isdone)
                        return layer;
        }
        return 0;
}

---