Write a Blog >>
ICPC 2021
Tue 18 - Thu 20 May 2021
co-located with ICSE 2021
Wed 19 May 2021 10:50 - 11:00 at ICPC Main Room - Refactoring Chair(s): Eunjong Choi

Code review plays an important role in software quality control. A typical review process would involve a careful check of a piece of code in an attempt to find defects and other quality issues/violations. One type of issues that may impact the quality of the software is code smells - i.e., bad programming practices that may lead to defects or maintenance issues. Yet, little is known about the extent to which code smells are identified during code reviews. To investigate the concept behind code smells identified in code reviews and what actions reviewers suggest and developers take in response to the identified smells, we conducted an empirical study of code smells in code reviews using the two most active OpenStack projects (Nova and Neutron). We manually checked 19,146 review comments obtained by keywords search and random selection, and got 1,190 smell-related reviews to study the causes of code smells and actions taken against the identified smells. Our analysis found that 1) code smells were not commonly identified in code reviews, 2) smells were usually caused by violation of coding conventions, 3) reviewers usually provided constructive feedback, including fixing (refactoring) recommendations to help developers remove smells, and 4) developers generally followed those recommendations and actioned the changes. Our results suggest that 1) developers should closely follow coding conventions in their projects to avoid introducing code smells, and 2) review-based detection of code smells is perceived to be a trustworthy approach by developers, mainly because reviews are context-sensitive (as reviewers are more aware of the context of the code given that they are part of the project’s development team).

Conference Day
Wed 19 May

Displayed time zone: Amsterdam, Berlin, Bern, Rome, Stockholm, Vienna change

10:40 - 11:00
RefactoringResearch at ICPC Main Room
Chair(s): Eunjong ChoiKyoto Institute of Technology
Understanding Architecture Erosion: The Practitioners' Perceptive
Ruiyin LiWuhan University, China; University of Groningen, The Netherlands, Peng LiangWuhan University, Mohamed Soliman, Paris AvgeriouUniversity of Groningen, The Netherlands
Pre-print Media Attached
Understanding Code Smell Detection via Code Review: A Study of the OpenStack CommunityIEEE/TCSE Distinguished Paper Award
Xiaofeng HanWuhan University, China, Amjed TahirMassey University, Peng LiangWuhan University, Steve CounsellBrunel University London, Yajing Luo
Pre-print Media Attached

Information for Participants
Wed 19 May 2021 10:40 - 11:00 at ICPC Main Room - Refactoring Chair(s): Eunjong Choi
Info for room ICPC Main Room:

Go directly to this room on Clowdr