search

pascalalfadian / SIAModels

Java classes representing objects available in UNPAR's Sistem Informasi Akademik
MIT License
0 stars 16 forks source link

Penambahan deskripsi mata kuliah, Author, dan interface HasPraktikum dan HasResponsi #4

Closed johanesadriano closed 8 years ago

johanesadriano commented 8 years ago

Mohon di review,pak Terimakasih

pascalalfadian commented 8 years ago

Terima kasih. Mohon diperbaiki:

  1. Deskripsi mata kuliah dan tag @author diisikan di javadoc kelas, bukan dengan membuat method getDeskripsi(). Perhatikan lagi contoh implementasi untuk AIF102 di issue 8
  2. Jangan import untuk kelas/interface yang tidak terpakai. Contoh, di kelas AIF103, kalian import HasPraktikum dan HasResponsi, tetapi tidak dipakai di dalam
  3. Mata Kuliah PBWL, silabus dikosongkan dulu saja (kalau ada).
pascalalfadian commented 8 years ago

Mohon pastikan dulu tidak ada compile error pada source code kamu.

Tips: Buat project eclipse dari source code tersebut. Caranya:

  1. Buka eclipse
  2. New Project (Java Project)
  3. Beri nama "SIAModels"
  4. Location project diset ke direktori yang sudah diclone tadi
  5. Biarkan eclipse mendeteksi source code yang sudah ada

Setelah itu, silahkan lihat error-error yang ada dan perbaiki.

pascalalfadian commented 8 years ago

Kelas AIF204 masih ada method getDeskripsi() mohon dihapus.

pascalalfadian commented 8 years ago

Saya masih menemukan kesalahan yang sama di AIF314. Mohon diperiksa sendiri dulu sebelum pull request, pastikan kode dalam kualitas yang baik.

Saya juga menemukan untuk kuliah yang dosennya lebih dari satu, kamu ganti @author ke @authors. Apakah @authors didukung oleh javadoc? Mohon dicek.

johanesadriano commented 8 years ago

Mohon maaf atas kesalahan-kesalahan kami. Kelas-kelas sudah diperiksa dan diperbaiki untuk author dan method getDeskripsi(). Terimakasih pak.

pascalalfadian commented 8 years ago

Mohon perbaiki: