DEV Community

Cover image for Code Smell 236 - Unwrapped Lines
Maxi Contieri
Maxi Contieri

Posted on • Originally published at maximilianocontieri.com

3

Code Smell 236 - Unwrapped Lines

Formatting plays a crucial role in creating code that is not only functional but also readable and maintainable. In this article, I will shed light on the code smell associated with neglecting proper line wrapping and explore why it's considered bad practice, too long, isn't it?

TL;DR: Extract and wrap your code

Problems

  • Hard to read code (especially in small devices)

  • Demeter's Law violation

Solutions

  1. Wrap the code to at most 75 characters

  2. Break and concatenate long strings. Compilers optimize them. Thinking there's a speed penalty is a premature optimization mistake.

  3. Don't use Abbreviations

Context

Unwrapped code formatting refers to the absence of line breaks or appropriate indentation.

It yields excessively long lines of code that extend beyond the typical width of a code editor or mobile screen.

While writing my last book, people read the code on small devices so line wrapping is critical.

Sample Code

Wrong

struct ExtraterrestrialSignal { signal_frequency: f64, signal_strength: f64, signal_duration: f64, }

fn perform_signal_processing_and_analysis(extraterrestrial_signal: &ExtraterrestrialSignal,
) {
    println!(
        "Extraterrestrial Signal processed - Frequency: {} Hz, Strength: {}, Duration: {} seconds", extraterrestrial_signal.signal_frequency,  extraterrestrial_signal.signal_strength, extraterrestrial_signal.signal_duration);

    if extraterrestrial_signal.signal_strength > 0.8 && extraterrestrial_signal.signal_duration > 10.0
    {
        println!("Potential Extraterrestrial Signal of interest!");
    } else {
        println!("Signal does not meet criteria for further investigation.");
    }
}
Enter fullscreen mode Exit fullscreen mode

Right

struct ExtraterrestrialSignal {
    signal_frequency: f64,
    signal_strength: f64,
    signal_duration: f64,
}

fn perform_signal_processing_and_analysis(
    extraterrestrial_signal: &ExtraterrestrialSignal,
) {
    println!(
        "Extraterrestrial Signal processed" + 
          "- Frequency: {} Hz, Strength: {}, Duration: {} seconds",
        extraterrestrial_signal.signal_frequency,
        extraterrestrial_signal.signal_strength,
        extraterrestrial_signal.signal_duration
    );

    if extraterrestrial_signal.signal_strength > 0.8
        && extraterrestrial_signal.signal_duration > 10.0
    {
        println!("Potential Extraterrestrial" 
            + " Signal of interest!");
    } else {
        println!("Signal does not meet criteria" 
            + " for further investigation.");
    }
}
Enter fullscreen mode Exit fullscreen mode

Detection

[X] Automatic

This is a formatting and syntactic smell

Tags

  • Formatting

Level

[X] Beginner

AI Assistants

AI assistants sometimes bring short code and not real production code.

You can use the assistants to wrap and format your code.

Conclusion

Ensuring readability is consistently paramount, with various facets warranting consideration.

When lines of code are excessively long, developers may find it challenging to understand the structure and flow of the code.

Relations

More Info

Clean Code Cookbook

Disclaimer

Code Smells are my opinion.

Credits

Photo by Olesya Yemets on Unsplash


The objective of cleaning is not just to clean, but to feel happiness living within that environment.

Marie Kondo


This article is part of the CodeSmell Series.

Build gen AI apps that run anywhere with MongoDB Atlas

Build gen AI apps that run anywhere with MongoDB Atlas

MongoDB Atlas bundles vector search and a flexible document model so developers can build, scale, and run gen AI apps without juggling multiple databases. From LLM to semantic search, Atlas streamlines AI architecture. Start free today.

Start Free

Top comments (0)

DevCycle image

Ship Faster, Stay Flexible.

DevCycle is the first feature flag platform with OpenFeature built-in to every open source SDK, designed to help developers ship faster while avoiding vendor-lock in.

Start shipping

👋 Kindness is contagious

Explore this compelling article, highly praised by the collaborative DEV Community. All developers, whether just starting out or already experienced, are invited to share insights and grow our collective expertise.

A quick “thank you” can lift someone’s spirits—drop your kudos in the comments!

On DEV, sharing experiences sparks innovation and strengthens our connections. If this post resonated with you, a brief note of appreciation goes a long way.

Get Started