At a glance, each developer seems to be working with a computer. But there is an invisible teamwork that decides not only the organization’s executive force but also code quality and personal growth. As I go through many teams and meet various colleagues, I think I’m playing a team sport. Even if they’re the same front-end developers they play different roles. Someone has fast hands, someone makes good design, someone is very good at making UI, someone is a good mentor, and someone acts as a mood maker. Even when making decisions, it’s almost like a fantasy to have everyone technically agree on something. Depending on the team’s situation, experience, capability, or someone’s influence, it is subject to change. Given the same problem, every team comes up with different solutions.

I think that’s why teamwork is important. It is the driving force to respond with flexibility and recover quickly even during a problem. I believe the basis of teamwork is a sense of security and trust.

An Environment Where an Individual is not Blamed

When there’s a problem, it’s easy to blame someone who was in charge of the work or the developer who wrote the code. But this is a very bad culture that hurts psychological safety. Moreover, if you end up thinking that it’s an individual’s fault or mistake, the team will experience the same problem again in the future because they’ll only remember ‘who’ caused the problem, and not the problem itself. They wouldn’t separate the problem from the person and wouldn’t go deeper into the problem itself. Then the team members will get anxious hoping that they won’t become the next target.

After solving a problem, the former head of the organization always said this to encourage us, “Don’t do anything if you want to avoid the problem. Legacy works well, so just leave it be and do not engage in new challenges. But we shouldn’t do that. We must continue to grow. If you have a problem, you can come up with measures to prevent it from happening again and try again.”

My former manager also started meetings where the whole team gathered to discuss solutions for a problem by saying, “We are not meeting to blame anyone and question them. Anyone here could have made this mistake and that’s why we’re meeting to come up with a solution to prevent the same mistake from happening again in our team.”

When it gets revealed that someone was the cause of the problem, that person will defend himself by instinct. What other people say about him will sound like criticism toward him and his abilities. So if a leader creates an atmosphere where the problem gets separated from the person, an individual will feel a sense of security, and the team will be able to solve the problem and establish preventive measures based on this.

A Retrospective Done with Your Head

A cool-headed retrospective done after separating the problem from the person is just like growing pains. It may be embarrassing from that person’s perspective, but there is a sense of security that the arrow is not directed at himself, so there’s a chance for the team to deal with the problem together and recover by transparently disclosing what has happened to solve the problem and to prevent it from happening again.

In Grab, an open retrospective is done in a few weeks after a hotfix. A post-report needs to be filled out in advance before the retrospective. It needs to include the time the problem was dealt with, the impact of the accident, the cause, and preventive measures with as much detail as possible. During the one-hour retrospective, dozens of developers gather around and ask each other sharp questions and suggest solutions, and display collective intelligence. There is no room for equivocation.

When I attended the retrospective for the first time, it was so straightforward that even someone who isn’t related to the incident would feel it was brutal. But as I kept on listening, I could see that everyone was there with the same purpose. This is not a hearing, but a strategy meeting. If you take out your emotions and retrospect with a cool head, you can grow together.

The Attitude to Grow Together

Whenever my old mentor talked about programming skills, many times he would speak with the word ‘team’ as the subject instead of an individual. The overall code quality or the level of code review follows the team’s average level of skill rather than the best developer in the team. So in order to better solve the problem, not only does one person need to be good, but the whole team needs to grow together. Developers who constantly hold study sessions and are eager to participate in them to share their knowledge must know this.

Rarely, there is a developer like this. As soon as he joins a team, he would point out all the wrong parts of the legacy, do code reviews only based on his own standards and criticize the other team members. There is a saying that legacy was the best option until yesterday. If you see something you can contribute to the team, I think it’s right for you to persuade your teammates and spread your knowledge to raise the team’s average. If you separate yourself from the team, nothing will be beneficial for both sides.

People can very well distinguish someone who wants to stand out alone from someone who is truly dedicated to the team. There is trust between people who treat each other with good intentions and kindness, while someone who wants to do things well alone harms teamwork. That’s why a senior developer once said that character is the most important thing he sees when hiring someone, and a famous coach once said that you should hire someone not based on their abilities, but based on their attitude.

UITableView는 당연히 사용할 줄 알아야 하는 필수적인 클래스인만큼 잘못 사용되는 경우도 잦은 클래스다. 컨텐츠가 화면에 다 담기지 않아 스크롤이 발생하는 레이아웃을 만들어야할 때 별 고민없이 테이블뷰가 사용되는 상황을 많이 봤다. 심지어 스크롤이 발생하지 않더라도 단지 같은 뷰가 여러번 반복된다는 이유로 테이블뷰를 사용하기도 한다. 하지만 테이블뷰를 사용하기로 쉽게 결정하기 전에, 정말 뷰를 재사용하는 컴포넌트가 필요한지 짚어봐야 한다.

테이블뷰의 장점: 구조화된 데이터 표현 및 메모리 효율

테이블뷰는 데이터가 일관된 구조를 가지고 있고 계층화되어 있을 때 가장 유용하다. iOS의 설정 앱이나 연락처 앱이 대표적인 예시다. 내비게이션 뷰컨트롤러와 함께 사용하면 계층적인 구조를 정말 잘 나타낼 수 있다. 그리고 뷰(UITableViewCell)를 재사용하기 때문에 보여줘야 할 데이터가 수 백, 수 천 개여도 화면을 채울 만큼만 인스턴스가 만들어지기 때문에 데이터의 사이즈가 커도 메모리 걱정을 하지 않아도 된다.

테이블뷰의 단점: 복잡한 코드 + ⍺

프로그래밍과 관련된 거의 모든 결정에는 트레이드오프가 발생하는 거처럼, 테이블뷰도 뚜렷한 단점이 있다. 뷰 인스턴스를 재사용하는 장점을 얻는 대신 구조가 더 복잡해지고 코드가 길어지고 버그에 취약해진다.

UITableView를 설정해주고, UITableViewCell을 상속하고, DataSource와 Delegate를 구현해야 한다. 또한 셀 인스턴스가 재사용되면서 발생할 수 있는 이상한 동작을 방지하기 위해 추가되는 코드가 소소하지만 꽤 많다. 셀에 필요한 UI 데이터(이미지 등)를 비동기로 불러와야 경우, 외부의 사건(유저 액션 등)으로 인해 셀의 UI가 바뀌어야 하는 경우 등등에 추가적인 검사 로직을 넣어줘야 한다. 게다가 뷰의 레이아웃이나 크기를 동적으로 바꾸거나 애니메이션을 넣는 간단한 일도 테이블뷰가 연관되면 잘 안되고 귀찮아질 때가 많다.

테이블뷰 필요없는 경우 🔴

글의 시작 부분에서 언급한 오용이란 장점을 제대로 누리지 못하는 상황을 말한다. 대표적으로 셀이 재사용될 필요가 거의 없거나 재사용이 안되는 경우다. 테이블 열의 갯수가 적으면 메모리 걱정을 할 필요가 없다. 또한 테이블뷰의 열이 10개인데 10개가 다 다른 종류의 셀이라면 아무것도 재사용되지 않는다. 굳이 테이블뷰를 쓸 이유가 없다.

테이블뷰 필요한 경우 🟢

반대로 테이블뷰를 꼭 써야하는 경우는 데이터가 많을 때다. ‘많다’를 판별하는 절대 기준이 있는건 아니다. 하지만 너무 보수적으로 잡을 필요는 없다고 생각한다.

고민이 필요한 경우 🟠

개발하다보면 중간지대의 상황도 자주 맞닥뜨린다. 데이터는 적당히 많으면서 셀 종류도 두어개, 많게는 10개 정도. 유저 인터랙션도 있고 그에 따른 상태 변화와 UI 변화도 있는 경우. 그럴땐 뷰 재사용이 꼭 필요한지, 아니면 간단하고 안전한 코드가 더 이득일지 판단을 내려야 한다. 경험적으로 두 페이지가 넘어가지 않는 경우나, 필요한 뷰들을 한꺼번에 생성해도 메인쓰레드가 블락되지 않는 정도라면 간결한 코드의 장점이 더 크다.

대안: UIStackView + UIScrollView

뷰 재사용이 필수적인 요구사항이 아니라면 스택뷰 단독이나 스택뷰+스크롤뷰 조합을 이용해 훨씬 간결한 코드로 UI를 만들수 있다. 이 방식은 뷰를 직접 참조하여 들고 있을 수 있어서 언제든 간단하게 변화를 줄수도 있고, 테이블뷰처럼 의무적으로 호출해야 하는 메서드도 없어서 실수도 덜하고 디버깅도 쉽다.

5월 17일부터 20일까지 오후 7시-8시 반, 4일 동안 진행하는 단기 스터디를 함께할 스터디원을 모집합니다.

주제

Building Mobile Apps at Scale을 읽고, 책에 나오는 사례를 위주로 각자의 실제 경험을 나누는 시간을 가지려고 합니다. 저자는 앱 개발을 하면서 겪을 수 있는 다양한 문제와 해결법 몇 가지를 알려주긴 하지만 너무 짧고 간단하게 설명하고 있는 부분들이 아쉬워서 스터디를 계획하게 됐습니다. 실제로 문제를 겪고 해결했던 사례를 공유하면서 더 자세히 파고 들어가보면 어떨까 합니다.

장소

온라인 화상미팅

참여 방법

요일별로 주제가 다르니 책을 읽고 나서 관심있는 주제나 관련 경험이 있는 주제에 해당하는 요일에 구글폼 제출해주시면 개별 연락드려서 참여 확정을 하겠습니다.

요일별 주제

Day 1(월요일): 앱의 특수성으로 인한 어려움 (세부 주제 및 신청 폼)
Day 2(화요일): 앱이 커지면서 발생하는 어려움 (세부 주제 및 신청 폼)
Day 3(수요일): 팀이 커지면서 발생하는 어려움 및 크로스 플랫폼 관련 (세부 주제 및 신청 폼)
Day 4(목요일): 앱을 고도화할 때 발생하는 어려움 (세부 주제 및 신청 폼)

TDD는 설계 방법론이 아니다 글을 읽고 쓴 글입니다.

TDD는 안해봤지만 1년 동안 진하게 유닛 테스트 추가하고 레거시 코드를 테스트 가능하게 리팩토링 하면서 느낀 점과 유사하다. 요약하자면

1. 테스트 가능한 코드는 강한 결합을 회피하고 의존성 역전 원칙만큼은 지키게 해준다.
2. 단일 책임 원칙과 인터페이스 분리 원칙은 알아서 챙겨야 한다.

작년에는 코드를 테스트 가능하게 만드는 것부터 시작했다. RIBs는 워낙 기본 구조가 잘게 분리되어 있고 각 요소가 단일 책임 원칙을 지키고 있어서 그에 맞춰 짜기만 해도 중요 로직은 어느정도 테스트 가능하다. MVC를 제외한 대부분의 아키텍처가 비슷할 것이다. 그래서 모바일 개발 환경에서는 아키텍처의 손이 뻗지 않은 그 외 15-20프로 정도의 코드에서만 개발자의 설계 역량이 중요한거 같다.

그런데 그 15-20프로의 코드가 매우 중요하다. 앱의 디자인이나 기능과 유사하게 코드도 템플릿화되어 쉽게 복사되고 퍼진다. 어느 정도 코드베이스가 쌓이면 완전히 새로운 설계를 하게 되는 경우는 드물고 기존에 있는 비슷한 기능을 하는 코드를 찾아 쓰게 된다. 그리고 대부분의 사람들은 기존의 코드를 손쉽게 복붙해서 가져간다.

잘못 설계된 공통 모듈이 그에 의존하고 있는 모듈 수십 개과 수십 명의 개발자에게 안 좋은 영향을 주는걸 봤고, 누군가가 집어 넣은 잘못 설계된 구조가 스멀스멀 퍼져나가는걸 목격했다. 불필요한 의존성이 덤으로 딸려오는 바람에 인터페이스 분리 원칙도 못지키게 되고 빌드 시간도 늘어난다. 또한 누군가가 만든 테스트 불가능한 구조가 다른 개발자들이 별 생각없이 가져가 쓰면서 점점 증식되기도 한다. 코드가 안좋아지는 것 뿐아니라 의미없는 코드 리뷰어가 강제되어 리뷰 받으려는 사람, 리뷰 해야하는 사람의 시간을 허비하게 만들었다.

그래서 공통 모듈을 만들거나, 새로운 구조를 도입할때는 앞으로 이 코드가 오랫동안 남아서 미칠 영향을 고려하여 특히나 책임감을 가지고 설계해야 한다. 구글은 코드 리뷰 대원칙에서 이와 같이 말한다. ‘개발자는 맡은 일을 완수하면서도 코드 품질을 개선시켜야 한다. 꾸준히 개선하지 않으면 품질은 절대 나아지지 않는다.’