v 0.1.2 (colorify)

[RaminNietzsche]

چک می کتن ببینم ترمینال رنگ ساپورت می کنه یا نه اگر آره من یک پالت ۸تایی استاندارد در نظر می گیرم طبق همون رنگ ها رو میرم اگر صاپورت هم نکنه همه متغیر ها نال رد میشه در نتیجه خروجی ها درست چاپ میشه :)
فایل اضافی هم که گفتی منطقی بود پاک کردم
@bijanebrahimi

[bijanebrahimi]

@RaminNietzsche
-‫‪‫‪‫‫رنگبندی خروجی اگه هدفش خواناتر کردن خروجی باشه، در درجه اول باید اختباری باشه و چیزی ناخواسته تو لاگ نریزه. تعبیه کردن سویچ‫ color به عنوان مثال.
-‫اسکریپت برای دستور اشتباه instadl -u خطای نادرست بر‌میگردونه. تصور فرض کردن نام‌کاربری به عنوان دومین پارامتر ورودی‫ اشتباهه. این مثال و بالاییش نشون میده مسیر درست استفاده از getopt عه
-قبلا هم فک کنم اشاره کرده بودم که رنگ‌بندی تو خروجی اگه هم قراره رخ بده بهتره در جای مخصوصش رخ بده نه بر روی کل خط. مثلا‫ کاری که دستور ls با سوییچ color=auto انجام میده مثال خوبیه. فقط فایل‌های اجرایی یا .... مثلا تصور سفید بودن رنگ foreground تصور صحیحی نیست تو تابع show_msg

• ‫آیا هنوز سوییچ u تعداد عکس نادرست بر‌میگردونه؟ اینو چک نکردم هنوز

‫آپدیت:

• ‫تابع download_all هیچ‌وقت هیچ خطایی رو برنمی‌گردونه. حتی وضعیت خروج رو هم تعیین نمی‌کنه و پیش‌فرض همیشه بدون خطا اجرا میشه. این مشکل داره.

[bijanebrahimi]

‫این کد پیشنهادی من برای چک کردن آرگومان‌هاست. البته هنوز download_all مشکل داره ولی خب یکی از مشکلات رو حل می‌کنه.

while getopts "hu:" opt; do
    case "$opt" in
    h)
        show_help
        show_usage
        exit 0
        ;;
    u)  username=1
        download_all $username
        exit 0
        ;;
    esac
done