אין ספק שפיתרון הרבה מהבעיות בקוד דורש מחשבה, מיומנות, יצירתיות, היכרות עם עולם התוכן ושלל מיומנויות של מתכנתים חכמים. אבל הרבה פעמים דווקא הדברים הקטנים הם אלה שיכולים לשנות את החיים שלנו ביום יום.
הנה דוגמה לשינוי קוד פשוט מסוג זה. היה לי פרויקט knex בו לכל טבלה בבסיס הנתונים היה קובץ Repository שכלל קוד לפעולות בסיסיות עם הטבלה. לדוגמה עבור טבלה מדומיינת של משימות היה לנו קובץ שנראה בערך כך:
function create(db, task) {
return db('tasks').insert(task).then(([createdId]) => ({ ...task, id: createdId }));
}
function select(db, ...columns) {
return db('tasks').select(columns);
}
function destroy(db, task) {
return db('tasks').where('id', task.id).del();
}
function update(db, which, what) {
return db('tasks').where('id', which.id).update(what);
}
module.exports = { create, select, destroy, update };
אנשים אהבו את זה כי בתוכנית הראשית היית יכול לכתוב דברים כמו:
const newTask = await tasks.create(db, { text: 'new task', done: false });
await tasks.update(db, newTask, { done: true });
console.log(await tasks.select(db, '*'));
await tasks.destroy(db, newTask);
אבל הבעיה שכל מי שרצה להוסיף טבלה חדשה היה צריך לכתוב גם קובץ Repository, וכמעט כולם כללו את אותן פעולות בסיסיות, בנוסף לפעולות ספציפיות של אותה טבלה.
ה Refactoring במקרה הזה היה קסום כמו שהיה פשוט. יוצרים קובץ אחד בשם repo.js עם הקוד הבא:
module.exports = function(name) {
function create(db, item) {
return db(name).insert(item).then(([createdId]) => ({ ...item, id: createdId }));
}
function select(db, ...columns) {
return db(name).select(columns);
}
function destroy(db, item) {
return db(name).where('id', item.id).del();
}
function update(db, which, what) {
return db(name).where('id', which.id).update(what);
}
return { create, select, destroy, update };
}
ועכשיו מי שרוצה רק את הפעולות הבסיסיות מסתפק בשורה אחת:
module.exports = require('./repo')('tasks');
וגם מי שצריך יותר יכול לקחת את 4 הפעולות מ repo ולהוסיף עליהן פעולות חדשות.
וכן - הדוגמה שלי עברה ניקוי. בפרויקט אמיתי חיבור הקבצים לתשתית אחת דורש יותר עבודה ותמיד יהיה יותר מסובך מכתיבת קובץ אחד נוסף. ועדיין, מספיק בן אדם אחד שיארגן מחדש את התשתית כדי שכל המערכת תזוז יותר מהר.
(נ.ב. אם אתם כבר עובדים ב knex ורוצים להרחיב אותו עם יכולות של ORM שווה לבדוק את פרויקט Bookshelf.js שעושה בדיוק את זה)
