Urgent, please! I want to create a startscreen for my pong game. What am I doing wrong ?

Answers

• TfGuy44

  January 2017 Answer ✓

  You aren't formatting your code properly for the forums. Edit your post, select your code, and press Ctrl + o.
• TfGuy44

  January 2017 Answer ✓

  You have two variables called startscreen. One is an image. The other is a int. You probably only want the image one.
• Chrisir

  January 2017 edited January 2017 Answer ✓

  Hello,

  I formatted the code a little and made some corrections :

  • e.g. it's loadImage with a small start letter

  • and image, not img

  • I inserted 20 in a few points because that is the radius of the ball (it looks better that way since it is not leaving the screen or overlap / reaching over the border/ paddle).

  • Text align:

  text("hit: " + hit, 7, 40); is now with textAlign(LEFT); which looks better

  • Also you had one closing } here:

    if (stage == 2) { 
    
            background(100, 200, 100);
    
    } // I am wrong!!!
    

  which isolated the whole part after it inside draw() from this.

  Instead we want the } at the end of draw().

  • Remember you can use ctrl-t in processing (not in the forum) to get auto indent. Good.

  Best, Chrisir ;-)

  PImage startscreen; 
  // int startscreen; 
  
  int stage; // stage of the program  
  
  int base=20; 
  int x, y; 
  int hit = 0; 
  int miss = 0; 
  
  int changeX=-5; 
  int changeY=-5;
  
  int gameOver=0;
  
  void setup() { 
    stage = 1; 
    startscreen = loadImage("2teniscourt.jpg"); 
    image(startscreen, 0, 0, 600, 600); 
    size(600, 600); 
    x=(int)random(23, width-23); 
    y=height-base-23;
  } 
  
  void draw() { 
    if (stage==1) { 
      //background (0); 
      image(startscreen, 600, 600);    
      textAlign(CENTER); 
      textSize(45); 
      fill(66, 104, 244); 
      text("PONG GAME", width/2, 170); 
      text("Press any key to start", width/2, 170+77); 
      if (keyPressed == true) { 
        stage = 2;
      }
    } else if (stage == 2) { 
  
      background(100, 200, 100);
  
      if (gameOver==0) { 
        background(0); 
        textAlign(LEFT); 
        fill(244, 66, 66); 
        text("hit: " + hit, 7, 40); 
        fill(66, 104, 244); 
        rect(mouseX, height-base, 150, base); 
        ellipse(x, y, 40, 40); 
        //move
        x=x+changeX; 
        y=y+changeY; 
        // we use 20 here which is the radius of the ball  
        if (x<20 | x>width-20) { 
          changeX=-changeX;
        } 
        if (y<20) { 
          changeY=-changeY;
        } 
        if (y>height-base-20) { 
          //check whether it is falling inside the rectangle or not
          if (x>mouseX && x<mouseX+200) { 
            changeY = -changeY; 
            y-=6; 
            //bounce back 
            hit += 1;
          } else { 
            // game over
            gameOverSplash();
          }
        }
      } else { 
        background(244, 78, 66); 
        fill(66, 104, 244); 
        textAlign(CENTER); 
        text("Game Over!", width/2, height/2); 
        text("CLICK MOUSE TO RESTART", width/2, height/2+60);
      }//else
    }//stage 2
  }// func 
  
  void gameOverSplash()
  { 
    gameOver=1;
  } 
  
  void mouseClicked() {
    if (gameOver==1) {
      changeY=-changeY; 
      hit +=1; 
      gameOver=0;
    }
  }
  
• Chrisir

  January 2017 Answer ✓

  you have stage 1 and 2

  now you can name your stages by saying

  int stageStartscreen = 1; 
  int stageGame = 2; 
  

  and use those like

  if (stage==stageStartscreen) { 
  

  or

  stage = stageGame;
  

  Also, since gameover is also stage you can alter the structure and think of gameover as a 3rd stage:

  int stageGameOver = 3; 
  

  now you don't need a confusing if (gameOver==0) { inside the if (stage == 2) { anymore...

  now you can say in draw():

  if (stage==stageStartscreen) { 
  //....
  }
  else if (stage==stageGame) { 
  //....
  }
  else if(stage==stageGameOver) { 
  //....
  }
  

  or even use switch (you'll need constants for this, see below):

  switch(stage)  {
  
      case stageStartscreen:
      //...
      break; 
  
      case stageGame:
      //...
      break; 
  
      case stageGameOver:
      // ....
      break;
  
  } // switch 
  

  BTW

  since those names never change (only stage does, not its names) we can mark them as constants (with final):

  final int stageStartscreen = 1; 
  final int stageGame = 2; 
  final int stageGameOver = 3; 
  
• supsup

  January 2017

  you're unbelieveble, thank you so much:)
• koogs

  January 2017 Answer ✓

  continued here

  https://forum.processing.org/two/discussion/20049/hi-i-want-to-add-the-pong-game-below-a-speed-function-which-makes-game-speed-increase-with-10-hits