使用两个不同的构造函数创建单个对象

Question

我有一个Card类，其中包含两个构造函数，您可以在下面的代码中看到。问题是每个Card对象包含2个字符串和2个图像，因为我想要保存这些对象，我必须保存图像＆＃39 ; s路径为字符串然后检索它们，因此是2个构造函数。但是，使用一些打印语句，我发现无论何时检索到一张卡，都会创建2个单独的对象：正常的对象和另一个无效的对象。我知道这是因为我使用第二个中的第一个构造函数创建了一个新卡。有没有办法可以避免这种情况，以至于我没有2个独立的物体？如果可能的话，我想找到一种只使用一个构造函数的方法。这是我的课程：

Card.java：

package com.spdesigns.dokkancardspreview.model;

import javafx.scene.image.Image;

public class Card {

private String mName;
private String mDescription;
private Image mMainImage;
private Image mSecondaryImage;
private String mMainImagePath;
private String mSecondaryImagePath;

public Card(String name, String description, Image mainImage, Image secondaryImage) {
    mName = name;
    mDescription = description;
    mMainImage = mainImage;
    mSecondaryImage = secondaryImage;
}

public Card(String name, String description , String mainImagePath, String secondaryImagePath) {
   Card newCardToAdd =  new Card(name,description,new Image(mainImagePath),new Image(secondaryImagePath));
}

@Override
public String toString() {
    return mName + " | " + mDescription;
}

public Image getmMainImage() {
    return mMainImage;
}

public Image getmSecondaryImage() {
    return mSecondaryImage;
}

public String getName() {
    return mName;
}

public String getDescription() {
    return mDescription;
}
}

home.java：

package com.spdesigns.dokkancardspreview.controllers;

import com.spdesigns.dokkancardspreview.model.Card;

import javafx.beans.property.ListProperty;
import javafx.beans.property.SimpleListProperty;
import javafx.collections.FXCollections;
import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.control.Button;
import javafx.scene.control.ListView;
import javafx.scene.image.Image;
import javafx.scene.image.ImageView;

import java.io.*;
import java.util.List;
import java.net.URL;
import java.util.ArrayList;
import java.util.ResourceBundle;

public class home implements Initializable {

private Card hit = new Card("Hit","Expanding Possibility",
        new Image("/images/hit_main.jpg") , new Image("/images/hit_secondary.jpg"));

private Card goku = new Card("Goku SSJ3","Everlasting legend",
        new Image("/images/gokussj3_main.jpg") , new Image("/images/gokussj3_secondary.jpg"));

private boolean clickedAgain = false;

@FXML
private Button arrowButton;
@FXML
private ImageView imageView;

@FXML
private ImageView arrow;

@FXML
private ListView listView;

protected List<Card> testingList = new ArrayList<Card>();

    protected ListProperty<Card> listProperty = new SimpleListProperty<Card>();


@Override
public void initialize(URL location, ResourceBundle resources) {
    addCard(hit);
    addCard(goku);
    //testingList.add("test2");

    listView.itemsProperty().bind(listProperty);
    // wrapping our list in an observable list and then pass that observableList to the ListProperty isntance
    listProperty.set(FXCollections.observableArrayList(testingList));

    // Handle listView selection changes
    listView.getSelectionModel().selectedItemProperty().addListener(((observable, oldValue, newValue) -> {
        Card currentlySelectedCard = listProperty.get(listView.getSelectionModel().getSelectedIndex());
        System.out.printf("ListView item clicked! Value retrieved: %s\n", currentlySelectedCard);
        imageView.setImage(new Image(currentlySelectedCard.getmMainImage().impl_getUrl()));
        arrow.setVisible(true);
        arrowButton.setVisible(true);
    }));

    arrow.translateYProperty().set(283f);
    arrowButton.translateYProperty().set(283f);
    arrow.setRotate(180);
    arrow.setVisible(false);
    arrowButton.setVisible(false);
}

public void handleShowDetails(ActionEvent actionEvent) {
    System.out.println("Button Clicked!");
    Card currentlySelectedCard = listProperty.get(listView.getSelectionModel().getSelectedIndex());
    if(clickedAgain) {
        imageView.setImage(new Image(currentlySelectedCard.getmMainImage().impl_getUrl()));
        arrow.setRotate(180);
        clickedAgain = false;
    } else {
        imageView.setImage(new Image(currentlySelectedCard.getmSecondaryImage().impl_getUrl()));
        arrow.setRotate(360);
        clickedAgain = true;
    }
}

// Saving
public void exportTo(String fileName) {
    try(
            FileOutputStream fos = new FileOutputStream(fileName);
            PrintWriter writer = new PrintWriter(fos);
    ){
        for(int i =0;i<testingList.size()-1;i++) {
            writer.printf("%s|%s|%s|%s\n",testingList.get(i).getName(),testingList.get(i).getDescription(),
                    testingList.get(i).getmMainImage().impl_getUrl(),testingList.get(i).getmSecondaryImage().impl_getUrl());
            System.out.println(testingList.get(i).toString());
        }
    } catch (IOException ioe) {
        System.out.printf("Problem saving: %s/n", fileName);
        ioe.printStackTrace();
    }
}

// Loading
public void importFrom(String fileName) {
    try(
            FileInputStream fis = new FileInputStream(fileName);
            BufferedReader reader = new BufferedReader(new InputStreamReader(fis));
    ){
        String line;
        while((line = reader.readLine()) != null) {
            String[] args = line.split("\\|");
            this.addCard(new Card(args[0],args[1],args[2],args[3]));
        }
    } catch (IOException ioe) {
        System.out.printf("Problem loading: %S\n" , fileName);
        ioe.printStackTrace();
    }
    int i = 0;
    for (Card card : testingList) {
        System.out.printf("%s loaded\n",testingList.get(i).toString());
        i++;
    }
    System.out.println("Loading Successful!");
}

public void addCard(Card card) {
    testingList.add(card);
}

// DEBUG purposes
public void printTestingList() {
    for (Card card : testingList) {
        System.out.println(card.toString());
    }
}
}

MAIN.JAVA：

package com.spdesigns.dokkancardspreview;

import com.spdesigns.dokkancardspreview.controllers.home;
import javafx.application.Application;
import javafx.fxml.FXMLLoader;
import javafx.scene.Parent;
import javafx.scene.Scene;
import javafx.stage.Stage;

import java.io.File;

public class Main extends Application {

private home controller;
private File file = new File("CardsCollection.txt");

@Override
public void start(Stage primaryStage) throws Exception{
    FXMLLoader loader = new FXMLLoader(getClass().getResource("/fxml/home.fxml"));
    Parent root = loader.load();
    controller = loader.getController();
    primaryStage.setTitle("Dokkan Battle Card Preview");
    primaryStage.setScene(new Scene(root, 900, 700));
    primaryStage.setResizable(false);
    // Loading cards
    primaryStage.show();
    try {
        if(!file.exists()) {
            file.createNewFile();
        }
        controller.importFrom("CardsCollection.txt");
    } catch (NullPointerException npe) {
        System.out.println("Error loading file!");
        npe.printStackTrace();
    }
}

@Override
public void stop() {
    System.out.println("App is closing!");
    // Saving before exiting
    try {
        controller.exportTo("CardsCollection.txt");
    } catch (NullPointerException npe) {
        System.out.println("Problem saving file!");
        npe.printStackTrace();
    }
   // controller.printTestingList();
}

public static void main(String[] args) {
    launch(args);
}
}

Answer 1

变化

public Card(String name, String description , String mainImagePath, String secondaryImagePath) {
   Card newCardToAdd =  new Card(name,description,new Image(mainImagePath),new Image(secondaryImagePath));
}

到

public Card(String name, String description , String mainImagePath, String secondaryImagePath) {
   this(name,description,new Image(mainImagePath),new Image(secondaryImagePath));
}

Answer 2

代码的问题在于此构造函数：

public Card(String name, String description , String mainImagePath, String secondaryImagePath) {
   Card newCardToAdd =  new Card(name,description,new Image(mainImagePath),new Image(secondaryImagePath));
}

如你所见，你在那里创建一张新卡 newCardToAdd ，让新实例完全错误初始化......

在这种情况下你做的是从另一个构建函数调用...在你的情况下，你的2个cosntructors应该是这样的：

public Card(String name, String description, Image mainImage, Image secondaryImage) {
    mName = name;
    mDescription = description;
    mMainImage = mainImage;
    mSecondaryImage = secondaryImage;
}

public Card(String name, String description, String mainImagePath, String secondaryImagePath) {
    this(name, description, new Image(mainImagePath), new Image(secondaryImagePath));
}

Answer 3

如果你永远不需要将Image直接传递给类（只有图像路径），那么单个构造函数就足够了：

public Card(String name, String description, 
               String mainImagePath, String secondaryImagePath) {
    mName = name;
    mDescription = description;
    mMainImagePath = mainImagePath;
    mSecondaryImagePath = secondaryImagePath;
    mMainImage = new Image(mainImagePath);
    mSecondaryImage = new Image(secondaryImagePath);
}