реклама на сайте
подробности

 
 
> Индусский код, много букв
juvf
сообщение Nov 7 2017, 12:05
Сообщение #1


Профессионал
*****

Группа: Свой
Сообщений: 1 261
Регистрация: 14-05-09
Из: Челябинск
Пользователь №: 49 045



Вброс .... на вентелятор.
Столкнулся с одним кодом.... предлагается в 5 шагов поднять обмен по LoRa чуть-ли не с нулевым порогом вхождения в программирование... есть проект LowCostLoRaGw.

Конечно всё шоколадно. На проводочках ардуино+рфм95 и распбери+рфм95, по 5 шагов и вот тебе готовая система сбора данных. На базе Sx1272.cpp даже есть библа для абдурино ардуино.

Решил велосипед не изобретать и портировать этот код себе, благо дело сырци есть. Компилятор дал тонну ворнингов. Я уже молчу за такой код
Код
bool SerialPi::find(const char *target){
    findUntil(target,NULL);
}
я уже молча испоравляю ворнинги где int сравнивается с unsigned int. Но не удержаляся после подобнойго кода (благо дело ворнинг показал)
Код
/* Prints data to the serial port as human-readable ASCII text followed
* by a carriage retrun character '\r' and a newline character '\n' */
void SerialPi::println(float f, int precission){
    const char *str1="%.";
    char * str2;
    char * str3;
    char * message;
    sprintf(str2,"%df",precission);             //ошибка
    asprintf(&str3,"%s%s",str1,str2);               //утечка памяти
    sprintf(message,str3,f);                     //ошибка

    const char *newline="\r\n";
    char * msg = NULL;
    asprintf(&msg,"%s%s",message,newline);              //утечка памяти
    unistd::write(sd,msg,strlen(msg));
}

Всё это великолепие есть тут
автор проекта какой-то индус CongDuc Pham
автор этого кода, судя по коментам в файле arduPi_pi2.cpp некая Libelium Comunicaciones Distribuidas

на сайте Libelium чего только нет. И для автомобилей что-то делают, и для сетей, и для МЕДИЦИНЫ!!! И такой говнокод пишут. УЖАС!!!

Весь arduPi_pi2.cpp исчерпан вызовами asprintf() и ни разу нет вызова free(). Конечно без free такой код будет работать, за каждый вызов будет утекать несколько байт памяти. Но сколько времени непрерывно проработает такой шлюз? неделю? Месяц? Год? Не удевляюсь, почему винда, после недели непрерывной работы начинает тормозить и съедает 8 Гб озу с закрытыми приложениями. А потом начинаются срачи на темы, типа "Почему до сих пор все сидят на древних языках вроде С и С++", и что с++ говноязык, а жава тру!!!
Go to the top of the page
 
+Quote Post
 
Start new topic
Ответов
kolobok0
сообщение Nov 7 2017, 18:24
Сообщение #2


практикующий тех. волшебник
*****

Группа: Участник
Сообщений: 1 190
Регистрация: 9-09-05
Пользователь №: 8 417



Цитата(juvf @ Nov 7 2017, 15:05) *
Вброс .... на вентелятор.
...


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

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

я о чём - везде можно найти косяки. Где больше, где меньше. Где впарили, где работают на своё имя. Где лишь бы взлетело(кстати для продажников - это время, оборот, результат),
где приёмка для жизнеобоспечения...
И индусы тут увы не всегда худшее. Есть и на наших просторах деятели.

удачи усем
(круглый)
Go to the top of the page
 
+Quote Post

Сообщений в этой теме
- juvf   Индусский код   Nov 7 2017, 12:05
- - iosifk   Цитата(juvf @ Nov 7 2017, 15:05) Вброс .....   Nov 7 2017, 13:11
- - AlexandrY   Цитата(juvf @ Nov 7 2017, 14:05) Всё это ...   Nov 7 2017, 13:37
|- - Эдди   Цитата(AlexandrY @ Nov 7 2017, 16:37) Для...   Nov 7 2017, 20:10
|- - mantech   Цитата(Эдди @ Nov 7 2017, 23:10) К сожале...   Nov 8 2017, 17:46
- - juvf   Цитата(iosifk @ Nov 7 2017, 14:11) Почему...   Nov 7 2017, 13:38
|- - AlexandrY   Цитата(juvf @ Nov 7 2017, 15:38) умышленн...   Nov 7 2017, 13:51
||- - juvf   Цитата(AlexandrY @ Nov 7 2017, 18:51) Вы ...   Nov 7 2017, 13:53
||- - AlexandrY   Цитата(juvf @ Nov 7 2017, 15:53) Это рабо...   Nov 7 2017, 14:03
||- - juvf   Цитата(AlexandrY @ Nov 7 2017, 19:03) Это...   Nov 7 2017, 14:36
||- - iosifk   Цитата(juvf @ Nov 7 2017, 17:36) с этим с...   Nov 7 2017, 14:54
||- - juvf   Цитата(iosifk @ Nov 7 2017, 19:54) Тут ес...   Nov 7 2017, 15:12
||- - iosifk   Цитата(juvf @ Nov 7 2017, 18:12) я вашу т...   Nov 7 2017, 15:24
|- - iosifk   Цитата(juvf @ Nov 7 2017, 16:38) Ну выкла...   Nov 7 2017, 14:09
- - x893   CongDuc Pham может он и индус, но работает во Фран...   Nov 7 2017, 20:27
- - juvf   ЦитатаНасчет printf: как по мне, так использование...   Nov 9 2017, 04:23
- - Эдди   Ой, про абдуринщиков вообще не надо. У них из 10 ...   Nov 9 2017, 05:35


Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 29th July 2025 - 00:40
Рейтинг@Mail.ru


Страница сгенерированна за 0.01357 секунд с 7
ELECTRONIX ©2004-2016