Shell Programming and Scripting

#!/bin/bash
#This will create blank lyric files, as used by mocp, for all mp3s in a direcory

files=(ls *.mp3)
for song in $files
 echo "${song%%*(.mp3)}"
 touch "${song%%*(.mp3)}"
done

#!/bin/bash
#This will create blank lyric files, as used by mocp, for all mp3s in a direcory

files=(ls *.mp3)
for song in $files
do
 echo "${song%%*(.mp3)}"
 touch "${song%%*(.mp3)}"
done

files=(ls *.mp3)                # this will define & assign an array: (ls a.mp3 b.mp3 c.mp3 ...). You'd need to use it like an array in the for loop, then. 
                                # you might prefer command substitution: files=$(ls *.mp3)
                                # or let the shell do the work for you: files=*.mp3
for song in $files              # this will expand to the first array element: ls. If you want the .mp3 files as well, use array expansion: ${files[@]} if you don't like above proposals
 echo "${song%%*(.mp3)}"        # Why the * and the parentheses? You want to remove nothing but ".mp3", and once only, so one "%" is sufficient
 touch "${song%%*(.mp3)}"       # "${song%.mp3}" will do the job for you. "%*.mp3"  would erase the entire file name, the parentheses can't be matched
                                # and thus stop the expansion.  
                                # You are right in double quoting the file names which will handle spaces in file names, then.
done

#!/bin/bash
#This will create blank lyric files, as used by mocp, for all mp3s in a directory

files=*.mp3
for song in ${files[*]}
do
 echo "${song%.mp3}"
 touch "${song%.mp3}"
done