Изпечатай

[neter]

Не се прави така прозорец с прогрес в zenity. Така, както си го направил в момента, създава нов прозорец за прогрес за всеки файл, което води до мигане на прозорци, липса на запълващ се прогрес и загуба на смисъла да го има. Промени реда, започващ с IFS, така:

Код

GeSHi (Bash):
IFS=$'\n' image=( `$gui --file-selection  --multiple --separator=$'\n' --filename=$HOME/ --file-filter='Image Files (jpg,png,bmp) | *.jpg *.png *.bmp' --file-filter='All files | *' --title="Select an image files"` )

Целта на добавените скоби е да се докара масивът до стандартен shell масив, за да можем да броим стойностите му и така да си изчисляваме прогреса. Редактирал съм и title-а - поправена граматическа грешка и множествено число за съответствие с multiple-а.

След това замени целия for цикъл с това:

Код

GeSHi (Bash):
step=$((100/${#image[@]}))
pos=0
 
(
for i in ${image[@]}; do
    echo "# Current file:\n$i"
    $converter -format $format -quality $scale -scale $resolution "$i"
    pos=$(($pos+1))
    echo $(($step*$pos))
done
) | $gui --progress --percentage=0 --no-cancel --auto-close --title="Нещо за заглавие"

Текстът в прозорец с прогрес на zenity се сменя с echo, съдържащо символ # в началото на стринга. С това echo изписваме името на файла, който се обработва в момента. Позицията на прогреса се сменя с echo, съдържащо цифра за процента, на който прогресът трябва да се позиционира в дадения момент. Това echo трябва да съдържа само цифра и го слагаме след обработката на файла, за да отговаря на приключилата бройка файлове.

[Rosen88]

Не се прави така прозорец с прогрес в zenity. Така, както си го направил в момента, създава нов прозорец за прогрес за всеки файл, което води до мигане на прозорци, липса на запълващ се прогрес и загуба на смисъла да го има. Промени реда, започващ с IFS, така:

Код

GeSHi (Bash):
IFS=$'\n' image=( `$gui --file-selection  --multiple --separator=$'\n' --filename=$HOME/ --file-filter='Image Files (jpg,png,bmp) | *.jpg *.png *.bmp' --file-filter='All files | *' --title="Select an image files"` )

Целта на добавените скоби е да се докара масивът до стандартен shell масив, за да можем да броим стойностите му и така да си изчисляваме прогреса. Редактирал съм и title-а - поправена граматическа грешка и множествено число за съответствие с multiple-а.

След това замени целия for цикъл с това:

Код

GeSHi (Bash):
step=$((100/${#image[@]}))
pos=0
 
(
for i in ${image[@]}; do
    echo "# Current file:\n$i"
    $converter -format $format -quality $scale -scale $resolution "$i"
    pos=$(($pos+1))
    echo $(($step*$pos))
done
) | $gui --progress --percentage=0 --no-cancel --auto-close --title="Нещо за заглавие"

Текстът в прозорец с прогрес на zenity се сменя с echo, съдържащо символ # в началото на стринга. С това echo изписваме името на файла, който се обработва в момента. Позицията на прогреса се сменя с echo, съдържащо цифра за процента, на който прогресът трябва да се позиционира в дадения момент. Това echo трябва да съдържа само цифра и го слагаме след обработката на файла, за да отговаря на приключилата бройка файлове.

За граматическите грешки не споря с английския не съм добре . А за прогреса не бях сигурен как да го направя тук, въпреки че на друго място съм го правил по следния начин.

Код

GeSHi (Bash):
detect () {
# needs some time to register
(
echo "20" ; sleep 1
echo "40" ; sleep 2
echo "60" ; sleep 2
echo "80" ; sleep 2
echo "90" ; sleep 2
DEVICE=$(dmesg| tail -n 1|grep removable|grep -o "\[sd*\]"|grep -o sd[a-z])
 
if [ "$DEVICE" = "" ]; then DEVTEXT="No device is detected"; else DEVTEXT="Detected new removable device /dev/$DEVICE"; fi
 
echo "# $DEVTEXT"
) |\
$DIALOG --width=400 --height=200 --title="$TITLE" --progress --text="Waiting for device to register"
#
DEVICE=$(dmesg| tail -n 1|grep removable|grep -o "\[sd*\]"|grep -o sd[a-z])
#
if [ "$DEVICE" = "" ]; then
#
$DIALOG --title="Device not detected..." --question --ok-label="Try again" --cancel-label="Exit now" --text="No new usb device was detected..."
 
	if [[ $? -eq 1 ]]; then
	exit 1
	else
	detect
	fi
fi
}
detect
 

Ако е нещо подобно ще се опитам да го запомня твоят метод тъй като изглежда по-прост.

[Rosen88]

Струва ми се че сега нищо не прави освен броенето на прогрес. Ааа всъщност е презаписала имената вместо да създаде нови файлове. Това е една от спънките..

https://github.com/sandikata/ROKO__/blob/master/imagemagick-zenity

Много искам да ги понауча тези неща за да не те занимавам ама с английския като съм зле а и не мога да намеря свестен материал на български или поне руски...

[neter]

Ааа всъщност е презаписала имената вместо да създаде нови файлове. Това е една от спънките..

По-нагоре ти бях обърнал внимание на това.

А наясно ли си, че ползвайки mogrify, вместо convert, ще имаш различен брой остатъчни файлове в зависимост от форматите на входния и изходния файл? При convert оригиналният файл остава наличен, докато резултатът се записва в друг файл. При mogrify резултатът ще се запише в друг файл, ако форматът на резултата е друг (например, входящ jpg файл, изходящ png файл), но ако форматите са еднакви (например, и входящият, и изходящият са jpg), то резултатът ще се запише върху оригинала. Няма по-добър или по-лош метод - зависи какво целиш.

П.П.: От примера, който си дал, не виждам каква е разликата между "моя" метод и "твоя" метод. Парче код, принтиращо числа за стойностите на прогреса, засилено към zenity. В този пример ти нямаш множество команди, които можеш да разграничиш в стъпки, с които да пълниш прогреса, а имаш единично действие, на което искаш да определиш максимално време за изпълнение, и това си се прави точно с поредица от sleep-ове, всеки от които е стъпка от прогреса. Е, аз бих вкарал sleep-овете и dmesg-а в цикъл, за да не ги пиша няколко пъти и за да проверявам dmesg-а след всеки sleep, за да не е нужно всеки път да се чака целия timeout, ако устройството е засечено по-рано, ама пак си зависи какво целиш.
А пък единично действие с неизвестно време за изпълнение, на което няма да се залага timeout и трябва да се изчака да приключи, се засилва към zenity с пулсиращ прогрес, тъй като няма нищо на стъпки, които да пълнят прогреса, а прозорецът трябва да остане отворен точно толкова време, колкото трае изпълнението на действието. Опитът за изкуствено създаване на стъпки в такива случаи води до некоректна информация за потребителя.