bash array unset geht nur einmal ?...

[ren22]

hallo ich versuche gerade ein script zu schreiben was alle .dep dateien in ein array 'cat' -ted und wenn das jeweilige array element am ende der schleife ist, soll dieses array element mit 'unset' gelöscht werden was aber nur bei "lcd4linux" klappt .. den rest löscht das script nicht .. kann mir wer sagen wo der Fehler liegt?

Code: Alles auswählen

#!/bin/bash

EXTENSIONS=(lcd4linux libpng libgd libusb-compat)
EXTENSIONFILESSOURCE=/mnt/sda1/tce/optional

#
#get the size of the extensions
#
set -x

function1 () {
for i in ${EXTENSIONS[@]}; do

    if [ -f $EXTENSIONFILESSOURCE/$i.tcz.dep ];then #if .dep file exists write it into an array

<------>echo "dep"
    fi

       
       unset EXTENSIONS[$i] #remove current array-element from array


echo "-------------"
echo ${EXTENSIONS[@]}
echo ${#EXTENSIONS[@]}
sleep 1
done

}

while [ $(echo ${#EXTENSIONS[@]}) -gt 0 ] ; do
function1
done

echo "-------------"
echo ${EXTENSIONS[@]}
echo ${#EXTENSIONS[@]}

ausgabe:

Code: Alles auswählen

./depfinder.sh 
dep
-------------
libpng libgd libusb-compat
3
-------------
libpng libgd libusb-compat
3
-------------
libpng libgd libusb-compat
3
dep
-------------
libpng libgd libusb-compat
3 ...CTRL+C

wie man sieht "lcd4linux" wird gelöscht aber der rest wird nicht gelöscht und bleibt in einer endlos schleife hängen
vielen Dank

[smutbert]

i nimmt bei dir die einzelnen Werte des arrays an, zum Löschen würdest du aber imho den Index benötigen.

[gms]

Vorsicht auch beim Löschen über den Index, denn dabei entstehen Lücken im Index

Code: Alles auswählen

gms@sirius:~$ EXTENSIONS=(lcd4linux libpng libgd libusb-compat)
gms@sirius:~$ set | grep EXTENSIONS
EXTENSIONS=([0]="lcd4linux" [1]="libpng" [2]="libgd" [3]="libusb-compat")
gms@sirius:~$ unset EXTENSIONS[2]
gms@sirius:~$ set | grep EXTENSIONS
EXTENSIONS=([0]="lcd4linux" [1]="libpng" [3]="libusb-compat")
gms@sirius:~$ echo ${#EXTENSIONS[@]}
3

[Cae]

Code: Alles auswählen

while [ $(echo ${#EXTENSIONS[@]}) -gt 0 ] ; do

Meinten Sie folgendes?

Code: Alles auswählen

while [ ${#EXTENSIONS[@]} -gt 0 ] ; do

echo und Subshell erzeugt an der Stelle genau denselben Output, ist aber deutlich verschwurbelter (und hat mehr Overhead, ist also langsamer).

Generell ist noch zu sagen, dass du sprechende Namen verwenden solltest, bei function1() weiss niemand, was gemeint ist, unset_if_subdepends() oder aehnlich ist erheblich aussagekraeftiger (zumindest scheint der Code mal etwas in dieser Richtung zu tun zu sollen).

Gruss Cae

[rendegast]

indexed array <-> assoziativer array

Code: Alles auswählen

#!/bin/bash


#declare -a EXTENSIONS  # indexed array
    #EXTENSIONS=(lcd4linux libpng libgd libusb-compat)          # ein indexed array

declare -A EXTENSIONS   # assoziativer array
    EXTENSIONS[lcd4linux]=lcd4linux
    EXTENSIONS[libpng]=libpng
    EXTENSIONS[libgd]=libgd
    EXTENSIONS[libusb-compat]=libusb-compat

echo  EXTENSIONS@ ${EXTENSIONS[@]}

for i in lcd4linux libpng libgd libusb-compat; do

echo ii $i
echo  AAA EXTENSIONS[$i]
echo  AAA ${EXTENSIONS[$i]}

echo  AA0 ${EXTENSIONS[0]} AA1 ${EXTENSIONS[1]} AA2 ${EXTENSIONS[2]} AA3 ${EXTENSIONS[3]} AA4 ${EXTENSIONS[4]}
echo  AAlcd4linux ${EXTENSIONS[lcd4linux]} AAlibpng ${EXTENSIONS[libpng]} AAlibgd ${EXTENSIONS[libgd]} AAlibusb-compat ${EXTENSIONS[libusb-compat]

done