баг или я что-то не понимаю?

если я не поставлю в функцию printf(«bla bla bla\n») то она и не вызывается вовсе

А если сделать return s_fl?

встаил в функцию

    return (int)s_fl;

вызываю так:

    if (!bubble_sort()) {
        printf("bad\n");
    }

в выводе есть bad массив не отсортирован!!!!!!!

Что такое N_CHRCMP?

флаг и должен быть 0, сделал так

return i;

вызываю так:

    if (bubble_sort()) {
        printf("good\n");
    } else {
        printf("bad\n");
    }

хотя нет, флаг должен быть 1

номер символа в строке. по которому сортируем строки в массиве

если 0, то по первому символу, если 1 или 2 или n. то сооствественно по нному.

навверху есть вот это:

#define N_CHRCMP 0

Инициализируй s_fl нулём при объявлении.

добавил в первый пост ссылку на весь файл на всякий случай,

программа читает файл со строками, сортирует их и выводит все строки. в которых буква, по которой сортируются строки дублируется.

Попытался осознать алгоритм, поломал мозг.

Что-то это очень странный пузырёк какой-то. Я даже не удивляюсь, что этот код ничего не сортирует.

все, понял, он не инициализирован, по видимому его значение менялось (он был в другом месте). когда я делал printf, спасибо, это не баг, это фича.

А, понял, он его сортирует просто лишние раз стописят. Нормально.

отсекает лишние итерации, если таковые имеются, на википедии прочитал

читабельнее было бы так

s_fl == 1

Лабу что ли делаешь? Зачем extern, если у тебя переменные глобальные?

Да я знаю, что пузырьки разные бывают. Это я типа (неудачно) попытался посеять сомнения. Для пользы дела.

Ну, зачем плохому учить. Это не сильно лучше. Пусть через аргументы уж передаёт.

типа того.

я вообще-то не очень понимаю, что такое extern, сначала без них было, потом когда поудалял все отладочные printf, то программа перестала работать, я подумал, что это магия и начал шаманить :)

там через аргументы многовато передавать пришлось бы, вот и поделал все глобальными

Это если в другом объектном файле определена переменная (или функция или ещё что). А у тебя в том же.

Кстати я попался на твое сравнение - инициализировать надо единичкой. ты так и сделал?

угу, единичкой и сравнение попраил, не s_fl != 0, а s_fl == 1

На плюсах это выглядит так

void buble_sort(int *array, int counter){

bool sorted=false;
  while (!sorted){
	sorted=true;
	for (int i = 0; i < (counter-1); i++) {
	
		if ( array[i] > array[i+1]) 
		{
			int tmp = array[i];
			array[i] = array[i+1];
			array[i+1] = tmp;
			sorted=false;
		}
	}
   }

}

может я что-то не так делаю?

Читай на досуге K&R

На плюсах

4.2! Это — чистый си.

в плюсах, есть встроенные средства сортировки, хотя в си возможно тоже. где-то натыкался

дак и потом во внешнем цикле тоже желательно ставить счетчик, поскольку за счет него можно уменьшить количество внутренних итераций, а флаг количество внутренних не уменьшает, в то время, как после каждого прохода количество внутренних должно редуцироваться, ну не должно (желательно), просто один проход становится лишним, поскольку самый большой элемент уже наверху.

так что как не крути, а плюсы не нужны :)

bool в Си? ну-ну...

qsort в си. std::sort в Си++. Но ни там ни там оно не реализовано, как buble_sort.

Не нукай!

#include <stdbool.h>

мой пример на Си переписать - 2 буковки поменять :)

ну это понятно.

бгг...

это вообще-то _Bool, в С++ bool это встроенный тип - ему не нужны никакие заголовки. да и ключ -std=C99 нужно скормить компайлеру.

С++ bool это встроенный тип - ему не нужны никакие заголовки

И? Это не делает тот код кодом на С++.

ключ -std=C99 нужно скормить компайлеру

ключ -std=gnu99 нужен лишь потому, что ты итератор цикла объявил в цикле, а не заранее.