Se tastiera è dichiarato nel main, è LOCALE e ovviamente non è visibile in altri metodi. Serve cambiare un pochino le cose.
riempiZaino così come si vede è comunque sbagliato. Fai un for sugli indici ma ad ogni elemento assegni lo stesso x, sovrascrivendo altro che c'era prima.
Inoltre cerca di usare nomi di variabili chiari e comprensibili, non x, j, h, y, ecc... Quando finisci le lettere dell'alfabeto ...che fai?![]()