DEV Community

Cover image for Code Smell 244 - Incomplete Error information
Maxi Contieri
Maxi Contieri

Posted on • Originally published at maximilianocontieri.com

4 1 1 1 1

Code Smell 244 - Incomplete Error information

You show an error and provide no useful information

TL;DR: Help yourself and others with correction information

Problems

  • Debugging and maintenance challenge.

  • Fail Fast Principle violation

  • Debugging complex situations.

Solutions

  1. Add all the relevant information to solve the solution

Context

When you are reporting an error, either via an information text in the UI, by processing an API request, or by creating a test assertion, you need to provide an exit (a possible solution).

This is very relevant when dealing with complex scenarios, large objects, or arrays with minimal mistakes.

Sample Code

Wrong

VALID_COLUMNS = ['name', 'gender', 'email']

def process_API_information(data):
    invalid_columns = []
    for column in data.keys():
        if column not in VALID_COLUMNS:
            invalid_columns.append(column)

    assert not invalid_columns, "Invalid columns detected."  
    # No details were provided about which columns are invalid


data = {'name': 'John', 'gender': 'Pangender', 
        'age': 47, 'email': 'john@example.com'}
process_API_information(data)
Enter fullscreen mode Exit fullscreen mode

Right

VALID_COLUMNS = ['name', 'gender', 'email']

def process_API_information(data):
    invalid_columns = [
        column for column in data.keys() if column not in VALID_COLUMNS
    ]

    if invalid_columns:
        raise ValueError(
            f"Invalid columns detected: {', '.join(invalid_columns)}"
        )  # Shows WHICH columns are invalid

data = {'name': 'John', 'gender': 'Pangender', 
        'age': 47, 'email': 'john@example.com'}
process_API_information(data)
Enter fullscreen mode Exit fullscreen mode

Detection

[X] Semi-Automatic

This is a semantic smell. You can warn the developers on error texts that do not include variables.

Tags

  • Errors

Level

[X] Beginner

AI Assistants

AI assistants usually miss this kind of help and provide hardcoded error messages.

Conclusion

You need to always think about how to help your end users.

It might be yourself.

Relations

Disclaimer

Code Smells are my opinion.

Credits

Photo by Brett Jordan on Unsplash


Information shared by an object might or might not be part of the structure of that object. That is, the object might compute the information, or it might delegate the request for information to another object.

Rebecca Wirfs Brooks


This article is part of the CodeSmell Series.

Image of Timescale

Timescale – the developer's data platform for modern apps, built on PostgreSQL

Timescale Cloud is PostgreSQL optimized for speed, scale, and performance. Over 3 million IoT, AI, crypto, and dev tool apps are powered by Timescale. Try it free today! No credit card required.

Try free

Top comments (0)

Billboard image

The Next Generation Developer Platform

Coherence is the first Platform-as-a-Service you can control. Unlike "black-box" platforms that are opinionated about the infra you can deploy, Coherence is powered by CNC, the open-source IaC framework, which offers limitless customization.

Learn more

👋 Kindness is contagious

Please leave a ❤️ or a friendly comment on this post if you found it helpful!

Okay