친구의 조언 따라 코드 수정하기

M E M O2018.08.29 14:54

중국 친구가 내 코드를 보고 문제를 알려 주었다.

코드 스타일에 대해 고민하며 만든 적이 없기에, 한번 고쳐 보았다.

  • 문제점

1. assert/ensure point is not null    (포인터에 Null 값 넣은 것)                                  
2. keys like name or nickname should be put in macros. dont hard code it   (키 이름을 하드코딩)


Before

After

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
 
 
struct JsonKey
{
    const QString Name{"name"}, Nickname{"nickname"};
    const QString Content{"content"}, Subtype{"subtype"}, Status{"Status"};
};
 
 
QJsonObject JsonManager::createMediumJSON()
{
   Node<JsonMediumList *> *tempMediumNode;
   tempMediumNode = ListMedium->first();
 
   QJsonObject rootObj,json;
   JsonKey jsonKey;
  
   json.insert(jsonKey.Name, tempMediumNode->getData()->getName());
   json.insert(jsonKey.Nickname, tempMediumNode->getData()->getNickname());
   json.insert(jsonKey.Content, tempMediumNode->getData()->getContent());
   json.insert(jsonKey.Subtype, tempMediumNode->getData()->getSubtype());
   json.insert(jsonKey.Status, recordStatus._PENDING);
  
   rootObj.insert("data",json);
 
   return rootObj;
 
}
cs


작성자

Posted by 비타오백

관련 글

댓글 영역

티스토리 툴바