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

 
 
> Индусский код, много букв
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
Ответов
AlexandrY
сообщение Nov 7 2017, 13:37
Сообщение #2


Ally
******

Группа: Модераторы
Сообщений: 6 232
Регистрация: 19-01-05
Пользователь №: 2 050



Цитата(juvf @ Nov 7 2017, 14:05) *
Всё это великолепие есть тут

Для github-а это просто идеальный проект.
Все познается в сравнении.
Покажите свой код на github.
Go to the top of the page
 
+Quote Post
Эдди
сообщение Nov 7 2017, 20:10
Сообщение #3


Знающий
****

Группа: Участник
Сообщений: 825
Регистрация: 16-04-15
Из: КЧР, Нижний Архыз
Пользователь №: 86 250



Цитата(AlexandrY @ Nov 7 2017, 16:37) *
Для github-а это просто идеальный проект.

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

Насчет printf: как по мне, так использование такой жути, как printf, malloc и т.п. оправдано лишь в "жирных" микроконтроллерах. В обычных (вроде STM32F0/1/2/3/4) им не место. Аналогично с плавающей точкой: если нет в камне их аппаратной поддержки, нечего их туда лепить! Иначе получается абдуринщество чистой воды!

Сообщение отредактировал Эдди - Nov 7 2017, 20:13
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
|- - 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
- - kolobok0   Цитата(juvf @ Nov 7 2017, 15:05) Вброс .....   Nov 7 2017, 18:24
- - 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 Текстовая версия Сейчас: 19th July 2025 - 18:32
Рейтинг@Mail.ru


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