[Batch] Se me cae el script

[Eleкtro]

De que me hablas? no se que habras leido.. yo no puse nada.. jajaja, me voy chauuu

XDDDDDDD

[VivaElPirateo]

Lo del doble post se me había olvidado: es algo que está siempre en regla en todos los foros...sorry por eso!!!

Código [Seleccionar] Expandir

set /p "op=Ingrese una opcion: "
Echo "%OP%" | FINDSTR "\<[1-8]\>" Error de seleccion pause >NUL|| (Goto :Select)
Goto :Opcion%OP%

Al hacer lo de las comillas dobles en %OP% me resuelve lo del &, pero no lo de la comilla "

Millón de gracias por las respuestas sociates!!!!!!!!

Igual necesito ponerle un mensaje de error para que el usuario sepa que se está equivocando en ingresar los caracteres.

Por más que intento hacerlo, no lo consigo.

Ayuda, porfa!!!!!!!

[$Edu$]

Lo de que haga el mensaje de error no se como, pero mejore el codigo, anulando lo de la comilla, ahora si no da error nunca, eso espero.

Código (dos) [Seleccionar] Expandir


:menu
cls
echo+
echo+
echo              1.- Ver datos de autor.
echo              2.- Mirar mensaje.
echo              3.- Salir del programa.
echo+
echo+
set "opcion="
set /p "opcion= Elija a donde desea ir: "
if not defined opcion (goto menu)
if "%opcion:"=%"=="1" (goto :autor)
if "%opcion:"=%"=="2" (goto :mensaje)
if "%opcion:"=%"=="3" (goto :salir) 
goto :menu


Hice que verifique el valor de opcion pero sin las comillas. Total todo cuenta como error.

Tal vez haciendo lo de FINDSTR y buscando si existe la comilla ", que muestre el mismo error que cuando no se ingresa los datos correctos, es decir, hacer una captura de esa exepcion en batch, para q antes de hacer los ifs verifique si existe la comilla. Yo no se usar FINDSTR, pero Elektro tal vez lo hace.

Aunque.. para mi, repito, no mostrar ningun mensaje, asi como hace este ultimo codigo que deje, queda mas profesional, pienso yo.

[Eleкtro]

Yo creo q lo profesional es mostrar un mensaje de error siempre que el código pueda dar algún error incorregible, En caso contrario, Si es un código perfecto que no puede dar lugar a errores, Pues sin mensaje de error como dice $Edu$

Y con este code de abajo, ya no puede dar error, Pero ahi te dejo para que pongas el mensaje de error a tu gusto:

Código (dos) [Seleccionar] Expandir

@echo off

:Select
CLS

REM Reseteamos el valor de la variable...
Set op=Foro.ElHacker.Net

set /p "op=Ingrese una opcion: "

REM Reemplacamiento del string. (Le borramos las comillas que contenga la variable. No es necesario hacer esto con el resto de caracteres.)
Set "Op=%OP:^"=%" & REM "

REM Esta linea devuelve directamente al menú principal si no se introduce el intervalo de números 1 a 8.
Echo "%OP%" | FINDSTR "\<[1-8]\>" >NUL || (Goto :Select)

REM Esta linea manda un mensaje de error si no se introduce el intervalo de números 1 a 8, y devuelve al menú principal.
REM Echo "%OP%" | FINDSTR "\<[1-8]\>" >NUL || (Echo: Tu mensaje de error & Ping -n 4 localhost >NUL & Goto :Select)

Goto :Opcion%OP%

:Opcion1
Echo Has elegido la Opcion 1 & pause & Goto :Select
:Opcion2
Echo Has elegido la Opcion 2 & pause & Goto :Select
:Opcion3
pause
etc...

Saludos

[VivaElPirateo]

Ahora tengo lo siguiente:

Código [Seleccionar] Expandir

set "var="   
set /p "var=               Ingrese un numero entre el 1 y el 8 : "



if not defined opcion (goto :inicio)

if "%var:"%"=="1" (goto :alfa)
if "%var:"%"=="2" (goto :beta)
if "%var:"%"=="3" (goto :gamma)
if "%var:"%"=="4" (goto :epsilon)
if "%var:"%"=="5" (goto :omega)
if "%var:"%"=="6" (goto :uno)
if "%var:"%"=="7" (goto :dos)
if "%var:"%"=="8" (goto :tres)



echo off
:alfa
color 2e
cls
echo...


Todo OK con lo de la validación de errores, pero cuando intento pasar a la opción alfa, por ejemplo, no me dirige hacia ahí.

Cuál esl el drama???

[Eleкtro]

Código (dos) [Seleccionar] Expandir

if "%var:"%"=="1" (goto :alfa)
if "%var:"%"=="2" (goto :beta)
if "%var:"%"=="3" (goto :gamma)
...

Todo OK con lo de la validación de errores, pero cuando intento pasar a la opción alfa, por ejemplo, no me dirige hacia ahí.

No, Nada está OK. El código que has puesto ahora no da errores porque no está comprobando ninguna variable.
Estás usando mal la syntaxis, Estás escribiendo una comilla doble donde no debe haber comillas, Las comprobaciones no se puede efectuar.

Se te ha explicado como hacerlo, Sobretodo en mi última respuesta.
Si tienes más dudas con este código, Mejor deberias pensar en coger un manual de Batch y empezar a practicar, Porque de verdad que ya has recibido toda la ayuda posible para este problema, Te lo dijimos TODO y te lo dimos todo hecho, Ahi tienes los códigos que funcionan, No hay más misterio.


PD:

Código (dos) [Seleccionar] Expandir

:: BAD
if "%var:"%"=="1" (goto :alfa)

:: GOOD
if "%var%"=="1" (goto :alfa)

Un saludo.

[$Edu$]

Creo que te has equivocado Elektro, esas comillas era por lo de que no de errores y entonces yo las remplazaba por nada.

El codigo ya lo deje y es este:

Código (dos) [Seleccionar] Expandir


:menu
cls
echo+
echo+
echo              1.- Ver datos de autor.
echo              2.- Mirar mensaje.
echo              3.- Salir del programa.
echo+
echo+
set "opcion="
set /p "opcion= Elija a donde desea ir: "
if not defined opcion (goto menu)
if "%opcion:"=%"=="1" (goto :autor)
if "%opcion:"=%"=="2" (goto :mensaje)
if "%opcion:"=%"=="3" (goto :salir) 
goto :menu



Fijate bien lo que copias y no copies a media, mas vale trata de entenderlo y entonces no te equivocaras. No se porque por ejemplo pones un echo off por ahi en tu codigo..

[Eleкtro]

Creo que te has equivocado Elektro, esas comillas era por lo de que no de errores y entonces yo las remplazaba por nada.

No, no me habia equivocado, Tú si que estás haciendo bien tu code, Al agregarle el signo del  = estás eliminando temporalmente las comillas para hacer la comprobacion, Pero en el code que he citado no hace eso, No funciona xD, El tuyo sí.

Código (dos) [Seleccionar] Expandir

if "%opcion:"=%"=="1" (goto :autor)

Código (dos) [Seleccionar] Expandir

if "%opcion:"%"=="1" (goto :autor)

Saludos

[$Edu$]

Ahhh no veia bien xDD

[VivaElPirateo]

Mil gracias socitos!!!!

Ahora sí que me pegué la escurrida y me di cuenta del error.


Millón de gracias, mis buenos foreros!!!




Otra consulta:

Cómo la hago para poner más de 9 opciones???

Ya que cuando intento poner 10 ó más, me genera problemas el código:

Código [Seleccionar] Expandir

set "op="
set /p "op=               Ingrese una opcion entre el 1 y el 10 : "

Set "op=%op:^"=%" & REM "

echo "%op%" | FINDSTR "\<[1-9]\>" >NUL || (Goto :beta)

Goto :op%op%


La tengo que dejar en 9 para no tener problemas.


No sé si me responda a mí mismo con esto:

Código [Seleccionar] Expandir

echo "%op%" | FINDSTR "\<[1-9] || [10] \>" >NUL || (Goto :beta)