Skip to content
Snippets Groups Projects
Commit 7996dbf7 authored by Nico Hautakoski's avatar Nico Hautakoski
Browse files

Korjattu

parent 8d82d6f6
No related branches found
No related tags found
No related merge requests found
# Default ignored files
/shelf/
/workspace.xml
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="ProjectRootManager" version="2" project-jdk-name="19" project-jdk-type="JavaSDK">
<output url="file://$PROJECT_DIR$/out" />
</component>
</project>
\ No newline at end of file
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="ProjectModuleManager">
<modules>
<module fileurl="file://$PROJECT_DIR$/oop-demo3-esimerkit.iml" filepath="$PROJECT_DIR$/oop-demo3-esimerkit.iml" />
</modules>
</component>
</project>
\ No newline at end of file
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
<component name="VcsDirectoryMappings">
<mapping directory="" vcs="Git" />
</component>
</project>
\ No newline at end of file
...@@ -2,28 +2,28 @@ import java.util.ArrayList; ...@@ -2,28 +2,28 @@ import java.util.ArrayList;
import java.util.HashMap; import java.util.HashMap;
public class Verkkokauppa { public class Verkkokauppa {
private ArrayList<Asiakas> asiakkaat; private HashMap<String, Asiakas> asiakkaat;
private ArrayList<Tuote> tuotteet; private HashMap<String, Tuote> tuotteet;
private ArrayList<Myyja> myyjat; private HashMap<String, Myyja> myyjat;
private ArrayList<Ostotapahtuma> tapahtumat; private ArrayList<Ostotapahtuma> tapahtumat;
public Verkkokauppa() { public Verkkokauppa() {
asiakkaat = new ArrayList<>(); asiakkaat = new HashMap<>();
tuotteet = new ArrayList<>(); tuotteet = new HashMap<>();
myyjat = new ArrayList<>(); myyjat = new HashMap<>();
tapahtumat = new ArrayList<>(); tapahtumat = new ArrayList<>();
} }
public void lisaaAsiakas(Asiakas asiakas) { public void lisaaAsiakas(Asiakas asiakas) {
asiakkaat.add(asiakas); asiakkaat.put(asiakas.getAsiakasNumero(), asiakas);
} }
public void lisaaTuote(Tuote tuote) { public void lisaaTuote(Tuote tuote) {
tuotteet.add(tuote); tuotteet.put(tuote.getNimi(), tuote);
} }
public void lisaaMyyja(Myyja myyja) { public void lisaaMyyja(Myyja myyja) {
myyjat.add(myyja); myyjat.put(myyja.getTunniste(), myyja);
} }
public void lisaaTapahtuma(Ostotapahtuma tapahtuma) { public void lisaaTapahtuma(Ostotapahtuma tapahtuma) {
...@@ -37,8 +37,8 @@ public class Verkkokauppa { ...@@ -37,8 +37,8 @@ public class Verkkokauppa {
* @return true, jos asiakas löytyi ja poistettiin, muuten false * @return true, jos asiakas löytyi ja poistettiin, muuten false
*/ */
public boolean poistaAsiakas(Asiakas asiakas) { public boolean poistaAsiakas(Asiakas asiakas) {
if (asiakkaat.contains(asiakas)) { if (asiakkaat.containsValue(asiakas)) {
asiakkaat.remove(asiakas); asiakkaat.remove(asiakas.getAsiakasNumero());
return true; return true;
} }
return false; return false;
...@@ -51,8 +51,8 @@ public class Verkkokauppa { ...@@ -51,8 +51,8 @@ public class Verkkokauppa {
* @return true, jos tuote löytyi ja poistettiin, muuten false * @return true, jos tuote löytyi ja poistettiin, muuten false
*/ */
public boolean poistaTuote(Tuote tuote) { public boolean poistaTuote(Tuote tuote) {
if (tuotteet.contains(tuote)) { if (tuotteet.containsValue(tuote)) {
tuotteet.remove(tuote); tuotteet.remove(tuote.getNimi());
return true; return true;
} }
return false; return false;
...@@ -65,8 +65,8 @@ public class Verkkokauppa { ...@@ -65,8 +65,8 @@ public class Verkkokauppa {
* @return true, jos myyjä löytyi ja poistettiin, muuten false * @return true, jos myyjä löytyi ja poistettiin, muuten false
*/ */
public boolean poistaMyyja(Myyja myyja) { public boolean poistaMyyja(Myyja myyja) {
if (myyjat.contains(myyja)) { if (myyjat.containsValue(myyja)) {
myyjat.remove(myyja); myyjat.remove(myyja.getTunniste());
return true; return true;
} }
return false; return false;
...@@ -94,7 +94,7 @@ public class Verkkokauppa { ...@@ -94,7 +94,7 @@ public class Verkkokauppa {
* @return pyydetyn asiakkaan tai null, jos asiakasta ei löydy * @return pyydetyn asiakkaan tai null, jos asiakasta ei löydy
*/ */
public Asiakas annaAsiakas(String asiakasnumero) { public Asiakas annaAsiakas(String asiakasnumero) {
for (Asiakas asiakas : asiakkaat) { for (Asiakas asiakas : asiakkaat.values()) {
if (asiakas.getAsiakasNumero().equals(asiakasnumero)) { if (asiakas.getAsiakasNumero().equals(asiakasnumero)) {
return asiakas; return asiakas;
} }
...@@ -110,7 +110,7 @@ public class Verkkokauppa { ...@@ -110,7 +110,7 @@ public class Verkkokauppa {
* @return pyydetyn tuotteen tai null, jos tuotetta ei löydy * @return pyydetyn tuotteen tai null, jos tuotetta ei löydy
*/ */
public Tuote annaTuote(String nimi) { public Tuote annaTuote(String nimi) {
for (Tuote tuote : tuotteet) { for (Tuote tuote : tuotteet.values()) {
if (tuote.getNimi().equals(nimi)) { if (tuote.getNimi().equals(nimi)) {
return tuote; return tuote;
} }
...@@ -126,7 +126,7 @@ public class Verkkokauppa { ...@@ -126,7 +126,7 @@ public class Verkkokauppa {
* @return pyydetyn myyjän tai null, jos myyjää ei löydy * @return pyydetyn myyjän tai null, jos myyjää ei löydy
*/ */
public Myyja annaMyyja(String tunniste) { public Myyja annaMyyja(String tunniste) {
for (Myyja myyja : myyjat) { for (Myyja myyja : myyjat.values()) {
if (myyja.getTunniste().equals(tunniste)) { if (myyja.getTunniste().equals(tunniste)) {
return myyja; return myyja;
} }
...@@ -149,7 +149,6 @@ public class Verkkokauppa { ...@@ -149,7 +149,6 @@ public class Verkkokauppa {
return null; return null;
} }
/** /**
* Palauttaa kaikki asiakkaat yhtenä merkkijonona. * Palauttaa kaikki asiakkaat yhtenä merkkijonona.
* *
...@@ -160,7 +159,7 @@ public class Verkkokauppa { ...@@ -160,7 +159,7 @@ public class Verkkokauppa {
// \t on tabulaattori, joka tasaa tulosteen // \t on tabulaattori, joka tasaa tulosteen
// sarakkeisiin; \n tekee rivinvaihdon // sarakkeisiin; \n tekee rivinvaihdon
StringBuilder s = new StringBuilder("Asiakasnumero\tNimi\t\t\tOstoja\tKanta-asiakas\n"); StringBuilder s = new StringBuilder("Asiakasnumero\tNimi\t\t\tOstoja\tKanta-asiakas\n");
for (Asiakas asiakas : asiakkaat) { for (Asiakas asiakas : asiakkaat.values()) {
s.append(asiakas.getAsiakasNumero() + "\t\t\t"); s.append(asiakas.getAsiakasNumero() + "\t\t\t");
s.append(asiakas.getNimi() + "\t"); s.append(asiakas.getNimi() + "\t");
s.append(asiakas.getOstojaTehty() + "\t\t"); s.append(asiakas.getOstojaTehty() + "\t\t");
...@@ -183,7 +182,7 @@ public class Verkkokauppa { ...@@ -183,7 +182,7 @@ public class Verkkokauppa {
// \t on tabulaattori, joka tasaa tulosteen // \t on tabulaattori, joka tasaa tulosteen
// sarakkeisiin; \n tekee rivinvaihdon // sarakkeisiin; \n tekee rivinvaihdon
StringBuilder s = new StringBuilder("Tunniste\tNimi\t\t\tProvisiot\n"); StringBuilder s = new StringBuilder("Tunniste\tNimi\t\t\tProvisiot\n");
for (Myyja myyja : myyjat) { for (Myyja myyja : myyjat.values()) {
s.append(myyja.getTunniste() + "\t\t"); s.append(myyja.getTunniste() + "\t\t");
s.append(myyja.getNimi() + "\t\t"); s.append(myyja.getNimi() + "\t\t");
s.append(myyja.getProvisiot() + "\n"); s.append(myyja.getProvisiot() + "\n");
...@@ -201,7 +200,7 @@ public class Verkkokauppa { ...@@ -201,7 +200,7 @@ public class Verkkokauppa {
// \t on tabulaattori, joka tasaa tulosteen // \t on tabulaattori, joka tasaa tulosteen
// sarakkeisiin; \n tekee rivinvaihdon // sarakkeisiin; \n tekee rivinvaihdon
StringBuilder s = new StringBuilder("Nimi\tSaldo\t\t\tHinta\tVirtuaalinen\n"); StringBuilder s = new StringBuilder("Nimi\tSaldo\t\t\tHinta\tVirtuaalinen\n");
for (Tuote tuote : tuotteet) { for (Tuote tuote : tuotteet.values()) {
s.append(tuote.getNimi() + "\t"); s.append(tuote.getNimi() + "\t");
s.append(tuote.getSaldo() + "\t\t"); s.append(tuote.getSaldo() + "\t\t");
s.append(tuote.getHinta() + "\t"); s.append(tuote.getHinta() + "\t");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment