Начальная программа C

Я предполагаю, что под уборщиком вы подразумеваете "элегантный и модный".

Это выглядит шикарно для меня:

#include <stdio.h>
#include <cs50.h>

int main(void) {
    // get user input and set to variable
    printf("Height: ");
    int height = GetInt();
    int hm2 = height - 2;

    int j, k;
    for(int i = 1 ; i < height; i++) {
        // create n-1 spaces
        for(k = hm2; k > (i-1); k--)
            printf("%c", ' ');      

        // create n+1 hash tags
        for(j = 0; j < (i+1); j++)
            printf("#");

        printf("\n");
    }
    return 0;
}

Тем не менее, не слишком зацикливайтесь на причудливом коде. Хотя хорошо, если вы работаете с другими или сами с собой. Ваш пример выглядел нормально.

Теперь, что касается оптимизации, об этом стоит побеспокоиться. Просто помните, что чрезмерная оптимизация потенциально может сломать вашу программу.

На всеобщее обозрение: вот как выглядит «стиль и отсутствие читабельности» :)

i = 0;
while (i++ < height*height)
    printf ("%c%s", (i-1)/height < height-(i-1)%height-1 ? ' ' : '#',
    i % height ? "" : "\n");

Практически невозможно увидеть, что делает код, не запустив его. Если должно быть последующее упражнение, его трудно переписать, чтобы сформировать, скажем, пирамиду с четными сторонами. Я бы, вероятно, выбросил это и начал снова с основ, прежде чем снова объединить его в маленького монстра, такого как этот.

(позже) Немного более аккуратно поставить i++ в конце, поэтому два раза (i-1) обменивается на немного более сложный тест конца строки:

i = 0;
do
    printf ("%c%s", i/height < height-i%height-1 ? ' ' : '#',
    i % height==height-1 ? "\n" : "");
while (++i < height*height);

Я думаю, что под более чистым и лучшим способом вы подразумеваете идеальную форму прямоугольной треугольной пирамиды.
Для этого вы должны сделать следующее
Изменить

printf("Height: ");  

to

printf("Height: \n\n");  

и

for(i = 1 ; i < height; i++)  

to

for(i = 0 ; i < height; i++)   

Посмотрите пример вывода.

Вот предложение:

#include <stdio.h>
#include <cs50.h>

int main(void) {
    //initialize variables
    int height, hm2, j, k, i;

    printf("Height: \n");
    // Get user input
    height = GetInt();
    hm2 = height - 1;
    for(i = 0; i < height; i++) {
        // create n spaces
        for(k = hm2; k > i; k--)
            printf("%c", ' ');      

        // create n+1 hash tags
        for(j = 0; j < i+1; j++)
            printf("#");

        printf("\n");
    }
    return 0;
}

Результат, если пользователь ввел 5 для высоты:

Height: 
    #
   ##
  ###
 ####
#####

Пара вещей, которые я рассмотрел с этой версией кода:

- В C рекомендуется объявлять все переменные отдельно от присвоения им значения и присваивать значения позже. Некоторые компиляторы могут вызвать эту ошибку, если вы объявляете и присваиваете значение в цикле for: «ошибка: начальные объявления цикла for разрешены только в режиме C99». Эти изменения считаются с тем, что я предоставил.

//initialize variables                                                            
int height, hm2, j, k, i;

-Я добавил новую строку здесь

printf("Height: \n");

-Вместо hm2 = высота - 2 я изменил на:

hm2 = height - 1;

-Первый цикл, теперь мы даем значение i и устанавливаем его на 0, чтобы соответствовать другим изменениям, которые были сделаны:

for(i = 0; i < height; i++) {

-Для цикла создания n пробелов я изменил его на:

for(k = hm2; k > i; k--)

-Наконец удалены скобки (в данном случае не нужны) в последнем цикле for:

for(j = 0; j < i+1; j++)

Ваше здоровье