Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Menambahkan materi basic Object Oriented Programing #106

Merged
merged 8 commits into from Nov 27, 2021

Conversation

kondiak
Copy link
Collaborator

@kondiak kondiak commented Oct 29, 2021

Closes

Deskripsi (Description)

  • saya menambahkan materi basic baru berupa pengenalan OOP
  • saya memperbaiki materi basic yang sudah ada dalam materi Perulangan

Contributor Requirements (Syarat Kontributor)

  • saya sudah membaca (I have read) CONTRIBUTING dan sudah menyetujui semua
  • saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat
  • saya sudah membuat artikel README.md tentang materi yang saya buat
  • saya menggunakan bahasa indonesia untuk memberikan penjelasan dari kode yang saya buat

Environment

saya menggunakan (I am using):

  • os = windows
  • php version = 8.0.8
  • base url = localhost:8085

Testing

  • Codesniffer PSR-12 phpcs
  • Codesniffer autofix phpcbf
  • Unit testing PHPUnit

Maintainer

@bellshade/php-team
@bellshade/docs-team

  1. Kali ini saya tambahkan sedikit mengenai infinite loop, walaupun dalam daftar materi tidak ada, saya merasa perlu untuk membuat ini karena hampir semua penulis kode pernah mengalami hal ini.
  2. Saya mencoba menjelaskan materi mengenai OOP, pada saat ini baru sebatas pengenalan, untuk itu saya tidak mentag issue yang bersangkutan

@kondiak
Copy link
Collaborator Author

kondiak commented Oct 29, 2021

Mohon untuk di periksa penyampaian mengenai OOP yang saya tulis, berhubung saya kesulitan menjelaskan apa itu OOP sesederhana mungkin dan mudah dimengerti. Untuk itu mohon masukan dari rekan-rekan team bellshade maupun para contributor.

sebelumnya atas masukan yang diberikan saya ucapkan terima kasih.

@kondiak kondiak mentioned this pull request Oct 29, 2021
8 tasks
Copy link
Member

@wildanie12 wildanie12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saya rasa udah cukup bagus untuk penjelasan fundamentalnya tentang basic pemrograman berorientasi objek.

Mungkin @bellshade/docs-team perlu mengkoreksi beberapa typo yang ada saja.

@wildanie12 wildanie12 linked an issue Oct 30, 2021 that may be closed by this pull request
@mergify mergify bot requested a review from a team October 30, 2021 02:37
@mergify mergify bot requested review from a team October 30, 2021 04:22
Copy link

@Vins2106 Vins2106 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mergify mergify bot requested a review from a team October 30, 2021 05:10
@Vins2106 Vins2106 removed request for a team October 30, 2021 05:11
@mergify mergify bot requested review from a team October 30, 2021 05:12
Copy link
Member

@dhafitf dhafitf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Menurut KBBI, kata yang benar adalah pemrograman bukan pemograman.
Serta ada beberapa typo. Mohon diperbaiki.

basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/5_perulangan/README.md Show resolved Hide resolved
basics/5_perulangan/README.md Show resolved Hide resolved
basics/5_perulangan/README.md Show resolved Hide resolved
basics/5_perulangan/README.md Show resolved Hide resolved
basics/5_perulangan/README.md Show resolved Hide resolved
@mergify mergify bot requested a review from a team October 30, 2021 06:09
@wildanie12
Copy link
Member

Saya ambil alih om @kondiak. Saya pingin sedikit kontribusi juga ke pengenalan OOP ini juga lewat PR ini.
Sekalin saya fix typonya.
btw.. kebanyakan udah mantep om. 👍

@wildanie12 wildanie12 self-assigned this Oct 30, 2021
@kondiak
Copy link
Collaborator Author

kondiak commented Oct 30, 2021

Saya ambil alih om @kondiak. Saya pingin sedikit kontribusi juga ke pengenalan OOP ini juga lewat PR ini. Sekalin saya fix typonya. btw.. kebanyakan udah mantep om. 👍

maksud om @wildanie12 , perbaikannya om yang ngerjain gitu ya, sekalian sama lanjutan nya gak?, kalau iya saya titip nih class sama object, nanggung udah rampung, kalau mau di sih

@kondiak
Copy link
Collaborator Author

kondiak commented Nov 23, 2021

Bagus tuh om @wildanie12, cakeppppp

@mergify mergify bot requested review from azqilana and removed request for Vins2106, AdhyWiranto44, danz63, dipras, noukode, msazzuhair and yezki November 23, 2021 05:27
@kondiak
Copy link
Collaborator Author

kondiak commented Nov 23, 2021

Halo om @kondiak. Disini saya merombak beberapa hasil kerjaan saya dengan om Kondiak.. Karena saya rasa ada yang lebih baik lagi tentang fundamental metode pemrograman OOP. Jadi PR yang terakhir adalah hasil kerja antara saya dan om Kondiak.

Lalu saya rasa untuk kelas dan objek (Issue #108) lebih baik diikutkan untuk PR selanjutnya, om @kondiak jika mau bisa juga submit PR lagi untuk issue ini om.

Untuk #108 mungkin harus di perbaharui juga om, soalnya tema udah beda, kalau di awal bahas nya rumah makan, harus konsisten ke belakang rumah makan, soalnya kalau berubah di tiap sub materi, pemahaman akan paradigma OOP takutnya gak sampai

@wildanie12
Copy link
Member

@

Halo om @kondiak. Disini saya merombak beberapa hasil kerjaan saya dengan om Kondiak.. Karena saya rasa ada yang lebih baik lagi tentang fundamental metode pemrograman OOP. Jadi PR yang terakhir adalah hasil kerja antara saya dan om Kondiak.
Lalu saya rasa untuk kelas dan objek (Issue #108) lebih baik diikutkan untuk PR selanjutnya, om @kondiak jika mau bisa juga submit PR lagi untuk issue ini om.

Untuk #108 mungkin harus di perbaharui juga om, soalnya tema udah beda, kalau di awal bahas nya rumah makan, harus konsisten ke belakang rumah makan, soalnya kalau berubah di tiap sub materi, pemahaman akan paradigma OOP takutnya gak sampai

Tentu om.. Issue akan di evaluasi lagi dan diperbarui. Sebelumnya mohoh maaf yah klo sebelumnya aktifitas di bellshade PHP agak menurun, tapi sekarang mulai ditingkatkan lagi. Lalu maaf juga yah klo topik analoginya saya rubah. Semuanya harapannya bisa mempermudah bagi yang menggunakan repo ini buat belajar.

@kondiak
Copy link
Collaborator Author

kondiak commented Nov 23, 2021

@

Halo om @kondiak. Disini saya merombak beberapa hasil kerjaan saya dengan om Kondiak.. Karena saya rasa ada yang lebih baik lagi tentang fundamental metode pemrograman OOP. Jadi PR yang terakhir adalah hasil kerja antara saya dan om Kondiak.
Lalu saya rasa untuk kelas dan objek (Issue #108) lebih baik diikutkan untuk PR selanjutnya, om @kondiak jika mau bisa juga submit PR lagi untuk issue ini om.

Untuk #108 mungkin harus di perbaharui juga om, soalnya tema udah beda, kalau di awal bahas nya rumah makan, harus konsisten ke belakang rumah makan, soalnya kalau berubah di tiap sub materi, pemahaman akan paradigma OOP takutnya gak sampai

Tentu om.. Issue akan di evaluasi lagi dan diperbarui. Sebelumnya mohoh maaf yah klo sebelumnya aktifitas di bellshade PHP agak menurun, tapi sekarang mulai ditingkatkan lagi. Lalu maaf juga yah klo topik analoginya saya rubah. Semuanya harapannya bisa mempermudah bagi yang menggunakan repo ini buat belajar.

Perubahan untuk jadi lebih baik pasti.!!!.

Copy link
Contributor

@azqilana azqilana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Saya sangat suka dengan analogi yang ada tp ada sedikit perubahan yang saya ajukan pada analogi yang sudah di ketik tolong di pertimbangkan untuk melakukan perubahan sesuai request dari saya om @wildanie12

basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
basics/14_oop_dasar/README.md Outdated Show resolved Hide resolved
@mergify mergify bot requested a review from a team November 23, 2021 13:54
Co-authored-by: Muhammad Azqilana <mazqulunu@gmail.com>
Copy link
Collaborator

@feri-irawan feri-irawan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sip om, LGTM 👍🏻

@mergify mergify bot requested a review from a team November 23, 2021 14:36
@wildanie12
Copy link
Member

Saya sudah memperbarui dokumentasi pada file markdown README.md. Semua koreksi request diterapkan kecuali yang kedua tentang ide paragraf analogi metode prosedural. Namun paragraf itu juga diperbarui dan diperingkas agar bisa lebih mudah dibaca. Terakhir saya koreksi juga kata diasingkan, saya rasa disendirikan lebih sering umum dipake.

Silahkan direview lagi @bellshade/php-team @azqilana

@azqilana
Copy link
Contributor

Saya sudah memperbarui dokumentasi pada file markdown README.md. Semua koreksi request diterapkan kecuali yang kedua tentang ide paragraf analogi metode prosedural. Namun paragraf itu juga diperbarui dan diperingkas agar bisa lebih mudah dibaca. Terakhir saya koreksi juga kata diasingkan, saya rasa disendirikan lebih sering umum dipake.

Silahkan direview lagi @bellshade/php-team @azqilana

oke om saysa periksa

Copy link
Contributor

@azqilana azqilana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oke om lgtm

@mergify mergify bot requested a review from a team November 26, 2021 07:46
@mergify
Copy link
Contributor

mergify bot commented Nov 27, 2021

terima kasih atas kontribusinya @kondiak !

@mergify mergify bot merged commit ab0cdce into bellshade:main Nov 27, 2021
@kondiak kondiak deleted the Cesium branch December 9, 2021 01:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1. Pengenalan paradigma OOP
7 participants