if (!ptr) VS if (ptr == 0) [С++] - Священные войны - Страница 2

@Bers · 29.06.2011, 22:41 **[ТС]**

Студворк — интернет-сервис помощи студентам

Сообщение от sandye51

Не по теме:

может тогда Вам лучше психологией заниматься, хоть на 1 флудера тут меньше станет)

Мой код привентивно безопасен. Как насчёт вас?

Надеюсь, мне никогда не придётся с вами работать.

oxotnik · 29.06.2011, 22:51

Сообщение от Bers

Допустим у меня "валидность" указателя определяется совсем другими критериям

хорошо, заменим нерусское слово "валидность" на словосочетание "нельзя использовать"
if(!prt) // если указатель нельзя использовать
вот только не надо демагогии по поводу "бывают моменты когда нельзя но очень хочется"

@Bers · 29.06.2011, 22:52 **[ТС]**

Сообщение от oxotnik

хорошо, заменим нерусское слово "валидность" на словосочетание "нельзя использовать"
if(!prt) // если указатель нельзя использовать
вот только не надо демагогии по поводу "бывают моменты когда нельзя но очень хочется"

Отрицание != нельзя. Как вы не поймёте?

oxotnik · 29.06.2011, 22:57

Сообщение от Bers

Отрицание != нельзя. Как вы не поймёте?

отрицание без равно пишется
http://www.cyberguru.ru/progra... age30.html
посему, разводя демагогию, результат проверки указателя является отрицательным и его (указатель) нельзя использовать

@AzaKendler · 29.06.2011, 23:01

Bers, а метафора про неоднозначность неплохая. насчет ! так очень многие и читают и пишут.
типа сокращение. вот тут мне недавно напомнили про лямбда выражения - они так сказать тоже пример сокращений и в принципе тоже способны запутать при определенных обстоятельствах. как вы считате использование лямбда выражений вне закона? или не страшно?

Добавлено через 1 минуту
oxotnik, ответ в тему. спасибо

@easybudda · 29.06.2011, 23:22

Сообщение от sandye51

если у кого-то не получается быстро понять что происходит - это уже его проблемы

Золотые слова, полностью согласен!

Сообщение от Bers

То ли Макконнел, то ли Робинсон приводили пример, как данная запись может привести к трудно-уловимой ошибке.

Не будьте голословны! Потрудитесь указать пример, иначе это только ваше утверждение.

Сообщение от Bers

это у вас. Как насчёт других? Допустим у меня "валидность" указателя определяется совсем другими критериями (тут есть варианты, например первый long данных - указатель на таблицу валидности данных).
Когда я читаю код, который писал не я, то что бы понять что этот код делает, я стараюсь думать так, как думает компилятор при компиляции. То бишь, в уме выполняю целый ряд преобразований.
Думаю, другие кодеры поступают так же. Ну так вот, чем меньше таких умственных потуг, тем меньше риск понять код не правильно. Меньше риск ошибки.

Не знаю, как там у вас и "других кодеров", я например читая чужой код пытаюсь понять логику, а не представить себе то, во что его компилятор превратит. Кстати, компилятор вообще не думает, просто обрабатывает текст и переводит его в машинные коды, опять же в строгом соответствии с логикой его (компилятора) разработчиков.
Ну а если у вас конструкция if ( ! ptr ) такие затруднения вызывает, думаю, вам больше бейсик подойдёт - там всё просто и понятно. Ну разве что лишних буковок много.

Сообщение от Bers

Как насчёт вас?

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

Собственно по теме: на сколько я знаю, более менее вменяемого способа определить, содержит указатель осмысленное значение, или просто мусор, не существует. В большинстве *NIX систем есть функция malloc_usable_size(), которая должна бы 0 возвращать, но это уже ближе к маразматическому шаманству...

Evg · 29.06.2011, 23:28

По части стилевых правок. Я всегда пишу "if (p == NULL)" или "if (p != NULL)", потому что над таким кодом не надо задумываться, в то время как после 8-часового рабочего дня в код "if (p)" надо ещё втыкать. Записи "if (p)" и "if (!p)" только в случае, если p - имеет булевский тип. Ну или в Си тип int, но с булевским смыслом (да/нет). Если int содержит некую исчислимую величину, то использую записи "if (p != 0)" и "if (p == 0)".

Для сравнения указателя использую именно NULL, а не 0. Почему так, написал в Неочевидные ответы на простые вопросы в разделе 3

@prZzZ · 29.06.2011, 23:36

Сообщение от Bers

Когда я встречаю вот такие записи: if (!ptr)

Bers, Думаю, у знающих язык выражение

C
1
while(*d++ = *s++)

не вызовет каких либо затруднений. А ориентироваться на новичков - глупо. Всё равно, что всех взрослых посадить на трехколесные велосипеды. Малыши ведь не могут водить машину.
С очень гибкий, мощный и компактный язык.

@Bers · 30.06.2011, 06:18 **[ТС]**

Сообщение от Evg

По части стилевых правок. Я всегда пишу "if (p == NULL)" или "if (p != NULL)", потому что над таким кодом не надо задумываться, в то время как после 8-часового рабочего дня в код "if (p)" надо ещё втыкать. Записи "if (p)" и "if (!p)" только в случае, если p - имеет булевский тип. Ну или в Си тип int, но с булевским смыслом (да/нет). Если int содержит некую исчислимую величину, то использую записи "if (p != 0)" и "if (p == 0)".

Для сравнения указателя использую именно NULL, а не 0. Почему так, написал в Неочевидные ответы на простые вопросы в разделе 3

Очень рад, что хотя бы один человек разделяет мою точку зрения.

Добавлено через 19 минут
И даже скажу больше: по своему опыту я заметил, что программист, который позволят себе писать в код типа if(ptr) {....}, или if(!ptr) {...} ,
так же, зачастую употребляет подобные записи и в более сложных конструкциях. А вот в них иногда уже приходится очень втыкать.

И тогда фраза:

Сообщение от easybudda

если у кого-то не получается быстро понять что происходит - это уже его проблемы

Приобретает совсем иной смысл: "я пишу как захочу, а как вы будите после меня сношаццо - ваши трудности"
Сквозит пренебрежительным отношением к членам своей команды.

Поиск ошибок в таком коде - геммор. На каждом брек-поинте приходится: впираццо, что тут есть по факту, а что подразумевал Великий_Программист_который_ложил_на_нас _с_большим_прибором.

А вот в более читабельном (очевидном) коде сам риск возникновения ошибки значительно ниже.

Добавлено через 46 минут

Сообщение от prZzZ

С очень гибкий, мощный и компактный язык.

while((*s!=*d) && !(*d)) d++;

Только честно, сколько времени у вас ушло на то, что бы прочитать эту запись?
У меня 2 секунды на чтение самого кода, и ещё 3 секунды на то, что бы удостовериться, что автор подразумевал именно то, что тут написано. Итого - 5 секунд на одну строчку.
Теперь представьте себе, какого это, искать семантическую ошибку в коде, который весь в таком вот безобразном стиле выполнен.
Имена переменных и те - не говорящие.

Итак, вы готовы? Засекайте время

Представленный ниже код - нерабочий. Найдите ошибку, сделайте так, что бы он заработал.

А потом скажите, сколько времени у вас ушло.

Задача:
Вывести сообщение "МОЖНО", если из букв введеной строки X можно составить введенную строку Y, при условии, что каждую букву строки X можно использовать один раз; и сообщение "НЕЛЬЗЯ" в противном случае.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
/*
подразумеваем, что в строке используются только печатные символы, то есть от 32-го до 255-го
*/
char *x="букварь";
char *y="фраза";
char buf[4096];
 
char *d=buf;
char *s=x;
while(*d++=*s++);
/******** start **********/
s=y;
d=buf;
while((*s!=*d) && !(*d)) d++;
while((*s==*d) && !(*s)){
s++;
*d="\x1";
d=buf;
while((*s!=*d) && !(*d)) d++;
}
if(*s)printf("НЕЛЬЗЯ\n");else printf("МОЖНО\n");
/******** finish **********/

@easybudda · 30.06.2011, 10:48

Сообщение от Bers

Представленный ниже код - нерабочий.

Представленный код - "для дурдома арифметика" (с) и вникать в него никакого вообще желания.
А вот так вроде работает

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int cmp(const void * a, const void * b){
    return *(char*)a - *(char*)b;
}
 
int test(const char * a, const char * b){
    return ( ! *b ) ? 1 : ( ! *a ) ? 0 : ( *b == *a ) ? test(++a, ++b) : test(++a, b);
}   
 
int main(void){
    char x[BUFSIZ], y[BUFSIZ];
    
    printf("X: ");
    scanf("%s", x);
    printf("Y: ");
    scanf("%s", y);
    
    qsort(x, strlen(x), 1, cmp);
    qsort(y, strlen(y), 1, cmp);
    printf("%s\n", ( test(x, y) ) ? "AGA" : "XPEH!");
    
    exit(0);
}

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

Vourhey · 30.06.2011, 11:26

Сравнение с NULL повышает читабельность кода. Меньше понтов с сокращениями записи, больше читабельности не в ущерб производительности. Я бы написал с == NULL.

@easybudda · 30.06.2011, 11:29

Сообщение от Bers

while((*s!=*d) && !(*d)) d++;

Кстати, сто'ит только присмотреться:

Code
1
2
пока((то, на что указывает s не равно тому, на что указывает d) и (то, на что указывает d, равно нулю)) 
  переместить указатель d на следующий символ.

Какое на фиг d++ если и так !(*d)

Evg · 30.06.2011, 11:36

Сообщение от Bers

Приобретает совсем иной смысл: "я пишу как захочу, а как вы будите после меня сношаццо - ваши трудности"
Сквозит пренебрежительным отношением к членам своей команды

По малолетсву мне тоже казалось, что зачем писать в 5 букв, когда можно сэкономить 1 букву и написать только 4. Со временем твёрдо убедился в том, что программа в первую очередь должна быть понятна стороннему человеку. А страдать фигнёй типа сокращения количества букв или оптимизировать вместо компилятора в ущерб читабельности кода - этим пусть студенты занимаются, которые пишут код в одиночку. Работая в команде так делать, на мой взгляд, нельзя

@Bers · 30.06.2011, 11:38 **[ТС]**

Сообщение от easybudda

Представленный код - "для дурдома арифметика" (с) и вникать в него никакого вообще желания.

Конечно не охото. И не потому, что дурдом, а потому, что искать ошибку в таком плохо читаемом коде тяжело. Сразу даже не понятно, какой там используется алгоритм. Не понятно, где именно программа делает "не то, что нужно".

Хотя с другой стороны, если разобраться, то этот "дурдом для арифметики" в общем то, особенно не отличается от вашего

Ваш вариант - такой же ужассный в плане читабельности кода.
Идея отсортировать массивы строк конечно прикольная

Но взгляните на ваш test()
Как такое вообще можно читать?
Условие в условии, в условии! Тройная вложенность в условии, и это все в одной строке!
Это уже шесть вариантов развития событий нужно держать в голове одновременно! Шесть состояний системы.
Рекурсия мой бедный мозг добила.

Проще поставить брек-поинт, и посмотреть в пошаговом режиме, чем рисковать "прочитать неправильно".

Если бы вы сознательно допустили в коде какую нить хитрую ошибку, наверняка какой нибудь ТяжолыйБудда тоже сказал бы "в этот дурдом вникать совсем не хочется".

А если у вас проект уже содержит 100500 строк такого вот нечитабельного ужаса, и тут на тебе, бац! Ни с того ни с сего падаем в рантайме. Тому, кто будит искать ошибку проще сразу застрелиться.

C++
1
2
3
4
int test(const char * a, const char * b)
{
    return ( ! *b ) ? 1 : ( ! *a ) ? 0 : ( *b == *a ) ? test(++a, ++b) : test(++a, b);
}

Эпус достоин того, что бы увековечить его на обоях в моей комнате

Теперь кто нить в гости придёт из товарищей, буду всех спрашивать: шо за фигня твориццо в этом коде?

@easybudda · 30.06.2011, 11:39

Сообщение от Vourhey

Сравнение с NULL повышает читабельность кода. Меньше понтов с сокращениями записи, больше читабельности не в ущерб производительности. Я бы написал с == NULL.

На производительность оно, конечно, никаким боком не влияет, но раз уж есть в С унарный оператор !, не вижу ни одной причины им не пользоваться. Когда знаешь, что он делает, просто читаешь так же ( если выражение имеет нулевое значение ). Никаких проблем с читабельностью. А что до понтов, то самый понтовый язык, какой знаю - это PERL. Вот уж, где понт на понте и понтом погоняет

Когда первый раз на этом языке программу видишь, думаешь - марсиане с новым годом поздравление прислали

За то, когда разберёшься - оказывается, это так удобно... Так, что, я за короткие записи - букв меньше, смысл всё так же ясен...

Vourhey · 30.06.2011, 11:46

Сообщение от easybudda

но раз уж есть в С унарный оператор !, не вижу ни одной причины им не пользоваться.

В С много операторов, что позволяет испортить читабельность кода до бесконечности.

Сообщение от easybudda

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

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

@easybudda · 30.06.2011, 11:53

Сообщение от Bers

Хотя с другой стороны, если разобраться, то этот "дурдом для арифметики" в общем то, особенно не отличается от вашего

Ещё как отличается.

Сообщение от Bers

Но взгляните на ваш test()
Как такое вообще можно читать?

Да прям по пунктам. Имеем: упорядоченный набор букв a, который должен включать в себя упорядоченный набор букв b.
Смотрим, что происходит

Code
1
2
3
4
если буквы в b закончились, значит всё нашлось, возвращаем 1,
иначе, если в b ещё есть буквы, а в наборе a нет, возвращаем 0,
если a и b указывают на одинаковую букву, ищем букву, следующую за *b в наборе букв, следующих за *a
иначе продолжаем искать букву *b в наборе букв, следующих за *a

с какого места не понятно?

Сообщение от Bers

Это уже шесть вариантов развития событий нужно держать в голове одновременно! Шесть состояний системы.
Рекурсия мой бедный мозг добила.

Вот говорю же - надо в смысл вникать. А повышение наглядности кода с целью не задумываться, что в нём происходит - верный путь к пропуску очевидных ошибок. Проходили уже когда-то тут на форуме...

Добавлено через 1 минуту

Сообщение от Vourhey

Я несколько лет с перлом работаю и код у нас очень грамотно выстроен и легко читается.

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

Vourhey · 30.06.2011, 11:57

Сообщение от easybudda

но наверняка при одном обязательном условии - язык всё-таки нужно знать..

Язык знать нужно, никто не спорит. Но при возможности код должен быть написан так, чтобы читался он, как книжка. В этот код даже функцию можно добавить isValidPointer, что еще повысит читабельность кода и для некритичной системы не скажется на производительности.

@iama · 30.06.2011, 12:03

Сообщение от Vourhey

В этот код даже функцию можно добавить isValidPointer, что еще повысит читабельность кода и для некритичной системы не скажется на производительности.

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

это была простая обучающая программа, которая показывала, как проверить, инициализирован ли указатель NULL'ом. Имхо, в том случае уж точно такая запись не мешала читабельности кода, читалось элементарно:

Code
1
2
3
4
если указатель нулевой то
  выводим "Нет"
иначе 
  выводим "Да"

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

Vourhey · 30.06.2011, 12:18

Сообщение от iama

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

Если брать учебный пример, то, в принципе, пофиг. Но в нормальной проге я бы так (!ptr) писать не стал.

Новые блоги и статьи Все статьи Все блоги /
[golang] Угол между стрелками часов alhaos 12.05.2026 По заданным значениям часа и минуты необходимо определить значение меньшего угла между стрелками аналогового циферблата часов. import "math" func angleClock(hour int, minutes int) float64 { . . .	Debian 13: Установка Lazarus QT5 ВитГо 09.05.2026 Эта инструкция моя компиляция инструкций volvo https:/ / www. cyberforum. ru/ blogs/ 203668/ 10753. html и его же старой инструкции по установке Lazarus с gtk2. . .	Нейросеть на алгоритме "эстафета хвоста" как перспектива. Hrethgir 06.05.2026 На десерт, когда запущу сервер. Статья тут https:/ / habr. com/ ru/ articles/ 1030914/ . Автор я сам, нейросеть только помогает в вопросах которые мне не известны - не знаю людей которые знали-бы. . .	Асинхронный приём данных из COM-порта Argus19 01.05.2026 Асинхронный приём данных из COM-порта Купил на aliexpress термопринтер QR701. Он оказался странным. Поключил к Arduino Nano. Был очень удивлён. Наотрез отказывается печатать русские буквы. Чтобы. . .
попытка написать игровой сервер на C++ pyirrlicht 29.04.2026 попытка написать игровой сервер на плюсах с открытым бесконечным миром. возможно получится прикрутить интерпретатор питон для кастомизации игровой логики. что есть на текущий момент:. . .	Контроль уникальности выбранного документа-основания при изменении реквизита Maks 28.04.2026 Алгоритм из решения ниже разработан на примере нетипового документа "ЗаявкаНаРемонтСпецтехники", разработанного в КА2. Задача: уведомлять пользователя, если указанная заявка (документ-основание). . .	Благородство как наказание Maks 24.04.2026 У хорошего человека отношения с женщинами всегда складываются трудно. А я человек хороший. Заявляю без тени смущения, потому что гордиться тут нечем. От хорошего человека ждут соответствующего. . .	Валидация и контроль данных табличной части документа перед записью Maks 22.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в КА2. Задача: контроль и валидация данных табличной части документа перед записью с учетом регламента компании. . .

@AzaKendler 214 / 116 / 14 Регистрация: 30.05.2011 Сообщений: 1,772 Записей в блоге: 15
	29.06.2011, 23:01
	Bers, а метафора про неоднозначность неплохая. насчет ! так очень многие и читают и пишут. типа сокращение. вот тут мне недавно напомнили про лямбда выражения - они так сказать тоже пример сокращений и в принципе тоже способны запутать при определенных обстоятельствах. как вы считате использование лямбда выражений вне закона? или не страшно? Добавлено через 1 минуту oxotnik, ответ в тему. спасибо 0

Evg 21281 / 8305 / 637 Регистрация: 30.03.2009 Сообщений: 22,660 Записей в блоге: 30
	29.06.2011, 23:28
	По части стилевых правок. Я всегда пишу "if (p == NULL)" или "if (p != NULL)", потому что над таким кодом не надо задумываться, в то время как после 8-часового рабочего дня в код "if (p)" надо ещё втыкать. Записи "if (p)" и "if (!p)" только в случае, если p - имеет булевский тип. Ну или в Си тип int, но с булевским смыслом (да/нет). Если int содержит некую исчислимую величину, то использую записи "if (p != 0)" и "if (p == 0)". Для сравнения указателя использую именно NULL, а не 0. Почему так, написал в Неочевидные ответы на простые вопросы в разделе 3 0

Vourhey Почетный модератор 7393 / 2639 / 281 Регистрация: 29.07.2006 Сообщений: 13,696
	30.06.2011, 11:26
	Сравнение с NULL повышает читабельность кода. Меньше понтов с сокращениями записи, больше читабельности не в ущерб производительности. Я бы написал с == NULL. 0