So weird

abdelilah

abdelilah

Hello everyone

I have to make a script which creates as much folders as declared by the user, the problem is that I've made a loop which doesn't seem to work can you please help :

Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
while [ $nmbrf>0 ]
	do mkdir "foo$nmbrf"
	expr $nmbrf-1
done

Thanks in advance
 
Homework, so we don't want to give it away... Run your script with -x, or add some echo or printf statements to show the values of the loop index.
 
"The Variable : what it is and why we need them"
by UNIX god.​

( translated from octal to human from the 1967 filmstrip )
In the beginning the art of accessing memory addresses the indigenous people used the ancient technique of interpolation to create, record, update, and destroy a little primitive memory binding known simply as a variable.

The variable which initially held a numerical value would eventually be endowed to hold a whole array of interesting values. In the early days when the variable was of no use any more one of the most respected of all of the indigenous tribe known only as the 'garbage collector' would free() the variable by sacrificing it to the natives' deity known simply as the heap.

Though the native scripture warned them of Undefined Behaviour ... stack overflow ... hellfire ... segfault

Seriously though break down what isn't working.

Start with your loop and counter as SirDice pointed out.

To do that experiment with expr() directly in the command line. build a counter you can call line by line. (READ THE MAN PAGE)

Also take the time to investigate the man page for sh(). I realise it's a long man page but it will be your best resource once you know the language( much like how our o'reilly books used to be) Hint: look at Quoting, Parameter Expansion, Command Substitution, Flow-Control Constructs and test()... did I mention READ THE MAN PAGES.

The Almquist Bourne shell is a great shell to work with. It takes some getting used to.

For further tutorials and best practices I have always enjoyed reading the stuff on this site:

http://www.shelldorado.com/

a more direct link to save you a couple clicks off of Heiner's site: http://www.shelldorado.com/shelltips/

Look at the beginner section.

After you get your bearings( in say 45 minutes) take the time to learn what NOT to do with the shell:

http://partmaps.org/era/unix/award.html
http://www.shelldorado.com/articles/ignorantsguide.html

Good luck! and don't forget READ THE man() PAGES!

~
 
Hello

I've made a slight modification but the loop isn't effective since my tests stops on the lower value :

Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
echo $nmbrf
while [ $nmbrf>0 ]
	#do mkdir "foo$nmbrf"
do	
	echo foo$nmbrf
	expr $nmbrf - 1
done

Result :
Code: 
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4
3
foo4

What is wrong with this loop, since the expr is working correctly.
 
Hello again

I've made another change but I still have the same bug :
Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
echo $nmbrf
#while [ $nmbrf > 0 ]
while test $nmbrf -gt 0
	#do mkdir "foo$nmbrf"
do
echo foo$nmbrf
expr $nmbrf - 1
done
 
Always indent code properly, even temporary code.

expr is not doing what you think it's doing. Add another echo after the expr line to show the value of $nmbrf at that point.
 
Here again it shows that :
Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
#echo $nmbrf
#while [ $nmbrf > 0 ]
while [ $nmbrf -gt 0 ]
	#do mkdir "foo$nmbrf"
do
#echo foo$nmbrf
	expr $nmbrf - 1
	echo "$nmbrf"
done

Terminal :
Code: 
4
3
4
3
4
3
4
3
4
3
^C

I really think that c programming is much more easier, please help.
 
The echo command shows a decreased value so I guess expr done the job
I've already done that but it crashes the scripts.
Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
#echo $nmbrf
#while [ $nmbrf > 0 ]
while [ $nmbrf -gt 0 ]
	#do mkdir "foo$nmbrf"
do
#echo foo$nmbrf
	`expr $nmbrf - 1`
	echo "$nmbrf"
done

Here it is :
Code: 
./scriptDevoir: line 10: 3: command not found
4
./scriptDevoir: line 10: 3: command not found
4
./scriptDevoir: line 10: 3: command not found
4
./scriptDevoir: line 10: 3: command not found
4
^C
 
Thank you, but it is just the beginning :
Code: 
#!/bin/sh
echo "Veuillez saisir un nombre"; read nmbr
nmbrf=$nmbr
#echo $nmbrf
#while [ $nmbrf > 0 ]
while [ $nmbrf -gt 0 ]
	#do mkdir "foo$nmbrf"
do
#echo foo$nmbrf
	nmbrf=`expr $nmbrf - 1`
	echo "$nmbrf"
done

Code: 
Veuillez saisir un nombre
4
3
2
1
0
 
jalla said:
One last hint
Code: 
nmbrf=`expr $nmbrf - 1`
Click to expand...

abdelilah said:
Thank you, but it is just the beginning
Click to expand...

abdelilah please note how the backticks work. They are opening up a new shell in the background and inheriting everything defined in the parent shell. The subshell evaluates the command expression to be returned back to the parents variable.

simple decrementing counter example
Code: 
(~) %  N=42; echo "=> ${N}"                                           
=> 42
(~) %  dec_N() {N=`expr $N + -1`; printf "=> N is now %d\n" $N;}      
(~) %  dec_N                                                          
=> N is now 41
(~) %  dec_N                                                          
=> N is now 40
(~) %  dec_N                                                          
=> N is now 39

Here is my english version of your program to show you how it can be accomplished with functions. Also note I only use one variable instead of two.

Code: 
#!/bin/sh

get_NUM() {
        echo "Please enter a number: "
        read NUM
}

NUM_countdown() {
        while [ $NUM -gt 0 ]; do
                echo ${NUM}
                NUM=`expr $NUM + -1`
        done
        printf "LIFTOFF!\n"
}
get_NUM
NUM_countdown
 
You must log in or register to reply here.
Back
Top