# Проверка за администраторски права
if [ `id -u` != "0" ]; then
echo "Добре дошли в инсталатора на Gentoo Linux"
echo "За да изпълните този скрипт ви трябва root."
echo "Моля стартирайте скрипта като root."
fi
Сложи и еxit, за да прекъснеш скрипта при изпълнение на условието. В противен случай на потребителя ще се изпише, че трябва да е root, но скриптът ще продължи изпълнението си.
echo "Изберете дял за SWAP (Допълнителна виртуална памет)"
read SWAP
Всички подобни комплекти от echo и read по-добре да бъдат комплект от printf и read. Пример
GeSHi (Bash):
printf "Изберете дял за SWAP (Допълнителна виртуална памет): "
read SWAP
echo "Изберете дял за root (/)"
echo "Например : /dev/sda1"
read BOOTPART
echo "Изберете дял за SWAP (Допълнителна виртуална памет)"
read SWAP
mkswap $SWAP
swapon $SWAP
Ами ако потребителят въведе невалидно или несъществуващо устройство? Добави проверка за валидност на данните и коректност на изпълнението към всяко действие в скрипта. В зависимост от получените резултати (или твоите предпочитания) за действието, скриптът може да пита отново за данни или направо да спре изпълнението си.
4. Изнеси всякакви текстове и линкове като променливи в началото на скрипта, или по-добре в отделни файлове, от които да ги четеш. Гледай в кода да имаш възможно най-малко неща, които може да претърпят промяна в даден момент.
5. Оправи си печатните и граматическите грешки.
Не съм гледал за синтактични грешки в кода ти, но ако има такива, ще си проличи. Опитай се да проиграеш всички варианти, които ти хрумнат, за да провериш максимално скрипта си за бъгове.
P.S.: Едно допълнение към ядовете ти от предните страници на темата. Никъде и никога не слагай имена на твои променливи, започващи с цифра. Имената на променливите трябва да започват с буква. Иначе ще има много да си скубеш косата