Проблема конструктора, сеттера, геттера в создании объекта
Я создаю класс, чтобы делать автомобили, основываясь на нескольких переменных, таких как год (год назад), марка (марка), цвет (couleur), одометр (километраж), продано (продавец), трансмиссия (автомат), комментарий (автокомпонент) и число дверей (nbrPortes).
Аргументы внутри кода приведены на французском языке, потому что я учусь во французском университете, поэтому, пожалуйста, потерпите меня.
Мой код должен иметь сеттеры, конструкторы и методы клонирования в точности такими, какие они есть сейчас (навязаны). Я сделал геттеры и инкремент для IDnumber и compteurvendue (количество проданных автомобилей) самостоятельно.
Вот что я имею в качестве кода:
public class Automobile {
private static final String[] COULEURS = { "Autre", "Noir", "Blanc", "Bleu Nuit", "Bleu Clair", "Vert Pomme", "Vert Bouteille", "Taupe", "Argent", "Sable", "Gris Charbon", "Gris Clair", "Orange", "Rouge", "Jaune", "Brun" };
private static final String[] MARQUES = { "Autre", "Mazda", "Toyota", "Ford", "GM", "Hyunday", "BMW", "SAAB", "Honda", "Mitsubishi", "Mercedes", "KIA", "Wolkswagen"};
public static final int COULEUR_DEF = 8;
public static final int COULEUR_MIN = 0;
public static final int COULEUR_MAX = COULEURS.length - 1;
public static final int MARQUE_DEF = 4;
public static final int MARQUE_MIN = 0;
public static final int MARQUE_MAX = MARQUES.length - 1;
public static final double KILO_DEFAUT = 55000;
public static final double KILO_MIN = 15000;
public static final double KILO_MAX = 140000;
public static final int DEUX_PORTES = 2;
public static final int QUATRE_PORTES = 4;
public static final int PORTES_DEFAUT = QUATRE_PORTES;
public static final boolean AUTO_DEF = true;
public static final int ANNEE_MIN = 1997;
public static final int ANNEE_MAX = 2016;
public static final int ANNEE_DEFAUT = 2007;
public static final String COMM_DEFAUT = "";
public static final boolean VENDUE_DEF = false;
//private variables
private int marque;
private int annee;
private int couleur;
private boolean a;
private boolean automatique;
private double k;
private double kilometrage;
private int p;
private int nbrPortes;
private String c;
private String commentaire;
private boolean v;
private boolean vendue;
private int compteurvendue = 0;
private int IDnumber = 0;
// CONSTRUCTORS. I do not know I have to make 2, I am thinking for the purpose of the cloning without copying certain variables? I'm confused about this.
public Automobile (int marque, int annee, int couleur, boolean automatique, double kilometrage) {
if (marque>= MARQUE_MIN && marque <= MARQUE_MAX) {
this.marque = marque;
} else {
this.marque = MARQUE_DEF;
}
if (annee>= ANNEE_MIN && annee <= ANNEE_MAX) {
this.annee = annee;
} else {
this.annee = ANNEE_DEFAUT;
}
if (couleur>= COULEUR_MIN && couleur <= COULEUR_MAX) {
this.couleur = couleur;
} else {
this.couleur = COULEUR_DEF;
}
if (a == true || a == false) {
this.automatique = a;
} else {
this.automatique = AUTO_DEF;
}
if (k >= KILO_MIN && k <= KILO_MAX) {
this.kilometrage = k;
} else {
this.kilometrage = KILO_DEFAUT;
}
this.IDnumber = IDnumber + 1;
}
public Automobile (int marque, int annee, int couleur, boolean automatique, double kilometrage,
int nbrPortes, String commentaire, boolean vendue){
if (marque>= MARQUE_MIN && marque <= MARQUE_MAX) {
this.marque = marque;
} else {
this.marque = MARQUE_DEF;
}
if (annee>= ANNEE_MIN && annee <= ANNEE_MAX) {
this.annee = annee;
} else {
this.annee = ANNEE_DEFAUT;
}
if (couleur>= COULEUR_MIN && couleur <= COULEUR_MAX) {
this.couleur = couleur;
} else {
this.couleur = COULEUR_DEF;
}
if (a == true || a == false) {
this.automatique = a;
} else {
this.automatique = AUTO_DEF;
}
if (k >= KILO_MIN && k <= KILO_MAX) {
this.kilometrage = k;
} else {
this.kilometrage = KILO_DEFAUT;
}
if (p == DEUX_PORTES || p == QUATRE_PORTES) {
this.nbrPortes = p;
} else {
this.nbrPortes = PORTES_DEFAUT;
}
if (c != "") {
this.commentaire = c;
} else {
this.commentaire = COMM_DEFAUT;
}
if (v == true || v == false) {
this.vendue = v;
} else {
this.vendue = VENDUE_DEF;
}
this.IDnumber = IDnumber + 1; //The problem is that the IDnumber is 1 for every new instance I create, wether I have the this. in front or not.
}
//HERE I AM SUPPOSED TO PUT IN A CLASS GETTER...I have not managed to make it work in any way
public static Automobile getAutomobile() {
return null; //just put a null return in the meantime so I can compile
}
//GETTERS
public int getAnnee() {
return annee;
}
public int getMarque() {
return marque;
}
public int getCouleur() {
return couleur;
}
public int getNbrPortes() {
return p;
}
public double getKilometrage() {
return k;
}
public boolean getVendue() {
return v;
}
public String getCommentaire() {
return c;
}
public boolean getAutomatique() {
return a;
}
//Setters. According to the imposed settings, Not supposed to have any error message or correction in case of wrong input, it just goes right through?
public void setAnnee ( int annee ) {
//A COMPLETER
if (annee >= ANNEE_MIN && annee <= ANNEE_MAX) {
this.annee = annee;
}
}
public void setMarque (int marque){
//A COMPLETER
if (marque >= MARQUE_MIN && marque <= MARQUE_MAX) {
this.marque = marque;
}
}
public void setCouleur (int couleur) {
//A COMPLETER
if (couleur >= COULEUR_MIN && couleur <= COULEUR_MAX){
this.couleur = couleur;
}
}
public void setNbrPortes (int p) {
//A COMPLETER
if (p == DEUX_PORTES || p == QUATRE_PORTES){
this.p = p;
}
}
public void setKilometrage (double k) {
//A COMPLETER
if (k >= KILO_MIN && k <= KILO_MAX) {
this.k = k;
}
}
// This is supposed to count if I select a car as sold, and counts how many are sold in total (So I can later show it to the user)
public void setVendue (boolean v) {
if (v == true || v == false) {
this.v = v;
}
if (v = true) {
compteurvendue = compteurvendue + 1;
}
}
//Sets comment as long as it is not null
public void setCommentaire (String c){
if (c != null) {
this.c = c;
}
}
public void setAutomatique (boolean a){
if (a == true || a == false) {
this.a = a;
}
}
//Supposed to be able to create a clone of the car with all the same variables except the comment and sold that are reverted to default, and the IDnumber which will be a new one (+1)
public Automobile cloner () {
Automobile Clone = new Automobile(marque,annee,couleur,a,k,p,COMM_DEFAUT,VENDUE_DEF);
return Clone;
}
} // Automobile
Есть несколько частей моего кода, которые в настоящее время не работают:
Первый конструктор для Automobile принимает мои переменные, когда я тестирую его в среде BlueJ, но логические значения (vendue и automatique) всегда выдаются в виде FALSE, а комментарий всегда выводится в виде COMM_DEFAUT (пустая строка "").
Второй конструктор выводит все в моем экземпляре как значение по умолчанию, даже если я ввожу допустимые значения. логические значения и комментарий по-прежнему имеют ту же проблему.
Я не понимаю, как создать "класс getter"... и какова его цель в этом случае
IDnumber всегда получается как 1. Он не сохраняется в памяти между тем, как я создаю каждый новый объект.
Я чувствую, что мой метод клонирования верен, но я не могу вызвать его или проверить, чтобы создать даже автомобиль DEFAULT.
*** Пожалуйста, имейте в виду, что помимо приватных переменных и методов получения, заголовки всех сеттеров, конструкторов, финальных переменных и заголовка клонера накладываются и не могут быть изменены (что делает это намного сложнее),
Что я могу сделать, чтобы решить эти разные проблемы с моим кодом? Большое спасибо, так как это сообщество уже очень помогло!
1 ответ
Первый конструктор Automatique и комментарии: там как опечатка или вопрос копирования / вставки:)
if (a == true || a == false) {
**this.automatique = a;**
} else {
this.automatique = AUTO_DEF;
}
что делает указанная линия? на самом деле это равно
this.automatique = this.a;
неавтоматический параметр из конструктора.
тогда зачем вообще иметь две переменные a и automatique? (то же самое и для других). затем
if (a == true || a == false)
не имеет никакого смысла вообще. логическое a может быть только истинным или ложным, так что в результате этого, если?
Итак, удалите ненужные дубликаты переменных. затем перейдите по этой схеме:
// holder
private int marque;
//constructor: sure there will be all other parameters as well
public Automobile (int marque)
{
//if it is needed to check between MIN and MAx do it as in your code. (not for boolean :) )
this.marque = marque;
}
// getter
public int getMarque()
{
return this.marque;
}
// setter
public void setMarque(int marque)
{
//if it is needed to check between MIN and MAx do it as in your code.
this.marque = marque;
}
// cloner
public Automobile clone()
{
// sure there will be all other parameters and logic as well
Automobile clonedAuto = new Automobile(this.marque);
return clonedAuto;
}
вот и все.