Почему к сожалению? Это рулез! Там и другие еще вещи рулят...

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

>Почему к сожалению? Это рулез! Там и другие еще вещи рулят...

провоцирует написание "грязного кода".

Без навороченной IDE в таком исходнике искать нужную переменную что выковыривать изюм из булочек

Очень странно. Я вот так в жабке живу годами, без особо навороченных IDE. Если функции не очень большого размера - "все найдется" (с) "Яндекс". И как это провоцирует "грязный код"? Наоборот, для меня это стимулирует аккуратность, переменная объявляется, когда она реально нужна. Впрочем, в С вопрос с совместимостью (со старыми компиляторами) стоит остро, поэтому приходится пока забыть про эту фичу...

>Без навороченной IDE в таком исходнике искать нужную переменную что выковыривать изюм из булочек

если сделать грамотно, вот почищу код покажу - там ничего сложного нет, Я ж скрин то сделал - мол работа - а не покрасоваться чем либо, просто показать как работа происходит тут у нас.

Ладно, давайте забъем на стиль. Почти договорились, что он gnu - и о моей жалобе про пробелы тоже забудем. Что у нас с query?

а что - сказал же она освобождаеться после того как использована.

Это происходит внутри той же функции?

Дейстительно мало пробелов. Просто давно не видел код где между токенами вообще нет пробелов, как-то в глаза бросается. Думаю если там везде пробелы расставить и комментарию нормальный отступ дать, то и скобка не будет так заметна ;)

да конечно.

Альфекс, а вот у меня ещё запоздалый вопрос. Только сейчас в голову пришло. А зачем, собственно, было выходить из jed-а чтобы сделать скрин с помощью import???

PS. Я кажется уже начинаю понимать, почему ты по отдельному терминалу с емаксом на каждый файл запускаешь :-))

Очень может быть:)

Вот за такое просто убивать надо (тот коллега, который учил меня когда-то хорошим манерам в С/С++, так бы и поступил)! Это я без эмоций, образно:))). Почему она не локальная? Зачем искусственно снижать производительность?

"УЗНАЮ БРАТА КОЛЮ" (с) Ильф и Петров Праильно! Я тоже доооолго мутил всякую попсу... ну там иконки, окошки, бЫкгроунды пока в конце концов это не стало отжирать туеву хучу времени... и в конце концов пришел к фб-консоли невз№"%енных размеров . Кесарю - кесарево, кодеру - кодерово. Там кстати ктото про цвета ругнулся типа глаза ломит ? - Батенька, будете пальцевать пару ночей напролет красивые буковки пастельных цветов будут круто сливатся в серобуромалиновые полосы... А синтаксис ...... А для чего компилятор ? :))))))))))

блин ну сделал еще один Я скрин показал как Я работаю специально для тебя.

тогда просто ступил.

кто сказал что она не локальная? она локальная просто на скрине не видно.

-std=c99

#include <stdio.h>
#include <string.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>

int main( int argc, char **argv )
{
 for (int i=0;i<3;i++)  {
    char a[10]="qwerty";
  for (int i=0;i<3;i++) {
    char a[10]="йцукенг";
   for (int i=0;i<3;i++){
     char a[10]="абырвалг";
   if (i > 1)
       break;
   }
   printf("a[%i]=%c\n",i,a[i]);
  }
 }
 return 0;
}

for (int i=0;i<3;i++) - не прокатит 
int i; // один раз объявил и все - зачем так то?
for (i=0;i<3;i++)
 и тд

Епт!? И как оно отработает?

2Альпекс

А если будет: 
    char query[255];
Неужели работать будет хуже?

и работает.

да и зачем так ?

Массив локальный! char query[256] - и все. Никаких calloc (кстати, в Вашем случае даже malloc, теоретически, более эффективен - ну не нужно Вам обнуление массива, если следующий оператор sprintf).

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

Ну, это же надо постараться так написать. Мы же не будем стараться, правда?

ну блин, производительность сильно не упадет, есть другие задачи - решаю их, а такие мелочи блин.

Запостил скрин на лор - время потерял =) это точно.

Нет, коллега. Это вопрос принципиальный. Производительность падает в бесконечное кол-во раз - потому что выделение локального массива не стОит НИЧЕГО (и мне грустно Вам об этом сообщать). А выделение памяти стОит, и даже не очень мало. Так что это, наряду с snprintf - САМОЕ страшное в Вашем скриншоте (ну, если считать, что багу в SQL вы отловите при первом же прогоне).

>sS ну если тут говорить о читабельности, то это не читабельно не потому что переменные локальны в цикле, а потому что инденты сделаны через зад.

Дык специально показываю \КАК\ не нужно делать

BTW: вопросами стиля должна заниматься IDE

>for (int i=0;i<3;i++) - не прокатит

c Subj еще как прокатит ...

багу не отловил в SQL - кстати эта ф-ция вполне себе работает.

>> "вопросами стиля должна заниматься IDE"

Ну так emacs как раз отлично занимаецца вопросами стиля:

<Ctl+X> <H> <Ctl+Alt+\>

При биндингах по-умолчанию :-)

customer_id - строка или число? Вообще, я мускул давно не трогал - но то ли Оракл, то ли Мелкомягкий сервер - не любят чисел в кавычках... Хотя, при сравнении - может, и они догадаются привести типы. При присвоении (в смысле, INSERT), помню, были грабли...

Число. Там же в строке формата видно %d.

int там вроде - не помню , можно глянуть - но ничего работает кстати.

`customer_id` int(11) default '0',

там так.

Из %d никак не следует, какой тип данных реально используется в базе.

да там база такая что, вообщем не Я ее делал, жутко мне становиться все больше и больше когда deadline подходит =)

Тем более - пофиксите query:)

И очень прошу выяснить правила использования "тся"/"ться"/"цца":) Один, другой раз пропускаешь, но когда это становится системой - начинает напрягать...

>потому что выделение локального массива не стОит НИЧЕГО

это если omit frame pointer

А это что - единственная локальная переменная в этом куске кода?

> это так называемый ALGOL стиль - иногда бывает удобно ... хотя так уже действительно редко кто пишет
У меня на столе книга:
Н. Джехани "Программирование на языке Си" "Радио и Связь", 1988; Bell Labs, 1985.

Там всё так написано. Я по ней учил Си сто лет назад. И объявления переменных не в начале функции или хотя бы блока меня нервируют %)

Зато и помойки с ними нет...

блин, никогда ещё не видел столько флейма в скринах, сколько на скрины Альфекса =)

не надо рассказывать - было, да и тут флейм то нормальный;

кстати почти переписал чтобы на тредах работало

блин почему в виндах fork(), wait() - нету ? =(( убогость эти винды.

http://sources.redhat.com/pthreads-win32/

А почему никто не сказал, что в данном случае скобки после if'ов вовсе не нужны?

Ну, это было бы брюзжание второго порядка:) В некоторых coding standards (в некоторых конторах) применение скобок обязательно даже в таких случаях. Чтобы все было ровно, пострижено и посыпано песком...

А вот интересно, такое объявление переменной, это плохой стиль?

for( int i=0;i<100;i++){ .................. }

А вообще надо бы тоже сюда кусок своего кода запостить, мне критика понравилась!

кстати ANSI консрукции

if()
{

}
else
{

}


for(;;)
{

}


наиболее понятны и читабельны

А разве ANSI специфицирует правила написания текста на Си?

Рулят правила Kernel Style, да и они более рациональны - более читабельно и компактно. Вот только мне кажется, что восемь отступов у них - это слишком... Четырех хватило бы за глаза.

Однако если бы я писал программу в рамках GNU, то писал бы по их требованиям...

> А вот интересно, такое объявление переменной, это плохой стиль?

> for( int i=0;i<100;i++){ .................. }

Для C++ это однозначно хороший стиль. Думаю что и для C99 тоже (хотя всегда можно найти противников).

> А вообще надо бы тоже сюда кусок своего кода запостить, мне критика понравилась!

А расплачиваться как будешь? Профессиональные ананимусы критикуют бесплатно только Код великих Эстонских Гуру, таких как Альфекс. ;)))

ну да. по этому поводу даже в kdevelop3 есть соответсвующее форматирование.

откр./закр. скобки на одном уровне, причём закр. ещё и подписываются . А тот бардак типа

if(){ for(){ if(){ printf(""); } } }

читать одна радость, особенно когда такая конструкция на 5 страниц, со вложенными else if,switch,return, ещё if ....