4.4 Review

An empirical study of the impact of modern code review practices on software quality

Journal

EMPIRICAL SOFTWARE ENGINEERING
Volume 21, Issue 5, Pages 2146-2189

Publisher

SPRINGER
DOI: 10.1007/s10664-015-9381-9

Keywords

Code review; Software quality

Funding

  1. Natural Sciences and Engineering Research Council of Canada (NSERC)
  2. JSPS KAKENHI Grant [24680003, 25540026]
  3. Grants-in-Aid for Scientific Research [24680003, 25540026] Funding Source: KAKEN

Ask authors/readers for more resources

Software code review, i.e., the practice of having other team members critique changes to a software system, is a well-established best practice in both open source and proprietary software domains. Prior work has shown that formal code inspections tend to improve the quality of delivered software. However, the formal code inspection process mandates strict review criteria (e.g., in-person meetings and reviewer checklists) to ensure a base level of review quality, while the modern, lightweight code reviewing process does not. Although recent work explores the modern code review process, little is known about the relationship between modern code review practices and long-term software quality. Hence, in this paper, we study the relationship between post-release defects (a popular proxy for long-term software quality) and: (1) code review coverage, i.e., the proportion of changes that have been code reviewed, (2) code review participation, i.e., the degree of reviewer involvement in the code review process, and (3) code reviewer expertise, i.e., the level of domain-specific expertise of the code reviewers. Through a case study of the Qt, VTK, and ITK projects, we find that code review coverage, participation, and expertise share a significant link with software quality. Hence, our results empirically confirm the intuition that poorly-reviewed code has a negative impact on software quality in large systems using modern reviewing tools.

Authors

I am an author on this paper
Click your name to claim this paper and add it to your profile.

Reviews

Primary Rating

4.4
Not enough ratings

Secondary Ratings

Novelty
-
Significance
-
Scientific rigor
-
Rate this paper

Recommended

Article Computer Science, Software Engineering

Towards reducing the time needed for load testing

Hammam M. AlGhamdi, Cor-Paul Bezemer, Weiyi Shang, Ahmed E. Hassan, Parminder Flora

Summary: Load testing of large-scale systems is crucial but time-consuming. Previous work has successfully reduced execution time but missed important information by ignoring combinations of performance metrics. In this paper, a new approach is proposed to detect when new combinations of observed performance metrics are no longer exercised, thereby reducing the execution time of load tests.

JOURNAL OF SOFTWARE-EVOLUTION AND PROCESS (2023)

Article Chemistry, Analytical

Implementation of a Miniaturized Planar Tri-Band Microstrip Patch Antenna for Wireless Sensors in Mobile Applications

Ahmed Saad Elkorany, Alyaa Nehru Mousa, Sarosh Ahmad, Demyana Adel Saleeb, Adnan Ghaffar, Mohammad Soruri, Mariana Dalarsson, Mohammad Alibakhshikenari, Ernesto Limiti

Summary: In this paper, a planar patch antenna for mobile communication applications operating at multiple frequencies is proposed. The antenna demonstrates good performance and can be utilized in various mobile applications such as digital communication systems, WiMAX, and WLAN.

SENSORS (2022)

Article Infectious Diseases

Genotyping of foot-and-mouth disease viruses collected in Sudan between 2009 and 2018

Yazeed A. Raouf, Jemma Wadsworth, Abdelghani Bin-Tarif, Ashley R. Gray, Mohammed Habiela, Ameera A. Almutalb, Hanan Yousif, Maysa Ragab, Wefag Alfouz, Nussiba H. Ahmed, Inas Ibrahim, Ahmed M. Hassan, Markos Tibbo, Ahmad M. Almajali, Cornelis van Maanen, Nicholas A. Lyons, Donald P. King, Nick J. Knowles

Summary: This study aims to understand the epidemiological patterns of foot-and-mouth disease (FMD) in Sudan and its connections to neighboring countries by analyzing the genetic sequences of FMD viruses collected from samples in Sudan. The study found that FMDV lineages were maintained within Sudan and showed connections to FMD outbreaks in neighboring countries in East and North Africa. This study highlights the importance of continued FMD surveillance and improving our understanding of epidemiological risks in the region.

TRANSBOUNDARY AND EMERGING DISEASES (2022)

Article Biochemical Research Methods

Evaluation of resonant scanning as a high-speed imaging technique for two-photon imaging of cortical vasculature

Annie Zhou, Alankrit Tomar, Ahmed M. Hassan, Andrew K. Dunn, Shaun A. Engelmann, Samuel A. Mihelic

Summary: We present a simple and affordable two-photon microscope design that features both galvo-galvo and resonant-galvo scanning capabilities. We quantitatively compare the signal-to-noise ratios and imaging speeds of these two scanning modes in murine neurovascular imaging. Under shot-noise limited detection, both scanning modes perform as expected and achieve comparable signal-to-noise ratios. Resonant-galvo scanning can achieve the desired signal-to-noise ratios in less acquisition time when higher excitation power is used. Under equal excitation power and total pixel dwell time, galvo-galvo scanning outperforms resonant-galvo scanning in image quality when detection deviates from being shot-noise limited.

BIOMEDICAL OPTICS EXPRESS (2022)

Article Engineering, Chemical

Improving the Dynamic Performance of a Variable Speed DFIG for Energy Conversion Purposes Using an Effective Control System

Mahmoud A. Mossa, Mahmoud K. Abdelhamid, Ahmed A. Hassan, Nicola Bianchi

Summary: The present paper introduces an effective control system to enhance the dynamics of a doubly fed induction generator (DFIG). The control system is evaluated and compared with other control techniques, including stator voltage-oriented control (SVOC), model predictive current control (MPCC), and model predictive direct torque control (MPDTC). The results show that the proposed predictive voltage control (PVC) algorithm outperforms the other techniques in terms of dynamic response, control structure simplicity, and computational burden reduction.

PROCESSES (2022)

Article Medicine, General & Internal

Rapid and Reliable Detection of SARS-CoV-2 Using Direct RT-LAMP

Sherif A. El-Kafrawy, Mai M. El-Daly, Ahmed M. Hassan, Steve M. Harakeh, Thamir A. Alandijany, Esam Azhar

Summary: In this study, a rapid, simple, and sensitive RT-LAMP assay for the detection of SARS-CoV-2 in clinical samples was established. The assay is suitable for point of care detection in public hospitals, medical centers in rural areas, and in transportation hubs.

DIAGNOSTICS (2022)

Article Management

Corporate internet reporting in Egypt: a pre- and peri-uprising analysis

Ahmed Hassan, Mohamed Elmaghrabi, Bruce Burton, Theresa Dunne

Summary: The purpose of this study is to provide a detailed descriptive account and analysis of corporate internet reporting (CIR) practices among non-financial companies listed on the Egyptian Exchange (EGX) at two points in time - December 2010 (pre) and December 2013 (peri) political and social unrest in Egypt. The findings suggest that listed companies in Egypt have embraced internet as a disclosure channel, with significant increase in the extent of these practices over the investigated period. The variations in CIR practices were dependent on changing institutional actors and the time factor was identified as particularly important for inducing diffusion of corporate practices during periods of institutional change.

INTERNATIONAL JOURNAL OF ORGANIZATIONAL ANALYSIS (2023)

Article Computer Science, Software Engineering

Upstream bug management in Linux distributions An empirical study of Debian and Fedora practices

Jiahuei Lin, Haoxiang Zhang, Bram Adams, Ahmed E. Hassan

Summary: This paper empirically studies how high-severity bugs are fixed in upstream packages for two Linux distributions, Debian and Fedora. The results show that most bugs in Debian do not have explicit information about being fixed, and upstream fixed bugs require users to wait longer and provide more information compared to fixing them locally. Additionally, the number of bug comment links and the similarity score between upstream and distribution bug reports are important factors for the likelihood of a bug being fixed upstream. Traceability tools are needed to make upstream fixes easier and lower the cost of upstream bug management.

EMPIRICAL SOFTWARE ENGINEERING (2022)

Article Virology

A potential association between obesity and reduced effectiveness of COVID-19 vaccine-induced neutralizing humoral immunity

Arwa A. Faizo, Fadi S. Qashqari, Sherif A. El-Kafrawy, Osamah Barasheed, Majed N. Almashjary, Mohammed Alfelali, Asma A. Bawazir, Boshra M. Albarakati, Soud A. Khayyat, Ahmed M. Hassan, Thamir A. Alandijany, Esam Azhar

Summary: This study investigated the effectiveness of COVID-19 vaccines in individuals with different degrees of obesity and found a slight reduction in the rate and titer of neutralizing antibodies among obese individuals. This suggests that obesity may affect the effectiveness of COVID-19 vaccines.

JOURNAL OF MEDICAL VIROLOGY (2023)

Article Computer Science, Software Engineering

On the Discoverability of npm Vulnerabilities in Node.js Projects

Mahmoud Alfadel, Diego Elias Costa, Emad Shihab, Bram Adams

Summary: The reliance on vulnerable dependencies poses a significant threat to software systems. Many dependency vulnerabilities remain undisclosed for a long time until they are discovered and publicly known. In our large-scale empirical study of 6,546 Node.js applications, we found that the majority of the affected applications rely on undisclosed vulnerabilities, but a significant percentage (4.63%) are exposed to dependencies with public vulnerabilities. The main reason for the exposure to public vulnerabilities is the lack of dependency updates and failure to patch available fixes by application maintainers. Additionally, we found that applications remain affected by public vulnerabilities for an average of 103 days. To address these issues, we developed DepReveal, a tool that helps developers better understand vulnerabilities in their application dependencies and plan project maintenance.

ACM TRANSACTIONS ON SOFTWARE ENGINEERING AND METHODOLOGY (2023)

Article Computer Science, Software Engineering

A Grounded Theory of Cross-Community SECOs: Feedback Diversity Versus Synchronization

Armstrong Foundjem, Ellis E. Eghan, Bram Adams

Summary: This paper performs a socio-technical analysis of cross-community collaboration in the OpenStack SECO. The study finds that distributors play a significant role in the sustainability of the SECO, with diversity and synchronization being important aspects.

IEEE TRANSACTIONS ON SOFTWARE ENGINEERING (2023)

Article Computer Science, Information Systems

Breast Cancer Diagnosis Using Support Vector Machines Optimized by Whale Optimization and Dragonfly Algorithms

Ahmed S. Elkorany, Mohamed Marey, Khaled M. Almustafa, Zeinab F. Elsharkawy

Summary: In this paper, two automated breast cancer classification approaches based on the hybridization of the Whale Optimization Algorithm and Dragonfly Algorithm with Radial Basis Function Kernel Support Vector Machines were proposed. The effectiveness of these approaches was tested on two breast cancer databases and compared to other classifiers. The results showed improved performance compared to previous classification methods.

IEEE ACCESS (2022)

Article Chemistry, Physical

One-step construction of Y, C, and O tridoped g-C3N4 as a bifunctional photocatalyst for H2 evolution and organic pollutant degradation under visible light irradiation

Ahmed E. Hassan, Mai S. A. Hussien, Mohamed Hammad Elsayed, Mohamed Gamal Mohamed, Shiao-Wei Kuo, Ho-Hsiu Chou, Ibrahim S. Yahia, Genxinag Wang, Zhenhai Wen

Summary: In this study, a novel type of tridoped g-C3N4 bifunctional photocatalyst was prepared via a simple and low-cost thermal polymerization technique. The photocatalyst exhibited a significant enhancement in photocatalytic redox efficiency for H-2 production and degradation of pollutants under visible light. The study also investigated the influence of Y3+ concentration on the photocatalytic performance and electronic structure of the photocatalyst.

SUSTAINABLE ENERGY & FUELS (2022)

Article Construction & Building Technology

Mandatory Policy, Innovations and the Renewable Energy Debate: A Case Study on Building Integrated Photovoltaics

Daniel Efurosibina Attoye, Kheira Anissa Tabet Aoul, Ahmed Hassan

Summary: This study investigates the challenges of adopting innovations in buildings and renewable energy technology, highlighting the conflicts between the need for innovation and the financial burden and complex adoption processes. Through interviews with residents in the UAE, the study focuses on the limited adoption of Building Integrated Photovoltaics (BIPV) and identifies multiple barriers to its adoption. The findings also reveal a debate between proponents of mandatory policies for innovation adoption and those who argue that these policies are ineffective due to existing barriers. To address this debate, the study proposes a systematic approach with specific drivers and supporting policies to guide a stakeholder-driven renewable energy transition.

BUILDINGS (2022)

Article Computer Science, Information Systems

Detection of Kidney Cancer Using Circularly Polarized Patch Antenna Array

Demyana A. Saleeb, Rehab M. Helmy, Nihal F. F. Areed, Mohamed Marey, Khaled Mohamad Almustafa, Ahmed S. Elkorany

Summary: This paper proposes the use of a circularly polarized patch antenna array and microwave techniques to detect kidney cancer. The method is safe, compact, fast, inexpensive, and non-invasive, with no ionizing radiation during measurement.

IEEE ACCESS (2022)

No Data Available