check logic not finished