Define "its not working". Just from quickly inspecting your code, I'd highly recommend using a LayoutManager rather than a null layout. See A Visual Guide to Layout Managers (The Java™ Tutorials >...