nodejs תקינות פונקציה

[eido]

האם הפונקציה הזו תקינה? או שחסרים/מיותרים בה דברים?
gpt מביא רשימה שאני ממש לא מסכים איתה.
אשמח לשמוע את דעתכם.

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

export async function addAd(params) {
    let command = [];
    let placeholders = [];
    let values = [];

    // מחפש/מפרסם - חובה
    if (params.type && checkType(params.type)) {
    command.push("type");
    placeholders.push("?");
    values.push(params.type);
    }
    else { return { error: "Invalid type" }; }

    // מקצוע - חובה
    if (params.profession && checkProfession(params.profession)) {
        command.push(", profession");
        placeholders.push(", ?");
        values.push(params.profession);
    }
    else { return { error: "Invalid profession" }; }

    // מינימום/מקסימום מחיר - לא חובה, אבל אם קיים חייב להיות מספר חיובי ושלם
    if (params.price) {
      if (checkNumber(params.price)) {
        command.push((params.type === "מחפש") ? ", max_payment" : ", min_payment");
        placeholders.push(", ?");
        values.push(params.price);
        }
      else { return { error: "Invalid price" }; }
    }

    // תדירות צינתוק  - חובה רק במחפש, אסור שיהיה במפרסם
    if (params.type === "מחפש") {
       if (params.ringMode && checkRingMode(params.ringMode)) {
          command.push(", ring_mode");
          placeholders.push(", ?");
          values.push(params.ringMode);
          }
        else { return { error: "Invalid ring mode" }; }
      }
    
    if (params.path) {
        command.push(", recording_path");
        placeholders.push(", ?");
        values.push(params.path);
    }

    let connection;
    connection = await mysql.createConnection({ host, user, password, database });
    const [results] = await connection.execute(`INSERT INTO ads (${command.join('')}) VALUES (${placeholders.join('')})`,
        values
    );

    // אזור - לא חובה, אבל אם קיים חייב להיות תקין
    if (params.area) {
      if (checkArea(params.area)) {
          const [results1] = await connection.execute(`INSERT INTO ad_cities (ad_id, city) VALUES (?, ?)`,
         [results.insertId, params.area]);
         return results1;
      }
      else { return { error: "Invalid area" }; }
      }
    // עיר - לא חובה, אבל אם קיים חייב להיות תקין 
    else if (params.city) {
      if (Array.isArray(params.city)) {
        for (const city of params.city) {
          if (checkCity(city)) {
            await connection.execute(`INSERT INTO ad_cities (ad_id, city) VALUES (?, ?)`,
              [results.insertId, city]);
          } 
          else { return { error: `Invalid city: ${city}` }; }
        }
      } 
      } else {const [results1] = await connection.execute(`INSERT INTO ad_cities (ad_id, city) VALUES (?, ?)`,
            [results.insertId, "all"]);
            return results1;
        }
}

[dovid]

לא צריך לשפר את הפונקציה, צריך ללכת ללמוד JS (בסיסי!).
יכול להיות שאתה חושב שבעידן הAI לא צריך ללמוד, ויכול להיות שאתה צודק,
אבל אני דורש שלפני שאתה תפרסם פה הודעות נוספות, לך תלמד JS בסיסי במדריך עדכני רציני (וכך גם בכל שפה שעליה אתה רוצה לשאול). אני לא דורש את זה מכל שואל פה, אבל במקרה שלך מדובר בתסמונת "אני יודע" עקבית מידי.

[eido]

@dovid אם זה יועיל

[eido]

@dovid האמת שחוץ מאת

else { return { error: "Invalid type" }; }

ואת החיבור למסד נתונים כתבתי לבד, בלי AI...

אולי תמקד אותי איפה הבעיה?

זה לא קוד מורכב מידי, סתם קוד שבודק שהפרמטר קיים -> בודק תקינות -> מוסיף לפקודת הsql.

איזה חלק פספסתי כאן?

בעצם, בחלק הזה במה js שונה משאר השפות?

[dovid]

@eido כתב בnodejs תקינות פונקציה:

אולי תמקד אותי איפה הבעיה?
איזה חלק פספסתי כאן?

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