Finished etapa2 #2
Conversation
| import java.util.Random; | ||
| import java.util.Scanner; | ||
|
|
||
| public class Course extends Activity { |
There was a problem hiding this comment.
codul nu compileaza, Activity pare sa nu mai existe?
There was a problem hiding this comment.
Am adaugat acum manual clasele care lipseau din etapa2 fata de etapa1. Nu stiu de ce s-a intamplat sa nu mai fie.. La mine compileaza local. Daca inca nu este in regula cu pull request si github, pot trimite arhiva.. Imi cer scuze, tot am probleme cu folosirea github-ului :(
There was a problem hiding this comment.
acum pare sa fie ok, sa compileze :) baby steps, daca iti propui, inveti si git/github mai bine cu timpul :) scopul e sa va obisnuiti de pe acum
|
|
| } | ||
| return instance; | ||
| } | ||
|
|
There was a problem hiding this comment.
ca sa fie singleton, ai nevoie de private constructor, aici lipseste
| import java.util.ArrayList; | ||
| import java.util.List; | ||
|
|
||
| public class CourseSingleton { |
There was a problem hiding this comment.
acesta se doreste a fi serviciul singleton de scriere + citire pentru un curs?
- nu este singleton pentru ca ii lipseste constructorul private
- ar trebui redenumite clasele in CourseReadService/CourseWriteService si sa ai pastrate separate metodele de read/write, cerinta era sa aveti 2 servicii singleton generice, unul pentru scriere, unul pentru citire. Daca reuseai sa le faci generice, nu iti trebuiau clase separate pentru fiecare entitate in parte.
| import java.util.List; | ||
|
|
||
| public class ScheduleSingleton { | ||
| public static ScheduleSingleton instance = null; |
There was a problem hiding this comment.
acelasi comentariu ca si pentru CourseSingleton
| public class StudentSingleton { | ||
| public static StudentSingleton instance = null; | ||
|
|
||
| private List<Student> students = new ArrayList<Student>(); |
There was a problem hiding this comment.
acelasi comentariu ca si pentru CourseSingleton
| import java.util.List; | ||
|
|
||
| public class TeacherSingleton { | ||
| public static TeacherSingleton instance = null; |
There was a problem hiding this comment.
acelasi comentariu ca si pentru CourseSingleton
|
daca dresti sa fixezi comentariile pt 1p dedicat acestui lucru, te rog sa o faci si apoi sa faci merge in main si sa lucrezi pt etapa 3 (la etapa 3 nu e necesar PR, veti prezenta in demo chiar si din main branch - cum doriti, pe 30/31 mai in functie de ziua pe care ai ales-o). Mult spor, Andreea! |
Multumesc! Ce deadline am pentru a fixa comentariile? |
No description provided.