Crear mi propia función cp en bash

8

Para una tarea, se me pide que escriba inteligentemente una función bash que tenga la misma funcionalidad básica que la función cp(copiar). Solo tiene que copiar un archivo a otro, por lo que no se copian varios archivos a un nuevo directorio.

Como soy nuevo en el lenguaje bash, no puedo entender por qué mi programa no funciona. La función original pide sobrescribir un archivo si ya está presente, así que intenté implementarlo. Falla.

Parece que el archivo falla en varias líneas, pero lo más importante es que comprueba si el archivo a copiar ya existe ( [-e "$2"]). Aun así, todavía muestra el mensaje que se supone que se activará si se cumple esa condición (El nombre del archivo ...).

¿Alguien podría ayudarme a arreglar este archivo, posiblemente proporcionando alguna información útil en mi comprensión básica del lenguaje? El código es el siguiente.

#!/bin/sh
echo "file variable: $2"
if [-e file]&> /dev/null
then
    echo "The file name already exists, want to overwrite? (yes/no)"
    read  | tr "[A-Z]" "[a-z]"
    if [$REPLY -eq "yes"] ; then
        rm "$2"
        echo $2 > "$2"
        exit 0
    else
        exit 1
    fi
else
    cat $1 | $2
    exit 0
fi
timmaay92
fuente
17
Comience con www.shellcheck.net
steeldriver
2
Si esto fuera una revisión de código: nunca lo hagas [ ... ] &> /dev/null. Las pruebas admitidas por [ ... ]son siempre silenciosas, solo se produce un resultado de sintaxis. Silenciar tal prueba es simplemente cavar tu propia tumba.
muru
1
Un punto: cat $1 | $2"canaliza" la salida del primer comando al comando en su segunda variable. Pero ese es un nombre de archivo, no el nombre de un comando para ejecutar.
Floris
Debe revisar la sintaxis básica del [comando en su if.
Barmar
Y necesita aprender la diferencia entre canalizar |y redirigir a un archivo con >. Estos son problemas básicos de sintaxis que ya deberían haberse cubierto en su clase.
Barmar

Respuestas:

26

La cputilidad sobrescribirá felizmente el archivo de destino si ese archivo ya existe, sin preguntar al usuario.

Una función que implementa la cpcapacidad básica , sin usar cpsería

cp () {
    cat "$1" >"$2"
}

Si desea solicitar al usuario antes de sobrescribir el destino (tenga en cuenta que puede que no sea conveniente hacerlo si un shell no interactivo llama a la función):

cp () {
    if [ -e "$2" ]; then
        printf '"%s" exists, overwrite (y/n): ' "$2" >&2
        read
        case "$REPLY" in
            n*|N*) return ;;
        esac
    fi

    cat "$1" >"$2"
}

Los mensajes de diagnóstico deben ir al flujo de error estándar. Esto es con lo que hago printf ... >&2.

Tenga en cuenta que realmente no necesitamos rmel archivo de destino ya que la redirección lo truncará. Si nosotros nos queremos rmen primer lugar, a continuación, usted tendría que comprobar si se trata de un directorio, y si lo es, poner el archivo de destino dentro de ese directorio, tal y como cpharía. Esto está haciendo eso, pero aún sin explícito rm:

cp () {
    target="$2"
    if [ -d "$target" ]; then
        target="$target/$1"
    fi

    if [ -d "$target" ]; then
        printf '"%s": is a directory\n' "$target" >&2
        return 1
    fi

    if [ -e "$target" ]; then
        printf '"%s" exists, overwrite (y/n): ' "$target" >&2
        read
        case "$REPLY" in
            n*|N*) return ;;
        esac
    fi

    cat "$1" >"$target"
}

También puede asegurarse de que la fuente realmente exista, que es algo que cp lo hace ( cattambién lo hace, por lo que puede omitirse por completo, por supuesto, pero hacerlo crearía un archivo de destino vacío):

cp () {
    if [ ! -f "$1" ]; then
        printf '"%s": no such file\n' "$1" >&2
        return 1
    fi

    target="$2"
    if [ -d "$target" ]; then
        target="$target/$1"
    fi

    if [ -d "$target" ]; then
        printf '"%s": is a directory\n' "$target" >&2
        return 1
    fi

    if [ -e "$target" ]; then
        printf '"%s" exists, overwrite (y/n): ' "$target" >&2
        read
        case "$REPLY" in
            n*|N*) return ;;
        esac
    fi

    cat "$1" >"$target"
}

Esta función no utiliza "bashismos" y debería funcionar en todos los shshells.

Con un poco más de ajustes para admitir múltiples archivos fuente y una -imarca que activa la solicitud interactiva al sobrescribir un archivo existente:

cp () {
    local interactive=0

    # Handle the optional -i flag
    case "$1" in
        -i) interactive=1
            shift ;;
    esac

    # All command line arguments (not -i)
    local -a argv=( "$@" )

    # The target is at the end of argv, pull it off from there
    local target="${argv[-1]}"
    unset argv[-1]

    # Get the source file names
    local -a sources=( "${argv[@]}" )

    for source in "${sources[@]}"; do
        # Skip source files that do not exist
        if [ ! -f "$source" ]; then
            printf '"%s": no such file\n' "$source" >&2
            continue
        fi

        local _target="$target"

        if [ -d "$_target" ]; then
            # Target is a directory, put file inside
            _target="$_target/$source"
        elif (( ${#sources[@]} > 1 )); then
            # More than one source, target needs to be a directory
            printf '"%s": not a directory\n' "$target" >&2
            return 1
        fi

        if [ -d "$_target" ]; then
            # Target can not be overwritten, is directory
            printf '"%s": is a directory\n' "$_target" >&2
            continue
        fi

        if [ "$source" -ef "$_target" ]; then
            printf '"%s" and "%s" are the same file\n' "$source" "$_target" >&2
            continue
        fi

        if [ -e "$_target" ] && (( interactive )); then
            # Prompt user for overwriting target file
            printf '"%s" exists, overwrite (y/n): ' "$_target" >&2
            read
            case "$REPLY" in
                n*|N*) continue ;;
            esac
        fi

        cat -- "$source" >"$_target"
    done
}

Su código tiene mal if [ ... ]espaciado (necesita espacio antes y después [, y antes ]). Tampoco debe intentar redirigir la prueba /dev/nullya que la prueba en sí no tiene salida. Además, la primera prueba debe usar el parámetro posicional $2, no la cadena file.

Usando case ... esaccomo lo hice, evitas tener que poner en minúscula / mayúscula la respuesta del usuario que usa tr. En bash, si hubiera querido hacer esto de todos modos, una forma más económica de hacerlo hubiera sido usar REPLY="${REPLY^^}"(para mayúsculas) o REPLY="${REPLY,,}"(para minúsculas).

Si el usuario dice "sí", con su código, la función coloca el nombre de archivo del archivo de destino en el archivo de destino. Esto no es una copia del archivo fuente. Debe pasar al bit de copia real de la función.

El bit de copia es algo que ha implementado usando una tubería. Una tubería se utiliza para pasar datos desde la salida de un comando a la entrada de otro comando. Esto no es algo que debamos hacer aquí. Simplemente invoque caten el archivo fuente y redirija su salida al archivo de destino.

Lo mismo está mal con tu llamado de trantes. readestablecerá el valor de una variable, pero no produce salida, por lo que canalizar readcualquier cosa no tiene sentido.

No se necesita una salida explícita a menos que el usuario diga "no" (o la función se encuentra con alguna condición de error como en bits de mi código, pero como es una función que uso en returnlugar de exit).

Además, dijiste "función", pero tu implementación es un script.

Eche un vistazo a https://www.shellcheck.net/ , es una buena herramienta para identificar partes problemáticas de scripts de shell.


Usar cates solo una forma de copiar el contenido de un archivo. Otras formas incluyen

  • dd if="$1" of="$2" 2>/dev/null
  • Usar cualquier utilidad similar a un filtro que se pueda hacer para que simplemente pase datos, por ejemplo, sed "" "$1" >"2"or awk '1' "$1" >"$2"o tr '.' '.' <"$1" >"$2"etc.
  • etc.

El truco es hacer que la función copie los metadatos (propiedad y permisos) del origen al destino.

Otra cosa a tener en cuenta es que la función que he escrito se comportará de manera bastante diferente a cpsi el objetivo es algo así como, /dev/ttypor ejemplo (un archivo no regular).

Kusalananda
fuente
para agregar una precisión interesante sobre cómo funciona:: cat "$1" >"$2" funcionará 2 veces: el shell ve por primera vez > "$2", lo que significa: crear-o-clobber (= vacío) el archivo "$ 2", y redirigir la salida estándar del comando a ese archivo. Luego, inicia el comando: cat "$1"y redirige su stdout a un archivo "$2"(ya vaciado o creado vacío).
Olivier Dulac
¡Una de las mejores respuestas que he visto aquí! Aprendí sobre ambas -efcomparaciones de archivos y local -a.
cabeza de jardín
@gardenhead Sí, -eftoma avisos si los dos archivos son iguales (también se encarga de los enlaces) y local -acrea una variable de matriz local.
Kusalananda
Muchas gracias! Muy buena respuesta elaborada, muy útil en la comprensión del lenguaje también
timmaay92