xb4dc0d3

from rest_framework import serializer

# Bad (if you search "result" maybe the related result more than 100 occurence)
result = serializer.serialize(data_mahasiswa, 100)

# Good
result_json_mahasiswa = serializer.serialize(data_mahasiswa, 100)

xb4dc0d3

# Bad
def getPotentialEnergy(mass, height):
  return mass * 9.8 * height

# Good
GRAVITATIONAL_CONSTANT = 9.8
def getPotentialEnergy(mass, height):
  return mass * GRAVITATIONAL_CONSTANT * height

xb4dc0d3

# Bad (not consistent variable and function name)
@api_view(["GET"])
@permission_classes([IsAuthenticated])
def get_all_mahasiswa(request):
  mahasiswa = Mahasiswa.objects.all()
  json_mahasiswa = MahasiswaDetailSerializer(mahasiswa, many=True)
  return Response(json_mahasiswa.data)
  
@api_view(["GET"])
@permission_classes([IsAuthenticated])

xb4dc0d3

'''
Classic Mine Sweeper Game 
'''

# Bad
def getThem():
  lists = []
  for i in theList:
    if i[0] = 4:
      lists.add(i)

xb4dc0d3

# Bad
def create_user_mahasiswa(self, name, username, student_no, telephone_num, gender):
    self.name = name
    self.username = username
    self.student_no = student_no
    self.telephone_num = telephone_num
    self.gender = gender
    
    
# Good

xb4dc0d3

def invitation(user):
  # ...

# Bad
def invite_supervisor_lembaga(lists_supervisor_lembaga: List[SupervisorLembaga]):
    '''
    Filter supervisor lembaga yang aktif dan kirim undangan
    '''
    for supevisor_lembaga in lists_supervisor_lembaga:
        if supervisor_lembaga.active:

xb4dc0d3

# Bad
class Invitation:
    def handle(self) -> None:
        # Do something...

message = Invitation()
# handle untuk apa?
message.handle()

# Good

xb4dc0d3

# Bad
name = "Cristiano Rolando"
def split_into_first_and_last_name():
  global name
  name = name.split()

# ketika fungsi dieksusi akan merusak dari global variabel name
split_into_first_and_last_name()
print(name)

xb4dc0d3

# Bad
class LembagaViewByNama(RetrieveAPIView):

    permission_classes = (IsAuthenticated, )
    authentication_class = JSONWebTokenAuthentication

    def get(self, request, nama_lembaga):
        nama = Lembaga.objects.get(nama_lembaga=nama_lembaga)
        if nama != None: # jika semakin banyak conditional maka akan semakin sulit
            serializer = LembagaSerializer(nama, context=serializer_context)

xb4dc0d3

#!/bin/sh
# From https://www.hiroom2.com/2017/09/24/parrotsec-3-8-docker-engine-en/

set -e

# Install dependencies.
sudo apt install -y curl apt-transport-https \
     software-properties-common ca-certificates

# Install docker.