0

ゲームを作っています。Block クラスを拡張すると、エラーが表示されます。

エラー: 暗黙のスーパー コンストラクター Block() が定義されていません。別のコンストラクターを明示的に呼び出す必要があります

コード:

ゲーム.java:

package lt.projecturanium;

import java.awt.BorderLayout;
import java.awt.Canvas;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.image.BufferStrategy;
import java.util.HashMap;

import javax.swing.JFrame;

import lt.projecturanium.blocks.Block;
import lt.projecturanium.blocks.BlockRectangle;
import lt.projecturanium.entity.Player;
@SuppressWarnings("unused")

public class Game extends Canvas implements Runnable{

    private static final long serialVersionUID = 1L;

    private static JFrame _frame;
    public static Game _instance;

    private static final String TITLE = "Project Uranium";
    private static final int WIDTH = 650;
    private static final int HEIGHT = WIDTH * 3 / 4;

    private static final int UPDATE_RATE = 50;
    private static final int RENDER_RATE = 100;

    public static HashMap<Block, Coordinates> blocks = new HashMap<Block, Coordinates>();

    public int rectx = 0;
    public int recty = 0;
    public int rectID = 0;

    public boolean hitted = false;

    public float interpolation;

    public static final Dimension SIZE = new Dimension(WIDTH, HEIGHT);

    private Thread _thread;

    private boolean _running;

    private int _totalTicks = 0;
    private int _tps = 0;
    private int _fps = 0;

    public Game()
    {
        _instance = this;
        setPreferredSize(SIZE);
        setMinimumSize(SIZE);
        setMaximumSize(SIZE);

        _frame = new JFrame(TITLE);

        _frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        _frame.setLayout(new BorderLayout());
        _frame.add(_instance, BorderLayout.CENTER);
        _frame.pack();

        _frame.setResizable(false);
        _frame.setLocationRelativeTo(null);
        _frame.setVisible(true);
        createBufferStrategy(2);
        blocks.put(new Block(new BlockRectangle(200)), new Coordinates(30, 50));
    }
    public synchronized void start()
    {
        _running = true;
        _thread = new Thread(this, TITLE+"_main");
        _thread.start();
    }
    public synchronized void stop()
    {
        _running = false;
        if (_thread != null)
        {
            try {
                _thread.join();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }
    public void paint(Graphics g) {
        super.paint(g);  // fixes the immediate problem.
        Graphics2D g2 = (Graphics2D) g;
        g2.drawString("FPS: " + _fps + "\n TPS: " + _tps, 10, 10);
        if (hitted)
        {
            recty = 0;
            rectx += 21;
            rectID++;
            blocks.put(new Block(new BlockRectangle(rectID)), new Coordinates(rectx, recty));
            hitted = false;
        }
        recty++;
        g2.drawImage(Player.getTexture(), 60, 60, null);
        g2.drawRect(rectx, recty, 20, 20);
        g2.setColor(new Color(101, 67, 33));
        g2.fillRect(0, 430, getWidth(), getHeight());
        g2.setColor(new Color(0, 100, 0));
        g2.fillRect(0, 420, getWidth(), 10);
        g2.setColor(Color.BLACK);
        if (recty == (419 - 20))
        {   
            hitted = true;
        }
    }
    public void run() {
        double lastUpdateTime = System.nanoTime();
        double lastRenderTime = lastUpdateTime;
        final int ns = 1000000000;
        final double nsPerUpdate = (double) ns / UPDATE_RATE;
        final double nsPerRender = (double) ns / RENDER_RATE;
        final int maxUpdatesBeforeRender = 5;

        int lastSecond = (int) (lastUpdateTime / ns);
        int tickCount = 0;
        int renderCount = 0;
        while (_running) {

          long currTime = System.nanoTime();
          int tps = 0;

          while ((currTime - lastUpdateTime) > nsPerUpdate && tps < maxUpdatesBeforeRender) {
            update();
            tickCount++;
            _totalTicks++;
            tps++;
            lastUpdateTime += nsPerUpdate;
            interpolation = Math.min(1.0F, (float) ((currTime - lastUpdateTime) / nsPerUpdate));
            render(interpolation, getGraphics());
          }

          if (currTime - lastUpdateTime > nsPerUpdate) {
            lastUpdateTime = currTime - nsPerUpdate;
          }
          if (currTime - lastRenderTime == maxUpdatesBeforeRender + 1)
          {
              render(interpolation, getGraphics());
          }
          renderCount++;
          lastRenderTime = currTime;

          int currSecond = (int) (lastUpdateTime / ns);
          if (currSecond > lastSecond) {
            _tps = tickCount;
            _fps = renderCount;
            tickCount = 0;
            renderCount = 0;
            lastSecond = currSecond;
            _frame.setTitle(TITLE + " | TPS: " + _tps + " | FPS: "+ _fps);

          }

          while (currTime - lastRenderTime < nsPerRender && currTime - lastUpdateTime < nsPerUpdate) {
            Thread.yield();
            try {
              Thread.sleep(1);
            } catch (InterruptedException e) {
              e.printStackTrace();
            }
            currTime = System.nanoTime();
          }
        }   
      }
    public void update()
    {
        _frame.pack();
    }
    public void render(float interp, Graphics g)
    {
        BufferStrategy myStrategy = getBufferStrategy(); 
        Graphics gra = myStrategy.getDrawGraphics();
        paint(gra);
        g.dispose();
        myStrategy.show();
        //System.out.println("Grass x: " + blocks.get("grass").getX() + " y: " + blocks.get("grass").getY());
        System.out.println("Stone x: " + blocks.get(new Block(new BlockRectangle(rectID))).getX() + " y: " + blocks.get(new Block(new BlockRectangle(rectID))).getY());
    }
}

Block.java:

package lt.projecturanium.blocks;

public class Block {
    private Block block;
    public Block (Block block){
        this.block = this;
    }
    public Block getBlock() {
        return block;
    }
    public Block getBlockById(int id)
    {
        return block;
    }
}

BlockRectangle.java:

package lt.projecturanium.blocks;

import java.awt.Image;
import java.io.IOException;
import java.util.HashMap;

import javax.imageio.ImageIO;

import lt.projecturanium.Game;

public class BlockRectangle extends Block{
    private int id;
    private static HashMap<Integer, BlockRectangle> rects = new HashMap<Integer, BlockRectangle>();
    public BlockRectangle(int id)
    {
        this.id = id;
    }
    public int getID()
    {
        return this.id;
    }
    public static BlockRectangle getByID(int id)
    {
        return rects.get(id);
    }
    public static Image getTexture()
    {
        try{        
            return ImageIO.read(Game._instance.getClass().getClassLoader().getResource("../res/player.png"));   
        }
        catch(IOException e)
        {
            e.printStackTrace();
        }
        return null;
    }
}

座標.java:

package lt.projecturanium;

public class Coordinates {
    private int x;
    private int y;
    public Coordinates(int x, int y)
    {
        this.x = x;
        this.y = y;
    }
    public int getX()
    {
        return this.x;
    }
    public int getY()
    {
        return this.y;
    }
}
4

6 に答える 6

1
public class BlockRectangle extends Block

BlockRectangleBlockを拡張します。したがって、BlockRectangleのコンストラクターでは、まずsuper()を介してBlockのコンストラクターを呼び出す必要があります。

public BlockRectangle(int id){
    //This line is optional if Block has an empty constructor.
    super([...]);
    this.id = id;
}

Blockに空のコンストラクターがある場合、それは暗黙的に呼び出されます。そうではありません。したがって、明示的に自分でsuper()を呼び出す(または空のコンストラクターを定義する) 必要があります。

: Blockにコンストラクターがまったくない場合、空のコンストラクターが暗黙的に作成されます。

于 2013-09-16T12:24:37.150 に答える
0

コンストラクターを定義したため、Java はデフォルトのコンストラクターを提供しなくなりました。したがって、nullary コンストラクターを自分で定義する必要があります。

public Block()
{
}

また、なぜそれ自体への参照を保持しているのかわかりませんBlock。クラスにあるメソッドを考えると、Block継承についてもっと学びたいと思うかもしれません。悪用していると思います。

于 2013-09-16T12:22:02.373 に答える
0

デフォルトのコンストラクターは、クラスに他のコンストラクターを定義していない場合にのみ存在します。サブクラスのコンストラクターは、他のコンストラクターが手動で呼び出されない場合、暗黙のうちに super() を呼び出します。つまり、デフォルトのコンストラクターを呼び出そうとします。

この問題は、Block クラスでパラメーターなしのコンストラクターを自分で作成するか、各サブクラスで super を介して既存のコンストラクターを呼び出すことで解決できます。

于 2013-09-16T12:26:29.313 に答える
0

別のコンストラクターを明示的に定義しない限り、Java クラスには暗黙的なデフォルト コンストラクターがあります。これはあなたがBlockクラスで行ったことです。

コンストラクターを持つサブクラスがある場合、スーパークラスのデフォルトのコンストラクターが暗黙的に呼び出されます。

public BlockRectangle(int id)
{
    // Tries to call super() implictly which doesn't exist anymore
    this.id = id;
}

スーパークラスで追加のデフォルト コンストラクターを明示的に定義することで、これを修正できます。

public Block()
{
}

または、デフォルト以外のスーパークラス コンストラクターを明示的に呼び出すことによって

public BlockRectangle(int id)
{
    super(new Block());
    this.id = id;
}

別のトピックについて: あなたのコンストラクターはBlock本当に意味がありません:

public Block (Block block){
    this.block = this;
}

パラメータを無視し、フィールドを に設定していますthisthis参照はクラス内でいつでも利用できるため、意味がありません。

于 2013-09-16T12:28:32.743 に答える
0

コンストラクターが原因で、このエラーが発生しています

public BlockRectangle(int id)
    {
        this.id = id;
    }

super引数のないコンストラクターを暗黙的に呼び出します。Block(){}

クラスのコンストラクターを明示的に定義したため、コンパイラーはデフォルトの引数なしコンストラクターを提供しません。したがって、手動で作成する必要があります。

Block(){}クラスに引数なしのコンストラクターを追加するBlockと、問題が解決します。

于 2013-09-16T12:23:11.527 に答える