עזרה בקטע קוד C++

OdedDvir

@איש-ימיני זה בגלל שהיתה לי טעות בקוד, כמו שציין @yossiz. תיקנתי במקור והפכתי את ההשמה ל:

// insert into string and send it to other function
String segment = String((char *)buff);

ובעצם אפשר ישר כך:

doSomething(String((char *)buff));

yossiz

אני רואה בפורומים של ארדואינו שממליצים לא להשתמש ב-String

אולי יש מקומות שבהם זה בסדר להשתמש אבל צריך להיות מודע להשלכות

OdedDvir

@yossiz לכאורה אפשר לוותר על זה לגמרי, אבל אז צריך לשלוח גם את הגודל של המערך (כי אין null char בסוף)

// cast into char array and send it to other function
   doSomething((char *)buff, c);

...
void doSomething(char * segment, int size) {
 // do your thing
}

yossiz

@OdedDvir יש פה בעיה,
הקוד שלך עובד על כל פלח של 128 בייטים בנפרד,
הוא רוצה לאסוף את כל הפלחים למשתנה אחת
אבל האורך של המחרוזת המלאה לא ידועה מראש בצורה סטטית,
זה מחייב הקצאת זכרון דינאמי
משיטוט קצר ברשת נראה שזה נושא מתקדם ולא כזה פשוט בארדואינו,
מינימום צריך להכיר קצת את השפה והפלטפורמה לפני מימוש של דבר כזה

@איש-ימיני אם במקרה, ידוע מראש שהתשובה לא יארוך יותר מ-X תווים, או שלא תצטרך יותר מ-X תווים מהתשובה, זה יפשט בהרבה את הפתרון

yossiz

@OdedDvir כתב בעזרה בקטע קוד C++:

@איש-ימיני זה בגלל שהיתה לי טעות בקוד, כמו שציין @yossiz. תיקנתי במקור והפכתי את ההשמה ל:
// insert into string and send it to other function
String segment = String((char *)buff);
ובעצם אפשר ישר כך:
doSomething(String((char *)buff));

האם הקוד הזה מסתמך על ה-null byte כדי לציין את סוף המחרוזת? כי במקרה הזה אין null byte, או שיש במקום לא נכון...
עריכה: אני רואה עכשיו ששמת לב לנושא בפוסט אחר

איש ימיני

@OdedDvir כתב בעזרה בקטע קוד C++:

@yossiz לכאורה אפשר לוותר על זה לגמרי, אבל אז צריך לשלוח גם את הגודל של המערך (כי אין null char בסוף)
// cast into char array and send it to other function
   doSomething((char *)buff, c);

...
void doSomething(char * segment, int size) {
 // do your thing
}

הפתרון הזה עובד טוב מאוד.
עזרת לי מאוד.
תודה רבה.

OdedDvir

@yossiz שוב אתה צודק. הניסיון שלי עם c++ כבר חלוד... שמתי לב לבעיה הזו בפוסט האחרון שלי, כשהורדתי לגמרי את השימוש ב-String.
מעכשיו אני אשתדל לשבת בשקט עד ש @איש-ימיני יחכים אותנו מה הוא מבקש לעשות...

yossiz

@OdedDvir כתב בעזרה בקטע קוד C++:

מעכשיו אני אשתדל לשבת בשקט

ככל הנראה פתרת לו את הצורך לבינתיים (עד שזה יקרוס מסיבה עלומה ואז נשמע ממנו שוב...)

@איש-ימיני חשוב להבין ששפת ++C לא דומה כלל ל-php או vba

יש ב-++C הרבה הזדמנויות לעשות טעויות קטנות שלא תשים לב עליהם ואף אחד לא יתריע לך עד שפתאום משהו (שנראה) לא קשור לגמרי יתפוצץ לך בפנים ולא תבין מאיפה זה בא לך

כל זה תוצאה של חסרון "אבטחת זכרון" (memory safety) של השפה

איש ימיני

@OdedDvir
הקוד עובד די טוב, אבל יש קטע מוזר שהמקטע האחרון חוזר על עצמו.
זה הקוד הסופי כרגע:

while(http.connected() && (len > 0 || len == -1)) {
// get available data size
    size_t size = stream->available();
    if(size) {
        // read up to 128 byte
        int c = stream->readBytes(buff, ((size > sizeof(buff)) ? sizeof(buff) : size));
 
        // write it to Serial
        Serial.write(buff, c);
        doSomething((char *)buff, c);

        // decrease length
        if(len > 0) {
            len -= c;
        }
    }
    delay(1);
}

void doSomething(char * segment, int size) {
    result += segment;
}
Serial.println(result);

בעת שאני מדפיס את התוצאה

Serial.println(result);

נראה שהמקטע האחרון של ה 128 ביטים חוזר על עצמו.
אבל אני יכול לשים תו סיום ולהסתדר.
תודה בכל אופן.

OdedDvir

@איש-ימיני זה בגלל שהמקטע האחרון מכיל מידע שימושי פחות מ-128 תוים, ושאר התאים במערך מכילים שאריות מהמקטע הקודם. הפעולה:

    result += segment;

מוסיפה את כל המקטע בגלל שאין תו null. אתה צריך להוסיף רק את size התוים האחרונים. אפשר להתחכם כך:

if (size < 128)
   segment[size] = '\0';
result += segment;

למעשה זה קצת מטריד אותי שזה עבד בלי לקרוס, כי לא ברור לי מניין ידוע גודל המערך segment להיות 128, בכל האיטרציות הקודמות. הקוד הזה לא מניח את דעתי לגמרי. הפתרון הכי טוב זה להעתיק תמיד size תוים.
ובלאו הכי, נראה לי שהפעולה += מבצעת המרה לא מפורשת לאובייקט String. הכי טוב להגדיר מראש את הגודל המקסימלי של result, ולהשתמש בהעתקה ישירה ע"י memcpy.

// Globals
define MAX_LEN 1000
char result[MAX_LEN+1];  // save space for trailing null char
int resultSize = 0;

...
void doSomething(char * segment, int size) {
   int charsToCopy;
   charsToCopy = resultSize+size > MAX_LEN ? MAX_LEN-resultSize : size;
   if (charsToCopy > 0) {
      memcpy(result[resultSize], segment, charsToCopy);
      resultSize += charsToCopy;
      result[resultSize] = '\0'; // add trailing null char
   }
}

ככה זה יותר בטוח

תחומים - פורום חרדי מקצועי

עזרה בקטע קוד C++